The 2026 Drupal Business Survey is open. Your response will help shape the future of the Drupal business ecosystem.Hi !
when a new role is added, all existing nodes cannot have permissions saved for this role.
For example : create a node N, set some permission on it. Add a new role. Edit the node, and assign "edit" and "view" to the new role. Save it and re-edit it : the new permissions have not been saved...
This patch corrects this (patch against 5.x-1.x-dev)
Alex :)
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | patch_node_privacy_byrole_new_role_1.txt | 3.25 KB | alex_ost |
| #2 | patch_node_privacy_byrole_new_role_0.txt | 2.86 KB | alex_ost |
| patch_node_privacy_byrole_new_role.txt | 1.83 KB | alex_ost |
Comments
Comment #1
jondoesdrupal commentedGreat stuff, thanks for the patch. I'll apply it and get it commited when I get home tonight.
Cheers,
Jon
Comment #2
alex_ost commentedWell, glad that you didn't commit it, it's not quite complete :)
Here is another patch which includes the preceding one and adds some more : when editing a node (with a new role added), it fills up permissions for the new role with default permissions, instead of just denying everything.
BTW : node_privacy_byrole is a really great module, seems to me mandatory for any drupal site. Why isn't it included in core ???
Alex :)
Comment #3
alex_ost commentedSorry, wrong version... :/
Comment #4
jondoesdrupal commentedI feel the same way, I only came across this module a month or two ago and it's the exact simple access control I've always wanted. There hasn't been much development activity on it though which is no doubt why it's not had bigger uptake but hopefully now we'll get a bug free 5.x release and that'll raise it's profile.
I've not had a play with 6.x yet, but it'd probably be good to try and produce a release for that too.
Anyway, thanks for the second patch, I'll get it in place tonight.
Cheers,
Jon
Comment #5
jondoesdrupal commentedThanks Alex, patch has been committed and should be published in the dev release later on today.
Comment #6
jondoesdrupal commentedOnly committed this to head, forgot the 5-x branch, so will fix this tonight.