From 20807a7ae1f722a31a81607b710bc12ca7ea661d Mon Sep 17 00:00:00 2001
From: Isaac Machakata <isaac@caasi.co.zw>
Date: Sun, 7 Apr 2024 03:29:26 +0200
Subject: [PATCH] refactor: allow rejection of requisitions

---
 app/Controllers/Requisition.php            | 9 +++------
 app/Views/forms/authorize-requisitions.php | 8 ++++----
 2 files changed, 7 insertions(+), 10 deletions(-)

diff --git a/app/Controllers/Requisition.php b/app/Controllers/Requisition.php
index b739c88..fca30fc 100644
--- a/app/Controllers/Requisition.php
+++ b/app/Controllers/Requisition.php
@@ -186,7 +186,7 @@ public function viewUserReportsIndex()
 
     public function authorizeRequisitionsIndex()
     {
-        $isSupervisor = $this->account->Department == 'Supervisor';
+        $isSupervisor = $this->account->department == 'Supervisor';
         $allowedStatus = $isSupervisor ? 'Submitted' : 'Supervisor_Approved';
         self::$VIEW_PARAMS['statuses'] = [
             'Approve' => $isSupervisor ? 'Supervisor_Approved' : 'Finance_Disbursed',
@@ -203,7 +203,7 @@ public function authorizeRequisitionsIndex()
 
     public function authorizeRequisitions()
     {
-        $isSupervisor = $this->account->Department == 'Supervisor';
+        $isSupervisor = $this->account->department == 'Supervisor';
         $allowedStatus = $isSupervisor ? 'Submitted' : 'Supervisor_Approved';
         self::$VIEW_PARAMS['statuses'] = [
             'CanReject' => $isSupervisor
@@ -212,10 +212,7 @@ public function authorizeRequisitions()
         // this prevents the user from rejecting requisitions without the required rights
         if ($isSupervisor) {
             $allowedApprovals = 'Supervisor_Approved,Reject';
-            self::$VIEW_PARAMS['statuses'] = [
-                'Approve' => 'Supervisor_Approved',
-                'CanReject' => $this->account->Department == 'Supervisor'
-            ];
+            self::$VIEW_PARAMS['statuses']['Approve'] = 'Supervisor_Approved';
         } else {
             $allowedApprovals = 'Finance_Disbursed';
         }
diff --git a/app/Views/forms/authorize-requisitions.php b/app/Views/forms/authorize-requisitions.php
index 01a0f50..dd0e219 100644
--- a/app/Views/forms/authorize-requisitions.php
+++ b/app/Views/forms/authorize-requisitions.php
@@ -94,10 +94,10 @@
                                             <label class="btn btn-outline-primary" for="StatusSubmitted">Submitted</label>
                                             <input type="radio" class="btn-check" name="Status" value="<?= $statuses['Approve'] ?>" id="StatusApprove" autocomplete="off">
                                             <label class="btn btn-outline-primary" for="StatusApprove">Approve Requisition</label>
-                                            <?php if($statuses['CanReject']):?>
-                                            <input type="radio" class="btn-check" name="Status" id="StatusDismiss" value="<?= $statuses['Reject'] ?>" autocomplete="off">
-                                            <label class="btn btn-outline-primary" for="StatusDismiss">Cancel Requisition</label>
-                                            <?php endif;?>
+                                            <?php if ($statuses['CanReject']) : ?>
+                                                <input type="radio" class="btn-check" name="Status" id="StatusDismiss" value="Rejected" autocomplete="off">
+                                                <label class="btn btn-outline-primary" for="StatusDismiss">Reject Requisition</label>
+                                            <?php endif; ?>
                                         </div>
                                     </div>
                                     <div class="mb-2 col-12">