It would be very usefull to have permissions of nodeinblock edit, and not only the default with node edit permissions.

Scenario: Many users with edit own permissions, I don't want to give them the permissions to put some node in a block.

Thanks!.

Alek

CommentFileSizeAuthor
#7 901858.patch1009 bytesswentel

Comments

swentel’s picture

hrm, makes sense, never thought about that actually. Something like 'Configure nodes in block on node edit page' should be ok right ? (or something alike)

alek123’s picture

Yes!, that's it. :D

Thank you.

Alek

swentel’s picture

Status: Active » Fixed

Committed to dev version. The permission is called 'deny configuration on node form'. It acts as a restristrive permission (except for user 1). If the user has this permission, it won't have access to the nodes in block fieldset.

Status: Fixed » Closed (fixed)

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

thomas molck’s picture

Category: feature » bug
Status: Closed (fixed) » Active

hello, reopened the issue as bug report. the permission 'deny configuration on node form' hide the 'nodes in block' fieldset in the 'edit node form'. but if i try to submit a change on a node, i get a error message "Visibility field is required.". in this way, the permission blocks any node edit permission.

thomas molck’s picture

Version: 6.x-1.6 » 6.x-1.x-dev
swentel’s picture

StatusFileSize
new1009 bytes

Wow, and I was about to throw a new release today :) Patch attached and committed to CVS. Thanks for the report!

swentel’s picture

Status: Active » Fixed
thomas molck’s picture

Status: Fixed » Patch (to be ported)

thanks for the quick patch. unfortunately, it dont prevents the message "Visibility field is required."
for me, this change in line 285 of nodesinblock.module works:

- if ($node->nodesinblock_delta != -1 && empty($visibility)) {
+ if ($node->nodesinblock_delta != -1 && empty($visibility) && !user_access('deny configuration on node form')) {
thomas molck’s picture

i just realize, that my suggestion deactivate visibility validation for all users (incl. user 1), if 'deny configuration on node form' is set. anyway, it helps me. just want to give the hint to others who may read my comment.

swentel’s picture

Status: Patch (to be ported) » Closed (fixed)

Should now be fixed in D6 branch.