Edgewall Software
Modify

Opened 18 years ago

Closed 18 years ago

Last modified 18 years ago

#2885 closed defect (fixed)

Deletion of page versions not hooked by !IWikiChangeListener

Reported by: Alec Thomas Owned by: Jonas Borgström
Priority: normal Milestone: 0.10
Component: wiki system Version: devel
Severity: normal Keywords:
Cc: Branch:
Release Notes:
API Changes:
Internal Changes:

Description (last modified by Alec Thomas)

There are hooks for page deletion as well as modification and addition, but not deleting a version.

Attachments (2)

wiki-listener-delete-version.diff (2.0 KB ) - added by Alec Thomas 18 years ago.
Updated patch with additional check
wiki-listener-delete-version.2.diff (1.9 KB ) - added by Alec Thomas 18 years ago.
Check for version delete before calling, for backwards compatibility

Download all attachments as: .zip

Change History (9)

comment:1 by Alec Thomas, 18 years ago

Description: modified (diff)

comment:2 by Emmanuel Blot, 18 years ago

I might be wrong, but page deletion (or version deletion) are not tracked in the DB, so once the page has been deleted, there is no way to know that it has existed at an earlier time. It is therefore not possible for now to obtain this kind of events when the DB is parsed.

comment:3 by Emmanuel Blot, 18 years ago

I guess I've missed something if the submitted patch address this issue - please ignore my previous comment ;-)

comment:4 by anonymous, 18 years ago

Priority: normalhighest
Resolution: fixed
Status: newclosed
Type: defecttask

comment:5 by Emmanuel Blot, 18 years ago

Priority: highestnormal
Resolution: fixed
Status: closedreopened
Type: taskdefect

Reverting to the previous state, I guess this ticket has been spammed (if not, do no use "anonymous" to change the ticket status).

by Alec Thomas, 18 years ago

Updated patch with additional check

comment:6 by Alec Thomas, 18 years ago

Check for existence of wiki_page_version_deleted() before calling, for backwards compatibility.

by Alec Thomas, 18 years ago

Check for version delete before calling, for backwards compatibility

comment:7 by Alec Thomas, 18 years ago

Resolution: fixed
Status: reopenedclosed

Added in r3206.

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Jonas Borgström.
The resolution will be deleted. Next status will be 'reopened'.
to The owner will be changed from Jonas Borgström to the specified user.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.