@fleetbase/fleetops-engine 0.6.40 → 0.6.42

package/addon/routes/operations.js

@@ -1,3 +1,25 @@
 import Route from '@ember/routing/route';
+import { inject as service } from '@ember/service';
 
-export default class OperationsRoute extends Route {}
+export default class OperationsRoute extends Route {
+    @service('universe/menu-service') menuService;
+    @service universe;
+
+    beforeModel(transition) {
+        if (transition.intent && transition.intent.url) {
+            // Here we will check if it's an actual virtual route instead of operations
+            const intendedUrl = transition.intent.url;
+            const intentSegments = intendedUrl.split('/');
+            // Needs to match for section and slug
+            const section = intentSegments[2];
+            const slug = intentSegments[3];
+            // This is not a operations route check menu service for a virtual registration match
+            if (section !== 'operations') {
+                const menuItem = this.menuService.lookupMenuItem('engine:fleet-ops', slug, null, section);
+                if (menuItem) {
+                    return this.universe.transitionMenuItem('console.fleet-ops.virtual', menuItem);
+                }
+            }
+        }
+    }
+}

package/composer.json

@@ -1,6 +1,6 @@
 {
     "name": "fleetbase/fleetops-api",
-    "version": "0.6.40",
+    "version": "0.6.42",
     "description": "Fleet & Transport Management Extension for Fleetbase",
     "keywords": [
         "fleetbase-extension",

package/extension.json

@@ -1,6 +1,6 @@
 {
     "name": "Fleet-Ops",
-    "version": "0.6.40",
+    "version": "0.6.42",
     "description": "Fleet & Transport Management Extension for Fleetbase",
     "repository": "https://github.com/fleetbase/fleetops",
     "license": "AGPL-3.0-or-later",

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@fleetbase/fleetops-engine",
-    "version": "0.6.40",
+    "version": "0.6.42",
     "description": "Fleet & Transport Management Extension for Fleetbase",
     "fleetbase": {
         "route": "fleet-ops"

package/server/migrations/2026_04_20_000001_make_orchestrator_priority_nullable_on_orders_table.php

@@ -0,0 +1,50 @@
+<?php
+
+use Illuminate\Database\Migrations\Migration;
+use Illuminate\Support\Facades\DB;
+
+/**
+ * Make `orchestrator_priority` nullable on the orders table.
+ *
+ * The original migration (2026_04_08_000003) added this column as NOT NULL
+ * with a database-level default of 50.  When Eloquent explicitly passes NULL
+ * for the column (e.g. when a user submits the order form without filling in
+ * orchestrator constraints), the DB default is bypassed and MySQL raises:
+ *
+ *   SQLSTATE[23000]: Integrity constraint violation: 1048
+ *   Column 'orchestrator_priority' cannot be null
+ *
+ * This migration makes the column nullable so that existing installations
+ * that have already run the original migration are also protected.  The
+ * application layer (Order model + controllers) already coerces null to 50,
+ * so NULL values will not appear in practice; the nullable flag is a
+ * belt-and-suspenders safety net for any code path that may have been missed.
+ *
+ * NOTE: We use raw DB::statement() instead of Blueprint::change() because
+ * ->change() relies on Doctrine DBAL to introspect the existing column type,
+ * and Doctrine does not recognise MySQL's TINYINT as a mapped type
+ * ("Unknown column type tinyinteger requested"), causing the migration to
+ * crash on deployment.  The raw ALTER TABLE statement is portable across all
+ * MySQL/MariaDB versions supported by Fleetbase and requires no extra
+ * dependencies.
+ */
+return new class extends Migration {
+    public function up(): void
+    {
+        DB::statement(
+            'ALTER TABLE `orders` MODIFY COLUMN `orchestrator_priority` TINYINT UNSIGNED NULL DEFAULT 50'
+        );
+    }
+
+    public function down(): void
+    {
+        // Back-fill any NULLs before restoring the NOT NULL constraint.
+        DB::table('orders')
+            ->whereNull('orchestrator_priority')
+            ->update(['orchestrator_priority' => 50]);
+
+        DB::statement(
+            'ALTER TABLE `orders` MODIFY COLUMN `orchestrator_priority` TINYINT UNSIGNED NOT NULL DEFAULT 50'
+        );
+    }
+};

package/server/src/Http/Controllers/Api/v1/OrderController.php

@@ -299,6 +299,12 @@ class OrderController extends Controller
             $input['adhoc'] = Utils::isTrue($input['adhoc']) ? 1 : 0;
         }
 
+        // Ensure orchestrator_priority is never null — the column is NOT NULL
+        // and the DB default is bypassed when Eloquent receives an explicit null.
+        if (!isset($input['orchestrator_priority']) || !is_numeric($input['orchestrator_priority'])) {
+            $input['orchestrator_priority'] = 50;
+        }
+
         if (!isset($input['payload_uuid'])) {
             return response()->apiError('Attempted to attach invalid payload to order.');
         }
@@ -527,6 +533,12 @@ class OrderController extends Controller
             $order->dispatch();
         }
 
+        // Ensure orchestrator_priority is never null on update either —
+        // only apply the default when the key was explicitly sent as null/empty.
+        if (array_key_exists('orchestrator_priority', $input) && !is_numeric($input['orchestrator_priority'])) {
+            $input['orchestrator_priority'] = 50;
+        }
+
         // update the order
         $order->update($input);
         $order->flushAttributesCache();

package/server/src/Http/Controllers/Internal/v1/OrderController.php

@@ -126,6 +126,12 @@ class OrderController extends FleetOpsController
                             $input['order_config_uuid'] = $defaultOrderConfig->uuid;
                         }
                     }
+
+                    // Ensure orchestrator_priority is never null — the column is NOT NULL
+                    // and the DB default is bypassed when Eloquent receives an explicit null.
+                    if (!isset($input['orchestrator_priority']) || !is_numeric($input['orchestrator_priority'])) {
+                        $input['orchestrator_priority'] = 50;
+                    }
                 },
                 function (&$request, Order &$order, &$requestInput) {
                     $input                   = $request->input('order');

package/server/src/Models/Order.php

@@ -219,6 +219,20 @@ class Order extends Model
         'orchestrator_priority' => 'integer',
     ];
 
+    /**
+     * The model's default attribute values.
+     *
+     * Ensures that `orchestrator_priority` is never persisted as NULL even when
+     * the caller omits the field entirely (e.g. the order/form component does
+     * not require the user to fill in orchestrator constraints).  The value
+     * mirrors the database column default defined in the migration.
+     *
+     * @var array<string, mixed>
+     */
+    protected $attributes = [
+        'orchestrator_priority' => 50,
+    ];
+
     /**
      * The attributes excluded from the model's JSON form.
      *
@@ -619,6 +633,18 @@ class Order extends Model
         return data_get($this, 'updatedBy.name');
     }
 
+    /**
+     * Set the orchestrator_priority attribute.
+     *
+     * Coerces null or non-numeric values to the default priority of 50 so that
+     * the NOT NULL database constraint is never violated when a user submits
+     * the order form without filling in the orchestrator constraints section.
+     */
+    public function setOrchestratorPriorityAttribute($value): void
+    {
+        $this->attributes['orchestrator_priority'] = is_numeric($value) ? (int) $value : 50;
+    }
+
     /**
      * Set the order type attribute, which defaults to `default`.
      */