Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
This sounds like a common use case, especially since I think the setting "group manager full access" should usually be turned off.
Comment | File | Size | Author |
---|---|---|---|
#10 | og-default-role-member-1444516-7463892.patch | 2.51 KB | paolomainardi |
#2 | 1444516-role-grant-test-2.patch | 1.09 KB | RoySegall |
#1 | OG settings | Site-Install.jpg | 57.32 KB | amitaibu |
Comments
Comment #1
amitaibuCommitted.
Comment #2
RoySegall CreditAttribution: RoySegall commentedAttached test.
Comment #3
amitaibuCommitted test, thanks.
Comment #4
ohthehugemanatee CreditAttribution: ohthehugemanatee commentedThanks for the patch! It looks like it tries to detect which role to apply. It would be simpler to just add a form element to admin/config/group/settings to "apply a default role to the Group creator", with a pulldown to select which role you want to apply.
Comment #5
amitaibu> It would be simpler to just add a form element to admin/config/group/settings to "apply a default role to the Group creator", with a pulldown to select which role you want to apply.
It's doing exactly that.
Comment #6
ohthehugemanatee CreditAttribution: ohthehugemanatee commentedWell I feel foolish - I'd applied it against an earlier version of dev. Thanks! :)
Comment #8
paolomainardi CreditAttribution: paolomainardi commentedThis patch seems to be broken, i'm working on og "version = "7.x-2.2".
How to reproduce:
1) Group type: "Community"
2) Set as default role "administration member"
This what is saved in "og_group_manager_default_rids_node_community" variable:
RID 3 is the default administration member role created by og.
Lastly, what happens at insert/update of this og entity content type:
And when it fails:
As you can see we are going to check for "rid == 3" but new groups has new roles with new rid (in this case: rid=165) and it fails each time.
I'm going to write a patch.
Comment #9
paolomainardi CreditAttribution: paolomainardi commentedSeems that your patch does not take in charge groups with overridden rules.
Comment #10
paolomainardi CreditAttribution: paolomainardi commentedPatch attached.
Comment #12
amitaibuPlease open a new issue.
Comment #13
paolomainardi CreditAttribution: paolomainardi commentedAmitaibu, ok, but i don't understand why to open a new issue when this patch is strictly related to this one.