CHANGELOG.md

@@ -1,10 +1,3 @@
-## 1.2.0a54 (2026-01-14)
-
-### Fix
-
-- issymlinkoption is not available for an optiondescription
-- better error messages
-
 ## 1.2.0a53 (2026-01-05)
 
 ### Fix

pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "rougail"
-version = "1.2.0a54"
+version = "1.2.0a53"
 
 [tool.commitizen]
 name = "cz_conventional_commits"

rougail-base-pyproject.toml

@@ -4,7 +4,7 @@ requires = ["flit_core >=3.8.0,<4"]
 
 [project]
 name = "rougail-base"
-version = "1.2.0a54"
+version = "1.2.0a53"
 authors = [{name = "Emmanuel Garette", email = "gnunux@gnunux.info"}]
 readme = "README.md"
 description = "A consistency handling system that was initially designed in the configuration management"

rougail-pyproject.toml

@@ -4,7 +4,7 @@ requires = ["flit_core >=3.8.0,<4"]
 
 [project]
 name = "rougail"
-version = "1.2.0a54"
+version = "1.2.0a53"
 authors = [{name = "Emmanuel Garette", email = "gnunux@gnunux.info"}]
 description = "A consistency handling system that was initially designed in the configuration management"
 classifiers = [
@@ -18,7 +18,7 @@ classifiers = [
 dependencies = [
     "ruamel.yaml ~= 0.18.6",
     "pydantic ~= 2.9.2",
-    "rougail-base == 1.2.0a54",
+    "rougail-base == 1.2.0a53",
 ]
 
 [tool.flit.sdist]

src/rougail/__version__.py

@@ -1 +1 @@
-__version__ = "1.2.0a54"
+__version__ = "1.2.0a53"

src/rougail/user_data.py

@@ -56,12 +56,11 @@ class UserData:
         self.errors = []
         self.warnings = []
         self.invalid_user_data_error = invalid_user_data_error
-        self.unknown_user_data_error = unknown_user_data_error
         if self.invalid_user_data_error:
             self.invalids = self.errors
         else:
             self.invalids = self.warnings
-        if self.unknown_user_data_error:
+        if unknown_user_data_error:
             self.unknowns = self.errors
         else:
             self.unknowns = self.warnings
@@ -300,14 +299,7 @@ class UserData:
 
     def _display_value(self, option, value):
         if not self.show_secrets and option.type() == "password":
-            if not isinstance(value, list):
+            return "*" * 10
-                value = "*" * 10
-            else:
-                value = ["*" * 10 for val in value]
-        if isinstance(value, list):
-            value = display_list(value, add_quote=True)
-        else:
-            value = '"' + str(value) + '"'
         return value
 
     def _populate_error_warnings(self):
@@ -323,11 +315,11 @@ class UserData:
                     if value:
                         if self.invalid_user_data_error:
                             msg = _(
-                                    'it\'s a family so we cannot set the value {0}, it has been loading from {1}'
+                                    'is a family so we cannot set the value "{0}", it has been loading from {1}'
                                     )
                         else:
                             msg = _(
-                                    'it\'s a family so we cannot set the value {0}, it will be ignored when loading from {1}'
+                                    'is a family so we cannot set the value "{0}", it will be ignored when loading from {1}'
                                     )
                         self.invalids.append({msg.format(
                                 self._display_value(option, value),
@@ -335,14 +327,6 @@ class UserData:
                                 ): option._subconfig}
                         )
                     continue
-                if option.issymlinkoption():
-                    err = _('it\'s a symlink option so we cannot set the value {0}').format(self._display_value(option, value))
-                    if self.invalid_user_data_error:
-                        msg = _('{0}, it has been loading from {1}').format(err, options["source"])
-                    else:
-                        msg = _('{0}, it will be ignored when loading from {1}').format(err, options["source"])
-                    self.unknowns.append({msg: option._subconfig})
-                    continue
             except ConfigError as err:
                 self.invalids.append({
                     _("{0}, it has been loaded from {1}").format(err, options["source"]): option._subconfig}
@@ -355,7 +339,7 @@ class UserData:
                 continue
             except AttributeOptionError as err:
                 if err.code == "option-not-found":
-                    if self.unknown_user_data_error:
+                    if self.invalid_user_data_error:
                         msg = _(
                             'variable or family "{0}" does not exist, it has been loading from {1}'
                             )
@@ -408,7 +392,7 @@ class UserData:
                         display_name = option.description(with_quote=True)
                         if index is not None:
                             if path == err_path:
-                                if self.unknown_user_data_error:
+                                if self.invalid_user_data_error:
                                     msg = _(
                                         'variable {0} at index "{1}" is {2}, it has been loading from {3}'
                                     )
@@ -425,7 +409,7 @@ class UserData:
                                         ): option._subconfig}
                                 )
                             else:
-                                if self.unknown_user_data_error:
+                                if self.invalid_user_data_error:
                                     msg = _(
                                         'family {0} is {1}, {2} at index "{3}", it has been loading from {4}'
                                     )
@@ -444,7 +428,7 @@ class UserData:
                                 )
                         else:
                             if path == err_path:
-                                if self.unknown_user_data_error:
+                                if self.invalid_user_data_error:
                                     msg = _(
                                         "variable has propery {0}, it has been loading from {1}"
                                     )
@@ -458,7 +442,7 @@ class UserData:
                                         ): option._subconfig}
                                 )
                             else:
-                                if self.unknown_user_data_error:
+                                if self.invalid_user_data_error:
                                     msg = _(
                                         "family {0} has property {1}, so cannot access to {2}, it has been loading from {3}"
                                     )
@@ -475,29 +459,11 @@ class UserData:
                                         ): option._subconfig}
                                 )
                     else:
-                        if self.unknown_user_data_error:
-                            msg = _(
-                                "{0}, it has been loading from {1}"
-                            )
-                        else:
-                            msg = _(
-                                "{0}, it will be ignored when loading from {1}"
-                            )
                         self.unknowns.append({
-                            msg.format(err, options["source"]): option._subconfig}
+                            _("{0} in {1}").format(err, options["source"]): option._subconfig}
                         )
                 except LeadershipError as err:
-                    if self.unknown_user_data_error:
+                    self.unknowns.append({_("{0} in {1}").format(err, options["source"]): option._subconfig})
-                        msg = _(
-                            "{0}, it has been loading from {1}"
-                        )
-                    else:
-                        msg = _(
-                            "{0}, it will be ignored when loading from {1}"
-                        )
-                    self.unknowns.append({
-                        msg.format(err, options["source"]): option._subconfig}
-                    )
                 except ConfigError as err:
                     err.prefix = ""
                     if self.invalid_user_data_error:
@@ -508,7 +474,7 @@ class UserData:
                 except ValueError as err:
                     err.prefix = ""
                     type_ = option.type(translation=True)
-                    msg = _('the value {0} is an invalid {1}, {2}').format(
+                    msg = _('the value "{0}" is an invalid {1}, {2}').format(
                             self._display_value(option, value),
                             type_,
                             err,

tests/dictionaries/20_7help_family/makedict/after.json

@@ -1,10 +0,0 @@
-{
-    "rougail.family1.var": {
-        "owner": "default",
-        "value": null
-    },
-    "rougail.family2.var": {
-        "owner": "default",
-        "value": null
-    }
-}

tests/dictionaries/20_7help_family/makedict/base.json

@@ -1,4 +0,0 @@
-{
-    "rougail.family1.var": null,
-    "rougail.family2.var": null
-}

tests/dictionaries/20_7help_family/makedict/before.json

@@ -1,10 +0,0 @@
-{
-    "rougail.family1.var": {
-        "owner": "default",
-        "value": null
-    },
-    "rougail.family2.var": {
-        "owner": "default",
-        "value": null
-    }
-}

tests/dictionaries/20_7help_family/makedict/mandatory.json

@@ -1 +0,0 @@
-["rougail.family1.var", "rougail.family2.var"]

tests/dictionaries/20_7help_family/makedict/read_write.json

@@ -1,4 +0,0 @@
-{
-    "rougail.family1.var": null,
-    "rougail.family2.var": null
-}

tests/dictionaries/20_7help_family/tiramisu/base.py

@@ -1,18 +0,0 @@
-from tiramisu import *
-from tiramisu.setting import ALLOWED_LEADER_PROPERTIES
-from re import compile as re_compile
-from rougail.tiramisu import func, dict_env, load_functions, ConvertDynOptionDescription
-load_functions('../rougail-tests/funcs/test.py')
-try:
-    groups.namespace
-except:
-    groups.addgroup('namespace')
-ALLOWED_LEADER_PROPERTIES.add("basic")
-ALLOWED_LEADER_PROPERTIES.add("standard")
-ALLOWED_LEADER_PROPERTIES.add("advanced")
-option_3 = StrOption(name="var", doc="var", properties=frozenset({"basic", "mandatory"}), informations={'ymlfiles': ['../rougail-tests/structures/20_7help_family/rougail/00-base.yml'], 'type': 'string'})
-optiondescription_2 = OptionDescription(name="family1", doc="the first family", children=[option_3], properties=frozenset({"basic"}), informations={'ymlfiles': ['../rougail-tests/structures/20_7help_family/rougail/00-base.yml'], 'help': 'Multi line\n\nHelp\n\nWith useful information'})
-option_5 = StrOption(name="var", doc="var", properties=frozenset({"basic", "mandatory"}), informations={'ymlfiles': ['../rougail-tests/structures/20_7help_family/rougail/00-base.yml'], 'type': 'string'})
-optiondescription_4 = OptionDescription(name="family2", doc="the second family", children=[option_5], properties=frozenset({"basic"}), informations={'ymlfiles': ['../rougail-tests/structures/20_7help_family/rougail/00-base.yml'], 'help': 'Multi line\nHelp\nWith useful information'})
-optiondescription_1 = OptionDescription(name="rougail", doc="Rougail", group_type=groups.namespace, children=[optiondescription_2, optiondescription_4], properties=frozenset({"basic"}))
-option_0 = OptionDescription(name="baseoption", doc="baseoption", children=[optiondescription_1])

tests/dictionaries/20_7help_family/tiramisu/no_namespace.py

@@ -1,17 +0,0 @@
-from tiramisu import *
-from tiramisu.setting import ALLOWED_LEADER_PROPERTIES
-from re import compile as re_compile
-from rougail.tiramisu import func, dict_env, load_functions, ConvertDynOptionDescription
-load_functions('../rougail-tests/funcs/test.py')
-try:
-    groups.namespace
-except:
-    groups.addgroup('namespace')
-ALLOWED_LEADER_PROPERTIES.add("basic")
-ALLOWED_LEADER_PROPERTIES.add("standard")
-ALLOWED_LEADER_PROPERTIES.add("advanced")
-option_2 = StrOption(name="var", doc="var", properties=frozenset({"basic", "mandatory"}), informations={'ymlfiles': ['../rougail-tests/structures/20_7help_family/rougail/00-base.yml'], 'type': 'string'})
-optiondescription_1 = OptionDescription(name="family1", doc="the first family", children=[option_2], properties=frozenset({"basic"}), informations={'ymlfiles': ['../rougail-tests/structures/20_7help_family/rougail/00-base.yml'], 'help': 'Multi line\n\nHelp\n\nWith useful information'})
-option_4 = StrOption(name="var", doc="var", properties=frozenset({"basic", "mandatory"}), informations={'ymlfiles': ['../rougail-tests/structures/20_7help_family/rougail/00-base.yml'], 'type': 'string'})
-optiondescription_3 = OptionDescription(name="family2", doc="the second family", children=[option_4], properties=frozenset({"basic"}), informations={'ymlfiles': ['../rougail-tests/structures/20_7help_family/rougail/00-base.yml'], 'help': 'Multi line\nHelp\nWith useful information'})
-option_0 = OptionDescription(name="baseoption", doc="baseoption", children=[optiondescription_1, optiondescription_3])

tests/test_1_flattener.py

@@ -111,10 +111,7 @@ def load_rougail_object(test_dir, rougailconfig, multi=False, namespace=False):
         if namespace is False and not multi:
             return None
         rougailconfig['extra_namespaces'] = extra_namespaces
-    try:
     rougailconfig['tiramisu_cache'] = get_tiramisu_filename(test_dir, 'tmp', multi, namespace)
-    except:
-        rougailconfig['cli.tiramisu_cache'] = get_tiramisu_filename(test_dir, 'tmp', multi, namespace)
     rougailconfig['custom_types']['custom'] = CustomOption
     return Rougail(rougailconfig)
 

tests/test_others.py

@@ -35,10 +35,7 @@ def test_personalize_mode():
     rougailconfig['modes_level'] = ['level1', 'level2']
     rougailconfig['default_variable_mode'] = 'level1'
     rougailconfig['default_family_mode'] = 'level1'
-    try:
     rougailconfig['tiramisu_cache'] = None
-    except:
-        rougailconfig['cli.tiramisu_cache'] = None
     eolobj = Rougail(rougailconfig=rougailconfig)
     eolobj.run()
 

tests/test_types.py

@@ -17,10 +17,7 @@ def type_variable(test_name):
     rougailconfig = RougailConfig.copy()
     rougailconfig['types'] = [f'tests/types/types/{test_name}']
     rougailconfig['main_structural_directories'] = [f'tests/types/structures/{ test_name }']
-    try:
     rougailconfig['tiramisu_cache'] = str(tmp_file)
-    except:
-        rougailconfig['cli.tiramisu_cache'] = str(tmp_file)
     rougail = Rougail(rougailconfig=rougailconfig)
     config = rougail.run()
     #