Commit Graph

11 Commits

Author SHA1 Message Date
Tim Almdal
427e1130b2 Modified the notification get_subscribers to ignore any users that don't
have "view" access to the item the notification is being generated for.
Fix for ticket: #538.
2009-07-22 11:11:48 -07:00
Bharat Mediratta
5b3b675b6d Non-trivial changes to the event handling code:
1) The item_updated event no longer takes the old and new items.
   Instead we overload ORM to track the original data and make
   that available via the item.  This will allow us to move event
   publishing down into the API methods which in turn will give us
   more stability since we won't require each controller to remember
   to do it.

2) ORM class now tracks the original values.  It doesn't track
   the original relationships (no need for that, yet)

3) Added new events:
     item_deleted
     group_deleted
     user_deleted
2009-07-16 11:19:34 -07:00
Chad Kieffer
4d3cb760de Move watch notification menu from the view menu to under the Options menu. 2009-06-05 01:14:09 -06:00
Chad Kieffer
bc38505251 Update notify/watch eyeglasses icon with bullhorn icon. Rename css/image names from watch to notify. 2009-06-04 11:55:18 -07:00
Bharat Mediratta
5495037a3d Gee it's May already. Update copyright to 2009. 2009-05-13 20:04:58 +00:00
Tim Almdal
9e01c80ef4 Enable csrf validation on the add/remove notification controller 2009-02-02 13:28:42 +00:00
Andy Staudacher
60d3d6e704 code audit fixes + bug fix: don't call sendmail with an empty To list (in notifications module) 2009-02-02 07:32:45 +00:00
Tim Almdal
01d00dfd86 Change the item menu to allow the watch icon to change depending on
whether the item is currently being watched.  Currently the icon is
the same as the add icon and needs to be changed.
2009-02-02 03:55:33 +00:00
Tim Almdal
0eb593711b We no longer need a dialog to set or remove a watch, so remove it. 2009-02-02 03:21:29 +00:00
Tim Almdal
6f35829173 Simplify the setting of a notifications. Notifications are not only
set on a album. The notifications are implicitly active for all child
elements.

It now sends emails if the email address of the subscribed user has
been set.  No email, no attempt to send the notification.

Still to do, come up with better messages as the current ones are just
place holders.
2009-02-01 03:58:31 +00:00
Tim Almdal
38cf6ca403 The start of the notification module. At this point, the icon has been added to the item menu in the sidebar (both photo and album). There is a corresponding icon in themes/default/images that needs to be spruced up. You can add and remove notifications to albums and photos, but nothing happens under the covers for event handling. 2009-01-30 20:52:25 +00:00