From bea5d72d13d80d4ea1fd6f19dd528abeb9950eef Mon Sep 17 00:00:00 2001
From: Korina Cordero <cordero.korina@gmail.com>
Date: Fri, 14 Feb 2020 03:07:42 +0000
Subject: [PATCH] Modify the function names for walkin. #340

---
 config/routes/job_order.yaml          |  8 ++++----
 src/Controller/JobOrderController.php | 29 ++++++++++++++++++++++++---
 2 files changed, 30 insertions(+), 7 deletions(-)

diff --git a/config/routes/job_order.yaml b/config/routes/job_order.yaml
index d3ce6834..d5cd8dfc 100644
--- a/config/routes/job_order.yaml
+++ b/config/routes/job_order.yaml
@@ -208,21 +208,21 @@ jo_tracker:
 
 jo_walkin_form:
     path: /job-order/walk-in
-    controller: App\Controller\JobOrderController::walkinForm
+    controller: App\Controller\JobOrderController::walkInForm
     methods: [GET]
 
 jo_walkin_submit:
     path: /job-order/walk-in
-    controller: App\Controller\JobOrderController::walkinSubmit
+    controller: App\Controller\JobOrderController::walkInSubmit
     methods: [POST]
 
 jo_walkin_edit_form:
     path: /job-order/walk-in/{id}
-    controller: App\Controller\JobOrderController::walkinEditForm
+    controller: App\Controller\JobOrderController::walkInEditForm
     methods: [GET]
 
 jo_walkin_edit_submit:
     path: /job-order/walk-in/{id}
-    controller: App\Controller\JobOrderController::walkinEditSubmit
+    controller: App\Controller\JobOrderController::walkInEditSubmit
     methods: [POST]
 
diff --git a/src/Controller/JobOrderController.php b/src/Controller/JobOrderController.php
index 08b6dbe0..15d44e1d 100644
--- a/src/Controller/JobOrderController.php
+++ b/src/Controller/JobOrderController.php
@@ -966,7 +966,7 @@ class JobOrderController extends Controller
     /**
      * @Menu(selected="jo_walkin_form")
      */
-    public function walkinForm(EntityManagerInterface $em, JobOrderHandlerInterface $jo_handler)
+    public function walkInForm(EntityManagerInterface $em, JobOrderHandlerInterface $jo_handler)
     {
         $this->denyAccessUnlessGranted('jo_walkin.form', null, 'No access.');
 
@@ -983,7 +983,7 @@ class JobOrderController extends Controller
         return $this->render($template, $params);
     }
 
-    public function walkinSubmit(Request $req, JobOrderHandlerInterface $jo_handler)
+    public function walkInSubmit(Request $req, JobOrderHandlerInterface $jo_handler)
     {
         $this->denyAccessUnlessGranted('jo_walkin.form', null, 'No access.');
 
@@ -1010,7 +1010,7 @@ class JobOrderController extends Controller
     /**
      * @Menu(selected="jo_walkin_edit_form")
      */
-    public function walkinEditForm($id, EntityManagerInterface $em, JobOrderHandlerInterface $jo_handler)
+    public function walkInEditForm($id, EntityManagerInterface $em, JobOrderHandlerInterface $jo_handler)
     {
         $this->denyAccessUnlessGranted('jo_walkin.edit', null, 'No access.');
 
@@ -1027,5 +1027,28 @@ class JobOrderController extends Controller
         return $this->render($template, $params);
     }
 
+    public function walkInEditSubmit(Request $req, JobOrderHandlerInterface $jo_handler)
+    {
+        $this->denyAccessUnlessGranted('jo_walkin.edit', null, 'No access.');
+
+        $error_array = [];
+        $error_array = $jo_handler->processOneStepJobOrder($req, $id);
+
+        // check if any errors were found
+        if (!empty($error_array)) {
+            // return validation failure response
+            return $this->json([
+                'success' => false,
+                'errors' => $error_array
+            ], 422);
+        }
+
+
+        // return successful response
+        return $this->json([
+            'success' => 'Changes have been saved!'
+        ]);
+    }
+
 
 }