Make handlers test whether filters are callable and not if they exist

delivrance · delivrance · commit 4e293f23a92d · 2018-09-14T15:28:08.000+02:00
diff --git a/pyrogram/client/handlers/callback_query_handler.py b/pyrogram/client/handlers/callback_query_handler.py
@@ -49,6 +49,6 @@ def __init__(self, callback: callable, filters=None):
     def check(self, callback_query):
         return (
             self.filters(callback_query)
-            if self.filters
+            if callable(self.filters)
             else True
         )
diff --git a/pyrogram/client/handlers/deleted_messages_handler.py b/pyrogram/client/handlers/deleted_messages_handler.py
@@ -50,6 +50,6 @@ def __init__(self, callback: callable, filters=None):
     def check(self, messages):
         return (
             self.filters(messages.messages[0])
-            if self.filters
+            if callable(self.filters)
             else True
         )
diff --git a/pyrogram/client/handlers/message_handler.py b/pyrogram/client/handlers/message_handler.py
@@ -50,6 +50,6 @@ def __init__(self, callback: callable, filters=None):
     def check(self, message):
         return (
             self.filters(message)
-            if self.filters
+            if callable(self.filters)
             else True
         )

Original file line number	Diff line number	Diff line change
`@@ -49,6 +49,6 @@ def __init__(self, callback: callable, filters=None):`
`49`	`49`	`def check(self, callback_query):`
`50`	`50`	`return (`
`51`	`51`	`self.filters(callback_query)`
`52`		`- if self.filters`
	`52`	`+ if callable(self.filters)`
`53`	`53`	`else True`
`54`	`54`	`)`
Original file line number	Diff line number	Diff line change
`@@ -50,6 +50,6 @@ def __init__(self, callback: callable, filters=None):`
`50`	`50`	`def check(self, messages):`
`51`	`51`	`return (`
`52`	`52`	`self.filters(messages.messages[0])`
`53`		`- if self.filters`
	`53`	`+ if callable(self.filters)`
`54`	`54`	`else True`
`55`	`55`	`)`