Users with "Maintain own subscriptions" permissions can not delete nodes via the url:

notifications/subscription/%notifications_subscription/delete

Attached patch for it.

CommentFileSizeAuthor
0001-fix-delete-tab-perms.patch805 bytesthat0n3guy
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

webflo’s picture

webflo CreditAttribution: webflo commented

Uhm. Whats the difference between "unsubscribe" and "delete"?

that0n3guy’s picture

that0n3guy CreditAttribution: that0n3guy commented

I don't really know... is there a difference? If not why are both in code?

All I know is that at the time the delete button on the edit page didnt work (see issue #1307594) but the delete tab did. So I just gave the user perms using the above patch.

BarisW’s picture

BarisW CreditAttribution: BarisW commented
Status: Needs review » Reviewed & tested by the community

Patch works, thanks for sharing.

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented
Issue summary: View changes
Parent issue: » #2507375: Plan for Notifications v7.x-1.0 release

  • Nafes committed 86459a4 on 7.x-1.x authored by that0n3guy 
    Issue #1307582 by that0n3guy: Access issues for deleting subscriptions
Nafes’s picture

Nafes CreditAttribution: Nafes as a volunteer commented
Status: Reviewed & tested by the community » Fixed

Thank you for the patch, that0n3guy!

Status: Fixed » Closed (fixed)

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