fix service update

medical/gql_mutations.py

@@ -1,4 +1,5 @@
 import functools
+import logging
 from gettext import gettext as _
 from operator import or_
 
@@ -17,6 +18,8 @@
 from django.db import models
 from medical.utils import process_items_relations, process_services_relations
 
+
+logger = logging.getLogger(__name__)
 class ServiceCodeInputType(graphene.String):
     @staticmethod
     def coerce_string(value):
@@ -213,8 +216,8 @@ def do_mutate(cls, perms, user, **data):
         data['audit_user_id'] = user.id_for_audit
         from core.utils import TimeUtils
         data['validity_from'] = TimeUtils.now()
-        print("Create or Update Item or Service Mutation")
-        print(data)
+        logger.debug("Create or Update Item or Service Mutation")
+        logger.debug(data)
         update_or_create_item_or_service(data, user, cls.item_service_model)
         return None
 

medical/utils.py

@@ -3,19 +3,20 @@
 
 def process_child_relation(user, data_children, service_id, children, create_hook):
     claimed = 0
-    for data_elt in data_children:
-        elt_id = data_elt.pop('id') if 'id' in data_elt else None
-        if elt_id:
-            if create_hook == item_create_hook:
-                elt = ServiceItem.objects.get(id=elt_id)
+    if isinstance(data_children, list):
+        for data_elt in data_children:
+            elt_id = data_elt.pop('id') if 'id' in data_elt else None
+            if elt_id:
+                if create_hook == item_create_hook:
+                    elt = ServiceItem.objects.get(id=elt_id)
+                else:
+                    elt = ServiceService.objects.get(id=elt_id)
+                [setattr(elt, k, v) for k, v in data_elt.items()]
+                elt.save()
             else:
-                elt = ServiceService.objects.get(id=elt_id)
-            [setattr(elt, k, v) for k, v in data_elt.items()]
-            elt.save()
-        else:
-            print("Create Item or Service")
-            data_elt['audit_user_id'] = user.id_for_audit
-            create_hook(children, data_elt)
+                print("Create Item or Service")
+                data_elt['audit_user_id'] = user.id_for_audit
+                create_hook(children, data_elt)
     return claimed
 
 def item_create_hook(service_id, item):