Closed (fixed)
Project:
Revision Deletion
Version:
5.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
12 Aug 2009 at 21:53 UTC
Updated:
31 Aug 2009 at 18:34 UTC
The attached patch brings the 5.x version (will be ported to 6.x shortly) to compliance with the coding standards.
In addition, it makes, IMHO, the admin interface much cleaner (must clear the menu cache) by using radios and checkboxes (hence added a css file). I altered the intervals a bit. There were several strings that were not translatable so I wrapped them in t(). I added some descriptive text to the list.
One should not, as a friend recently discovered, allow a module like this to fire its cron code before the adopter is fully aware of what they're doing. So I set the default cron interval to 0, for "manual only" and added code to check for that.
| Comment | File | Size | Author |
|---|---|---|---|
| revision_deletion_5_01.patch | 10.94 KB | nancydru | |
| revision_deletion.css_.txt | 695 bytes | nancydru |
Comments
Comment #1
nancydruComment #2
Gman commentedThese look great, NancyDru. I admit I have neglected this module since the site that prompted its creation is no longer in use.
Thanks. -Greg
Comment #3
nancydruNo problem.
Comment #4
nancydruIncluded in the -rc1 versions.