Posted by linksunten on January 21, 2009 at 2:58am
| Project: | Comment Revisions |
| Version: | 6.x-2.x-dev |
| Component: | Code |
| Category: | task |
| Priority: | normal |
| Assigned: | thinkyhead |
| Status: | closed (fixed) |
Issue Summary
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!
| Attachment | Size |
|---|---|
| comment_revisions.module.diff | 9.23 KB |
Comments
#1
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.
#2
We found more bugs and hopefully fixed them.
IMC linksunten
#3
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.
#4