dates.YearLocator doesn't handle inverted axes

elpres · elpres · commit 9783e24f6bc8 · 2015-06-25T16:27:26.000+02:00
All other Locators are derived from RRuleLocator, which has the following check inside its `__call__` routine:

    try:
        dmin, dmax = self.viewlim_to_dt()
    except ValueError:
        return []

    if dmin &gt; dmax:
        dmax, dmin = dmin, dmax

YearLocator just gets values for `dmin` and `dmax` in the same manner as above and goes on using them without checking for ordering and swapping if necessary. This leads to only one tick on axes using YearLocator.
diff --git a/lib/matplotlib/dates.py b/lib/matplotlib/dates.py
@@ -698,10 +698,16 @@ def set_tzinfo(self, tz):
 
     def datalim_to_dt(self):
         dmin, dmax = self.axis.get_data_interval()
+        if dmin > dmax:
+            dmin, dmax = dmax, dmin
+
         return num2date(dmin, self.tz), num2date(dmax, self.tz)
 
     def viewlim_to_dt(self):
         vmin, vmax = self.axis.get_view_interval()
+        if vmin > vmax:
+            vmin, vmax = vmax, vmin
+
         return num2date(vmin, self.tz), num2date(vmax, self.tz)
 
     def _get_unit(self):
@@ -748,8 +754,6 @@ def __call__(self):
         return self.tick_values(dmin, dmax)
 
     def tick_values(self, vmin, vmax):
-        if vmin > vmax:
-            vmax, vmin = vmin, vmax
         delta = relativedelta(vmax, vmin)
 
         # We need to cap at the endpoints of valid datetime
@@ -820,9 +824,6 @@ def autoscale(self):
         Set the view limits to include the data range.
         """
         dmin, dmax = self.datalim_to_dt()
-        if dmin > dmax:
-            dmax, dmin = dmin, dmax
-
         delta = relativedelta(dmax, dmin)
 
         # We need to cap at the endpoints of valid datetime
