Hi,
I wonder if the following problem is possible to come around: In my setup with restricted access and taxonomy control, it seems that subscriptions module will simply not send any email to users, unless the original user that has posted an item also has the privilege to administer nodes in the node module. Granting this to all users that might contribute feels a bit unsafe. Has it something to do with some look-up of all nodes the subscriptions module has to perform when deciding what is new? I use Drupal 4.7.6, subscriptions 4.7.1.4.
Greetings
mk
| Comment | File | Size | Author |
|---|---|---|---|
| #8 | subscriptions.module.20070616.doc | 171.5 KB | rseiser |
Comments
Comment #1
milkes commented..so I switched to notify, somewhat simpler alternative that works with taxonomy access
Comment #2
roginald commentedI was having this same problem, I found that switching to cron based notifications actually fixed the issue. Hope this helps.
Comment #3
amanire commentedHow did you ever figure this out? It has been driving me nuts!
Comment #4
mike211175 commentedI have a similar problem but it is even worse: for me, not only does the person posting have to have "adminster nodes privileges", but even then, notices are only sent to users who have "adminster nodes privileges", which is obviously not a realistic option. Any help on this will be much appreciated.
Comment #5
salvissubscribing
Comment #6
Leeteq commentedsubscribing
Comment #7
rseiser commentedI have the same problem. It occurs in 4.7 and 5.1. I stepped through with the debugger, and found that the "administer nodes" privilege is one of the mechanisms that allows a notification to be sent. Hence, it gets sent when the administrator posts a forum topic.
A second mechanism to allow the notification occurs when the node access is tested. This one fails for any user, since the the node hasn't been created at this point, and the proper privileges haven't been added to the node_access table yet. Hence the "cron based" notification might work, since this might allow the node to be created first. This last part I have to verify. (Comment notification works by the way because here the node (top post) already exists.
Other than that it seems that there needs to be an additional check if the subscription user has access in general to a new node, or something like that. Or maybe first create the node (entry in node_access table), and then do the notification.
If someone has a proper solution please let me know.
Comment #8
rseiser commentedI now verified that using cron for sending notifications in general works. Therefore, I chose this setting.
I then found, however, that the cron comment-notification does not do all the checks the non-cron comment-notification does. So I had to add a few lines in the cron portion of the notification processing.
I also corrected a few things that I favor personally:
-If a comment is posted, it sends a teaser of the comment (if the user had chosen this setting). Before it sent a teaser of the node (top post), or it sent the full comment (depending on subscription type).
-Since the comment wasn't available for certain subscription types, I added the the comment_id as an additional parameter
I attached a MS Word file that should show the changes I made using the "compare" function. Maybe someone can make use of it, or let me know if I introduced any further errors.
This all does not solve the "non-cron" setting. Here still someone would have to see
-what can be done that the node_access-table entry doesn't exist at the time of notification.
-why the user who changed the node is checked for "administer nodes". Maybe this is not necessary.
Comment #9
justified2000 commentedsubscribing
Comment #10
rseiser commentedI tried to give the subscriptions module a higher weight (using the moduleweight module) to try if it gets executed after the node_access is created. No luck, the non-cron based subscription mode still didn't send out notifications when a node was created (only when updated or when a comment was created/updated). So I am still back to using the cron based notification. Does anyone have a similar problem?
Comment #11
gollyg commentedsubscribe
Comment #12
felixsmile commentedI think I have the same problem on Drupal 5.1 with version 1.9...
Comment #13
pukku commentedSame problem, same reason (I presume).
Comment #14
bdragon commented5.x-2.0 is a complete rewrite, therefore any bugs in it are new. All prior releases will be obsolete when 5.x-2.0 final is out. Closing references to bugs in prior releases for this reason. Please help test 5.x-2.0 betas. Thank you!