If user doesn't have access to the moderated revision there is no sense in redirecting him/her to the revision page.

Steps to reproduce (the most simple case):

  • Create node type A with enabled revision moderation.
  • Allow anonymous users to edit any nodes of type A (without any other content permissions, except for "View published content").
  • Create node of type A by admin.
  • Edit this node by anonymous user and save it.

Expected behavior:

  • Visitor is redirected back to the node/% page

Actual behavior:

  • Visitor is redirected to the node/%/revisions/%/view and see "Access denied".

Patch follows.

CommentFileSizeAuthor
#1 revisioning-1681254-1.patch1.32 KBdmitriy.trt

Comments

dmitriy.trt’s picture

Status: Active » Needs review
StatusFileSize
new1.32 KB

Patch attached.

rdeboer’s picture

Assigned: Unassigned » rdeboer

Thanks Dmitriy!
Makes sense. Hope to set aside some time for a batch of patches I want to apply soon.
Rik

rdeboer’s picture

Status: Needs review » Fixed

Patch applied (finally!).
Thanks Dmitriy!
Rik

Status: Fixed » Closed (fixed)

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