Fix default return of Collection.get_{cap,join}style

QuLogic · QuLogic · commit 2f4f08eff4a3 · 2023-05-03T19:58:00.000-04:00
If neither are specified at object creation, the default is to be
`None`. This broke `get_{cap,join}style` when the enum wrappers were
created as they assume the internal value is always an enum value.
diff --git a/lib/matplotlib/collections.py b/lib/matplotlib/collections.py
@@ -639,8 +639,16 @@ def set_capstyle(self, cs):
         """
         self._capstyle = CapStyle(cs)
 
+    @_docstring.interpd
     def get_capstyle(self):
-        return self._capstyle.name
+        """
+        Return the cap style for the collection (for all its elements).
+
+        Returns
+        -------
+        %(CapStyle)s or None
+        """
+        return self._capstyle.name if self._capstyle else None
 
     @_docstring.interpd
     def set_joinstyle(self, js):
@@ -653,8 +661,16 @@ def set_joinstyle(self, js):
         """
         self._joinstyle = JoinStyle(js)
 
+    @_docstring.interpd
     def get_joinstyle(self):
-        return self._joinstyle.name
+        """
+        Return the join style for the collection (for all its elements).
+
+        Returns
+        -------
+        %(JoinStyle)s or None
+        """
+        return self._joinstyle.name if self._joinstyle else None
 
     @staticmethod
     def _bcast_lwls(linewidths, dashes):
diff --git a/lib/matplotlib/collections.pyi b/lib/matplotlib/collections.pyi
@@ -52,9 +52,9 @@ class Collection(artist.Artist, cm.ScalarMappable):
     def set_linewidth(self, lw: float | Sequence[float]) -> None: ...
     def set_linestyle(self, ls: LineStyleType | Sequence[LineStyleType]) -> None: ...
     def set_capstyle(self, cs: CapStyleType) -> None: ...
-    def get_capstyle(self) -> Literal["butt", "projecting", "round"]: ...
+    def get_capstyle(self) -> Literal["butt", "projecting", "round"] | None: ...
     def set_joinstyle(self, js: JoinStyleType) -> None: ...
-    def get_joinstyle(self) -> Literal["miter", "round", "bevel"]: ...
+    def get_joinstyle(self) -> Literal["miter", "round", "bevel"] | None: ...
     def set_antialiased(self, aa: bool | Sequence[bool]) -> None: ...
     def set_color(self, c: ColorType | Sequence[ColorType]) -> None: ...
     def set_facecolor(self, c: ColorType | Sequence[ColorType]) -> None: ...
diff --git a/lib/matplotlib/tests/test_collections.py b/lib/matplotlib/tests/test_collections.py
@@ -620,6 +620,8 @@ def test_set_wrong_linestyle():
 
 @mpl.style.context('default')
 def test_capstyle():
+    col = mcollections.PathCollection([])
+    assert col.get_capstyle() is None
     col = mcollections.PathCollection([], capstyle='round')
     assert col.get_capstyle() == 'round'
     col.set_capstyle('butt')
@@ -628,6 +630,8 @@ def test_capstyle():
 
 @mpl.style.context('default')
 def test_joinstyle():
+    col = mcollections.PathCollection([])
+    assert col.get_joinstyle() is None
     col = mcollections.PathCollection([], joinstyle='round')
     assert col.get_joinstyle() == 'round'
     col.set_joinstyle('miter')
