diff options
| author | Kaalleen <36401965+kaalleen@users.noreply.github.com> | 2023-03-07 17:52:09 +0100 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2023-03-07 17:52:09 +0100 |
| commit | 9552291de5c7f1236542e9be3f1816e29701b7ed (patch) | |
| tree | 69f844be2a74df9cf20737f98186466e4dadfb69 | |
| parent | 90d3fcf69c7ccd16928aa26c6fa288aafcb2e05d (diff) | |
change method to update legacy fill_method (#2111)
| -rw-r--r-- | lib/elements/element.py | 9 | ||||
| -rw-r--r-- | lib/elements/fill_stitch.py | 15 | ||||
| -rw-r--r-- | lib/utils/param.py | 7 |
3 files changed, 16 insertions, 15 deletions
diff --git a/lib/elements/element.py b/lib/elements/element.py index 759574d3..d3b9f07d 100644 --- a/lib/elements/element.py +++ b/lib/elements/element.py @@ -72,6 +72,15 @@ class EmbroideryElement(object): elif legacy_tie == "False": self.set_param('ties', 3) + # convert legacy fill_method + legacy_fill_method = self.get_int_param('fill_method', None) + if legacy_fill_method == 1: + self.set_param('fill_method', 'contour_fill') + elif legacy_fill_method == 2: + self.set_param('fill_method', 'guided_fill') + elif legacy_fill_method == 3: + self.set_param('fill_method', 'legacy_fill') + # default setting for fill_underlay has changed if legacy_attribs and not self.get_param('fill_underlay', ""): self.set_param('fill_underlay', False) diff --git a/lib/elements/fill_stitch.py b/lib/elements/fill_stitch.py index c49f0b94..00a88d54 100644 --- a/lib/elements/fill_stitch.py +++ b/lib/elements/fill_stitch.py @@ -110,12 +110,12 @@ class FillStitch(EmbroideryElement): def auto_fill(self): return self.get_boolean_param('auto_fill', True) - _fill_methods = [ParamOption('auto_fill', _("Auto Fill"), 0), - ParamOption('contour_fill', _("Contour Fill"), 1), - ParamOption('guided_fill', _("Guided Fill"), 2), + _fill_methods = [ParamOption('auto_fill', _("Auto Fill")), + ParamOption('contour_fill', _("Contour Fill")), + ParamOption('guided_fill', _("Guided Fill")), ParamOption('meander_fill', _("Meander Fill")), ParamOption('circular_fill', _("Circular Fill")), - ParamOption('legacy_fill', _("Legacy Fill"), 3)] + ParamOption('legacy_fill', _("Legacy Fill"))] @property @param('fill_method', @@ -125,13 +125,6 @@ class FillStitch(EmbroideryElement): options=_fill_methods, sort_index=2) def fill_method(self): - # convert legacy values - legacy_method = self.get_int_param('fill_method', None) - if legacy_method in range(0, 4): - method = [method.id for method in self._fill_methods if method.legacy == legacy_method][0] - self.set_param('fill_method', method) - return method - return self.get_param('fill_method', 'auto_fill') @property diff --git a/lib/utils/param.py b/lib/utils/param.py index 19a369e1..162dcddc 100644 --- a/lib/utils/param.py +++ b/lib/utils/param.py @@ -1,9 +1,8 @@ class ParamOption: - def __init__(self, param_id=None, name=None, legacy=None, preview_image=None): + def __init__(self, param_id=None, name=None, preview_image=None): self.id: str = param_id self.name: str = name - self.legacy: int = legacy - self.preview_image: str = None + self.preview_image: str = preview_image def __repr__(self): - return "ParamOption(%s, %s, %s, %s)" % (self.id, self.name, self.legacy, self.preview_image) + return "ParamOption(%s, %s, %s)" % (self.id, self.name, self.preview_image) |