CVS edit link for mdupont

Hi,

I'm proposing a module allowing to force users to enter a revision log message when updating a node. Such a feature is useful on sites with several editors to easily keep track of the changes.

It is based on the following mini-module: http://drupal.org/node/369348
The plan is to turn it into a full-featured module, as some people requested it.

As my work involves developing websites with Drupal, I have a couple further projects of modules, and it would also be a way to publish some custom modules we could have to write for our customers.

Comments

mdupont’s picture

Status: Postponed (maintainer needs more info) » Needs review
StatusFileSize
new722 bytes

Attached is the archive containing the enforce_revlog module.

avpaderno’s picture

Issue tags: +Module review
dawehner’s picture

Status: Needs review » Reviewed & tested by the community

Module looks fine for me. I would like to have a setting per nodetype, but thats not part of the module.

avpaderno’s picture

I am not sure we accept modules that don't have a settings page as CVS application. If this would be the case, everybody would get a CVS account by writing (or copying) a single function module.

The main purpose of a CVS application is to understand if who is applying understood the basics on how to write a module, including secure code, coding standards, etc...

dawehner’s picture

Status: Reviewed & tested by the community » Needs work

Based on this updated the status to needs work

mdupont’s picture

Status: Needs work » Needs review
StatusFileSize
new1.12 KB

Updating the module to add:

  • A settings page
  • An help page

Please tell me if you need something else :-)

mdupont’s picture

StatusFileSize
new1.11 KB

Sorry, I didn't save the module file right so it had indentation issues. Please use this archive instead.

mdupont’s picture

StatusFileSize
new1.34 KB

I had some time to hack last night :-)

  • Added: per content type settings
  • Added: uninstallation routine
avpaderno’s picture

Status: Needs review » Fixed

It would also useful if the module would give a feedback to the user, setting the log field as required, in the cases he is not allowed to not write a log message.

Status: Fixed » Closed (fixed)
Issue tags: -Module review

Automatically closed -- issue fixed for 2 weeks with no activity.

avpaderno’s picture

Component: Miscellaneous » new project application
Assigned: Unassigned » avpaderno
Issue summary: View changes