diff options
| -rw-r--r-- | pyramid/config/adapters.py | 11 |
1 files changed, 11 insertions, 0 deletions
diff --git a/pyramid/config/adapters.py b/pyramid/config/adapters.py index 01438fd30..e9b0df614 100644 --- a/pyramid/config/adapters.py +++ b/pyramid/config/adapters.py @@ -70,6 +70,17 @@ class AdaptersConfiguratorMixin(object): if not predicates: return subscriber def subscriber_wrapper(*arg): + # We need to accept *arg and pass it along because zope + # subscribers are designed poorly. Notification will always call + # an associated subscriber with all of the objects involved in + # the subscription lookup, despite the fact that the event sender + # always has the option to attach those objects to the event + # object itself (and usually does). It would be much saner if the + # registry just used extra args passed to notify to do the lookup + # but only called event subscribers with the actual event object, + # or if we had been smart enough early on to always wrap + # subscribers in something that threw away the extra args, but + # c'est la vie. if all((predicate(*arg) for predicate in predicates)): return subscriber(*arg) if hasattr(subscriber, '__name__'): |