Fixed default value of Rect.rx/ry if only one is specified

CHANGELOG.rst

@@ -9,6 +9,7 @@ Unreleased
 - Support mmult import from reportlab < 3.5.61 (#316).
 - ``<symbol>`` nodes now only appears in documents when they are
   referenced, not on their initial appearance.
+- If only rx or ry is defined on a ``<rect>``, the value is used for both.
 
 1.2.1 (2022-01-27)
 ------------------

svglib/svglib.py

@@ -983,6 +983,10 @@ def convertRect(self, node):
             rx = width / 2
         if ry > (height / 2):
             ry = height / 2
+        if rx and not ry:
+            ry = rx
+        elif ry and not rx:
+            rx = ry
         return Rect(x, y, width, height, rx=rx, ry=ry)
 
     def convertCircle(self, node):

tests/test_basic.py

@@ -710,6 +710,20 @@ def test_rx_ry(self):
         assert rect.rx == 5
         assert rect.ry == 1.5
 
+    def test_rx_ry_lonely(self):
+        """If only rx or ry is present, the other default to the same value."""
+        converter = svglib.Svg2RlgShapeConverter(None)
+        node = svglib.NodeTracker(etree.XML(
+            '<rect rx="2.5" width="10" height="3" x="0" y="0"/>'
+        ))
+        rect = converter.convertRect(node)
+        assert rect.ry == 2.5
+        node = svglib.NodeTracker(etree.XML(
+            '<rect ry="1" width="10" height="3" x="0" y="0"/>'
+        ))
+        rect = converter.convertRect(node)
+        assert rect.rx == 1
+
     def test_strokewidth_0(self):
         """
         When strokeWidth = 0, we force strokeColor to None to avoid the stroke