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
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

aaronbauman created an issue. See original summary.

AaronBauman’s picture

AaronBauman
he/him
CreditAttribution: AaronBauman at Message Agency commented
Version: 7.x-1.5 » 7.x-1.x-dev
Status: Active » Needs review
FileSize
registration-backwards_compatible_permissions-2561117-1.patch1.33 KB

Something like this

bkno’s picture

bkno CreditAttribution: 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
CreditAttribution: Kingdutch as a volunteer commented
FileSize
registration-backwards_compatible_permissions-2561117-4.patch1.58 KB
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 CreditAttribution: Den Tweed commented
FileSize
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 CreditAttribution: 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
CreditAttribution: gcb at ThinkShout commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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