Bug fix for access. Add API logging to CMB services to avoid breakage. #377

config/cmb.services.yaml

@@ -227,3 +227,8 @@ services:
             $redis_prov: "@App\\Service\\RedisClientProvider"
             $loc_key: "%env(LOCATION_RIDER_ACTIVE_KEY)%"
             $status_key: "%env(STATUS_RIDER_KEY)%"
+
+    # API logging
+    App\EventSubscriber\LogSubscriber:
+        arguments:
+            $api_log_flag: "%env(API_LOGGING)%"

config/menu.yaml

@@ -122,10 +122,6 @@ main_menu:
       acl: jo_all.list
       label: View All
       parent: joborder
-    - id: jo_autoassign
-      acl: jo_autoassign.test
-      label: Autoassign Test
-      parent: joborder
 
     - id: support
       acl: support.menu

config/services.yaml

@@ -231,7 +231,7 @@ services:
     App\Service\InventoryManager:
         arguments:
             $api_url: "%env(INVENTORY_API_URL)%"
-            $api_ocp_key: "%env(INVENTORY_API_OCP)%" 
+            $api_ocp_key: "%env(INVENTORY_API_OCP)%"
             $api_auth_prefix: "%env(INVENTORY_API_AUTH_TOKEN_PREFIX)%"
             $api_auth_token: "%env(INVENTORY_API_AUTH_TOKEN)%"
 

src/Controller/CustomerController.php

@@ -181,15 +181,13 @@ class CustomerController extends Controller
 
     public function getCustomerVehicles(Request $req, CustomerHandlerInterface $cust_handler)
     {
-        /*
-        // TODO: fix
-        if ((!$this->isGranted('jo_onestep.form')) ||
+        if (!((!$this->isGranted('jo_onestep.form')) ||
             (!$this->isGranted('jo_walkin.form')) ||
-            (!$this->isGranted('jo_in.list'))) { 
+            (!$this->isGranted('jo_in.list'))))
+        { 
             $exception = $this->createAccessDeniedException('No access.');
             throw $exception;
         }
-        */
 
         $results = $cust_handler->getCustomerVehicles($req);