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 module interferes with Drupal 7 core's "Cancel own user account" permission.
When administerusersbyrole is enabled I get an access denied if I try to cancel my own account.
Comment | File | Size | Author |
---|---|---|---|
#4 | administerusersbyrole-cancel-own-account-1883856-4.patch | 821 bytes | skorzh |
Comments
Comment #1
smussbach CreditAttribution: smussbach commentedThis is a quite big one. No reactions? Pushing this to major as users should know this when planning to use this module.
Comment #2
skorzhMy patch should fix this bug.
Comment #4
skorzhTry to pass test
Comment #5
skorzhComment #6
skorzhany updates here?
Comment #7
skorzhStill no updates here?
Comment #8
Jason Dean CreditAttribution: Jason Dean commented#4 works great - thanks!
Comment #9
AdamPS CreditAttribution: AdamPS commentedI intend to fix this as part of #2378869: Meta-issue for Beta 2 release. Please sign up as a follower of that issue and see the patch there that I would like feedback on.
It's effectively a similar patch but I couldn't take your patch directly because there are so many issues to fix affecting the same lines of code.
Comment #10
skorzhNice to see what module will be maintained, but if you will use the code from my patch it would be pretty nice to see me as commit author somehow.
Comment #11
AdamPS CreditAttribution: AdamPS commented@korgik Sorry as I explained in the main Beta2 issue there will only be one commit for all the outstanding bugs (about 20 of them, all overlapping!) - it's the only way I can deal with the huge backlog in a practical amount of time. I'm afraid you will have to be satisfied with a warm glow inside for having done your bit to help:-).
It's not quite the same code as yours - basically I made a call to user_cancel_access (just 3 lines) which seems like the best solution as saves duplicating the code and is more future-proof.
What I really need now is people to test the new patch - I hope you can help out.
Comment #12
AdamPS CreditAttribution: AdamPS commentedFix now available in latest release