Problem/Motivation

The revisioning_require_update_to_publish variable uses different default value throughout the site causing confusion. It is defaulted to TRUE on the setting page, making one think it is enabled. It is then defaulted to FALSE when actually checking permissions.

Proposed resolution

The default should be TRUE for all references. A user without update permissions to an entity should not be able to change the published status (update) of that entity.

Remaining tasks

Patch attached, reviews needed.

Comments

jantoine’s picture

Status: Active » Needs review
StatusFileSize
new1.57 KB
rdeboer’s picture

Thanks for the patch jantoine.
Makes sense. Good pick-up!
Rik

rdeboer’s picture

Patch applied with attribute. Thanks Jon!
Rik

rdeboer’s picture

Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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