feat: Remove usage of window.Fleetbase, improve several order related functions

Author: roncodes

addon/components/customer/orders.js

@@ -42,7 +42,7 @@ export default class CustomerOrdersComponent extends Component {
 
     get modalsManager() {
         const owner = getOwner(this);
-        const application = typeof this.universe.getApplicationInstance === 'function' ? this.universe.getApplicationInstance() : window.Fleetbase;
+        const application = this.universe.getApplicationInstance();
         const modalsManager = application ? application.lookup('service:modals-manager') : owner.lookup('service:modals-manager');
         return modalsManager;
     }

addon/services/leaflet-routing-control.js

@@ -67,7 +67,7 @@ export default class LeafletRoutingControlService extends Service {
 
     #initializeRegistry() {
         const registry = 'registry:routing-controls';
-        const application = typeof this.universe?.getApplicationInstance === 'function' ? this.universe.getApplicationInstance() : window.Fleetbase;
+        const application = this.universe.getApplicationInstance();
         if (!application.hasRegistration(registry)) {
             application.register(registry, new RoutingControlRegistry(), { instantiate: false });
         }

addon/services/movement-tracker.js

@@ -122,6 +122,7 @@ export class EventBuffer {
 
 export default class MovementTrackerService extends Service {
     @service socket;
+    @service universe;
     @tracked channels = [];
     @tracked buffers = new Map();
 
@@ -131,7 +132,7 @@ export default class MovementTrackerService extends Service {
     }
 
     #getOwner(owner = null) {
-        return owner ?? window.Fleetbase ?? getOwner(this);
+        return owner ?? this.universe.getApplicationInstance() ?? getOwner(this);
     }
 
     #getBuffer(key, model, opts = {}) {

addon/services/route-optimization.js

@@ -57,7 +57,7 @@ export default class RouteOptimizationService extends Service {
 
     #initializeRegistry() {
         const registry = 'registry:route-optimization-engines';
-        const application = typeof this.universe?.getApplicationInstance === 'function' ? this.universe.getApplicationInstance() : window.Fleetbase;
+        const application = this.universe.getApplicationInstance();
         if (!application.hasRegistration(registry)) {
             application.register(registry, new RouteOptimizationRegistry(), { instantiate: false });
         }

server/src/Http/Controllers/Internal/v1/LiveController.php

@@ -139,6 +139,7 @@ function ($q) {
 
             if ($active) {
                 $query->whereHas('driverAssigned');
+                $query->whereNotIn('status', ['created', 'completed', 'expired', 'order_canceled', 'canceled', 'pending']);
             }
 
             if ($unassigned) {
@@ -166,7 +167,7 @@ public function drivers(Request $request)
 
         return LiveCacheService::remember('drivers', $cacheParams, function () use ($bounds) {
             $query = Driver::where(['company_uuid' => session('company')])
-                ->with(['user', 'vehicle', 'currentJob'])
+                ->with(['user', 'vehicle'])
                 ->applyDirectivesForPermissions('fleet-ops list driver');
 
             // Filter out drivers with invalid coordinates

server/src/Http/Filter/OrderFilter.php

@@ -116,7 +116,7 @@ public function active(bool $active = false)
             $this->builder->where(
                 function ($q) {
                     $q->whereHas('driverAssigned');
-                    $q->whereNotIn('status', ['created', 'canceled', 'order_canceled', 'completed']);
+                    $q->whereNotIn('status', ['created', 'completed', 'expired', 'order_canceled', 'canceled', 'pending']);
                 }
             );
         }

server/src/Http/Resources/v1/Order.php

@@ -84,6 +84,7 @@ public function toArray($request): array
             'pod_method'           => $this->pod_method,
             'pod_required'         => (bool) data_get($this, 'pod_required', false),
             'dispatched'           => (bool) data_get($this, 'dispatched', false),
+            'started'              => (bool) data_get($this, 'started', false),
             'adhoc'                => (bool) data_get($this, 'adhoc', false),
             'adhoc_distance'       => (int) $this->getAdhocDistance(),
             'distance'             => (int) $this->distance,

server/src/Models/Driver.php

@@ -262,12 +262,12 @@ public function vendor(): BelongsTo|Builder
 
     public function currentJob(): BelongsTo|Builder
     {
-        return $this->belongsTo(Order::class)->select(['id', 'uuid', 'public_id', 'payload_uuid', 'driver_assigned_uuid'])->without(['driver']);
+        return $this->belongsTo(Order::class)->without(['driver']);
     }
 
     public function currentOrder(): BelongsTo|Builder
     {
-        return $this->belongsTo(Order::class, 'current_job_uuid')->select(['id', 'uuid', 'public_id', 'payload_uuid', 'driver_assigned_uuid'])->without(['driver']);
+        return $this->belongsTo(Order::class, 'current_job_uuid')->without(['driver']);
     }
 
     public function jobs(): HasMany|Builder

server/src/Observers/OrderObserver.php

@@ -18,6 +18,20 @@ public function created(Order $order)
         $this->invalidateCache($order);
     }
 
+    /**
+     * Handle the Order "updating" event.
+     *
+     * This event is fired before the order is persisted to the database.
+     * It is used to mutate attributes as part of the same update operation
+     * without triggering additional save cycles.
+     *
+     * @param Order $order The order being updated
+     */
+    public function updating(Order $order): void
+    {
+        $this->ensureOrderStarted($order);
+    }
+
     /**
      * Handle the Order "updated" event.
      *
@@ -62,4 +76,40 @@ protected function invalidateCache(?Order $order = null): void
             Cache::forget("order:{$order->uuid}:tracker");
         }
     }
+
+    /**
+     * Detects when an order has just transitioned to the "started" status
+     * and initializes start-related fields.
+     *
+     * This method should be called during the "updating" lifecycle event
+     * to ensure that the changes are persisted as part of the same database
+     * update and do not trigger additional observer events.
+     *
+     * An order is considered "started" when:
+     * - The "status" attribute is being changed in the current update
+     * - The previous status was not "started"
+     * - The new status is "started"
+     *
+     * When these conditions are met, the order's start timestamp and
+     * started flag are set if they have not already been initialized.
+     *
+     * @param Order $order The order being evaluated for a start transition
+     */
+    protected function ensureOrderStarted(Order $order): void
+    {
+        if (
+            $order->isDirty('status')
+            && $order->getOriginal('status') === 'dispatched'
+            && $order->status === 'started'
+        ) {
+            // Only set defaults if not explicitly provided
+            if (is_null($order->started_at)) {
+                $order->started_at = now();
+            }
+
+            if (!$order->started) {
+                $order->started = true;
+            }
+        }
+    }
 }