Hi,

I'm the maintainer of the module Enforce revision log message, and author of http://drupal.org/node/369348. I've extended and replaced the mini-module with a more complete version including custom permissions, per content type settings, AJAX update of the node form, log textarea while reverting revisions, as well as installation/uninstallation routines. And today I've just stumbled upon Util, which includes its own version of enforce_revlog.

Currently, the enforce_revlog.module present in Util has the same name and is has less functionalities that my module. I'm here to discuss how we could avoid duplication, either by including my module into Util to replace the existing one, or, if my module is too large, removing your version of enforce_revlog and directing users to the more complete version.

Please have a look at my module and let's discuss.

Comments

nancydru’s picture

nancydru
she/her/hers
Primary language English
Location Boston
commented

That's probably where I got the code, although I seem to recall someone asking for it. At any rate, I sshould probably remove it from Util and tell people to use yours as a more complete solution. I'll need to look at my customer's site and see how to sneak it in.

mdupont’s picture

mdupont
Primary language French
Location Switzerland
commented

Any news on the issue? As you can read on #976676: Module conflicts with one of Util contributed submodules there is not only duplication between the 2 module but also a conflict as the two share the same name and as a result only one is listed in the module administration page.

I too think all is needed is to remove the enforce_revlog version in Util and to warn users that they should install the standalone Enforce revlog module if they need the feature.

Thanks!

nancydru’s picture

nancydru
she/her/hers
Primary language English
Location Boston
commented
Status: Active » Fixed

Removed from this module.

Status: Fixed » Closed (fixed)

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

jvieille’s picture

jvieille commented
Status: Closed (fixed) » Active

enforce_revlog.module is still part of the util module, and still conflicts with stand-alone enforce_revlog.module

nancydru’s picture

nancydru
she/her/hers
Primary language English
Location Boston
commented
Status: Active » Fixed

I hope I got it this time.

Status: Fixed » Closed (fixed)

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