Fix renaming legacy name records 1 and 2.

Author: fabiocaccamo

fontbro/font.py

@@ -931,24 +931,19 @@ def rename(self, family_name="", style_name="", style_flags=True):
         wws_family_name = family_name
         wws_subfamily_name = style_name
 
-        # family name, subfamily name and full name
-        family_name_parts = [family_name]
-        style_name_parts = style_name.split()
-        subfamily_name_parts = style_name.lower().split()
+        # family name, subfamily name and full name - fix legacy name records 1 and 2
         subfamily_names = ["regular", "italic", "bold", "bold italic"]
-        subfamily_name_default = subfamily_names[0]
-        for name_part in style_name_parts:
-            if name_part.lower() not in subfamily_names:
-                family_name_parts.append(name_part)
-                subfamily_name_parts.remove(name_part.lower())
-        if subfamily_name_default in subfamily_name_parts:
-            if len(subfamily_name_parts) > 1:
-                subfamily_name_parts.remove(subfamily_name_default)
-        subfamily_name_parts = subfamily_name_parts or [subfamily_name_default]
-        family_name = " ".join(family_name_parts)
-        subfamily_name = " ".join(subfamily_name_parts).lower()
-        if subfamily_name not in subfamily_names:
-            subfamily_name = subfamily_name_default
+        subfamily_name = style_name
+        style_name_lower = style_name.lower()
+        if style_name_lower not in subfamily_names:
+            subfamily_name = (
+                "Italic" if style_name_lower.endswith(" italic") else "Regular"
+            )
+            family_name_suffix = re.sub(
+                r"\ italic$", "", subfamily_name, flags=re.IGNORECASE
+            )
+            assert family_name_suffix
+            family_name = f"{typographic_family_name} {family_name_suffix}"
 
         subfamily_name = subfamily_name.title()
 

tests/test_rename.py

@@ -61,7 +61,7 @@ def test_rename_with_family_name_only_issue_0062(self):
         self.assertEqual(
             font.get_names(),
             {
-                "family_name": "Hugo Test Thin",
+                "family_name": "Hugo Test Regular",
                 "full_name": "Hugo Test Thin",
                 "postscript_name": "HugoTest-Thin",
                 "subfamily_name": "Regular",
@@ -112,7 +112,7 @@ def test_rename_with_style_name_containing_characters_not_allowed_in_postscript_
         font = self._get_font("/Tourney/Tourney-VariableFont_wdth,wght.ttf")
         font.rename(" Tourney Custom ", " [wdth-wght] ")
         names = font.get_names()
-        self.assertEqual(names[Font.NAME_FAMILY_NAME], "Tourney Custom [wdth-wght]")
+        self.assertEqual(names[Font.NAME_FAMILY_NAME], "Tourney Custom Regular")
         self.assertEqual(names[Font.NAME_SUBFAMILY_NAME], "Regular")
         self.assertEqual(names[Font.NAME_FULL_NAME], "Tourney Custom [wdth-wght]")
         self.assertEqual(names[Font.NAME_POSTSCRIPT_NAME], "TourneyCustom-wdth-wght")