"Don't create new revision" in revisions tab

neochief - May 31, 2009 - 19:20
Project:Vertical Tabs
Version:6.x-1.x-dev
Component:Code
Category:bug report
Priority:normal
Assigned:Dave Reid
Status:closed
Description

Hello.

When you don't have 'administer content' permission, in your node form there's a revision tab, and it doesn't contains the "Create a new revison" checkbox. But it is enabled by default under the hood, so, you will create a new revision on form submit.

So, the bug is that vertical tab contains "Don't create new revision" text, that is wrong, because you _are_ creating a new revision. Here's a patch to fix this bug.

AttachmentSize
vertical_tabs.patch692 bytes

#1

quicksketch - May 31, 2009 - 22:49
Status:needs review» needs work

Seems like the text should be omitted entirely if the option doesn't exist. This patch isn't necessary correct if the default setting for the content type is to not create a new revision and a user doesn't have administer content permission.

#2

neochief - June 1, 2009 - 00:10
Status:needs work» needs review

If default settings for content type is not to create revision and you have no administre rights, you'll see no revision tab at all. I tested it ;)

But I agree with your point about empty message. Here's a new patch.

AttachmentSize
vertical_tabs.patch 945 bytes

#3

alexanderpas - August 14, 2009 - 11:22
Status:needs review» reviewed & tested by the community

#4

Dave Reid - November 6, 2009 - 20:29
Assigned to:Anonymous» Dave Reid

Will get committed tonight.

#5

Dave Reid - November 6, 2009 - 23:41
Status:reviewed & tested by the community» fixed

Tested that the revision information tab doesn't come up if the user doesn't have admin nodes permission, tested with create story permission and access disabled on the 'Create new revision' checkbox, and got the bug as discovered and fixed with the patch. Committed to CVS (http://drupal.org/cvs?commit=285310). Thanks!

#6

System Message - November 20, 2009 - 23:50
Status:fixed» closed

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

 
 

Drupal is a registered trademark of Dries Buytaert.