Opened 9 years ago

Closed 5 years ago

#41 closed defect (fixed)

Recent Changes includes ancient edits if you don't have minor edits switched on

Reported by: kake Owned by: Nobody
Priority: critical Milestone:
Component: wiki-toolkit Version: 0.75
Keywords: Cc:

Description

I think this bug was probably introduced when I last overhauled the Recent Changes code; the test suite seems to have been inadequate.

If you list the Recent Changes with minor edits excluded, then it returns not only the most recent changes, but also some more ancient changes to nodes that have been edited recently. To reproduce this, just look at a reasonably active OpenGuides? install with minor edits excluded.

Attachments (1)

test_for_metadata_wasnt_recent_changes.diff (5.2 KB) - added by bob 8 years ago.
diff which adds tests which exposes this bug.

Download all attachments as: .zip

Change History (8)

comment:1 Changed 9 years ago by Dominic Hargreaves

Owner: changed from Dominic Hargreaves to Nobody

comment:2 Changed 9 years ago by Dominic Hargreaves

Priority: majorcritical

comment:3 Changed 9 years ago by Dominic Hargreaves

This is triggered by the following code in OpenGuides?:

            $criteria{metadata_was} = { edit_type => "Normal edit" }
              unless $minor_edits;
            my @rc = $self->{wiki}->list_recent_changes( %criteria );

comment:4 Changed 8 years ago by Dominic Hargreaves

Version: svn0.75

Changed 8 years ago by bob

diff which adds tests which exposes this bug.

comment:5 Changed 8 years ago by bob

i also have a fix for openguides to use is instead of was.

comment:6 Changed 8 years ago by Dominic Hargreaves

(In [503]) add more recent changes tests (references #41)

comment:7 Changed 5 years ago by kake

Resolution: fixed
Status: newclosed

(In [533]) Fixed SQL for recent changes when metadata_was/metadata_wasnt parameter supplied (fixes #41). Also removed bogus test after checking with bob (who wrote it).

Note: See TracTickets for help on using tickets.