refactor

src/rougail/annotator.py

@@ -39,7 +39,7 @@ modes = mode_factory()
 # that shall not be present in the exported (flatened) XML
 ERASED_ATTRIBUTES = ('redefine', 'exists', 'fallback', 'optional', 'remove_check', 'namespace',
                      'remove_condition', 'path', 'instance_mode', 'index', 'is_in_leadership',
-                     'level', 'submulti')  # , '_real_container')
+                     'level')  # , '_real_container')
 ERASED_CONTAINER_ATTRIBUTES = ('id', 'container', 'group_id', 'group', 'container_group')
 
 FORCE_CHOICE = {'oui/non': ['oui', 'non'],
@@ -405,9 +405,6 @@ class SpaceAnnotator(object):
         if leader_is_hidden:
             variable.frozen = True
             variable.force_default_on_freeze = True
-        # followers are multi
-        if not variable.multi:
-            raise CreoleDictConsistencyError(_('the variable {} in a group must be multi or submulti').format(variable.name))
         leader_space.variable.append(variable)  # pylint: disable=E1101
         self.paths.set_leader(namespace,
                               leader_family_name,
@@ -534,8 +531,6 @@ class SpaceAnnotator(object):
                                 variable.type = 'string'
                             if variable.type != 'symlink' and not hasattr(variable, 'description'):
                                 variable.description = variable.name
-                            if variable.submulti:
-                                variable.multi = 'submulti'
 
     def convert_auto_freeze(self):  # pylint: disable=C0111
         if hasattr(self.space, 'variables'):
@@ -732,21 +727,21 @@ class SpaceAnnotator(object):
         del self.space.constraints.fill
 
     def filter_separators(self):  # pylint: disable=C0111,R0201
-        # FIXME devrait etre dans la variable
         if not hasattr(self.space, 'variables'):
             return
         for family in self.space.variables.values():
-            if (hasattr(family, 'separators') and hasattr(family.separators, 'separator')):
+            if not hasattr(family, 'separators'):
-                space = family.separators.separator
+                continue
-                names = []
+            if hasattr(family.separators, 'separator'):
-                for idx, separator in enumerate(space):
+                for idx, separator in enumerate(family.separators.separator):
-                    namespace = self.paths.get_variable_namespace(separator.name)
+                    option = self.paths.get_variable_obj(separator.name)
-                    subpath = self.paths.get_variable_path(separator.name, namespace)
+                    if hasattr(option, 'separator'):
-                    separator.name = subpath
+                        subpath = self.paths.get_variable_path(separator.name,
-                    if separator.name in names:
+                                                               separator.namespace,
-                        raise CreoleDictConsistencyError(_('{} already has a separator').format(separator.name))
+                                                               )
-                    names.append(separator.name)
+                        raise CreoleDictConsistencyError(_('{} already has a separator').format(subpath))
-
+                    option.separator = separator.text
+            del family.separators
 
     def load_params_in_validenum(self, param):
         if param.type in ['string', 'python', 'number']:

src/rougail/data/rougail.dtd

@@ -101,7 +101,6 @@
 <!ATTLIST variable hidden (True|False) "False">
 <!ATTLIST variable disabled (True|False) "False">
 <!ATTLIST variable multi (True|False) "False">
-<!ATTLIST variable submulti (True|False) "False">
 <!ATTLIST variable redefine (True|False) "False">
 <!ATTLIST variable exists (True|False) "True">
 <!ATTLIST variable mandatory (True|False) "False">

src/rougail/loader.py

@@ -20,7 +20,6 @@ from .error import CreoleLoaderError
 
 
 FUNC_TO_DICT = ['valid_not_equal']
-KNOWN_TAGS = ['family', 'variable', 'separators', 'leader', 'property']
 
 
 class ConvertDynOptionDescription(DynOptionDescription):
@@ -31,22 +30,22 @@ class ConvertDynOptionDescription(DynOptionDescription):
                                 check_name=False)
 
 
-def convert_tiramisu_value(value, obj):
+def convert_boolean(value):
+    prop = {'True': True,
+            'False': False,
+            'None': None}
+    if value not in prop:
+        raise Exception('unknown value {} while trying to cast {} to boolean'.format(value, obj))
+    return prop[value]
+
+
+def convert_tiramisu_value(value, type_):
     """
     convertit les variables dans le bon type si nécessaire
     """
-    def _convert_boolean(value):
-        prop = {'True': True,
-                'False': False,
-                'None': None}
-        if value not in prop:
-            raise Exception('unknown value {} while trying to cast {} to boolean'.format(value, obj))
-        return prop[value]
-
     if value is None:
         return value
-    func = {IntOption: int,
+    func = CONVERT_OPTION[type_].get('func', None)
-            BoolOption: _convert_boolean}.get(obj, None)
     if func is None:
         return value
     if isinstance(value, list):
@@ -55,12 +54,12 @@ def convert_tiramisu_value(value, obj):
         return func(value)
 
 
-CONVERT_OPTION = {'number': dict(opttype=IntOption),
+CONVERT_OPTION = {'number': dict(opttype=IntOption, func=int),
                   'choice': dict(opttype=ChoiceOption),
                   'string': dict(opttype=StrOption),
                   'password': dict(opttype=PasswordOption),
                   'mail': dict(opttype=EmailOption),
-                  'boolean': dict(opttype=BoolOption, initkwargs={'default': True}),
+                  'boolean': dict(opttype=BoolOption, initkwargs={'default': True}, func=convert_boolean),
                   'symlink': dict(opttype=SymLinkOption),
                   'filename': dict(opttype=FilenameOption),
                   'date': dict(opttype=DateOption),
@@ -83,11 +82,37 @@ CONVERT_OPTION = {'number': dict(opttype=IntOption),
                   }
 
 
-class Elt:
+class BaseElt:
     def __init__(self, attrib):
         self.attrib = attrib
 
 
+class Common:
+    def populate_properties(self, child):
+        if child.get('type') == 'calculation':
+            kwargs = {'condition': child.attrib['source'],
+                      'expected':  ParamValue(child.attrib.get('expected'))}
+            if child.attrib['inverse'] == 'True':
+                kwargs['reverse_condition'] = ParamValue(True)
+            self.attrib['properties'].append((ParamValue(child.text), kwargs))
+        else:
+            self.attrib['properties'].append(child.text)
+
+    def build_properties(self):
+        for index, prop in enumerate(self.attrib['properties']):
+            if isinstance(prop, tuple):
+                action, kwargs = prop
+                kwargs['condition'] = ParamOption(self.storage.get(kwargs['condition']).get(), todict=True)
+                prop = Calculation(calc_value,
+                                   Params(action,
+                                          kwargs=kwargs))
+                self.attrib['properties'][index] = prop
+        if self.attrib['properties']:
+            self.attrib['properties'] = tuple(self.attrib['properties'])
+        else:
+            del self.attrib['properties']
+
+
 class PopulateTiramisuObjects:
     def __init__(self):
         self.storage = ElementStorage()
@@ -110,9 +135,19 @@ class PopulateTiramisuObjects:
                         if set(attr.itervalues()) == set(['True', 'False']):
                             self.booleans.append(attr.name)
 
+    def make_tiramisu_objects(self, xmlroot, eosfunc):
+        self.eosfunc = imp.load_source('eosfunc', eosfunc)
+
+        family = self.get_root_family()
+        for xmlelt in self.reorder_family(xmlroot):
+            self.iter_family(xmlelt,
+                             family,
+                             None,
+                             )
+
     def get_root_family(self):
-        family = Family(Elt({'name': 'baseoption',
+        family = Family(BaseElt({'name': 'baseoption',
-                             'doc': 'baseoption'}),
+                                 'doc': 'baseoption'}),
                         self.booleans,
                         self.storage,
                         self.eosfunc,
@@ -131,36 +166,24 @@ class PopulateTiramisuObjects:
                 xmlelts.append(xmlelt)
         return xmlelts
 
-    def make_tiramisu_objects(self, xmlroot, eosfunc):
-        self.eosfunc = imp.load_source('eosfunc', eosfunc)
-
-        family = self.get_root_family()
-        for xmlelt in self.reorder_family(xmlroot):
-            self.iter_family(xmlelt,
-                             family,
-                             None,
-                             )
-
     def iter_family(self,
                     child,
                     family,
                     subpath,
                     ):
-        if child.tag not in KNOWN_TAGS:
-            raise CreoleLoaderError(_('unknown tag {}').format(child.tag))
         if child.tag in ['family', 'leader']:
-            self.populate_family(family,
+            function = self.populate_family
-                                 child,
-                                 subpath,
-                                 )
-        elif child.tag == 'separators':
-            self.parse_separators(child)
         elif child.tag == 'variable':
-            self.populate_variable(child, subpath, family)
+            function = self.populate_variable
         elif child.tag == 'property':
-            self.parse_properties(family, child)
+            # property already imported with family
+            return
         else:
             raise Exception('unknown tag {}'.format(child.tag))
+        function(family,
+                 child,
+                 subpath,
+                 )
 
     def populate_family(self,
                         parent_family,
@@ -174,10 +197,9 @@ class PopulateTiramisuObjects:
                         self.booleans,
                         self.storage,
                         self.eosfunc,
+                        elt.tag == 'leader',
                         )
         self.storage.add(path, family)
-        if elt.tag == 'leader':
-            family.set_leader()
         parent_family.add(family)
         if len(elt) != 0:
             for child in elt:
@@ -186,14 +208,11 @@ class PopulateTiramisuObjects:
                                  path,
                                  )
 
-    def parse_separators(self,
+    def populate_variable(self,
-                         separators,
+                          family,
-                         ):
+                          elt,
-        for separator in separators:
+                          subpath,
-            elt = self.storage.get(separator.attrib['name'])
+                          ):
-            elt.add_information('separator', separator.text)
-
-    def populate_variable(self, elt, subpath, family):
         is_follower = False
         is_leader = False
         if family.is_leader:
@@ -214,16 +233,6 @@ class PopulateTiramisuObjects:
         self.storage.add(path, variable)
         family.add(variable)
 
-    def parse_properties(self, family, child):
-        if child.get('type') == 'calculation':
-            kwargs = {'condition': child.attrib['source'],
-                      'expected':  ParamValue(child.attrib.get('expected'))}
-            if child.attrib['inverse'] == 'True':
-                kwargs['reverse_condition'] = ParamValue(True)
-            family.attrib['properties'].append((ParamValue(child.text), kwargs))
-        else:
-            family.attrib['properties'].append(child.text)
-
     def build_path(self,
                    subpath,
                    elt,
@@ -248,22 +257,6 @@ class ElementStorage:
         return self.paths[path]
 
 
-class Common:
-    def build_properties(self):
-        for index, prop in enumerate(self.attrib['properties']):
-            if isinstance(prop, tuple):
-                action, kwargs = prop
-                kwargs['condition'] = ParamOption(self.storage.get(kwargs['condition']).get(), todict=True)
-                prop = Calculation(calc_value,
-                                   Params(action,
-                                          kwargs=kwargs))
-                self.attrib['properties'][index] = prop
-        if self.attrib['properties']:
-            self.attrib['properties'] = tuple(self.attrib['properties'])
-        else:
-            del self.attrib['properties']
-
-
 class Variable(Common):
     def __init__(self, elt, booleans, storage, is_follower, is_leader, eosfunc):
         self.option = None
@@ -282,185 +275,183 @@ class Variable(Common):
         self.is_follower = is_follower
         self.is_leader = is_leader
         self.object_type = convert_option['opttype']
-        self.populate_choice(elt)
+        if 'multi' in self.attrib and self.attrib['multi']:
-        for child in elt:
+            self.attrib['default'] = []
-            self.populate_property(child)
+            if self.is_follower:
-            self.populate_value(child)
+                self.attrib['default_multi'] = []
-            self.populate_check(child)
+        self.parse_children(elt)
+        if self.is_follower:
+            if self.attrib['multi'] is True:
+                self.attrib['multi'] = submulti
+            else:
+                self.attrib['multi'] = True
         if elt.attrib['type'] == 'symlink':
             self.attrib['opt'] = storage.get(self.attrib['opt'])
 
+    def parse_children(self,
+                       elt,
+                       ):
+        if elt.attrib['type'] == 'choice':
+            values = []
+        for child in elt:
+            if child.tag == 'property':
+                self.populate_properties(child)
+            elif child.tag == 'value':
+                if child.attrib.get('type') == 'calculation':
+                    self.calc_value(child)
+                else:
+                    self.populate_value(child, elt)
+            elif child.tag == 'check':
+                self.populate_check(child)
+            elif child.tag == 'choice':
+                value = child.text
+                if child.attrib['type'] == 'number':
+                    value = int(value)
+                values.append(value)
+        if elt.attrib['type'] == 'choice':
+            self.attrib['values'] = tuple(values)
+
     def populate_attrib(self,
                         elt,
                         ):
         for key, value in elt.attrib.items():
             if key == 'multi' and elt.attrib['type'] == 'symlink':
                 continue
-            if key == 'multi' and value == 'submulti':
+            if key in self.booleans:
-                value = submulti
-            elif key in self.booleans:
                 if value == 'True':
                     value = True
                 elif value == 'False':
                     value = False
                 else:
                     raise CreoleLoaderError(_('unknown value {} for {}').format(value, key))
-            if key in ['help', 'test']:
+            if key in ['help', 'test', 'separator']:
                 self.add_information(key, value)
             elif key != 'type':
                 self.attrib[key] = value
 
-    def populate_choice(self,
+    def calc_value(self, child):
-                        elt,
+        args = []
-                        ):
+        kwargs = {}
-        if elt.attrib['type'] == 'choice':
+        if child.text is not None and child.text.strip():
-            values = []
+            function = child.text.strip()
-            for child in elt:
+        else:
-                if child.tag == 'choice':
+            function = child.attrib['name']
-                    value = child.text
+            for param in child:
-                    if child.attrib['type'] == 'number':
+                self.parse_param(args,
-                        value = int(value)
+                                 kwargs,
-                    values.append(value)
+                                 param,
-            self.attrib['values'] = tuple(values)
+                                 )
+        self.attrib['default'] = {'function': getattr(self.eosfunc, function),
+                                  'args': args,
+                                  'kwargs': kwargs,
+                                  'warnings_only': False}
 
-    def populate_property(self, child):
+    def populate_value(self, child, elt):
-        if child.tag == 'property':
+        if "type" in child.attrib:
-            if child.get('type') == 'calculation':
+            type_ = child.attrib['type']
-                kwargs = {'condition': child.attrib['source'],
+        else:
-                          'expected':  ParamValue(child.attrib.get('expected'))}
+            type_ = elt.attrib['type']
-                if child.attrib['inverse'] == 'True':
+        value = convert_tiramisu_value(child.text, type_)
-                    kwargs['reverse_condition'] = ParamValue(True)
+        if self.is_follower:
-                self.attrib['properties'].append((ParamValue(child.text), kwargs))
+            if self.attrib['multi'] is True:
+                self.attrib['default_multi'].append(value)
             else:
-                self.attrib['properties'].append(child.text)
+                self.attrib['default_multi'] = value
-
+        elif self.attrib['multi'] is True:
-    def populate_value(self, child):
+            self.attrib['default'].append(value)
-        if child.tag == 'value':
+            if not self.is_leader:
-            if child.attrib.get('type') == 'calculation':
+                self.attrib['default_multi'] = value
-                if child.text is not None and child.text.strip():
+        else:
-                    self.attrib['default'] = (child.text.strip(),)
+            self.attrib['default'] = value
-                else:
-                    params = []
-                    for param in child:
-                        params.append(self.parse_param(param))
-                    self.attrib['default'] = (child.attrib['name'], params, False)
-            else:
-                if "type" in child.attrib:
-                    type_ = CONVERT_OPTION[child.attrib['type']]['opttype']
-                else:
-                    type_ = self.object_type
-                if self.attrib['multi'] is True and not self.is_follower:
-                    if 'default' not in self.attrib:
-                        self.attrib['default'] = []
-                    value = convert_tiramisu_value(child.text, type_)
-                    self.attrib['default'].append(value)
-                    if 'default_multi' not in self.attrib and not self.is_leader:
-                        self.attrib['default_multi'] = value
-                elif self.attrib['multi'] == submulti:
-                    if 'default' not in self.attrib:
-                        self.attrib['default'] = []
-                    value = convert_tiramisu_value(child.text, type_)
-                    if not self.is_follower:
-                        if not isinstance(value, list):
-                            dvalue = [value]
-                        else:
-                            dvalue = value
-                        self.attrib['default'].append(dvalue)
-                    if value and 'default_multi' not in self.attrib and not self.is_leader:
-                        self.attrib['default_multi'] = []
-                    if not self.is_leader:
-                        self.attrib['default_multi'].append(value)
-                else:
-                    value = convert_tiramisu_value(child.text, type_)
-                    if self.is_follower:
-                        self.attrib['default_multi'] = value
-                    else:
-                        self.attrib['default'] = value
 
     def populate_check(self, child):
-        if child.tag == 'check':
+        args = [ParamSelfOption()]
-            params = []
+        kwargs = {}
-            for param in child:
+        for param in child:
-                params.append(self.parse_param(param))
+            self.parse_param(args,
-            #check.params = params
+                             kwargs,
-            self.attrib['validators'].append((child.attrib['name'], params, child.attrib['warnings_only']))
+                             param,
+                             )
+        self.attrib['validators'].append({'function': getattr(self.eosfunc, child.attrib['name']),
+                                          'args': args,
+                                          'kwargs': kwargs,
+                                          'warnings_only': convert_boolean(child.attrib['warnings_only'])})
 
-    def parse_param(self, param):
+    def parse_param(self,
-        name = param.attrib.get('name', '')
+                    args,
+                    kwargs,
+                    param,
+                    ):
         if param.attrib['type'] == 'string':
-            value = param.text
+            value = ParamValue(param.text)
-        elif param.attrib['type'] == 'variable':
-            transitive = param.attrib.get('transitive', 'False')
-            if transitive == 'True':
-                transitive = True
-            elif transitive == 'False':
-                transitive = False
-            else:
-                raise CreoleLoaderError(_('unknown transitive boolean {}').format(transitive))
-            value = [param.text, transitive, param.attrib.get('suffix')]
         elif param.attrib['type'] == 'number':
-            value = int(param.text)
+            value = ParamValue(int(param.text))
+        elif param.attrib['type'] == 'variable':
+            transitive = convert_boolean(param.attrib.get('transitive', 'False'))
+            value = {'option': param.text,
+                     'notraisepropertyerror': transitive,
+                     'todict': param.text in FUNC_TO_DICT,
+                     }
+            if 'suffix' in param.attrib:
+                value['suffix'] = param.attrib['suffix']
         else:
             raise CreoleLoaderError(_('unknown param type {}').format(param.attrib['type']))
-        return(name, value)
+        if 'name' not in param.attrib:
+            args.append(value)
+        else:
+            kwargs[param.attrib['name']] = value
+
+    def build_calculator(self, key):
+        def build_param(param):
+            option = self.storage.get(param['option']).get()
+            if 'suffix' in param:
+                family = '.'.join(param['option'].split('.')[:-1])
+                return ParamDynOption(option,
+                                      param['suffix'],
+                                      self.storage.get(family).get(),
+                                      notraisepropertyerror=param['notraisepropertyerror'],
+                                      todict=param['todict'],
+                                      )
+            return ParamOption(option,
+                               notraisepropertyerror=param['notraisepropertyerror'],
+                               todict=param['todict'],
+                               )
+            return param
+        def build_calculation(value):
+            args = []
+            kwargs = {}
+            for param in value['args']:
+                if isinstance(param, dict):
+                    param = build_param(param)
+                args.append(param)
+            for key, param in value['kwargs'].items():
+                if isinstance(param, dict):
+                    param = build_param(param)
+                kwargs[key] = param
+            return Calculation(value['function'],
+                               Params(tuple(args),
+                                      kwargs=kwargs,
+                                      ),
+                               warnings_only=value['warnings_only'],
+                               )
+        if key in self.attrib:
+            values = self.attrib[key]
+            if isinstance(values, list):
+                self.attrib[key] = []
+                for value in values:
+                    if isinstance(value, dict):
+                        value = build_calculation(value)
+                    self.attrib[key].append(value)
+            else:
+                if isinstance(values, dict):
+                    values = build_calculation(values)
+                self.attrib[key] = values
 
     def add_information(self, key, value):
         if key in self.informations:
             raise CreoleLoaderError(_('key already exists in information {}').format(key))
         self.informations[key] = value
 
-    def build_calculator(self, key):
-        if key in self.attrib:
-            values = self.attrib[key]
-            if isinstance(values, list):
-                is_list = True
-            else:
-                is_list = False
-                values = [values]
-            ret = []
-            for value in values:
-                if isinstance(value, tuple):
-                    if key == 'validators':
-                        args = [ParamSelfOption()]
-                    else:
-                        args = []
-                    kwargs = {}
-                    if len(value) == 3:
-                        for param in value[1]:
-                            if isinstance(param[1], list):
-                                option = self.storage.get(param[1][0]).get()
-                                param_kwargs = {'notraisepropertyerror': param[1][1]}
-                                if value[0] in FUNC_TO_DICT:
-                                    param_kwargs['todict'] = True
-                                if not param[1][2]:
-                                    param_value = ParamOption(option,
-                                                              **param_kwargs,
-                                                              )
-                                else:
-                                    family = '.'.join(param[1][0].split('.', 3)[:2])
-                                    param_value = ParamDynOption(option,
-                                                                 param[1][2],
-                                                                 self.storage.get(family).get(),
-                                                                 **param_kwargs,
-                                                                 )
-                            else:
-                                param_value = ParamValue(param[1])
-                            if not param[0]:
-                                args.append(param_value)
-                            else:
-                                kwargs[param[0]] = param_value
-
-                    ret.append(Calculation(getattr(self.eosfunc, value[0]),
-                                           Params(tuple(args),
-                                           kwargs=kwargs)))
-                else:
-                    ret.append(value)
-            if not is_list:
-                self.attrib[key] = ret[0]
-            else:
-                self.attrib[key] = ret
-
-
     def get(self):
         if self.option is None:
             if self.object_type is SymLinkOption:
@@ -490,6 +481,7 @@ class Family(Common):
                  booleans,
                  storage,
                  eosfunc,
+                 is_leader=False,
                  ):
         self.option = None
         self.attrib = {}
@@ -504,6 +496,11 @@ class Family(Common):
                 self.add_information(key, value)
             else:
                 self.attrib[key] = value
+        if not isinstance(elt, BaseElt) and len(elt) != 0:
+            for child in elt:
+                if child.tag == 'property':
+                    self.populate_properties(child)
+        self.is_leader = is_leader
 
     def add(self, child):
         self.children.append(child)
@@ -513,9 +510,6 @@ class Family(Common):
             raise CreoleLoaderError(_('key already exists in information {}').format(key))
         self.informations[key] = value
 
-    def set_leader(self):
-        self.is_leader = True
-
     def get(self):
         if self.option is None:
             self.attrib['children'] = []

src/rougail/objspace.py

@@ -38,7 +38,7 @@ FORCE_REDEFINABLES = ('family', 'follower', 'service', 'disknod', 'variables')
 # CreoleObjSpace's elements that shall be forced to the UnRedefinable type
 FORCE_UNREDEFINABLES = ('value',)
 # CreoleObjSpace's elements that shall be set to the UnRedefinable type
-UNREDEFINABLE = ('submulti', 'multi', 'type')
+UNREDEFINABLE = ('multi', 'type')
 
 PROPERTIES = ('hidden', 'frozen', 'auto_freeze', 'auto_save', 'force_default_on_freeze',
               'force_store_value', 'disabled', 'mandatory')
@@ -46,17 +46,14 @@ CONVERT_PROPERTIES = {'auto_save': ['force_store_value'], 'auto_freeze': ['force
 
 RENAME_ATTIBUTES = {'description': 'doc'}
 
-INCOMPATIBLE_ATTRIBUTES = [['multi', 'submulti']]
 FORCED_TEXT_ELTS_AS_NAME = ('choice', 'property', 'value', 'target')
 
 CONVERT_EXPORT = {'Leadership': 'leader',
-                  'Separators': 'separators',
                   'Variable': 'variable',
                   'Value': 'value',
                   'Property': 'property',
                   'Choice': 'choice',
                   'Param': 'param',
-                  'Separator': 'separator',
                   'Check': 'check',
                   }
 
@@ -461,13 +458,6 @@ class CreoleObjSpace:
             if not (attr == 'name' and getattr(variableobj, 'name', None) != None):
                 setattr(variableobj, attr, val)
         keys = list(vars(variableobj).keys())
-        for incompatible in INCOMPATIBLE_ATTRIBUTES:
-            found = False
-            for inc in incompatible:
-                if inc in keys:
-                    if found:
-                        raise CreoleDictConsistencyError(_('those attributes are incompatible {}').format(incompatible))
-                    found = True
 
     def variableobj_tree_visitor(self,
                                  child,

tests/flattener_dicos/00load_autofreeze/result/00-base.xml

@@ -21,6 +21,5 @@
         <value type="string">non</value>
       </variable>
     </family>
-    <separators/>
   </family>
 </rougail>

tests/flattener_dicos/00load_autofreezeexpert/result/00-base.xml

@@ -21,6 +21,5 @@
         <value type="string">non</value>
       </variable>
     </family>
-    <separators/>
   </family>
 </rougail>

tests/flattener_dicos/00load_autosave/result/00-base.xml

@@ -12,6 +12,5 @@
         <value type="string">non</value>
       </variable>
     </family>
-    <separators/>
   </family>
 </rougail>

tests/flattener_dicos/00load_autosaveexpert/result/00-base.xml

@@ -12,6 +12,5 @@
         <value type="string">non</value>
       </variable>
     </family>
-    <separators/>
   </family>
 </rougail>

tests/flattener_dicos/00load_comment/result/00-base.xml

@@ -14,6 +14,5 @@
         <value type="string">non</value>
       </variable>
     </family>
-    <separators/>
   </family>
 </rougail>

tests/flattener_dicos/00load_notype/result/00-base.xml

@@ -19,6 +19,5 @@
         <value>non</value>
       </variable>
     </family>
-    <separators/>
   </family>
 </rougail>

tests/flattener_dicos/00load_save/result/00-base.xml

@@ -14,6 +14,5 @@
         <value type="string">non</value>
       </variable>
     </family>
-    <separators/>
   </family>
 </rougail>

tests/flattener_dicos/00load_subfolder/result/00-base.xml

@@ -24,6 +24,5 @@
         <value type="string">non</value>
       </variable>
     </family>
-    <separators/>
   </family>
 </rougail>

tests/flattener_dicos/01auto_base/result/00-base.xml

@@ -23,6 +23,5 @@
         <value type="string">non</value>
       </variable>
     </family>
-    <separators/>
   </family>
 </rougail>

tests/flattener_dicos/01auto_withoutparam/result/00-base.xml

@@ -21,6 +21,5 @@
         <value type="string">non</value>
       </variable>
     </family>
-    <separators/>
   </family>
 </rougail>

tests/flattener_dicos/01base_multi/result/00-base.xml

@@ -14,6 +14,5 @@
         <value type="string">non</value>
       </variable>
     </family>
-    <separators/>
   </family>
 </rougail>

tests/flattener_dicos/01base_submulti/makedict/base.json

@@ -1 +0,0 @@
-{"rougail.general.mode_conteneur_actif": [["non"]]}

tests/flattener_dicos/01base_submulti/result/00-base.xml

@@ -1,19 +0,0 @@
-<?xml version='1.0' encoding='UTF-8'?>
-<rougail>
-  <family doc="rougail" name="rougail">
-    <family doc="general" name="general">
-      <property>normal</property>
-      <variable doc="Redefine description" multi="submulti" name="mode_conteneur_actif" type="choice">
-        <choice type="string">oui</choice>
-        <choice type="string">non</choice>
-        <property>force_default_on_freeze</property>
-        <property>frozen</property>
-        <property>hidden</property>
-        <property>mandatory</property>
-        <property>normal</property>
-        <value type="string">non</value>
-      </variable>
-    </family>
-    <separators/>
-  </family>
-</rougail>

tests/flattener_dicos/01fill_autofreeze/result/00-base.xml

@@ -30,6 +30,5 @@
         <value type="string">non</value>
       </variable>
     </family>
-    <separators/>
   </family>
 </rougail>

tests/flattener_dicos/01fill_autosave/result/00-base.xml

@@ -21,6 +21,5 @@
         <value type="string">non</value>
       </variable>
     </family>
-    <separators/>
   </family>
 </rougail>

tests/flattener_dicos/01fill_base/result/00-base.xml

@@ -23,6 +23,5 @@
         <value type="string">non</value>
       </variable>
     </family>
-    <separators/>
   </family>
 </rougail>

tests/flattener_dicos/01fill_baseaccent/result/00-base.xml

@@ -23,6 +23,5 @@
         <value type="string">non</value>
       </variable>
     </family>
-    <separators/>
   </family>
 </rougail>

tests/flattener_dicos/01fill_mandatory/result/00-base.xml

@@ -18,6 +18,5 @@
         <value type="string">non</value>
       </variable>
     </family>
-    <separators/>
   </family>
 </rougail>

tests/flattener_dicos/01fill_number/result/00-base.xml

@@ -21,6 +21,5 @@
         <value type="string">non</value>
       </variable>
     </family>
-    <separators/>
   </family>
 </rougail>

tests/flattener_dicos/01fill_optional/result/00-base.xml

@@ -23,6 +23,5 @@
         <value type="string">non</value>
       </variable>
     </family>
-    <separators/>
   </family>
 </rougail>

tests/flattener_dicos/01separator_base/result/00-base.xml

@@ -3,7 +3,7 @@
   <family doc="rougail" name="rougail">
     <family doc="general" name="general">
       <property>normal</property>
-      <variable doc="No change" multi="False" name="mode_conteneur_actif" type="choice">
+      <variable doc="No change" multi="False" name="mode_conteneur_actif" separator="Établissement" type="choice">
         <choice type="string">oui</choice>
         <choice type="string">non</choice>
         <property>force_default_on_freeze</property>
@@ -14,8 +14,5 @@
         <value type="string">non</value>
       </variable>
     </family>
-    <separators>
-      <separator name="rougail.general.mode_conteneur_actif">Établissement</separator>
-    </separators>
   </family>
 </rougail>

tests/flattener_dicos/01separator_neverhidden/result/00-base.xml

@@ -3,7 +3,7 @@
   <family doc="rougail" name="rougail">
     <family doc="general" name="general">
       <property>normal</property>
-      <variable doc="No change" multi="False" name="mode_conteneur_actif" type="choice">
+      <variable doc="No change" multi="False" name="mode_conteneur_actif" separator="Établissement" type="choice">
         <choice type="string">oui</choice>
         <choice type="string">non</choice>
         <property>force_default_on_freeze</property>
@@ -14,8 +14,5 @@
         <value type="string">non</value>
       </variable>
     </family>
-    <separators>
-      <separator name="rougail.general.mode_conteneur_actif">Établissement</separator>
-    </separators>
   </family>
 </rougail>

tests/flattener_dicos/10autosave_hidden/result/00-base.xml

@@ -24,6 +24,5 @@
         </value>
       </variable>
     </family>
-    <separators/>
   </family>
 </rougail>

tests/flattener_dicos/10autosave_hidden_frozenifin/result/00-base.xml

@@ -25,6 +25,5 @@
         </value>
       </variable>
     </family>
-    <separators/>
   </family>
 </rougail>

tests/flattener_dicos/10check_base/result/00-base.xml

@@ -16,6 +16,5 @@
         <property>normal</property>
       </variable>
     </family>
-    <separators/>
   </family>
 </rougail>

tests/flattener_dicos/10check_option/result/00-base.xml

@@ -21,6 +21,5 @@
         <property>normal</property>
       </variable>
     </family>
-    <separators/>
   </family>
 </rougail>

tests/flattener_dicos/10check_optional/result/00-base.xml

@@ -18,6 +18,5 @@
         <property>normal</property>
       </variable>
     </family>
-    <separators/>
   </family>
 </rougail>

tests/flattener_dicos/10check_valid_differ/result/00-base.xml

@@ -21,6 +21,5 @@
         <value type="string">non</value>
       </variable>
     </family>
-    <separators/>
   </family>
 </rougail>

tests/flattener_dicos/10check_valid_differ_add/result/00-base.xml

@@ -36,6 +36,5 @@
         <value>oui</value>
       </variable>
     </family>
-    <separators/>
   </family>
 </rougail>

tests/flattener_dicos/10check_valid_differ_removecheck/result/00-base.xml

@@ -36,6 +36,5 @@
         <value>oui</value>
       </variable>
     </family>
-    <separators/>
   </family>
 </rougail>

tests/flattener_dicos/10check_valid_ipnetmask/result/00-base.xml

@@ -22,6 +22,5 @@
         <property>basic</property>
       </variable>
     </family>
-    <separators/>
   </family>
 </rougail>

tests/flattener_dicos/10leadership_append/00-base.xml

@@ -11,8 +11,8 @@
         </family>
         <family name="general1">
             <variable name="leader" type="string" description="leader" multi="True"/>
-            <variable name="follower1" type="string" description="follower1" multi="True"/>
+            <variable name="follower1" type="string" description="follower1"/>
-            <variable name="follower2" type="string" description="follower2" multi="True"/>
+            <variable name="follower2" type="string" description="follower2"/>
         </family>
     </variables>
 

tests/flattener_dicos/10leadership_append/01-base.xml

@@ -5,7 +5,7 @@
 
     <variables>
         <family name="general1">
-            <variable name="follower3" type="string" description="follower3" multi="True"/>
+            <variable name="follower3" type="string" description="follower3"/>
         </family>
     </variables>
 

tests/flattener_dicos/10leadership_append/result/00-base.xml

@@ -16,21 +16,21 @@
       <leader doc="leader" name="leader">
         <property>normal</property>
         <variable doc="leader" multi="True" name="leader" type="string"/>
-        <variable doc="follower1" multi="True" name="follower1" type="string">
+        <variable doc="follower1" multi="False" name="follower1" type="string">
           <property>mandatory</property>
           <property>normal</property>
           <value name="calc_val" type="calculation">
             <param name="valeur" transitive="False" type="string">valfill</param>
           </value>
         </variable>
-        <variable doc="follower2" multi="True" name="follower2" type="string">
+        <variable doc="follower2" multi="False" name="follower2" type="string">
           <property>mandatory</property>
           <property>normal</property>
           <value name="calc_val" type="calculation">
             <param transitive="False" type="variable">rougail.general1.leader.follower1</param>
           </value>
         </variable>
-        <variable doc="follower3" multi="True" name="follower3" type="string">
+        <variable doc="follower3" multi="False" name="follower3" type="string">
           <property>normal</property>
         </variable>
       </leader>

tests/flattener_dicos/10leadership_auto/00-base.xml

@@ -9,9 +9,9 @@
                 <value>non</value>
             </variable>
             <variable name="leader" type="string" description="leader" multi="True"/>
-            <variable name="follower1" type="string" description="follower1" multi="True" hidden="True"/>
+            <variable name="follower1" type="string" description="follower1" hidden="True"/>
-            <variable name="follower2" type="string" description="follower2" multi="True" hidden="True"/>
+            <variable name="follower2" type="string" description="follower2" hidden="True"/>
-            <variable name="follower3" type="string" description="follower3" multi="True" hidden="True"/>
+            <variable name="follower3" type="string" description="follower3" hidden="True"/>
         </family>
     </variables>
 

tests/flattener_dicos/10leadership_auto/result/00-base.xml

@@ -13,7 +13,7 @@
       <leader doc="leader" name="leader">
         <property>normal</property>
         <variable doc="leader" multi="True" name="leader" type="string"/>
-        <variable doc="follower1" multi="True" name="follower1" type="string">
+        <variable doc="follower1" multi="False" name="follower1" type="string">
           <property>force_default_on_freeze</property>
           <property>frozen</property>
           <property>hidden</property>
@@ -23,7 +23,7 @@
             <param name="valeur" transitive="False" type="string">valfill</param>
           </value>
         </variable>
-        <variable doc="follower2" multi="True" name="follower2" type="string">
+        <variable doc="follower2" multi="False" name="follower2" type="string">
           <property>force_default_on_freeze</property>
           <property>frozen</property>
           <property>hidden</property>
@@ -33,7 +33,7 @@
             <param transitive="False" type="variable">rougail.general.leader.follower1</param>
           </value>
         </variable>
-        <variable doc="follower3" multi="True" name="follower3" type="string">
+        <variable doc="follower3" multi="False" name="follower3" type="string">
           <property>force_default_on_freeze</property>
           <property>frozen</property>
           <property>hidden</property>

tests/flattener_dicos/10leadership_autoleader/00-base.xml

@@ -9,8 +9,8 @@
                 <value>non</value>
             </variable>
             <variable name="leader" type="string" description="leader" multi="True"/>
-            <variable name="follower1" type="string" description="follower1" multi="True" hidden="True"/>
+            <variable name="follower1" type="string" description="follower1" hidden="True"/>
-            <variable name="follower2" type="string" description="follower2" multi="True" hidden="True"/>
+            <variable name="follower2" type="string" description="follower2" hidden="True"/>
         </family>
     </variables>
 

tests/flattener_dicos/10leadership_autoleader/result/00-base.xml

@@ -13,7 +13,7 @@
       <leader doc="leader" name="leader">
         <property>normal</property>
         <variable doc="leader" multi="True" name="leader" type="string"/>
-        <variable doc="follower1" multi="True" name="follower1" type="string">
+        <variable doc="follower1" multi="False" name="follower1" type="string">
           <property>force_default_on_freeze</property>
           <property>frozen</property>
           <property>hidden</property>
@@ -23,7 +23,7 @@
             <param name="valeur" transitive="False" type="string">valfill</param>
           </value>
         </variable>
-        <variable doc="follower2" multi="True" name="follower2" type="string">
+        <variable doc="follower2" multi="False" name="follower2" type="string">
           <property>force_default_on_freeze</property>
           <property>frozen</property>
           <property>hidden</property>

tests/flattener_dicos/10leadership_autoleader_expert/00-base.xml

@@ -11,8 +11,8 @@
         </family>
         <family name="leadermode" mode="expert">
             <variable name="leader" type="string" description="leader" multi="True" hidden="True"/>
-            <variable name="follower1" type="string" description="follower1" multi="True"/>
+            <variable name="follower1" type="string" description="follower1"/>
-            <variable name="follower2" type="string" description="follower2" multi="True"/>
+            <variable name="follower2" type="string" description="follower2"/>
         </family>
     </variables>
 

tests/flattener_dicos/10leadership_autoleader_expert/result/00-base.xml

@@ -24,12 +24,12 @@
             <param name="valeur" transitive="False" type="string">valfill</param>
           </value>
         </variable>
-        <variable doc="follower1" multi="True" name="follower1" type="string">
+        <variable doc="follower1" multi="False" name="follower1" type="string">
           <property>force_default_on_freeze</property>
           <property>frozen</property>
           <property>expert</property>
         </variable>
-        <variable doc="follower2" multi="True" name="follower2" type="string">
+        <variable doc="follower2" multi="False" name="follower2" type="string">
           <property>force_default_on_freeze</property>
           <property>frozen</property>
           <property>expert</property>

tests/flattener_dicos/10leadership_autosaveexpert/00-base.xml

@@ -9,8 +9,8 @@
                 <value>non</value>
             </variable>
             <variable name="leader" type="string" description="leader" multi="True"/>
-            <variable name="follower1" type="string" description="follower1" multi="True"/>
+            <variable name="follower1" type="string" description="follower1"/>
-            <variable name="follower2" type="string" description="follower2" multi="True" mode="expert"/>
+            <variable name="follower2" type="string" description="follower2" mode="expert"/>
         </family>
     </variables>
 

tests/flattener_dicos/10leadership_autosaveexpert/result/00-base.xml

@@ -13,14 +13,14 @@
       <leader doc="leader" name="leader">
         <property>normal</property>
         <variable doc="leader" multi="True" name="leader" type="string"/>
-        <variable doc="follower1" multi="True" name="follower1" type="string">
+        <variable doc="follower1" multi="False" name="follower1" type="string">
           <property>mandatory</property>
           <property>normal</property>
           <value name="calc_val" type="calculation">
             <param name="valeur" transitive="False" type="string">valfill</param>
           </value>
         </variable>
-        <variable doc="follower2" multi="True" name="follower2" type="string">
+        <variable doc="follower2" multi="False" name="follower2" type="string">
           <property>mandatory</property>
           <property>expert</property>
           <value name="calc_val" type="calculation">

tests/flattener_dicos/10leadership_familyaccent/00-base.xml

@@ -9,8 +9,8 @@
                 <value>non</value>
             </variable>
             <variable name="leader" type="string" description="leader" multi="True"/>
-            <variable name="follower1" type="string" description="follower1" multi="True"/>
+            <variable name="follower1" type="string" description="follower1"/>
-            <variable name="follower2" type="string" description="follower2" multi="True"/>
+            <variable name="follower2" type="string" description="follower2"/>
         </family>
     </variables>
 

tests/flattener_dicos/10leadership_familyaccent/result/00-base.xml

@@ -13,14 +13,14 @@
       <leader doc="leader" name="leader">
         <property>normal</property>
         <variable doc="leader" multi="True" name="leader" type="string"/>
-        <variable doc="follower1" multi="True" name="follower1" type="string">
+        <variable doc="follower1" multi="False" name="follower1" type="string">
           <property>mandatory</property>
           <property>normal</property>
           <value name="calc_val" type="calculation">
             <param name="valeur" transitive="False" type="string">valfill</param>
           </value>
         </variable>
-        <variable doc="follower2" multi="True" name="follower2" type="string">
+        <variable doc="follower2" multi="False" name="follower2" type="string">
           <property>mandatory</property>
           <property>normal</property>
           <value name="calc_val" type="calculation">

tests/flattener_dicos/10leadership_leadermandatory/00-base.xml

@@ -9,8 +9,8 @@
                 <value>non</value>
             </variable>
             <variable name="leader" type="string" description="leader" multi="True" mandatory="True"/>
-            <variable name="follower1" type="string" description="follower1" multi="True"/>
+            <variable name="follower1" type="string" description="follower1"/>
-            <variable name="follower2" type="string" description="follower2" multi="True"/>
+            <variable name="follower2" type="string" description="follower2"/>
         </family>
     </variables>
 

tests/flattener_dicos/10leadership_leadermandatory/result/00-base.xml

@@ -15,14 +15,14 @@
         <variable doc="leader" multi="True" name="leader" type="string">
           <property>mandatory</property>
         </variable>
-        <variable doc="follower1" multi="True" name="follower1" type="string">
+        <variable doc="follower1" multi="False" name="follower1" type="string">
           <property>mandatory</property>
           <property>normal</property>
           <value name="calc_val" type="calculation">
             <param name="valeur" transitive="False" type="string">valfill</param>
           </value>
         </variable>
-        <variable doc="follower2" multi="True" name="follower2" type="string">
+        <variable doc="follower2" multi="False" name="follower2" type="string">
           <property>mandatory</property>
           <property>normal</property>
           <value name="calc_val" type="calculation">

tests/flattener_dicos/10leadership_mandatory/00-base.xml

@@ -9,8 +9,8 @@
                 <value>non</value>
             </variable>
             <variable name="leader" type="string" description="leader" multi="True"/>
-            <variable name="follower1" type="string" description="follower1" multi="True" mandatory="True"/>
+            <variable name="follower1" type="string" description="follower1" mandatory="True"/>
-            <variable name="follower2" type="string" description="follower2" multi="True"/>
+            <variable name="follower2" type="string" description="follower2"/>
         </family>
     </variables>
 

tests/flattener_dicos/10leadership_mandatory/result/00-base.xml

@@ -13,14 +13,14 @@
       <leader doc="leader" name="leader">
         <property>normal</property>
         <variable doc="leader" multi="True" name="leader" type="string"/>
-        <variable doc="follower1" multi="True" name="follower1" type="string">
+        <variable doc="follower1" multi="False" name="follower1" type="string">
           <property>mandatory</property>
           <property>normal</property>
           <value name="calc_val" type="calculation">
             <param name="valeur" transitive="False" type="string">valfill</param>
           </value>
         </variable>
-        <variable doc="follower2" multi="True" name="follower2" type="string">
+        <variable doc="follower2" multi="False" name="follower2" type="string">
           <property>mandatory</property>
           <property>normal</property>
           <value name="calc_val" type="calculation">

tests/flattener_dicos/10leadership_mandatoryfollower/00-base.xml

@@ -7,7 +7,7 @@
                 <value>oui</value>
             </variable>
 			<variable name="nut_monitor_netmask" type="netmask" description="Masque de l'IP du réseau de l'esclave" multi="True"/>
-            <variable name="nut_monitor_host" type="network" description="Adresse IP du réseau de l'esclave" mandatory="True" multi="True"/>
+            <variable name="nut_monitor_host" type="network" description="Adresse IP du réseau de l'esclave" mandatory="True"/>
         </family>
         <separators/>
     </variables>

tests/flattener_dicos/10leadership_mandatoryfollower/result/00-base.xml

@@ -16,12 +16,11 @@
       <leader doc="Masque de l'IP du réseau de l'esclave" name="nut_monitor_netmask">
         <property>normal</property>
         <variable doc="Masque de l'IP du réseau de l'esclave" multi="True" name="nut_monitor_netmask" type="netmask"/>
-        <variable doc="Adresse IP du réseau de l'esclave" multi="True" name="nut_monitor_host" type="network">
+        <variable doc="Adresse IP du réseau de l'esclave" multi="False" name="nut_monitor_host" type="network">
           <property>mandatory</property>
           <property>normal</property>
         </variable>
       </leader>
     </family>
-    <separators/>
   </family>
 </rougail>

tests/flattener_dicos/10leadership_multi/00-base.xml

@@ -11,11 +11,11 @@
         </family>
         <family name="general1">
             <variable name="leader" type="string" description="leader" multi="True"/>
-            <variable name="follower1" type="string" description="follower1" multi="True"/>
+            <variable name="follower1" type="string" description="follower1"/>
-            <variable name="follower2" type="string" description="follower2" multi="True"/>
+            <variable name="follower2" type="string" description="follower2"/>
             <variable name="leader1" type="string" description="leader" multi="True"/>
-            <variable name="follower11" type="string" description="follower1" multi="True"/>
+            <variable name="follower11" type="string" description="follower1"/>
-            <variable name="follower21" type="string" description="follower2" multi="True"/>
+            <variable name="follower21" type="string" description="follower2"/>
         </family>
     </variables>
 

tests/flattener_dicos/10leadership_multi/result/00-base.xml

@@ -16,14 +16,14 @@
       <leader doc="leader" name="leader">
         <property>normal</property>
         <variable doc="leader" multi="True" name="leader" type="string"/>
-        <variable doc="follower1" multi="True" name="follower1" type="string">
+        <variable doc="follower1" multi="False" name="follower1" type="string">
           <property>mandatory</property>
           <property>normal</property>
           <value name="calc_val" type="calculation">
             <param name="valeur" transitive="False" type="string">valfill</param>
           </value>
         </variable>
-        <variable doc="follower2" multi="True" name="follower2" type="string">
+        <variable doc="follower2" multi="False" name="follower2" type="string">
           <property>mandatory</property>
           <property>normal</property>
           <value name="calc_val" type="calculation">
@@ -34,10 +34,10 @@
       <leader doc="leader" name="leader1">
         <property>normal</property>
         <variable doc="leader" multi="True" name="leader1" type="string"/>
-        <variable doc="follower1" multi="True" name="follower11" type="string">
+        <variable doc="follower1" multi="False" name="follower11" type="string">
           <property>normal</property>
         </variable>
-        <variable doc="follower2" multi="True" name="follower21" type="string">
+        <variable doc="follower2" multi="False" name="follower21" type="string">
           <property>normal</property>
         </variable>
       </leader>

tests/flattener_dicos/10load_disabled_if_in/result/00-base.xml

@@ -33,6 +33,5 @@
         <value type="string">non</value>
       </variable>
     </family>
-    <separators/>
   </family>
 </rougail>

tests/flattener_dicos/10load_disabled_if_in_none/result/00-base.xml

@@ -31,6 +31,5 @@
         <value type="string">non</value>
       </variable>
     </family>
-    <separators/>
   </family>
 </rougail>

tests/flattener_dicos/10load_disabled_if_inaccent/result/00-base.xml

@@ -41,6 +41,5 @@
         <value type="string">non</value>
       </variable>
     </family>
-    <separators/>
   </family>
 </rougail>

tests/flattener_dicos/10load_disabledifin_fallback/result/00-base.xml

@@ -28,6 +28,5 @@
         <value type="string">non</value>
       </variable>
     </family>
-    <separators/>
   </family>
 </rougail>

tests/flattener_dicos/10load_disabledifin_whithouttype/result/00-base.xml

@@ -27,6 +27,5 @@
         <value type="string">non</value>
       </variable>
     </family>
-    <separators/>
   </family>
 </rougail>

tests/flattener_dicos/10load_emptyvalue/result/00-base.xml

@@ -10,6 +10,5 @@
         <property>normal</property>
       </variable>
     </family>
-    <separators/>
   </family>
 </rougail>

tests/flattener_dicos/10load_frozenifin/result/00-base.xml

@@ -31,6 +31,5 @@
         <value type="string">non</value>
       </variable>
     </family>
-    <separators/>
   </family>
 </rougail>

tests/flattener_dicos/10load_frozenifin_auto/result/00-base.xml

@@ -33,6 +33,5 @@
         <value type="string">non</value>
       </variable>
     </family>
-    <separators/>
   </family>
 </rougail>

tests/flattener_dicos/10load_frozenifin_multiparam/result/00-base.xml

@@ -38,6 +38,5 @@
         <value type="string">non</value>
       </variable>
     </family>
-    <separators/>
   </family>
 </rougail>

tests/flattener_dicos/10load_frozenifin_noexist/result/00-base.xml

@@ -32,6 +32,5 @@
         <value type="string">non</value>
       </variable>
     </family>
-    <separators/>
   </family>
 </rougail>

tests/flattener_dicos/10load_leadership/00-base.xml

@@ -11,8 +11,8 @@
         </family>
         <family name="general1">
             <variable name="leader" type="string" description="leader" multi="True"/>
-            <variable name="follower1" type="string" description="follower1" multi="True"/>
+            <variable name="follower1" type="string" description="follower1"/>
-            <variable name="follower2" type="string" description="follower2" multi="True"/>
+            <variable name="follower2" type="string" description="follower2"/>
         </family>
     </variables>
 

tests/flattener_dicos/10load_leadership/result/00-base.xml

@@ -16,14 +16,14 @@
       <leader doc="leader" name="leader">
         <property>normal</property>
         <variable doc="leader" multi="True" name="leader" type="string"/>
-        <variable doc="follower1" multi="True" name="follower1" type="string">
+        <variable doc="follower1" multi="False" name="follower1" type="string">
           <property>mandatory</property>
           <property>normal</property>
           <value name="calc_val" type="calculation">
             <param name="valeur" transitive="False" type="string">valfill</param>
           </value>
         </variable>
-        <variable doc="follower2" multi="True" name="follower2" type="string">
+        <variable doc="follower2" multi="False" name="follower2" type="string">
           <property>mandatory</property>
           <property>normal</property>
           <value name="calc_val" type="calculation">

tests/flattener_dicos/10load_leadership_default_multi/00-base.xml

@@ -9,10 +9,10 @@
             <value>non</value>
           </variable>
           <variable name="leader" type="string" description="leader" multi="True"/>
-          <variable name="follower1" type="string" description="follower1" multi="True">
+          <variable name="follower1" type="string" description="follower1">
            <value>value</value>
           </variable>
-          <variable name="follower2" type="string" description="follower2" multi="True"/>
+          <variable name="follower2" type="string" description="follower2"/>
         </family>
     </variables>
 

tests/flattener_dicos/10load_leadership_default_multi/result/00-base.xml

@@ -13,12 +13,12 @@
       <leader doc="leader" name="leader">
         <property>normal</property>
         <variable doc="leader" multi="True" name="leader" type="string"/>
-        <variable doc="follower1" multi="True" name="follower1" type="string">
+        <variable doc="follower1" multi="False" name="follower1" type="string">
           <property>mandatory</property>
           <property>normal</property>
           <value>value</value>
         </variable>
-        <variable doc="follower2" multi="True" name="follower2" type="string">
+        <variable doc="follower2" multi="False" name="follower2" type="string">
           <property>normal</property>
         </variable>
       </leader>

tests/flattener_dicos/10load_leadership_default_submulti/00-base.xml

@@ -11,10 +11,10 @@
           <variable name="leader" type="string" description="leader" multi="True">
             <value>leader</value>
           </variable>
-          <variable name="follower1" type="string" description="follower1" submulti="True">
+          <variable name="follower1" type="string" description="follower1" multi="True">
            <value>value</value>
           </variable>
-          <variable name="follower2" type="string" description="follower2" submulti="True">
+          <variable name="follower2" type="string" description="follower2" multi="True">
            <value>value1</value>
            <value>value2</value>
           </variable>

tests/flattener_dicos/10load_leadership_default_submulti/result/00-base.xml

@@ -16,12 +16,12 @@
           <property>mandatory</property>
           <value>leader</value>
         </variable>
-        <variable doc="follower1" multi="submulti" name="follower1" type="string">
+        <variable doc="follower1" multi="True" name="follower1" type="string">
           <property>mandatory</property>
           <property>normal</property>
           <value>value</value>
         </variable>
-        <variable doc="follower2" multi="submulti" name="follower2" type="string">
+        <variable doc="follower2" multi="True" name="follower2" type="string">
           <property>mandatory</property>
           <property>normal</property>
           <value>value1</value>

tests/flattener_dicos/10load_leadership_defaultmulti_leader/00-base.xml

@@ -11,8 +11,8 @@
           <variable name="leader" type="string" description="leader" multi="True">
            <value>value</value>
           </variable>
-	  <variable name="follower1" type="string" description="follower1" multi="True"/>
+	  <variable name="follower1" type="string" description="follower1"/>
-          <variable name="follower2" type="string" description="follower2" multi="True"/>
+          <variable name="follower2" type="string" description="follower2"/>
         </family>
     </variables>
 

tests/flattener_dicos/10load_leadership_defaultmulti_leader/result/00-base.xml

@@ -16,10 +16,10 @@
           <property>mandatory</property>
           <value>value</value>
         </variable>
-        <variable doc="follower1" multi="True" name="follower1" type="string">
+        <variable doc="follower1" multi="False" name="follower1" type="string">
           <property>normal</property>
         </variable>
-        <variable doc="follower2" multi="True" name="follower2" type="string">
+        <variable doc="follower2" multi="False" name="follower2" type="string">
           <property>normal</property>
         </variable>
       </leader>

tests/flattener_dicos/10load_leadership_description/00-base.xml

@@ -8,8 +8,8 @@
         </family>
         <family name='general1'>
             <variable name='leader' type='string' description="leader" multi="True"/>
-            <variable name='follower1' type='string' description="follower1" multi="True"/>
+            <variable name='follower1' type='string' description="follower1"/>
-            <variable name='follower2' type='string' description="follower2" multi="True"/>
+            <variable name='follower2' type='string' description="follower2"/>
         </family>
     </variables>
     <constraints>

tests/flattener_dicos/10load_leadership_description/result/00-base.xml

@@ -16,14 +16,14 @@
       <leader doc="other description" name="leader">
         <property>normal</property>
         <variable doc="leader" multi="True" name="leader" type="string"/>
-        <variable doc="follower1" multi="True" name="follower1" type="string">
+        <variable doc="follower1" multi="False" name="follower1" type="string">
           <property>mandatory</property>
           <property>normal</property>
           <value name="calc_val" type="calculation">
             <param name="valeur" transitive="False" type="string">valfill</param>
           </value>
         </variable>
-        <variable doc="follower2" multi="True" name="follower2" type="string">
+        <variable doc="follower2" multi="False" name="follower2" type="string">
           <property>mandatory</property>
           <property>normal</property>
           <value name="calc_val" type="calculation">

tests/flattener_dicos/10load_leadership_normalize_family/00-base.xml

@@ -11,8 +11,8 @@
         </family>
         <family name="general-1">
             <variable name="leader" type="string" description="leader" multi="True"/>
-            <variable name="follower1" type="string" description="follower1" multi="True"/>
+            <variable name="follower1" type="string" description="follower1"/>
-            <variable name="follower2" type="string" description="follower2" multi="True"/>
+            <variable name="follower2" type="string" description="follower2"/>
         </family>
     </variables>
 

tests/flattener_dicos/10load_leadership_normalize_family/result/00-base.xml

@@ -18,10 +18,10 @@
         <variable doc="leader" multi="True" name="leader" type="string">
           <property>mandatory</property>
         </variable>
-        <variable doc="follower1" multi="True" name="follower1" type="string">
+        <variable doc="follower1" multi="False" name="follower1" type="string">
           <property>normal</property>
         </variable>
-        <variable doc="follower2" multi="True" name="follower2" type="string">
+        <variable doc="follower2" multi="False" name="follower2" type="string">
           <property>normal</property>
         </variable>
       </leader>

tests/flattener_dicos/10load_leadership_submulti/00-base.xml

@@ -11,8 +11,8 @@
         </family>
         <family name="general1">
             <variable name="leader" type="string" description="leader" multi="True"/>
-            <variable name="follower1" type="string" description="follower1" multi="True"/>
+            <variable name="follower1" type="string" description="follower1"/>
-            <variable name="follower2" type="string" description="follower2" submulti="True"/>
+            <variable name="follower2" type="string" description="follower2" multi="True"/>
         </family>
     </variables>
 

tests/flattener_dicos/10load_leadership_submulti/result/00-base.xml

@@ -16,14 +16,14 @@
       <leader doc="leader" name="leader">
         <property>normal</property>
         <variable doc="leader" multi="True" name="leader" type="string"/>
-        <variable doc="follower1" multi="True" name="follower1" type="string">
+        <variable doc="follower1" multi="False" name="follower1" type="string">
           <property>mandatory</property>
           <property>normal</property>
           <value name="calc_val" type="calculation">
             <param name="valeur" transitive="False" type="string">valfill</param>
           </value>
         </variable>
-        <variable doc="follower2" multi="submulti" name="follower2" type="string">
+        <variable doc="follower2" multi="True" name="follower2" type="string">
           <property>mandatory</property>
           <property>normal</property>
           <value name="calc_val" type="calculation">

tests/flattener_dicos/10load_mandatoryifin/result/00-base.xml

@@ -27,6 +27,5 @@
         <value type="string">non</value>
       </variable>
     </family>
-    <separators/>
   </family>
 </rougail>

tests/flattener_dicos/10load_multivalue/result/00-base.xml

@@ -15,6 +15,5 @@
         <value type="string">oui</value>
       </variable>
     </family>
-    <separators/>
   </family>
 </rougail>

tests/flattener_dicos/10load_submulti_two_value/00-base.xml

@@ -1,23 +0,0 @@
-<?xml version='1.0' encoding='UTF-8'?>
-<rougail>
-
-    <services/>
-
-    <variables>
-        <family name="general">
-            <variable name="mode_conteneur_actif" type="oui/non" description="Description" submulti="True">
-                <value>non</value>
-                <value>oui</value>
-            </variable>
-        </family>
-        <separators/>
-    </variables>
-
-    <constraints>
-    </constraints>
-
-    <help/>
-
-</rougail>
-<!-- vim: ts=4 sw=4 expandtab
--->

tests/flattener_dicos/10load_submulti_two_value/makedict/base.json

@@ -1 +0,0 @@
-{"rougail.general.mode_conteneur_actif": [["non"], ["oui"]]}

tests/flattener_dicos/10load_submulti_two_value/result/00-base.xml

@@ -1,17 +0,0 @@
-<?xml version='1.0' encoding='UTF-8'?>
-<rougail>
-  <family doc="rougail" name="rougail">
-    <family doc="general" name="general">
-      <property>normal</property>
-      <variable doc="Description" multi="submulti" name="mode_conteneur_actif" type="choice">
-        <choice type="string">oui</choice>
-        <choice type="string">non</choice>
-        <property>mandatory</property>
-        <property>normal</property>
-        <value type="string">non</value>
-        <value type="string">oui</value>
-      </variable>
-    </family>
-    <separators/>
-  </family>
-</rougail>

tests/flattener_dicos/10valid_enum_accent/result/00-base.xml

@@ -23,6 +23,5 @@
         <value type="string">c</value>
       </variable>
     </family>
-    <separators/>
   </family>
 </rougail>

tests/flattener_dicos/10valid_enum_base/result/00-base.xml

@@ -22,6 +22,5 @@
         <value type="string">c</value>
       </variable>
     </family>
-    <separators/>
   </family>
 </rougail>

tests/flattener_dicos/10valid_enum_base_redefine/result/00-base.xml

@@ -29,6 +29,5 @@
         <value type="string">c</value>
       </variable>
     </family>
-    <separators/>
   </family>
 </rougail>

tests/flattener_dicos/10valid_enum_checkval_true/result/00-base.xml

@@ -12,6 +12,5 @@
         <value type="string">a</value>
       </variable>
     </family>
-    <separators/>
   </family>
 </rougail>

tests/flattener_dicos/10valid_enum_eosfunc/result/00-base.xml

@@ -11,6 +11,5 @@
         <value type="string">non</value>
       </variable>
     </family>
-    <separators/>
   </family>
 </rougail>

tests/flattener_dicos/10valid_enum_leader/00-base.xml

@@ -9,8 +9,8 @@
                 <value>non</value>
             </variable>
             <variable name="leader" type="string" description="leader" multi="True"/>
-            <variable name="follower1" type="string" description="follower1" multi="True"/>
+            <variable name="follower1" type="string" description="follower1"/>
-            <variable name="follower2" type="string" description="follower2" multi="True"/>
+            <variable name="follower2" type="string" description="follower2"/>
         </family>
         <separators/>
     </variables>

tests/flattener_dicos/10valid_enum_leader/result/00-base.xml

@@ -13,7 +13,7 @@
       <leader doc="leader" name="leader">
         <property>normal</property>
         <variable doc="leader" multi="True" name="leader" type="string"/>
-        <variable doc="follower1" multi="True" name="follower1" type="choice">
+        <variable doc="follower1" multi="False" name="follower1" type="choice">
           <choice type="string">a</choice>
           <choice type="string">b</choice>
           <choice type="string">c</choice>
@@ -21,11 +21,10 @@
           <property>normal</property>
           <value type="string">a</value>
         </variable>
-        <variable doc="follower2" multi="True" name="follower2" type="string">
+        <variable doc="follower2" multi="False" name="follower2" type="string">
           <property>normal</property>
         </variable>
       </leader>
     </family>
-    <separators/>
   </family>
 </rougail>

tests/flattener_dicos/10valid_enum_mandatory/result/00-base.xml

@@ -22,6 +22,5 @@
         <value type="string">a</value>
       </variable>
     </family>
-    <separators/>
   </family>
 </rougail>

tests/flattener_dicos/10valid_enum_multi/result/00-base.xml

@@ -19,6 +19,5 @@
         <value type="string">a</value>
       </variable>
     </family>
-    <separators/>
   </family>
 </rougail>

tests/flattener_dicos/10valid_enum_none/result/00-base.xml

@@ -22,6 +22,5 @@
         <value type="string">b</value>
       </variable>
     </family>
-    <separators/>
   </family>
 </rougail>

tests/flattener_dicos/10valid_enum_number/result/00-base.xml

@@ -22,6 +22,5 @@
         <value type="number">1</value>
       </variable>
     </family>
-    <separators/>
   </family>
 </rougail>

tests/flattener_dicos/10valid_enum_numberdefault/result/00-base.xml

@@ -22,6 +22,5 @@
         <value type="number">3</value>
       </variable>
     </family>
-    <separators/>
   </family>
 </rougail>

tests/flattener_dicos/10valid_enum_python/result/00-base.xml

@@ -20,6 +20,5 @@
         <value type="string">test</value>
       </variable>
     </family>
-    <separators/>
   </family>
 </rougail>

tests/flattener_dicos/10valid_enum_value/result/00-base.xml

@@ -12,6 +12,5 @@
         <value type="string">b</value>
       </variable>
     </family>
-    <separators/>
   </family>
 </rougail>

tests/flattener_dicos/11disabled_if_in_filelist/result/00-base.xml

@@ -54,6 +54,5 @@
         <property expected="oui" inverse="False" source="rougail.general.condition" type="calculation">disabled</property>
       </variable>
     </family>
-    <separators/>
   </family>
 </rougail>