Only write repr as str will invoke it implicitly

jsmnbom · jsmnbom · commit 6cc84b2c3247 · 2017-06-20T21:41:23.000+02:00
diff --git a/telegram/ext/filters.py b/telegram/ext/filters.py
@@ -68,15 +68,13 @@ def __or__(self, other):
     def __invert__(self):
         return InvertedFilter(self)
 
-    def __str__(self):
+    def __repr__(self):
         # Do not rely on classes overwriting __init__ to set a name
         # so we can keep backwards compatibility
         if not hasattr(self, 'name') or self.name is None:
             self.name = self.__class__.__name__
         return self.name
 
-    __repr__ = __str__
-
     def filter(self, message):
         raise NotImplementedError
 
@@ -95,11 +93,9 @@ def __init__(self, f):
     def filter(self, message):
         return not self.f(message)
 
-    def __str__(self):
+    def __repr__(self):
         return "<inverted {}>".format(self.f)
 
-    __repr__ = __str__
-
 
 class MergedFilter(BaseFilter):
     """Represents a filter consisting of two other filters.
@@ -122,12 +118,10 @@ def filter(self, message):
         elif self.or_filter:
             return self.base_filter(message) or self.or_filter(message)
 
-    def __str__(self):
+    def __repr__(self):
         return "<{} {} {}>".format(self.base_filter, "and" if self.and_filter else "or",
                                    self.and_filter or self.or_filter)
 
-    __repr__ = __str__
-
 
 class Filters(object):
     """
