"Add" overwrites presets (inkstitch#3896)

kaalleen · web-flow · commit ab7ff46b9117 · 2025-08-03T10:01:48.000+02:00
* style
* param presets: to not overwrite on add when name exists
diff --git a/lib/extensions/lettering_svg_font_to_layers.py b/lib/extensions/lettering_svg_font_to_layers.py
@@ -28,8 +28,6 @@
 
 from ..svg import PIXELS_PER_MM
 
-import unicodedata
-
 
 class LetteringSvgFontToLayers(InkstitchExtension):
     """
@@ -162,7 +160,7 @@ def set_guide_lines(self, font, scale_by) -> float:
 
         return baseline
 
-    def convert_glyph_to_layer(self, glyph, emsize, baseline, scale_by, hide_layer):
+    def convert_glyph_to_layer(self, glyph, emsize, baseline, scale_by, hide_layer):  # noqa C901
         """
         Converts a single glyph into a new SVG layer, applying scaling and coordinate transformation.
         """
@@ -198,7 +196,7 @@ def convert_glyph_to_layer(self, glyph, emsize, baseline, scale_by, hide_layer):
 
         unicode_char = glyph.get("unicode")
         if unicode_char is None:
-           
+
             glyph_name = glyph.get("glyph-name").split('.')
             if unicode_char is None:
                 if len(glyph_name) == 2:
diff --git a/lib/gui/presets.py b/lib/gui/presets.py
@@ -137,6 +137,7 @@ def add_preset(self, event, overwrite=False):
 
         if not overwrite and preset_name in self._load_presets():
             info_dialog(self, _('Preset "%s" already exists.  Please use another name or press "Overwrite"') % preset_name, caption=_('Preset'))
+            return
 
         self.store_preset(preset_name, self.parent.get_preset_data())
 
diff --git a/lib/lettering/font_info.py b/lib/lettering/font_info.py
@@ -40,11 +40,11 @@ def horiz_adv_x(self):
                 hax = float(hax)
 
             glyph_name = glyph.get('glyph-name', None)
-            
+
             if glyph_name is not None:
                 glyph_name = glyph_name.split('.')
                 if len(glyph_name) == 2:
-                    if ord(unicode_char[0]) >= 57344 :
+                    if ord(unicode_char[0]) >= 57344:
                         unicode_char = glyph_name[0]
                     typographic_feature = glyph_name[1]
                     unicode_char += f'.{typographic_feature}'
