Fix Filters.regex failing on non-text message (python-telegram-bot#1158)

Dmitry Grigoryev · jsmnbom · commit 8acff56145fc · 2018-08-26T20:39:01.000+02:00
* Fix python-telegram-bot#1115 * Improve regex filter test
diff --git a/telegram/ext/filters.py b/telegram/ext/filters.py
@@ -198,7 +198,9 @@ def __init__(self, pattern):
         # the matched groups and groupdict to the context object.
 
         def filter(self, message):
-            return bool(self.pattern.search(message.text))
+            if message.text:
+                return bool(self.pattern.search(message.text))
+            return False
 
     class _Reply(BaseFilter):
         name = 'Filters.reply'
diff --git a/tests/test_filters.py b/tests/test_filters.py
@@ -68,6 +68,9 @@ def test_filters_regex(self, message):
         message.text = 'i love python'
         assert Filters.regex(r'.\b[lo]{2}ve python')(message)
 
+        message.text = None
+        assert not Filters.regex(r'fail')(message)
+
     def test_filters_reply(self, message):
         another_message = Message(1, User(1, 'TestOther', False), datetime.datetime.now(),
                                   Chat(0, 'private'))
