FIX: back to isinstance

jklymak · jklymak · commit a6651fee85f8 · 2018-03-13T19:19:20.000-07:00
diff --git a/lib/matplotlib/axis.py b/lib/matplotlib/axis.py
@@ -1568,7 +1568,7 @@ def set_major_formatter(self, formatter):
 
         ACCEPTS: A :class:`~matplotlib.ticker.Formatter` instance
         """
-        if not hasattr(formatter, 'format_data'):
+        if not isinstance(formatter, mticker.Formatter):
             raise TypeError("formatter argument should be instance of "
                     "matplotlib.ticker.Formatter")
         self.isDefault_majfmt = False
@@ -1582,7 +1582,7 @@ def set_minor_formatter(self, formatter):
 
         ACCEPTS: A :class:`~matplotlib.ticker.Formatter` instance
         """
-        if not hasattr(formatter, 'format_data'):
+        if not isinstance(formatter, mticker.Formatter):
             raise TypeError("formatter argument should be instance of "
                     "matplotlib.ticker.Formatter")
         self.isDefault_minfmt = False
@@ -1596,7 +1596,7 @@ def set_major_locator(self, locator):
 
         ACCEPTS: a :class:`~matplotlib.ticker.Locator` instance
         """
-        if not hasattr(locator, 'tick_values'):
+        if not isinstance(locator, mticker.Locator):
             raise TypeError("formatter argument should be instance of "
                     "matplotlib.ticker.Locator")
         self.isDefault_majloc = False
@@ -1610,7 +1610,7 @@ def set_minor_locator(self, locator):
 
         ACCEPTS: a :class:`~matplotlib.ticker.Locator` instance
         """
-        if not hasattr(locator, 'tick_values'):
+        if not isinstance(locator, mticker.Locator):
             raise TypeError("formatter argument should be instance of "
                     "matplotlib.ticker.Locator")
         self.isDefault_minloc = False
