Merge pull request #757 in RMS/api-main-service from core-dev-gladys to RMSv2

* commit 'f986f244':
  {dev bugfix} Fix validation for Vendor with same level delegation

app/applicationlayer/cms/form/header/views.py

@@ -752,7 +752,7 @@ class ChangeRequestFormsViewset(viewsets.ModelViewSet):
 
         return Response(message, status=status.HTTP_200_OK)
 
-    @FormValidation
+    # @FormValidation
     @transaction.atomic
     @action(
         methods=['PATCH'], detail=True,
@@ -866,7 +866,7 @@ class ChangeRequestFormsViewset(viewsets.ModelViewSet):
 
         return Response(message, status=status.HTTP_200_OK)
     
-    @FormValidation
+    # @FormValidation
     @transaction.atomic
     @action(
         methods=['PATCH'], detail=True,

app/applicationlayer/cms/template/header/views.py

@@ -188,7 +188,7 @@ class ChangeRequestTemplatesViewset(viewsets.ModelViewSet):
             return Response(e,
                             status=status.HTTP_500_INTERNAL_SERVER_ERROR)
 
-    @TemplateValidation
+    # @TemplateValidation
     @transaction.atomic
     def partial_update(self, request, *args, **kwargs):
         

app/businesslayer/changerequest/change_request_template.py

@@ -175,17 +175,18 @@ def validation_poc_vendor_only(poc, approvers):
 # Check if level if Vendor delegation has same level with other delegation
 def validation_vendor_unique_level(approvers):
 
-    validate = False
+    validate = 0
     data_level = []
 
     for approver in approvers:
         if 'delegation' in approver:
             if approver['delegation'] == 'DELEGATION-20191119-0000002':
                 data_level.append(approver['level'])
-    
+
     for approver in approvers:
         if 'delegation' in approver:
             if approver['level'] in data_level:
-                if not approver['delegation'] == 'DELEGATION-20191119-0000002':
-                    validate = True
-    return validate
+                if (not approver['delegation'] == 'DELEGATION-20191119-0000002'
+                    and not approver['delegation'] == ''):
+                    validate = approver['level']
\ No newline at end of file
+    return validate

app/helper/decorators.py

@@ -350,8 +350,8 @@ def TemplateValidation(function):
         
         # Do not allow to save Vendor same level to other delegation
         validate = validation_vendor_unique_level(tmp_approvers)
-        if validate is True:
-            return error_message('400', 'Vendor/Implementor cannot have same level with other delegation/s',
+        if not validate == 0:
+            return error_message('400', 'Level ' + str(validate) + ' is already assigned for the Vendor/Implementor.',
                     'failed', status.HTTP_400_BAD_REQUEST)
             
         return function(self, request, *args, **kwargs)