After updating to version 7.x-1.5, all my event registrations give "access denied" since none of the new permissions get assigned to any roles - even those who were previously allowed to register.

There should be a hook_update that assigns the new permissions as applicable in order to preserve backwards compatibility.

CommentFileSizeAuthor
#5 registration-backwards_compatible_permissions-2561117-5.patch1.6 KBden tweed
#4 interdiff.txt721 byteskingdutch
#4 registration-backwards_compatible_permissions-2561117-4.patch1.58 KBkingdutch
#2 registration-backwards_compatible_permissions-2561117-1.patch1.33 KBaaronbauman

Comments

aaronbauman created an issue. See original summary.

aaronbauman’s picture

aaronbauman
he/him
commented
Version: 7.x-1.5 » 7.x-1.x-dev
Status: Active » Needs review
StatusFileSize
new registration-backwards_compatible_permissions-2561117-1.patch1.33 KB

Something like this

bkno’s picture

bkno commented

Realised today that registrations had not been working for a few days as a result of this...

For anyone else wondering why their registration button isn't appearing, add the new "Create new registration" and "Register self" permissions to your user role(s).

kingdutch’s picture

kingdutch
he/him
Primary language English
commented
StatusFileSize
new registration-backwards_compatible_permissions-2561117-4.patch1.58 KB
new interdiff.txt721 bytes

The patch in #2 will fail if the user already fixed the permissions themselves (Which is what I had done before trying this patch).

The attached patch fixes this.

den tweed’s picture

den tweed commented
StatusFileSize
new registration-backwards_compatible_permissions-2561117-5.patch1.6 KB

Patch in #4 was failing against newest codebase, also removed the unused registration_permission_list call

chris matthews’s picture

chris matthews commented
Status: Needs review » Reviewed & tested by the community

The patch in #5 is 3 years old, but still applied cleanly to 7.x-1.x-dev. Although this patch is probably not needed anymore, it would preserve backwards compatibility so changing the status to RTBC.

  • gcb committed c6e60bd on 7.x-1.x authored by TwEeD 
    Issue #2561117 by Kingdutch, aaronbauman, TwEeD, bkno, Chris Matthews: "...
gcb’s picture

gcb
he/him
Location Portland, OR
commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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