PCHR-4019: Improve the code

uk.co.compucorp.civicrm.hrleaveandabsences/js/src/leave-absences/shared/controllers/request.controller.js

@@ -99,7 +99,7 @@ define([
         .then(initIsSelfRecord)
         .then(function () {
           return $q.all([
-            initRoles(),
+            initRole(),
             loadAbsencePeriods(),
             loadStatuses()
           ]);
@@ -616,37 +616,51 @@ define([
      *
      * @return {Promise}
      */
-    function initRoles () {
+    function initRole () {
       role = 'staff';
 
-      // If the user is creating or editing their own leave, they will be
-      // treated as a staff regardless of their actual role.
-      if (vm.isSelfRecord) {
-        return checkIfContactIsSelfLeaveApprover()
-          .then(function (isSelfLeaveApprover) {
-            if (!isSelfLeaveApprover) {
-              return;
-            }
-
-            role = 'admin';
-            vm.isSelfLeaveApprover = true;
-            vm.canManage = true;
-          });
-      }
+      return (vm.isSelfRecord
+        ? initRoleBasedOnSelfLeaveApproverState()
+        : initRoleBasedOnPermissions())
+        .finally(function () {
+          vm.canManage = vm.isRole('manager') || vm.isRole('admin');
+        });
+    }
+
+    /**
+     * Initiates the user role based on their self leave approver state.
+     * If the user is creating or editing their own leave request, they will be
+     * treated as an "admin".
+     *
+     * @return {Promise}
+     */
+    function initRoleBasedOnSelfLeaveApproverState () {
+      return checkIfContactIsSelfLeaveApprover()
+        .then(function (isSelfLeaveApprover) {
+          if (!isSelfLeaveApprover) {
+            return;
+          }
+
+          role = 'admin';
+          vm.isSelfLeaveApprover = true;
+        });
+    }
 
+    /**
+     * Initiates user role based on their permissions
+     *
+     * @return {Promise}
+     */
+    function initRoleBasedOnPermissions () {
       return checkPermissions(sharedSettings.permissions.admin.administer)
         .then(function (isAdmin) {
           isAdmin && (role = 'admin');
         })
         .then(function () {
-          // (role === 'staff') means it is not admin so need to check if manager
-          return (role === 'staff') && checkPermissions(sharedSettings.permissions.ssp.manage)
-            .then(function (isManager) {
-              isManager && (role = 'manager');
-            });
+          return (role !== 'admin') && checkPermissions(sharedSettings.permissions.ssp.manage);
         })
-        .finally(function () {
-          vm.canManage = vm.isRole('manager') || vm.isRole('admin');
+        .then(function (isManager) {
+          isManager && (role = 'manager');
         });
     }
 

uk.co.compucorp.civicrm.hrleaveandabsences/js/test/shared/controllers/request.controller.spec.js

@@ -1488,7 +1488,7 @@
                     expect(controller.request.status_id).toEqual(approvalStatus);
                   });
 
-                  it('calls corresponding API end points', function () {
+                  it('calls the corresponding API endpoint', function () {
                     expect(LeaveRequestAPI.create).toHaveBeenCalled();
                   });
                 });