Closed (fixed)
Project:
Revision Moderation
Version:
master
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
31 Jan 2007 at 11:34 UTC
Updated:
26 Feb 2007 at 22:15 UTC
Jump to comment: Most recent file
Comments
Comment #1
toemaz commentedI was able to reproduce this problem. I believe this can be solved with a fix, implementing the form_alter hook.
But perhaps it would be better to look into the core for a nice solution.
Comment #2
toemaz commentedA quick work around without changing the revision_moderation module: add this module to your system and enable it. It will make the log window disappear.
It's of course not a nice long term solution. Perhaps a very simple one: a setting to display the log textarea or not. Or perhaps we can create a contrib module for a separte log system. Let me explain that last one.
This revision_moderation module is supposed to give users the possiblity to change the content. This change can vary: sometimes it is a minor change, sometimes a bigger change. For my type of website, it would be nice that users are able to select the type of change they made from a combo box. I know it seems strange but it some cases this might be usefull:
Would this be usefull in any kind for your type of website?
Also, I must check whether there isn't already something like this on the CVS.
Comment #3
toemaz commentedI forgot to attach the module. Here it is.
Comment #4
toemaz commentedI guess we can leave it as a contrib module. No need to clutter the revision_moderation module.
Comment #5
webchickhi, toemaz.
In 4.7.x, the "log" field was only added to books and pages. This is not very useful, since you basically always want the log field whenever there are revisions present, in order to track why changes were made. So that is why the module adds log fields to all node types.
In 5.x, however, log fields are now auto-appended to *all* content types when revisions are enabled (although node administrators always see the field). So this functionality would no longer be needed in a 5.x port of the module.
Hope that helps clear things up.
Comment #6
toemaz commentedThx for the clarification!
Comment #7
webchickIn looking at this more closely, I see the bug the original poster was talking about... the log field shows up for all node types, regardless. It should only show up on types that have revisions enabled.
This has now been fixed in the 4.7 branch.
Comment #8
marcoBauli commentedwebchick, thanks, yep that's what i meant to say (sorry for my non-motherlanguageness), great to hear is fixed now! cheers
Comment #9
webchickmarcob: No, your English is fine, I got distracted by the other comments. :)
No problem!
Comment #10
(not verified) commented