only send notifications if a user has access to the node
greggles - June 13, 2009 - 00:49
| Project: | Comment Notify |
| Version: | 5.x-2.x-dev |
| Component: | Code |
| Category: | task |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | patch (to be ported) |
Jump to:
Description
Originally by aclight.
I'd love a review on this before committing. It also needs a 5.x port, but that shouldn't be bad.
| Attachment | Size |
|---|---|
| comment_notify_6x_private.patch | 4.67 KB |
#1
@greggles: if this is still open I'll take a look soon... btw (before digging in), does this cover the case where users can create subscriptions to content types they don't have access to? Anyway, i'll check this out - but curious about this case
#2
oops - greggles - scratch my comment above (it's way too early in the morning - I was actually looking for this case in the notifications module... I'm using both on the same site :p )
#3
Now committed - http://drupal.org/cvs?commit=277504