Merge pull request #26595 from meeseeksmachine/auto-backport-of-pr-26591-on-v3.8.x

QuLogic · web-flow · commit a7e3142c5b66 · 2023-08-24T18:58:55.000-04:00
Backport PR #26591 on branch v3.8.x (Fix ToolBase.figure property setter.)
diff --git a/lib/matplotlib/backend_tools.py b/lib/matplotlib/backend_tools.py
@@ -118,16 +118,15 @@ def __init__(self, toolmanager, name):
         lambda self: self._figure.canvas if self._figure is not None else None,
         doc="The canvas of the figure affected by this tool, or None.")
 
-    @property
-    def figure(self):
-        """The Figure affected by this tool, or None."""
-        return self._figure
-
-    @figure.setter
-    def figure(self, figure):
+    def set_figure(self, figure):
         self._figure = figure
 
-    set_figure = figure.fset
+    figure = property(
+        lambda self: self._figure,
+        # The setter must explicitly call self.set_figure so that subclasses can
+        # meaningfully override it.
+        lambda self, figure: self.set_figure(figure),
+        doc="The Figure affected by this tool, or None.")
 
     def _make_classic_style_pseudo_toolbar(self):
         """
