pyegeria 5.3.9.9.7__py3-none-any.whl → 5.4.0__py3-none-any.whl

md_processing/md_processing_utils/common_md_proc_utils.py

@@ -5,13 +5,12 @@ import os
 import sys
 from typing import List
 
-from rich.console import Console
+from loguru import logger
 from rich import print
+from rich.console import Console
 
-
-
-from md_processing.md_processing_utils.common_md_utils import (get_current_datetime_string, print_msg,
-                                                               get_element_dictionary, update_element_dictionary,
+from md_processing.md_processing_utils.common_md_utils import (get_current_datetime_string, get_element_dictionary,
+                                                               update_element_dictionary,
                                                                split_tb_string, str_to_bool, )
 from md_processing.md_processing_utils.extraction_utils import (process_simple_attribute, extract_attribute,
                                                                 get_element_by_name)
@@ -20,6 +19,11 @@ from md_processing.md_processing_utils.message_constants import (ERROR, INFO, WA
 from pyegeria import EgeriaTech
 from pyegeria._globals import DEBUG_LEVEL
 
+log_format = "P {time} | {level} | {function} | {line} | {message} | {extra}"
+logger.remove()
+logger.add(sys.stderr, level="INFO", format=log_format, colorize=True)
+logger.add("debug_log.log", rotation="1 day", retention="1 week", compression="zip", level="TRACE", format=log_format,
+           colorize=True)
 # Constants
 EGERIA_WIDTH = int(os.environ.get("EGERIA_WIDTH", "200"))
 EGERIA_USAGE_LEVEL = os.environ.get("EGERIA_USAGE_LEVEL", "Basic")
@@ -29,6 +33,7 @@ debug_level = DEBUG_LEVEL
 global COMMAND_DEFINITIONS
 
 
+@logger.catch
 def process_provenance_command(file_path: str, txt: [str]) -> str:
     """
     Processes a provenance object_action by extracting the file path and current datetime.
@@ -46,7 +51,8 @@ def process_provenance_command(file_path: str, txt: [str]) -> str:
     return provenance
 
 
-def parse_user_command(egeria_client: EgeriaTech, object_type: str, object_action: str, txt: str,
+@logger.catch
+def parse_upsert_command(egeria_client: EgeriaTech, object_type: str, object_action: str, txt: str,
                        directive: str = "display") -> dict:
     parsed_attributes, parsed_output = {}, {}
 
@@ -56,7 +62,7 @@ def parse_user_command(egeria_client: EgeriaTech, object_type: str, object_actio
     display_name = ""
     labels = {}
 
-    command_spec = get_command_spec(object_type)
+    command_spec = get_command_spec(f"Create {object_type}")
     attributes = command_spec.get('Attributes', [])
     command_display_name = command_spec.get('display_name', None)
     command_qn_prefix = command_spec.get('qn_prefix', None)
@@ -69,7 +75,7 @@ def parse_user_command(egeria_client: EgeriaTech, object_type: str, object_actio
     parsed_output['reason'] = ""
 
     msg = f"\tProcessing {object_action} on a {object_type}  \n"
-    print_msg(ALWAYS, msg, debug_level)
+    logger.info(msg)
 
     # get the version early because we may need it to construct qualified names.
     version = process_simple_attribute(txt, {'Version', "Version Identifier", "Published Version"}, INFO)
@@ -81,29 +87,32 @@ def parse_user_command(egeria_client: EgeriaTech, object_type: str, object_actio
             for_update = attr[key].get('inUpdate', True)
             level = attr[key].get('level', 'Basic')
             msg = (f"___\nProcessing `{key}` in `{object_action}` on a `{object_type}` "
-                  f"\n\twith usage level: `{EGERIA_USAGE_LEVEL}` and  attribute level `{level}` and for_update `{for_update}`\n")
-            print_msg(INFO, msg, debug_level)
+                   f"\n\twith usage level: `{EGERIA_USAGE_LEVEL}` and  attribute level `{level}` and for_update `"
+                   f"{for_update}`\n")
+            logger.trace(msg)
             if for_update is False and object_action == "Update":
-                console.print(f"Attribute `{key}`is not allowed for `Update`", highlight=True)
+                logger.trace(f"Attribute `{key}`is not allowed for `Update`", highlight=True)
                 continue
             if EGERIA_USAGE_LEVEL == "Basic" and level != "Basic":
-                console.print(f"Attribute `{key}` is not supported for `{EGERIA_USAGE_LEVEL}` usage level. Skipping.",  highlight=True)
+                logger.trace(f"Attribute `{key}` is not supported for `{EGERIA_USAGE_LEVEL}` usage level. Skipping.",
+                            highlight=True)
                 continue
-            if EGERIA_USAGE_LEVEL == "Advanced" and level in [ "Expert", "Invisible"]:
-                console.print(f"Attribute `{key}` is not supported for `{EGERIA_USAGE_LEVEL}` usage level. Skipping.",  highlight=True)
+            if EGERIA_USAGE_LEVEL == "Advanced" and level in ["Expert", "Invisible"]:
+                logger.trace(f"Attribute `{key}` is not supported for `{EGERIA_USAGE_LEVEL}` usage level. Skipping.",
+                            highlight=True)
                 continue
             if EGERIA_USAGE_LEVEL == "Expert" and level == "Invisible":
-                console.print(f"Attribute `{key}` is not supported for `{EGERIA_USAGE_LEVEL}` usage level. Skipping.",  highlight=True)
+                logger.trace(f"Attribute `{key}` is not supported for `{EGERIA_USAGE_LEVEL}` usage level. Skipping.",
+                            highlight=True)
                 continue
 
-
             if attr[key].get('input_required', False) is True:
                 if_missing = ERROR
             else:
                 if_missing = INFO
 
             # lab = [item.strip() for item in re.split(r'[;,\n]+',attr[key]['attr_labels'])]
-            lab =split_tb_string(attr[key]['attr_labels'] )
+            lab = split_tb_string(attr[key]['attr_labels'])
             labels: set = set()
             labels.add(key.strip())
             if key == 'Display Name':
@@ -111,7 +120,6 @@ def parse_user_command(egeria_client: EgeriaTech, object_type: str, object_actio
             if lab is not None and lab != [""]:
                 labels.update(lab)
 
-
             default_value = attr[key].get('default_value', None)
 
             style = attr[key]['style']
@@ -124,7 +132,8 @@ def parse_user_command(egeria_client: EgeriaTech, object_type: str, object_actio
             elif style == 'QN':
                 parsed_attributes[key] = proc_el_id(egeria_client, command_display_name, command_qn_prefix, labels, txt,
                                                     object_action, version, if_missing)
-                if key == 'Qualified Name' and parsed_attributes[key]['value'] and parsed_attributes[key]['exists'] is False:
+                if key == 'Qualified Name' and parsed_attributes[key]['value'] and parsed_attributes[key][
+                    'exists'] is False:
                     parsed_output['exists'] = False
             elif style == 'ID':
                 parsed_attributes[key] = proc_el_id(egeria_client, command_display_name, command_qn_prefix, labels, txt,
@@ -139,19 +148,22 @@ def parse_user_command(egeria_client: EgeriaTech, object_type: str, object_actio
 
             elif style == 'Reference Name':
                 parsed_attributes[key] = proc_ids(egeria_client, key, labels, txt, object_action, if_missing)
-                if ((if_missing==ERROR) and parsed_attributes[key].get("value", None) and
-                        parsed_attributes[key]['exists'] is False):
+                if ((if_missing == ERROR) and parsed_attributes[key].get("value", None) and parsed_attributes[key][
+                    'exists'] is False):
                     msg = f"Reference Name `{parsed_attributes[key]['value']}` is specified but does not exist"
-                    print_msg(ERROR, msg, debug_level)
+                    logger.error(msg)
                     parsed_output['valid'] = False
                     parsed_output['reason'] += msg
+                elif parsed_attributes[key]['valid'] is False:
+                    parsed_output['valid'] = False
+                    parsed_output['reason'] += parsed_attributes[key]['reason']
 
             elif style == 'GUID':
                 parsed_attributes[key] = proc_simple_attribute(txt, object_action, labels, if_missing)
             elif style == 'Ordered Int':
                 parsed_attributes[key] = proc_simple_attribute(txt, object_action, labels, if_missing)
             elif style == 'Simple Int':
-                parsed_attributes[key] = proc_simple_attribute(txt, object_action, labels, if_missing, default_value)
+                parsed_attributes[key] = proc_simple_attribute(txt, object_action, labels, if_missing, default_value, "int")
             elif style == 'Simple List':
                 parsed_attributes[key] = proc_simple_attribute(txt, object_action, labels, if_missing, default_value)
                 name_list = parsed_attributes[key]['value']
@@ -167,65 +179,239 @@ def parse_user_command(egeria_client: EgeriaTech, object_type: str, object_actio
 
             elif style == 'Reference Name List':
                 parsed_attributes[key] = proc_name_list(egeria_client, key, txt, labels, if_missing)
-
+                if (parsed_attributes[key].get("value", None) and (
+                        parsed_attributes[key]['exists'] is False or parsed_attributes[key]['valid'] is False)):
+                    msg = (f"A Reference Name in `{parsed_attributes[key].get('name_list', None)}` is specified but "
+                           f"does not exist")
+                    logger.error(msg)
+                    parsed_output['valid'] = False
+                    parsed_output['reason'] += msg
             else:
-                msg = f"Unknown attribute style: {style}"
-                print_msg(ERROR, msg, debug_level)
+                msg = f"Unknown attribute style: {style} for key `{key}`"
+                logger.error(msg)
                 sys.exit(1)
                 parsed_attributes[key]['valid'] = False
                 parsed_attributes[key]['value'] = None
             if key == "Display Name":
                 display_name = parsed_attributes[key]['value']
 
-
-
-            value = parsed_attributes[key].get('value',None)
+            value = parsed_attributes[key].get('value', None)
 
             if value is not None:
                 # if the value is a dict, get the flattened name list
-                value = parsed_attributes[key].get('name_list', None) if isinstance(value,(dict, list)) else value
-
+                value = parsed_attributes[key].get('name_list', None) if isinstance(value, (dict, list)) else value
                 parsed_output['display'] += f"\n\t* {key}: `{value}`\n\t"
 
 
+    parsed_output['attributes'] = parsed_attributes
+
+    if display_name is None:
+        msg = f"No display name or name identifier found"
+        logger.error(msg)
+        parsed_output['valid'] = False
+        parsed_output['reason'] = msg
+        return parsed_output
+
+
     if parsed_attributes.get('Parent ID', {}).get('value', None) is not None:
         if (parsed_attributes['Parent Relationship Type Name']['value'] is None) or (
                 parsed_attributes['Parent at End1']['value'] is None):
             msg = "Parent ID was found but either Parent `Relationship Type Name` or `Parent at End1` are missing"
-            print_msg(ERROR, msg, debug_level)
+            logger.error(msg)
             parsed_output['valid'] = False
             parsed_output['reason'] = msg
         if parsed_attributes['Parent Relationship Type Name'].get('exists', False) is False:
             msg = "Parent ID was found but does not exist"
-            print_msg(ERROR, msg, debug_level)
+            logger.error(msg)
             parsed_output['valid'] = False
             parsed_output['reason'] = msg
 
     if directive in ["validate", "process"] and object_action == "Update" and not parsed_output[
         'exists']:  # check to see if provided information exists and is consistent with existing info
-        msg = f"Update request invalid, Term {display_name} does not exist\n"
-        print_msg(ERROR, msg, debug_level)
+        msg = f"Update request invalid, element `{display_name}` does not exist\n"
+        logger.error(msg)
         parsed_output['valid'] = False
     if directive in ["validate", "process"] and not parsed_output['valid'] and object_action == "Update":
         msg = f"Request is invalid, `{object_action} {object_type}` is not valid - see previous messages\n"
-        print_msg(ERROR, msg, debug_level)
+        logger.error(msg)
 
     elif directive in ["validate",
                        "process"] and object_action == 'Create':  # if the object_action is create, check that it
         # doesn't already exist
         if parsed_output['exists']:
             msg = f"Element `{display_name}` cannot be created since it already exists\n"
-            print_msg(ERROR, msg, debug_level)
+            logger.error(msg)
             parsed_output['valid'] = False
         else:
-            msg = f"It is valid to create Element `{display_name}`"
-            print_msg(ALWAYS, msg, debug_level)
+            msg = f"Element `{display_name}` does not exist so it can be created\n"
+            logger.info(msg)
+
+
+    if parsed_output.get('qualified_name',None) and "* Qualified Name" not in parsed_output['display']:
+        parsed_output['display'] += f"\n\t* Qualified Name: `{parsed_output['qualified_name']}`\n\t"
+    if parsed_output.get('guid',None):
+        parsed_output['display'] += f"\n\t* GUID: `{parsed_output['guid']}`\n\t"
+
+    return parsed_output
+
+
+@logger.catch
+def parse_view_command(egeria_client: EgeriaTech, object_type: str, object_action: str, txt: str,
+                         directive: str = "display") -> dict:
+    parsed_attributes, parsed_output = {}, {}
+
+    parsed_output['valid'] = True
+    parsed_output['exists'] = False
+
+    labels = {}
+    if object_action in ["Unlink","Detach"]:
+        command_spec = get_command_spec(f"Link {object_type}")
+    else:
+        command_spec = get_command_spec(f"{object_action} {object_type}")
+    attributes = command_spec.get('Attributes', [])
+    command_display_name = command_spec.get('display_name', None)
+
+    parsed_output['reason'] = ""
+    parsed_output['display'] = ""
+
+    msg = f"\tProcessing {object_action} on  {object_type}  \n"
+    logger.info(msg)
+
+    # get the version early because we may need it to construct qualified names.
+
+    for attr in attributes:
+        for key in attr:
+            # Run some checks to see if the attribute is appropriate to the operation and usage level
+
+            level = attr[key].get('level', 'Basic')
+            msg = (f"___\nProcessing `{key}` in `{object_action}` on a `{object_type}` "
+                   f"\n\twith usage level: `{EGERIA_USAGE_LEVEL}` ")
+            logger.trace(msg)
+
+            if EGERIA_USAGE_LEVEL == "Basic" and level != "Basic":
+                logger.trace(f"Attribute `{key}` is not supported for `{EGERIA_USAGE_LEVEL}` usage level. Skipping.",
+                             highlight=True)
+                continue
+            if EGERIA_USAGE_LEVEL == "Advanced" and level in ["Expert", "Invisible"]:
+                logger.trace(f"Attribute `{key}` is not supported for `{EGERIA_USAGE_LEVEL}` usage level. Skipping.",
+                             highlight=True)
+                continue
+            if EGERIA_USAGE_LEVEL == "Expert" and level == "Invisible":
+                logger.trace(f"Attribute `{key}` is not supported for `{EGERIA_USAGE_LEVEL}` usage level. Skipping.",
+                             highlight=True)
+                continue
+
+            if attr[key].get('input_required', False) is True:
+                if_missing = ERROR
+            else:
+                if_missing = INFO
+
+            # lab = [item.strip() for item in re.split(r'[;,\n]+',attr[key]['attr_labels'])]
+            lab = split_tb_string(attr[key]['attr_labels'])
+            labels: set = set()
+            labels.add(key.strip())
+
+            if lab is not None and lab != [""]:
+                labels.update(lab)
+
+            # set these to none since not needed for view commands
+            version = None
+            command_qn_prefix = None
+
+            default_value = attr[key].get('default_value', None)
+
+            style = attr[key]['style']
+            if style in ['Simple', 'Dictionary', 'Comment']:
+                parsed_attributes[key] = proc_simple_attribute(txt, object_action, labels, if_missing, default_value)
+            elif style == 'Valid Value':
+                parsed_attributes[key] = proc_valid_value(txt, object_action, labels,
+                                                          attr[key].get('valid_values', None), if_missing,
+                                                          default_value)
+            elif style == 'QN':
+                parsed_attributes[key] = proc_el_id(egeria_client, command_display_name, command_qn_prefix, labels, txt,
+                                                    object_action, version, if_missing)
+                if key == 'Qualified Name' and parsed_attributes[key]['value'] and parsed_attributes[key][
+                    'exists'] is False:
+                    parsed_output['exists'] = False
+            elif style == 'ID':
+                parsed_attributes[key] = proc_el_id(egeria_client, command_display_name, command_qn_prefix, labels, txt,
+                                                    object_action, version, if_missing)
+
+                parsed_output['guid'] = parsed_attributes[key].get('guid', None)
+                parsed_output['qualified_name'] = parsed_attributes[key].get('qualified_name', None)
+                parsed_output['exists'] = parsed_attributes[key]['exists']
+                if parsed_attributes[key]['valid'] is False:
+                    parsed_output['valid'] = False
+                    parsed_output['reason'] += parsed_attributes[key]['reason']
+
+            elif style == 'Reference Name':
+                parsed_attributes[key] = proc_ids(egeria_client, key, labels, txt, object_action, if_missing)
+                if ((if_missing == ERROR) and parsed_attributes[key].get("value", None) is None):
+                    msg = f"Required parameter `{parsed_attributes[key]['value']}` is missing"
+                    logger.error(msg)
+                    parsed_output['valid'] = False
+                    parsed_output['reason'] += msg
+                elif parsed_attributes[key]['value'] and parsed_attributes[key]['exists'] is False:
+                    msg = f"Reference Name `{parsed_attributes[key]['value']}` is specified but does not exist"
+                    logger.error(msg)
+                    parsed_output['valid'] = False
+                    parsed_output['reason'] += msg
+
+            elif style == 'GUID':
+                parsed_attributes[key] = proc_simple_attribute(txt, object_action, labels, if_missing)
+            elif style == 'Ordered Int':
+                parsed_attributes[key] = proc_simple_attribute(txt, object_action, labels, if_missing)
+            elif style == 'Simple Int':
+                parsed_attributes[key] = proc_simple_attribute(txt, object_action, labels, if_missing, default_value, "int")
+            elif style == 'Simple List':
+                parsed_attributes[key] = proc_simple_attribute(txt, object_action, labels, if_missing, default_value)
+                name_list = parsed_attributes[key]['value']
+                # attribute = re.split(r'[;,\n]+', name_list) if name_list is not None else None
+                attribute = split_tb_string(name_list)
+                parsed_attributes[key]['value'] = attribute
+                parsed_attributes[key]['name_list'] = name_list
+            elif style == 'Parent':
+                parsed_attributes[key] = proc_simple_attribute(txt, object_action, labels, if_missing, default_value)
+            elif style == 'Bool':
+                parsed_attributes[key] = proc_bool_attribute(txt, object_action, labels, if_missing, default_value)
+
+            elif style == 'Reference Name List':
+                parsed_attributes[key] = proc_name_list(egeria_client, key, txt, labels, if_missing)
+                if (parsed_attributes[key].get("value", None) and (
+                        parsed_attributes[key]['exists'] is False or parsed_attributes[key]['valid'] is False)):
+                    msg = (f"A Reference Name in `{parsed_attributes[key].get('name_list', None)}` is specified but "
+                           f"does not exist")
+                    logger.error(msg)
+                    parsed_output['valid'] = False
+                    parsed_output['reason'] += msg
+            else:
+                msg = f"Unknown attribute style: {style}"
+                logger.error(msg)
+                sys.exit(1)
+                parsed_attributes[key]['valid'] = False
+                parsed_attributes[key]['value'] = None
+
+            value = parsed_attributes[key].get('value', None)
+
+            if value is not None:
+                # if the value is a dict, get the flattened name list
+                value = parsed_attributes[key].get('name_list', None) if isinstance(value, (dict, list)) else value
+                parsed_output['display'] += f"\n\t* {key}: `{value}`\n\t"
 
     parsed_output['attributes'] = parsed_attributes
+
+
+    if directive in ["validate", "process"] and not parsed_output['valid'] and object_action == "Update":
+        msg = f"Request is invalid, `{object_action} {object_type}` is not valid - see previous messages\n"
+        logger.error(msg)
+
+
     return parsed_output
 
 
-def proc_simple_attribute(txt: str, action: str, labels: set, if_missing: str = INFO, default_value=None) -> dict:
+@logger.catch
+def proc_simple_attribute(txt: str, action: str, labels: set, if_missing: str = INFO, default_value=None,
+                          simp_type: str = None) -> dict:
     """Process a simple attribute based on the provided labels and if_missing value.
        Extract the attribute value from the text and store it in a dictionary along with valid.
        If it doesn`t exist, mark the dictionary entry as invalid and print an error message with severity of if_missing.
@@ -245,27 +431,34 @@ def proc_simple_attribute(txt: str, action: str, labels: set, if_missing: str =
 
     if if_missing not in ["WARNING", "ERROR", "INFO"]:
         msg = "Invalid severity for missing attribute"
-        print_msg("ERROR", msg, debug_level)
+        logger.error(msg)
         return {"status": ERROR, "reason": msg, "value": None, "valid": False}
 
-    attribute = extract_attribute(txt, labels)
     if default_value == "":
         default_value = None
-    attribute = default_value if attribute is None else attribute
 
-    if attribute is None:
+    attribute = extract_attribute(txt, labels)
+
+    attribute = default_value if attribute is None else attribute.replace('\n', '')
 
+    if attribute is None:
         if if_missing == INFO or if_missing == WARNING:
             msg = f"Optional attribute with labels: `{labels}` missing"
             valid = True
+            logger.info(msg)
         else:
             msg = f"Missing attribute with labels `{labels}` "
             valid = False
-        print_msg(if_missing, msg, debug_level)
+            logger.error(msg)
         return {"status": if_missing, "reason": msg, "value": None, "valid": valid, "exists": False}
+
+    if attribute and simp_type == "int" :
+        attribute = int(attribute)
+
     return {"status": INFO, "OK": None, "value": attribute, "valid": valid, "exists": True}
 
 
+@logger.catch
 def proc_valid_value(txt: str, action: str, labels: set, valid_values: [], if_missing: str = INFO,
                      default_value=None) -> dict:
     """Process a string attribute to check that it is a member of the associated value values list.
@@ -288,22 +481,22 @@ def proc_valid_value(txt: str, action: str, labels: set, valid_values: [], if_mi
 
     if if_missing not in ["WARNING", "ERROR", "INFO"]:
         msg = "Invalid severity for missing attribute"
-        print_msg("ERROR", msg, debug_level)
+        logger.error(msg)
         return {"status": ERROR, "reason": msg, "value": None, "valid": False}
     if valid_values is None:
         msg = "Missing valid values list"
-        print_msg("ERROR", msg, debug_level)
+        logger.error(msg)
         return {"status": WARNING, "reason": msg, "value": None, "valid": False}
     if isinstance(valid_values, str):
         # v_values = [item.strip() for item in re.split(r'[;,\n]+', valid_values)]
         v_values = split_tb_string(valid_values)
     if not isinstance(v_values, list):
         msg = "Valid values list is not a list"
-        print_msg("ERROR", msg, debug_level)
+        logger.error(msg)
         return {"status": WARNING, "reason": msg, "value": None, "valid": False}
     if len(v_values) == 0:
         msg = "Valid values list is empty"
-        print_msg("ERROR", msg, debug_level)
+        logger.error(msg)
         return {"status": WARNING, "reason": msg, "value": None, "valid": False}
 
     attribute = extract_attribute(txt, labels)
@@ -314,21 +507,23 @@ def proc_valid_value(txt: str, action: str, labels: set, valid_values: [], if_mi
     if attribute is None:
         if if_missing == INFO or if_missing == WARNING:
             msg = f"Optional attribute with labels: `{labels}` missing"
+            logger.info(msg)
             valid = True
         else:
             msg = f"Missing attribute with labels `{labels}` "
             valid = False
-        print_msg(if_missing, msg, debug_level)
+            logger.error(msg)
         return {"status": if_missing, "reason": msg, "value": None, "valid": valid, "exists": False}
     else:
         if attribute not in v_values:
             msg = f"Invalid value for attribute `{labels}`"
-            print_msg(WARNING, msg, debug_level)
+            logger.warning(msg)
             return {"status": WARNING, "reason": msg, "value": attribute, "valid": False, "exists": True}
 
     return {"status": INFO, "OK": "OK", "value": attribute, "valid": valid, "exists": True}
 
 
+@logger.catch
 def proc_bool_attribute(txt: str, action: str, labels: set, if_missing: str = INFO, default_value=None) -> dict:
     """Process a boolean attribute based on the provided labels and if_missing value.
        Extract the attribute value from the text and store it in a dictionary along with valid.
@@ -349,7 +544,7 @@ def proc_bool_attribute(txt: str, action: str, labels: set, if_missing: str = IN
 
     if if_missing not in ["WARNING", "ERROR", "INFO"]:
         msg = "Invalid severity for missing attribute"
-        print_msg("ERROR", msg, debug_level)
+        logger.error(msg)
         return {"status": ERROR, "reason": msg, "value": None, "valid": False}
 
     attribute = extract_attribute(txt, labels)
@@ -362,11 +557,12 @@ def proc_bool_attribute(txt: str, action: str, labels: set, if_missing: str = IN
     if attribute is None:
         if if_missing == INFO or if_missing == WARNING:
             msg = f"Optional attribute with labels: `{labels}` missing"
+            logger.info(msg)
             valid = True
         else:
             msg = f"Missing attribute with labels `{labels}` "
             valid = False
-        print_msg(if_missing, msg, debug_level)
+            logger.error(msg)
         return {"status": if_missing, "reason": msg, "value": None, "valid": valid, "exists": False}
 
     if isinstance(attribute, str):
@@ -377,12 +573,13 @@ def proc_bool_attribute(txt: str, action: str, labels: set, if_missing: str = IN
             attribute = False
         else:
             msg = f"Invalid value for boolean attribute `{labels}`"
-            print_msg("ERROR", msg, debug_level)
+            logger.error(msg)
             return {"status": ERROR, "reason": msg, "value": attribute, "valid": False, "exists": True}
 
     return {"status": INFO, "OK": None, "value": attribute, "valid": valid, "exists": True}
 
 
+@logger.catch
 def proc_el_id(egeria_client: EgeriaTech, element_type: str, qn_prefix: str, element_labels: list[str], txt: str,
                action: str, version: str = None, if_missing: str = INFO) -> dict:
     """
@@ -422,7 +619,7 @@ def proc_el_id(egeria_client: EgeriaTech, element_type: str, qn_prefix: str, ele
 
     if element_name is None:
         msg = f"Optional attribute with label`{element_type}` missing"
-        print_msg("INFO", msg, debug_level)
+        logger.info(msg)
         identifier_output = {"status": INFO, "reason": msg, "value": None, "valid": False, "exists": False, }
         return identifier_output
 
@@ -436,18 +633,18 @@ def proc_el_id(egeria_client: EgeriaTech, element_type: str, qn_prefix: str, ele
 
     if unique is False:
         msg = f"Multiple elements named  {element_name} found"
-        print_msg("DEBUG-ERROR", msg, debug_level)
+        logger.error(msg)
         identifier_output = {"status": ERROR, "reason": msg, "value": element_name, "valid": False, "exists": True, }
         valid = False
 
     if action == "Update" and not exists:
         msg = f"Element {element_name} does not exist"
-        print_msg("DEBUG-ERROR", msg, debug_level)
+        logger.error(msg)
         identifier_output = {"status": ERROR, "reason": msg, "value": element_name, "valid": False, "exists": False, }
 
-    elif action == "Update" and exists:
+    elif action in ["Update", "View"] and exists:
         msg = f"Element {element_name} exists"
-        print_msg("DEBUG-INFO", msg, debug_level)
+        logger.info(msg)
         identifier_output = {
             "status": INFO, "reason": msg, "value": element_name, "valid": True, "exists": True,
             'qualified_name': q_name, 'guid': guid
@@ -455,7 +652,7 @@ def proc_el_id(egeria_client: EgeriaTech, element_type: str, qn_prefix: str, ele
 
     elif action == "Create" and exists:
         msg = f"Element {element_name} already exists"
-        print_msg("DEBUG-ERROR", msg, debug_level)
+        logger.error(msg)
         identifier_output = {
             "status": ERROR, "reason": msg, "value": element_name, "valid": False, "exists": True,
             'qualified_name': qualified_name, 'guid': guid,
@@ -463,7 +660,7 @@ def proc_el_id(egeria_client: EgeriaTech, element_type: str, qn_prefix: str, ele
 
     elif action == "Create" and not exists and valid:
         msg = f"{element_type} `{element_name}` does not exist"
-        print_msg("DEBUG-INFO", msg, debug_level)
+        logger.info(msg)
 
         if q_name is None and qualified_name is None:
             q_name = egeria_client.__create_qualified_name__(qn_prefix, element_name, version_identifier=version)
@@ -481,11 +678,13 @@ def proc_el_id(egeria_client: EgeriaTech, element_type: str, qn_prefix: str, ele
     return identifier_output
 
 
+@logger.catch
 def proc_ids(egeria_client: EgeriaTech, element_type: str, element_labels: set, txt: str, action: str,
              if_missing: str = INFO, version: str = None) -> dict:
     """
     Processes element identifiers from the input text using the labels supplied,
     checking if the element exists in Egeria, and validating the information.
+    Only a single element is allowed.
 
     Parameters
     ----------
@@ -519,7 +718,12 @@ def proc_ids(egeria_client: EgeriaTech, element_type: str, element_labels: set,
     value = None
 
     element_name = extract_attribute(txt, element_labels)
+
     if element_name:
+        if '\n' in element_name or ',' in element_name:
+            msg = f"Element name `{element_name}` appears to be a list rather than a single element"
+            logger.error(msg)
+            return {"status": ERROR, "reason": msg, "value": None, "valid": False, "exists": False, }
         q_name, guid, unique, exists = get_element_by_name(egeria_client, element_type, element_name)
         value = element_name
     else:
@@ -530,30 +734,30 @@ def proc_ids(egeria_client: EgeriaTech, element_type: str, element_labels: set,
     if exists is True and unique is False:
         # Multiple elements found - so need to respecify with qualified name
         msg = f"Multiple elements named  {element_name} found"
-        print_msg("DEBUG-ERROR", msg, debug_level)
+        logger.error(msg)
         identifier_output = {"status": ERROR, "reason": msg, "value": element_name, "valid": False, "exists": True, }
 
 
     elif action == EXISTS_REQUIRED or if_missing == ERROR and not exists:
         # a required identifier doesn't exist
         msg = f"Required {element_type} `{element_name}` does not exist"
-        print_msg("DEBUG-ERROR", msg, debug_level)
+        logger.error(msg)
         identifier_output = {"status": ERROR, "reason": msg, "value": element_name, "valid": False, "exists": False, }
-    elif value is None  and if_missing == INFO:
+    elif value is None and if_missing == INFO:
         # an optional identifier is empty
         msg = f"Optional attribute with label`{element_type}` missing"
-        print_msg("INFO", msg, debug_level)
+        logger.info(msg)
         identifier_output = {"status": INFO, "reason": msg, "value": None, "valid": True, "exists": False, }
     elif value and exists is False:
         # optional identifier specified but doesn't exist
         msg = f"Optional attribute with label`{element_type}` specified but doesn't exist"
-        print_msg("ERROR", msg, debug_level)
+        logger.error(msg)
         identifier_output = {"status": ERROR, "reason": msg, "value": value, "valid": False, "exists": False, }
 
     else:
         # all good.
         msg = f"Element {element_type} `{element_name}` exists"
-        print_msg("DEBUG-INFO", msg, debug_level)
+        logger.info(msg)
         identifier_output = {
             "status": INFO, "reason": msg, "value": element_name, "valid": True, "exists": True,
             "qualified_name": q_name, 'guid': guid
@@ -562,6 +766,7 @@ def proc_ids(egeria_client: EgeriaTech, element_type: str, element_labels: set,
     return identifier_output
 
 
+@logger.catch
 def proc_name_list(egeria_client: EgeriaTech, element_type: str, txt: str, element_labels: set,
                    if_missing: str = INFO) -> dict:
     """
@@ -595,12 +800,12 @@ def proc_name_list(egeria_client: EgeriaTech, element_type: str, txt: str, eleme
     id_list_output = {}
     elements = ""
     new_element_list = []
-
+    guid_list = []
     elements_txt = extract_attribute(txt, element_labels)
 
     if elements_txt is None:
         msg = f"Attribute with labels `{{element_type}}` missing"
-        print_msg("DEBUG-INFO", msg, debug_level)
+        logger.debug(msg)
         return {"status": if_missing, "reason": msg, "value": None, "valid": False, "exists": False, }
     else:
         # element_list = re.split(r'[;,\n]+', elements_txt)
@@ -610,13 +815,14 @@ def proc_name_list(egeria_client: EgeriaTech, element_type: str, txt: str, eleme
             # Get the element using the generalized function
             known_q_name, known_guid, el_valid, el_exists = get_element_by_name(egeria_client, element_type, element)
             details = {"known_q_name": known_q_name, "known_guid": known_guid, "el_valid": el_valid}
-            elements = f"{element} {elements}"  # list of the input names
+            elements += element + ", "  # list of the input names
 
             if el_exists and el_valid:
                 new_element_list.append(known_q_name)  # list of qualified names
+                guid_list.append(known_guid)
             elif not el_exists:
                 msg = f"No {element_type} `{element}` found"
-                print_msg("DEBUG-INFO", msg, debug_level)
+                logger.debug(msg)
                 valid = False
             valid = valid if el_valid is None else (valid and el_valid)
             exists = exists and el_exists
@@ -624,19 +830,22 @@ def proc_name_list(egeria_client: EgeriaTech, element_type: str, txt: str, eleme
 
         if elements:
             msg = f"Found {element_type}: {elements}"
-            print_msg("DEBUG-INFO", msg, debug_level)
+            logger.debug(msg)
             id_list_output = {
                 "status": INFO, "reason": msg, "value": element_details, "valid": valid, "exists": exists,
-                "name_list": new_element_list,
+                "name_list": new_element_list, "guid_list": guid_list,
                 }
         else:
             msg = f" Name list contains one or more invalid qualified names."
-            print_msg("DEBUG-INFO", msg, debug_level)
-            id_list_output = {"status": if_missing, "reason": msg, "value": elements, "valid": valid,
-                              "exists": exists, "dict_list" : element_details}
+            logger.debug(msg)
+            id_list_output = {
+                "status": if_missing, "reason": msg, "value": elements, "valid": valid, "exists": exists,
+                "dict_list": element_details, "guid_list": guid_list
+                }
         return id_list_output
 
 
+@logger.catch
 def update_term_categories(egeria_client: EgeriaTech, term_guid: str, categories_exist: bool,
                            categories_list: List[str]) -> None:
     """
@@ -689,36 +898,19 @@ def update_term_categories(egeria_client: EgeriaTech, term_guid: str, categories
                 egeria_client.add_term_to_category(term_guid, cat)
                 current_categories.append(cat)
                 msg = f"Added term {term_guid} to category {cat}"
-                print_msg("DEBUG-INFO", msg, debug_level)
+                logger.info(msg)
 
         for cat in current_categories:
             if cat not in to_be_cat_guids:
                 egeria_client.remove_term_from_category(term_guid, cat)
                 msg = f"Removed term {term_guid} from category {cat}"
-                print_msg("DEBUG-INFO", msg, debug_level)
+                logger.info(msg)
     else:  # No categories specified - so remove any categories a term is in
         for cat in current_categories:
             egeria_client.remove_term_from_category(term_guid, cat)
             msg = f"Removed term {term_guid} from category {cat}"
-            print_msg("DEBUG-INFO", msg, debug_level)
-
-def sync_data_dict_membership(egeria_client: EgeriaTech, in_data_dictionary_names: list , in_data_dictionary: dict,
-                              guid:str, object_type:str)->dict:
-    pass
-
-def sync_data_structure_membership(egeria_client: EgeriaTech, in_data_structure_names: list , in_data_structure: dict,
-                              guid:str, object_type:str)->dict:
+            logger.info(msg)
 
-    pass
 
-def sync_data_spec_membership(egeria_client: EgeriaTech, in_data_spec_names: list , in_data_spec: dict,
-                              guid:str, object_type:str)->dict:
-    pass
 
-def sync_term_links(egeria_client: EgeriaTech, term_guid: str, links_exist: bool,
-                   links_list: List[str]) -> None:
-    pass
 
-def sync_parent_data_field(egeria_client: EgeriaTech, term_guid: str, links_exist: bool,
-                   links_list: List[str]) -> None:
-    pass