We came across a scenario in our project, where we need to check logs for revision deletion if there is a content loss. Here is patch for adding log entried for revisions deleted.

CommentFileSizeAuthor
#3 3216761-3.patch3.73 KBguilhermevp
#3 verbose.png9.2 KBguilhermevp
node-revision-delete-verbose-logging.patch1.27 KBsharique

Comments

Sharique created an issue. See original summary.

adriancid’s picture

adriancid
Primary language Spanish
Location Qc, Canada 🇨🇦
commented

Thanks for the patch, I think will be better to add a configuration option to use this feature, can you work adding the configuration in the module configuration page?

guilhermevp’s picture

guilhermevp commented
StatusFileSize
new verbose.png9.2 KB
new 3216761-3.patch3.73 KB

Hi! I created the config in /admin/config/content/node_revision_delete.

1

Maybe the wording can be better, but it works.

Status: Needs review » Needs work

The last submitted patch, 3: 3216761-3.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

adriancid’s picture

adriancid
Primary language Spanish
Location Qc, Canada 🇨🇦
commented

You need to update the existing tests and add one to test if the log is working.

miiimooo’s picture

miiimooo
Location Europe
commented

Looking at this I wonder how useful it is: I guess if you have a copy of the db before deletion, you can look up the affected nodes based on revision id. But what would be nice is to also get the node id (or even title?)

adriancid’s picture

adriancid
Primary language Spanish
Location Qc, Canada 🇨🇦
commented
Status: Needs work » Closed (duplicate)
Related issues: +#3321626: Log revision deletion

Closing because we have a working patch in #3321626: Log revision deletion and I will give credit there