Merge branch '742-add-checking-if-jo-is-null-in-hubarrive' into 'master'

Resolve "Add checking if JO is null in hubArrive"

Closes #742

See merge request jankstudio/resq!856

src/Controller/CAPI/RiderAppController.php

@@ -725,16 +725,20 @@ class RiderAppController extends APIController
         // get rider's current job order
         $jo = $rider->getCurrentJobOrder();
 
-        // set delivery status
+        // check if JO is null
-        $jo->setDeliveryStatus(DeliveryStatus::RIDER_ARRIVE_HUB);
+        if ($jo)
+        {
+            // set delivery status
+            $jo->setDeliveryStatus(DeliveryStatus::RIDER_ARRIVE_HUB);
 
-        $timestamp_event = new JOEvent();
+            $timestamp_event = new JOEvent();
-        $timestamp_event->setDateHappen(new DateTime())
+            $timestamp_event->setDateHappen(new DateTime())
-            ->setTypeID(JOEventType::RIDER_ARRIVE_HUB)
+                ->setTypeID(JOEventType::RIDER_ARRIVE_HUB)
-            ->setJobOrder($jo)
+                ->setJobOrder($jo)
-            ->setRider($rider);
+                ->setRider($rider);
 
-        $em->persist($timestamp_event);
+            $em->persist($timestamp_event);
+        }
 
         // tag rider as available
         $rider->setAvailable(true);