Merge pull request #2136 from jestabro/with-defaults

T5319: remove workarounds for incorrect defaults in config-mode scripts

2 files changed, 60 insertions, 59 deletions

diff --git a/python/vyos/xml_ref/__init__.py b/python/vyos/xml_ref/__init__.py
index ad2130dca..02bbaffd8 100644
--- a/python/vyos/xml_ref/__init__.py
+++ b/python/vyos/xml_ref/__init__.py
@@ -13,8 +13,12 @@
 # You should have received a copy of the GNU Lesser General Public License
 # along with this library.  If not, see <http://www.gnu.org/licenses/>.
 
+from typing import Optional, Union, TYPE_CHECKING
 from vyos.xml_ref import definition
 
+if TYPE_CHECKING:
+    from vyos.config import ConfigDict
+
 def load_reference(cache=[]):
     if cache:
         return cache[0]
@@ -48,12 +52,12 @@ def is_leaf(path: list) -> bool:
 def cli_defined(path: list, node: str, non_local=False) -> bool:
     return load_reference().cli_defined(path, node, non_local=non_local)
 
-def from_source(d: dict, path: list) -> bool:
-    return load_reference().from_source(d, path)
-
 def component_version() -> dict:
     return load_reference().component_version()
 
+def default_value(path: list) -> Optional[Union[str, list]]:
+    return load_reference().default_value(path)
+
 def multi_to_list(rpath: list, conf: dict) -> dict:
     return load_reference().multi_to_list(rpath, conf)
 
@@ -68,8 +72,8 @@ def relative_defaults(rpath: list, conf: dict, get_first_key=False,
                                               get_first_key=get_first_key,
                                               recursive=recursive)
 
-def merge_defaults(path: list, conf: dict, get_first_key=False,
-                   recursive=False) -> dict:
-    return load_reference().merge_defaults(path, conf,
-                                           get_first_key=get_first_key,
-                                           recursive=recursive)
+def from_source(d: dict, path: list) -> bool:
+    return definition.from_source(d, path)
+
+def ext_dict_merge(source: dict, destination: Union[dict, 'ConfigDict']):
+    return definition.ext_dict_merge(source, destination)
diff --git a/python/vyos/xml_ref/definition.py b/python/vyos/xml_ref/definition.py
index d95d580e2..38e07f0a7 100644
--- a/python/vyos/xml_ref/definition.py
+++ b/python/vyos/xml_ref/definition.py
@@ -20,6 +20,45 @@ from typing import Optional, Union, Any, TYPE_CHECKING
 if TYPE_CHECKING:
     from vyos.config import ConfigDict
 
+def set_source_recursive(o: Union[dict, str, list], b: bool):
+    d = {}
+    if not isinstance(o, dict):
+        d = {'_source': b}
+    else:
+        for k, v in o.items():
+            d[k] = set_source_recursive(v, b)
+        d |= {'_source': b}
+    return d
+
+def source_dict_merge(src: dict, dest: dict):
+    from copy import deepcopy
+    dst = deepcopy(dest)
+    from_src = {}
+
+    for key, value in src.items():
+        if key not in dst:
+            dst[key] = value
+            from_src[key] = set_source_recursive(value, True)
+        elif isinstance(src[key], dict):
+            dst[key], f = source_dict_merge(src[key], dst[key])
+            f |= {'_source': False}
+            from_src[key] = f
+
+    return dst, from_src
+
+def ext_dict_merge(src: dict, dest: Union[dict, 'ConfigDict']):
+    d, f = source_dict_merge(src, dest)
+    if hasattr(d, '_from_defaults'):
+        setattr(d, '_from_defaults', f)
+    return d
+
+def from_source(d: dict, path: list) -> bool:
+    for key in path:
+        d  = d[key] if key in d else {}
+        if not d or not isinstance(d, dict):
+            return False
+    return d.get('_source', False)
+
 class Xml:
     def __init__(self):
         self.ref = {}
@@ -153,6 +192,15 @@ class Xml:
             return default.split()
         return default
 
+    def default_value(self, path: list) -> Optional[Union[str, list]]:
+        d = self._get_ref_path(path)
+        default = self._get_default_value(d)
+        if default is None:
+            return None
+        if self._is_multi_node(d) or self._is_tag_node(d):
+            return default.split()
+        return default
+
     def get_defaults(self, path: list, get_first_key=False, recursive=False) -> dict:
         """Return dict containing default values below path
 
@@ -212,43 +260,6 @@ class Xml:
             return False
         return True
 
-    def _set_source_recursive(self, o: Union[dict, str, list], b: bool):
-        d = {}
-        if not isinstance(o, dict):
-            d = {'_source': b}
-        else:
-            for k, v in o.items():
-                d[k] = self._set_source_recursive(v, b)
-            d |= {'_source': b}
-        return d
-
-    # use local copy of function in module configdict, to avoid circular
-    # import
-    #
-    # extend dict_merge to keep track of keys only in source
-    def _dict_merge(self, source, destination):
-        from copy import deepcopy
-        dest = deepcopy(destination)
-        from_source = {}
-
-        for key, value in source.items():
-            if key not in dest:
-                dest[key] = value
-                from_source[key] = self._set_source_recursive(value, True)
-            elif isinstance(source[key], dict):
-                dest[key], f = self._dict_merge(source[key], dest[key])
-                f |= {'_source': False}
-                from_source[key] = f
-
-        return dest, from_source
-
-    def from_source(self, d: dict, path: list) -> bool:
-        for key in path:
-            d  = d[key] if key in d else {}
-            if not d or not isinstance(d, dict):
-                return False
-        return d.get('_source', False)
-
     def _relative_defaults(self, rpath: list, conf: dict, recursive=False) -> dict:
         res: dict = {}
         res = self.get_defaults(rpath, recursive=recursive,
@@ -289,17 +300,3 @@ class Xml:
                 res = {}
 
         return res
-
-    def merge_defaults(self, path: list, conf: Union[dict, 'ConfigDict'],
-                       get_first_key=False, recursive=False) -> dict:
-        """Return config dict with defaults non-destructively merged
-
-        This merges non-recursive defaults relative to the config dict.
-        """
-        d = self.relative_defaults(path, conf, get_first_key=get_first_key,
-                                   recursive=recursive)
-        d, f = self._dict_merge(d, conf)
-        d = type(conf)(d)
-        if hasattr(d, '_from_defaults'):
-            setattr(d, '_from_defaults', f)
-        return d