Closed (fixed)
Project:
Project issue tracking
Version:
7.x-2.x-dev
Component:
Issues
Priority:
Major
Category:
Bug report
Assigned:
Issue tags:
Reporter:
Created:
6 Sep 2015 at 17:00 UTC
Updated:
30 Sep 2015 at 14:54 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
formatC'vt commentedi'm working on it
Comment #3
formatC'vt commenteddone, patch attached.
Comment #4
drummLooking at
node_node_access(), theadminister nodespermission isn't actually checked. I'll update this to match the update access checking there.Comment #5
drummNevermind, matching
node_node_access()would of course make this override affect nothing.Comment #6
formatC'vt commentedhm, i do some test on localhost and user without
administer nodespermission can't edit issues with closed comments.Comment #8
drummCommitted with a small change - only
NODE_ACCESS_DENYis returned if applicable. In case there are multiplehook_node_access()implementations acting, a more-targetted implementation is more-likely to avoid conflicts.administer nodesis indeed a good permission to check for overrides.Comment #9
drummNow deployed to Drupal.org.