Skip to content

Commit 4ee3a11

Browse files
GVRVgvangool
authored andcommitted
Ignore 'user' for the hook if user_override=False.
1 parent fb7eb69 commit 4ee3a11

File tree

1 file changed

+8
-6
lines changed

1 file changed

+8
-6
lines changed

rest_hooks/utils.py

Lines changed: 8 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -37,12 +37,14 @@ def find_and_fire_hook(event_name, instance, user_override=None):
3737

3838
filters = {'event': event_name}
3939

40-
if user_override:
41-
filters['user'] = user_override
42-
elif hasattr(instance, 'user'):
43-
filters['user'] = instance.user
44-
elif isinstance(instance, User):
45-
filters['user'] = instance
40+
# Ignore the user if the user_override is False
41+
if user_override is not False:
42+
if user_override:
43+
filters['user'] = user_override
44+
elif hasattr(instance, 'user'):
45+
filters['user'] = instance.user
46+
elif isinstance(instance, User):
47+
filters['user'] = instance
4648

4749
# NOTE: This is probably up for discussion, but I think, in this
4850
# case, instead of raising an error, we should fire the hook for

0 commit comments

Comments
 (0)