Convert units when returning Rect bbox

dstansby · dstansby · commit 9c13491c45d6 · 2017-08-25T09:14:24.000+01:00
diff --git a/lib/matplotlib/patches.py b/lib/matplotlib/patches.py
@@ -713,10 +713,7 @@ def _update_patch_transform(self):
                  maxes it very important to call the accessor method and
                  not directly access the transformation member variable.
         """
-        x0 = self.convert_xunits(self._x0)
-        y0 = self.convert_yunits(self._y0)
-        x1 = self.convert_xunits(self._x1)
-        y1 = self.convert_yunits(self._y1)
+        x0, y0, x1, y1 = self._convert_units()
         bbox = transforms.Bbox.from_extents(x0, y0, x1, y1)
         rot_trans = transforms.Affine2D()
         rot_trans.rotate_deg_around(x0, y0, self.angle)
@@ -729,6 +726,16 @@ def _update_x1(self):
     def _update_y1(self):
         self._y1 = self._y0 + self._height
 
+    def _convert_units(self):
+        '''
+        Convert bounds of the rectangle
+        '''
+        x0 = self.convert_xunits(self._x0)
+        y0 = self.convert_yunits(self._y0)
+        x1 = self.convert_xunits(self._x1)
+        y1 = self.convert_yunits(self._y1)
+        return x0, y0, x1, y1
+
     def get_patch_transform(self):
         self._update_patch_transform()
         return self._rect_transform
@@ -746,7 +753,7 @@ def get_xy(self):
         return self._x0, self._y0
 
     def get_width(self):
-        "Return the width of the  rectangle"
+        "Return the width of the rectangle"
         return self._width
 
     def get_height(self):
@@ -823,6 +830,7 @@ def set_bounds(self, *args):
         self.stale = True
 
     def get_bbox(self):
+        x0, y0, x1, y1 = self._convert_units()
         return transforms.Bbox.from_extents(self._x0, self._y0,
                                             self._x1, self._y1)
 
