We found some bugs we hopefully fixed with our patch. The handling of permissions to administer the module was not correctly implemented. The module forgot some of its variables.

Now you can can set the maximal user revisions to unlimited, too. And you can decide if you want log messages. And the buttons to delete/revert only appear if you have the neccessary rights (reducing the number links leading to an "access denied" page).

Thanks for this wonderful module, it will help us to maintain transparency!

CommentFileSizeAuthor
#2 comment_revisions.tar_.gz12.99 KBlinksunten
#1 comment_revisions.tar_.gz12.8 KBlinksunten
comment_revisions.module.diff9.23 KBlinksunten

Comments

linksunten’s picture

linksunten commented
Title: Patch for Comment revisions » New features
Category: bug » task
StatusFileSize
new comment_revisions.tar_.gz12.8 KB

We added the new feature to stash a revision, i.e. make it invisible. Of course, the current revision cannot be stashed and a stashed revision cannot be reverted.

linksunten’s picture

linksunten commented
Title: New features » New features & less bugs
StatusFileSize
new comment_revisions.tar_.gz12.99 KB

We found more bugs and hopefully fixed them.

IMC linksunten

thinkyhead’s picture

thinkyhead commented
Assigned: Unassigned » thinkyhead
Priority: Critical » Normal

Patch in the OP has been incorporated into current 5.x and 6.x releases. I'm not sure whether it's very valuable to include invisible (stashed) revisions. I'd personally like this module to be as simple as Node revisions, but don't mind additional enhancements as options. Maybe we'll put in all the bells and whistles, then put a "simple revisions" checkbox in the settings for those who want to set and forget...

For now, I'm reviewing all extant patches, and will include all the good stuff in incremental releases.

thinkyhead’s picture

thinkyhead commented
Title: New features & less bugs » New features & fewer bugs
Status: Needs review » Closed (fixed)