Closed (fixed)
Project:
Notifications
Version:
6.x-2.0
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
6 May 2009 at 11:53 UTC
Updated:
25 Jun 2009 at 21:12 UTC
Even if a user has permissions configured to allow him or her to manage his or her own subscriptions, selecting a checkbox in the 'Subscriptions' section of the 'Notifications' tab (under My account) and then trying to perform the delete action results in an error message about not having the required permissions. Here is a patch to fix this.
I changed the === to == because db_result will not necessarily return something of type integer. The only time this should be a problem is when there is an error, so that FALSE is returned by db_query. Then == will only match when $sids is empty, which should not be a problem.
| Comment | File | Size | Author |
|---|---|---|---|
| notifications_manage_subs.patch | 637 bytes | ozzin |
Comments
Comment #1
jose reyero commentedThanks for the patch.
Comment #3
sly5 commentedjust wanted xpress my feeling. Thanks 4 the patch ozzin, great job!