Add new convention for user agnostic hooks.

Author: GVRV

README.md

@@ -75,7 +75,7 @@ INSTALLED_APPS = (
 HOOK_EVENTS = {
     # 'any.event.name': 'App.Model.Action' (created/updated/deleted)
     'book.added':       'bookstore.Book.created',
-    'book.changed':     'bookstore.Book.updated',
+    'book.changed':     'bookstore.Book.updated+',
     'book.removed':     'bookstore.Book.deleted',
     # and custom events, no extra meta data needed
     'book.read':         'bookstore.Book.read',
@@ -87,6 +87,9 @@ HOOK_EVENTS = {
 class Book(models.Model):
     # NOTE: it is important to have a user property
     # as we use it to help find and trigger each Hook
+    # which is specific to users. If you want a Hook to
+    # be triggered for all users, add '+' to built-in Hooks
+    # or pass user_override=False for custom_hook events
     user = models.ForeignKey('auth.User')
     # maybe user is off a related object, so try...
     # user = property(lambda self: self.intermediary.user)

rest_hooks/utils.py

@@ -45,6 +45,10 @@ def find_and_fire_hook(event_name, instance, user_override=None):
             filters['user'] = instance.user
         elif isinstance(instance, User):
             filters['user'] = instance
+        else:
+            raise Exception(
+                '{} has no `user` property. REST Hooks needs this.'.format(repr(instance))
+            )
 
     # NOTE: This is probably up for discussion, but I think, in this
     # case, instead of raising an error, we should fire the hook for
@@ -71,8 +75,11 @@ def distill_model_event(instance, model, action, user_override=None):
         if auto:
             # break auto into App.Model, Action
             maybe_model, maybe_action = auto.rsplit('.', 1)
-            if model == maybe_model and action == maybe_action:
+            maybe_action = maybe_action.rsplit('+', 1)
+            if model == maybe_model and action == maybe_action[0]:
                 event_name = maybe_event_name
+                if len(maybe_action) == 2:
+                    user_override = True
 
     if event_name:
         find_and_fire_hook(event_name, instance, user_override=user_override)