@fleetbase/fleetops-engine 0.6.33 → 0.6.35

package/addon/components/order/details/proof.js

@@ -1,3 +1,30 @@
 import Component from '@glimmer/component';
+import { tracked } from '@glimmer/tracking';
+import { inject as service } from '@ember/service';
+import { task } from 'ember-concurrency-decorators';
 
-export default class OrderDetailsProofComponent extends Component {}
+export default class OrderDetailsProofComponent extends Component {
+    @service fetch;
+    @tracked proofs = [];
+
+    constructor(owner, { resource }) {
+        super(...arguments);
+        this.loadOrderProofs.perform(resource);
+    }
+
+    @task *loadOrderProofs(order) {
+        const proofs = yield this.fetch.get(`orders/${order.id}/proofs`);
+
+        this.proofs = proofs.map((proof) => ({
+            ...proof,
+            type: this.#getTypeFromRemarks(proof.remarks),
+        }));
+    }
+
+    #getTypeFromRemarks(remarks = '') {
+        if (remarks.endsWith('Photo')) return 'photo';
+        if (remarks.endsWith('Scan')) return 'scan';
+        if (remarks.endsWith('Signature')) return 'signature';
+        return undefined;
+    }
+}

package/composer.json

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

package/extension.json

@@ -1,6 +1,6 @@
 {
     "name": "Fleet-Ops",
-    "version": "0.6.33",
+    "version": "0.6.35",
     "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.33",
+    "version": "0.6.35",
     "description": "Fleet & Transport Management Extension for Fleetbase",
     "fleetbase": {
         "route": "fleet-ops"

package/server/src/Console/Commands/TestEmail.php

@@ -33,7 +33,7 @@ class TestEmail extends Command
     public function handle()
     {
         $email = $this->argument('email');
-        $type = $this->option('type');
+        $type  = $this->option('type');
 
         $this->info('Sending test email...');
         $this->info("Type: {$type}");
@@ -47,40 +47,40 @@ class TestEmail extends Command
 
                 default:
                     $this->error("Unknown email type: {$type}");
+
                     return Command::FAILURE;
             }
 
             $this->info('✓ Test email sent successfully!');
+
             return Command::SUCCESS;
         } catch (\Exception $e) {
             $this->error('Failed to send test email: ' . $e->getMessage());
+
             return Command::FAILURE;
         }
     }
 
     /**
      * Send a test customer credentials email.
-     *
-     * @param string $email
-     * @return void
      */
     private function sendCustomerCredentialsEmail(string $email): void
     {
         // Create a mock user
         $user = new User([
-            'name' => 'Test Customer',
+            'name'  => 'Test Customer',
             'email' => $email,
         ]);
 
         // Create a mock company
         $company = new Company([
-            'name' => 'Test Company',
+            'name'      => 'Test Company',
             'public_id' => 'test_company_123',
         ]);
 
         // Create a mock customer
         $customer = new Contact([
-            'name' => 'Test Customer',
+            'name'  => 'Test Customer',
             'email' => $email,
             'phone' => '+1234567890',
         ]);

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

@@ -8,7 +8,6 @@ use Fleetbase\FleetOps\Http\Resources\v1\Contact as ContactResource;
 use Fleetbase\FleetOps\Http\Resources\v1\DeletedResource;
 use Fleetbase\FleetOps\Models\Contact;
 use Fleetbase\Http\Controllers\Controller;
-use Fleetbase\Models\File;
 use Fleetbase\Models\User;
 use Fleetbase\Support\Utils;
 use Illuminate\Http\Request;
@@ -29,24 +28,13 @@ class ContactController extends Controller
         $input['phone'] = is_string($input['phone']) ? Utils::formatPhoneNumber($input['phone']) : $input['phone'];
         $input['type']  = empty($input['type']) ? 'contact' : $input['type'];
 
-        // Handle photo as either file id/ or base64 data string
-        $photo = $request->input('photo');
-        if ($photo) {
-            // Handle photo being a file id
-            if (Utils::isPublicId($photo)) {
-                $file = File::where('public_id', $photo)->first();
-                if ($file) {
-                    $input['photo_uuid'] = $file->uuid;
-                }
-            }
+        // Handle photo upload using FileResolverService
+        if ($request->has('photo')) {
+            $path = 'uploads/' . session('company') . '/contacts';
+            $file = app(\Fleetbase\Services\FileResolverService::class)->resolve($request->input('photo'), $path);
 
-            // Handle the photo being base64 data string
-            if (Utils::isBase64String($photo)) {
-                $path = implode('/', ['uploads', session('company'), 'contacts']);
-                $file = File::createFromBase64($photo, null, $path);
-                if ($file) {
-                    $input['photo_uuid'] = $file->uuid;
-                }
+            if ($file) {
+                $input['photo_uuid'] = $file->uuid;
             }
         }
 
@@ -101,29 +89,20 @@ class ContactController extends Controller
             ]);
         }
 
-        // Handle photo as either file id/ or base64 data string
-        $photo = $request->input('photo');
-        if ($photo) {
-            // Handle photo being a file id
-            if (Utils::isPublicId($photo)) {
-                $file = File::where('public_id', $photo)->first();
-                if ($file) {
-                    $input['photo_uuid'] = $file->uuid;
-                }
-            }
-
-            // Handle the photo being base64 data string
-            if (Utils::isBase64String($photo)) {
-                $path = implode('/', ['uploads', session('company'), 'customers']);
-                $file = File::createFromBase64($photo, null, $path);
-                if ($file) {
-                    $input['photo_uuid'] = $file->uuid;
-                }
-            }
+        // Handle photo upload using FileResolverService
+        if ($request->has('photo')) {
+            $photo = $request->input('photo');
 
             // Handle removal key
             if ($photo === 'REMOVE') {
                 $input['photo_uuid'] = null;
+            } else {
+                $path = 'uploads/' . session('company') . '/contacts';
+                $file = app(\Fleetbase\Services\FileResolverService::class)->resolve($photo, $path);
+
+                if ($file) {
+                    $input['photo_uuid'] = $file->uuid;
+                }
             }
         }
 

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

@@ -20,7 +20,6 @@ use Fleetbase\Http\Resources\Organization;
 use Fleetbase\LaravelMysqlSpatial\Types\Point;
 use Fleetbase\Models\Company;
 use Fleetbase\Models\CompanyUser;
-use Fleetbase\Models\File;
 use Fleetbase\Models\User;
 use Fleetbase\Models\UserDevice;
 use Fleetbase\Models\VerificationCode;
@@ -124,20 +123,10 @@ class DriverController extends Controller
         // create the driver
         $driver = Driver::create($input);
 
-        // Handle photo as either file id/ or base64 data string
-        $photo = $request->input('photo');
-        if ($photo) {
-            $file = null;
-            // Handle photo being a file id
-            if (Utils::isPublicId($photo)) {
-                $file = File::where('public_id', $photo)->first();
-            }
-
-            // Handle the photo being base64 data string
-            if (Utils::isBase64String($photo)) {
-                $path = implode('/', ['uploads', session('company'), 'drivers']);
-                $file = File::createFromBase64($photo, null, $path);
-            }
+        // Handle photo upload using FileResolverService
+        if ($request->has('photo')) {
+            $path = 'uploads/' . $company->uuid . '/drivers';
+            $file = app(\Fleetbase\Services\FileResolverService::class)->resolve($request->input('photo'), $path);
 
             if ($file) {
                 $user->update(['photo_uuid' => $file->uuid]);
@@ -218,20 +207,10 @@ class DriverController extends Controller
         $driver->update($input);
         $driver->flushAttributesCache();
 
-        // Handle photo as either file id/ or base64 data string
-        $photo = $request->input('photo');
-        if ($photo) {
-            $file = null;
-            // Handle photo being a file id
-            if (Utils::isPublicId($photo)) {
-                $file = File::where('public_id', $photo)->first();
-            }
-
-            // Handle the photo being base64 data string
-            if (Utils::isBase64String($photo)) {
-                $path = implode('/', ['uploads', session('company'), 'drivers']);
-                $file = File::createFromBase64($photo, null, $path);
-            }
+        // Handle photo upload using FileResolverService
+        if ($request->has('photo')) {
+            $path = 'uploads/' . session('company') . '/drivers';
+            $file = app(\Fleetbase\Services\FileResolverService::class)->resolve($request->input('photo'), $path);
 
             if ($file) {
                 $driver->user->update(['photo_uuid' => $file->uuid]);

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

@@ -308,7 +308,7 @@ class OrderController extends Controller
         }
 
         // load required relations
-        $order->load(['trackingNumber', 'driverAssigned', 'purchaseRate', 'customer', 'facilitator']);
+        $order->load(['trackingNumber', 'trackingStatuses', 'driverAssigned', 'vehicleAssigned', 'purchaseRate', 'customer', 'facilitator']);
 
         // Determine if order should be dispatched on creation
         $shouldDispatch = $request->boolean('dispatch') && $integratedVendorOrder === null;
@@ -352,7 +352,7 @@ class OrderController extends Controller
 
         // find for the order
         try {
-            $order = Order::findRecordOrFail($id);
+            $order = Order::findRecordOrFail($id, ['trackingNumber', 'driverAssigned', 'purchaseRate', 'customer', 'facilitator']);
         } catch (ModelNotFoundException $exception) {
             return response()->json(
                 [
@@ -518,6 +518,9 @@ class OrderController extends Controller
         $order->update($input);
         $order->flushAttributesCache();
 
+        // load required relations
+        $order->load(['trackingNumber', 'trackingStatuses', 'driverAssigned', 'vehicleAssigned', 'purchaseRate', 'customer', 'facilitator']);
+
         // response the order resource
         return new OrderResource($order);
     }
@@ -532,6 +535,7 @@ class OrderController extends Controller
         set_time_limit(180);
 
         $results = Order::queryWithRequest($request, function (&$query, $request) {
+            $query->with(['trackingStatuses', 'driverAssigned', 'vehicleAssigned', 'customer', 'facilitator']);
             $query->where('company_uuid', session('company'));
             $query->whereNotNull('payload_uuid');
 
@@ -725,7 +729,7 @@ class OrderController extends Controller
     {
         // find for the order
         try {
-            $order = Order::findRecordOrFail($id);
+            $order = Order::findRecordOrFail($id, ['trackingNumber', 'trackingStatuses', 'driverAssigned', 'vehicleAssigned', 'purchaseRate', 'customer', 'facilitator']);
         } catch (ModelNotFoundException $exception) {
             return response()->json(
                 [
@@ -805,7 +809,7 @@ class OrderController extends Controller
     public function dispatchOrder(string $id)
     {
         try {
-            $order = Order::findRecordOrFail($id);
+            $order = Order::findRecordOrFail($id, ['trackingNumber', 'trackingStatuses', 'driverAssigned', 'vehicleAssigned', 'purchaseRate', 'customer', 'facilitator']);
         } catch (ModelNotFoundException $exception) {
             return response()->json(
                 [
@@ -887,7 +891,7 @@ class OrderController extends Controller
         $assignAdhocDriver = $request->input('assign');
 
         try {
-            $order = Order::findRecordOrFail($id, ['payload.waypoints'], []);
+            $order = Order::findRecordOrFail($id, ['payload.waypoints', 'driverAssigned'], []);
         } catch (ModelNotFoundException $exception) {
             return response()->json(
                 [
@@ -1220,14 +1224,11 @@ class OrderController extends Controller
     public function setDestination(string $id, string $placeId)
     {
         try {
-            $order = Order::findRecordOrFail($id);
+            $order = Order::findRecordOrFail($id, ['payload.waypoints', 'payload.pickup', 'payload.dropoff', 'driverAssigned', 'vehicleAssigned', 'customer', 'facilitator']);
         } catch (ModelNotFoundException $exception) {
             return response()->apiError('Order resource not found.', 404);
         }
 
-        // Load required relations
-        $order->loadMissing(['payload.waypoints', 'payload.pickup', 'payload.dropoff']);
-
         // Get the order payload
         $payload = $order->payload;
 
@@ -1567,12 +1568,8 @@ class OrderController extends Controller
      *
      * @return \Feetbase\Models\File
      */
-    protected function storeProofPhoto(
-        Proof $proof,
-        UploadedFile|string $photo,
-        string $disk,
-        string $bucket,
-    ): File {
+    protected function storeProofPhoto(Proof $proof, UploadedFile|string $photo, string $disk, string $bucket): File
+    {
         $isFile      = $photo instanceof UploadedFile;
         $contents    = $isFile
             ? file_get_contents($photo->getRealPath())

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

@@ -27,7 +27,7 @@ class VehicleController extends Controller
     {
         // get request input
         $input = $request->only(['status', 'make', 'model', 'year', 'trim', 'type', 'plate_number', 'vin', 'meta', 'online', 'location', 'altitude', 'heading', 'speed']);
-        
+
         // make sure company is set
         $input['company_uuid'] = session('company');
 

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

@@ -136,7 +136,7 @@ class DriverController extends FleetOpsController
 
                     if ($input->has('user_uuid')) {
                         $user = User::where('uuid', $input->get('user_uuid'))->first();
-                        
+
                         // If user doesn't exist with provided UUID, create new user
                         if (!$user) {
                             $userInput = $input

package/server/src/Http/Requests/Internal/CreateDriverRequest.php

@@ -26,8 +26,8 @@ class CreateDriverRequest extends CreateDriverApiRequest
      */
     public function rules()
     {
-        $isCreating = $this->isMethod('POST');
-        $isCreatingWithUser = $this->filled('driver.user_uuid');
+        $isCreating                   = $this->isMethod('POST');
+        $isCreatingWithUser           = $this->filled('driver.user_uuid');
         $shouldValidateUserAttributes = $isCreating && !$isCreatingWithUser;
 
         return [
@@ -36,13 +36,13 @@ class CreateDriverRequest extends CreateDriverApiRequest
             'email'                  => [
                 Rule::requiredIf($shouldValidateUserAttributes),
                 Rule::when($this->filled('email'), ['email']),
-                Rule::when($shouldValidateUserAttributes, [Rule::unique('users')->whereNull('deleted_at')])
+                Rule::when($shouldValidateUserAttributes, [Rule::unique('users')->whereNull('deleted_at')]),
             ],
             'phone'                  => [
                 Rule::requiredIf($shouldValidateUserAttributes),
-                Rule::when($shouldValidateUserAttributes, [Rule::unique('users')->whereNull('deleted_at')])
+                Rule::when($shouldValidateUserAttributes, [Rule::unique('users')->whereNull('deleted_at')]),
             ],
-            
+
             // Optional fields
             'password'               => 'nullable|string|min:8',
             'drivers_license_number' => 'nullable|string|max:255',
@@ -56,7 +56,7 @@ class CreateDriverRequest extends CreateDriverApiRequest
             'location'               => ['nullable', new ResolvablePoint()],
             'latitude'               => ['nullable', 'required_with:longitude', 'numeric'],
             'longitude'              => ['nullable', 'required_with:latitude', 'numeric'],
-            
+
             // Photo/avatar
             'photo_uuid'             => 'nullable|exists:files,uuid',
             'avatar_uuid'            => 'nullable|exists:files,uuid',

package/server/src/Http/Resources/v1/Driver.php

@@ -50,7 +50,7 @@ class Driver extends FleetbaseResource
             'jobs'                          => $this->whenLoaded('jobs', fn () => $this->getJobs()),
             'vendor'                        => $this->whenLoaded('vendor', fn () => new Vendor($this->vendor)),
             'fleets'                        => $this->whenLoaded('fleets', fn () => Fleet::collection($this->fleets()->without('drivers')->get())),
-            'location'                      => $this->wasRecentlyCreated ? new Point(0, 0) : data_get($this, 'location', new Point(0, 0)),
+            'location'                      => $this->wasRecentlyCreated ? new Point(0, 0) : Utils::castPoint($this->location),
             'heading'                       => (int) data_get($this, 'heading', 0),
             'altitude'                      => (int) data_get($this, 'altitude', 0),
             'speed'                         => (int) data_get($this, 'speed', 0),
@@ -96,7 +96,7 @@ class Driver extends FleetbaseResource
             'vehicle'                => data_get($this, 'vehicle.public_id'),
             'current_job'            => data_get($this, 'currentJob.public_id'),
             'vendor'                 => data_get($this, 'vendor.public_id'),
-            'location'               => data_get($this, 'location', new Point(0, 0)),
+            'location'               => Utils::castPoint($this->location),
             'heading'                => (int) data_get($this, 'heading', 0),
             'altitude'               => (int) data_get($this, 'altitude', 0),
             'speed'                  => (int) data_get($this, 'speed', 0),

package/server/src/Http/Resources/v1/Index/Driver.php

@@ -2,6 +2,7 @@
 
 namespace Fleetbase\FleetOps\Http\Resources\v1\Index;
 
+use Fleetbase\FleetOps\Support\Utils;
 use Fleetbase\Http\Resources\FleetbaseResource;
 use Fleetbase\LaravelMysqlSpatial\Types\Point;
 use Fleetbase\Support\Http;
@@ -35,7 +36,7 @@ class Driver extends FleetbaseResource
             'phone'           => $this->phone,
             'photo_url'       => $this->photo_url,
             'status'          => $this->status,
-            'location'        => $this->wasRecentlyCreated ? new Point(0, 0) : data_get($this, 'location', new Point(0, 0)),
+            'location'        => $this->wasRecentlyCreated ? new Point(0, 0) : Utils::castPoint($this->location),
             'heading'         => (int) data_get($this, 'heading', 0),
             'altitude'        => (int) data_get($this, 'altitude', 0),
             'speed'           => (int) data_get($this, 'speed', 0),

package/server/src/Http/Resources/v1/Index/Place.php

@@ -34,7 +34,7 @@ class Place extends FleetbaseResource
             'city'         => $this->city,
             'country'      => $this->country,
             'avatar_url'   => $this->avatar_url,
-            'location'     => Utils::getPointFromMixed($this->location),
+            'location'     => Utils::castPoint($this->location),
 
             // Meta flag to indicate this is an index resource
             'meta'         => [

package/server/src/Http/Resources/v1/Index/Vehicle.php

@@ -2,8 +2,8 @@
 
 namespace Fleetbase\FleetOps\Http\Resources\v1\Index;
 
+use Fleetbase\FleetOps\Support\Utils;
 use Fleetbase\Http\Resources\FleetbaseResource;
-use Fleetbase\LaravelMysqlSpatial\Types\Point;
 use Fleetbase\Support\Http;
 
 /**
@@ -35,7 +35,7 @@ class Vehicle extends FleetbaseResource
             'year'            => $this->year,
             'photo_url'       => $this->photo_url,
             'status'          => $this->status,
-            'location'        => data_get($this, 'location', new Point(0, 0)),
+            'location'        => Utils::castPoint($this->location),
             'heading'         => (int) data_get($this, 'heading', 0),
             'altitude'        => (int) data_get($this, 'altitude', 0),
             'speed'           => (int) data_get($this, 'speed', 0),

package/server/src/Http/Resources/v1/Place.php

@@ -28,7 +28,7 @@ class Place extends FleetbaseResource
             'owner_uuid'            => $this->when(Http::isInternalRequest(), $this->owner_uuid),
             'owner_type'            => $this->when(Http::isInternalRequest(), $this->owner_type ? Utils::toEmberResourceType($this->owner_type) : null),
             'name'                  => $this->name,
-            'location'              => Utils::getPointFromMixed($this->location),
+            'location'              => Utils::castPoint($this->location),
             'address'               => $this->address,
             'address_html'          => $this->when(Http::isInternalRequest(), $this->address_html),
             'avatar_url'            => $this->avatar_url,

package/server/src/Http/Resources/v1/Vehicle.php

@@ -4,7 +4,6 @@ namespace Fleetbase\FleetOps\Http\Resources\v1;
 
 use Fleetbase\FleetOps\Support\Utils;
 use Fleetbase\Http\Resources\FleetbaseResource;
-use Fleetbase\LaravelMysqlSpatial\Types\Point;
 use Fleetbase\Support\Http;
 
 class Vehicle extends FleetbaseResource
@@ -132,7 +131,7 @@ class Vehicle extends FleetbaseResource
             'updated_at'             => $this->updated_at,
             'created_at'             => $this->created_at,
             // Location & telematics
-            'location'               => data_get($this, 'location', new Point(0, 0)),
+            'location'               => Utils::castPoint($this->location),
             'heading'                => (int) data_get($this, 'heading', 0),
             'altitude'               => (int) data_get($this, 'altitude', 0),
             'speed'                  => (int) data_get($this, 'speed', 0),
@@ -248,7 +247,7 @@ class Vehicle extends FleetbaseResource
             'updated_at'                 => $this->updated_at,
             'created_at'                 => $this->created_at,
             // Location & telematics
-            'location'                   => data_get($this, 'location', new Point(0, 0)),
+            'location'                   => Utils::castPoint($this->location),
             'heading'                    => (int) data_get($this, 'heading', 0),
             'altitude'                   => (int) data_get($this, 'altitude', 0),
             'speed'                      => (int) data_get($this, 'speed', 0),

package/server/src/Http/Resources/v1/VehicleWithoutDriver.php

@@ -4,7 +4,6 @@ namespace Fleetbase\FleetOps\Http\Resources\v1;
 
 use Fleetbase\FleetOps\Support\Utils;
 use Fleetbase\Http\Resources\FleetbaseResource;
-use Fleetbase\LaravelMysqlSpatial\Types\Point;
 use Fleetbase\Support\Http;
 
 class VehicleWithoutDriver extends FleetbaseResource
@@ -131,7 +130,7 @@ class VehicleWithoutDriver extends FleetbaseResource
             'updated_at'             => $this->updated_at,
             'created_at'             => $this->created_at,
             // Location & telematics
-            'location'               => data_get($this, 'location', new Point(0, 0)),
+            'location'               => Utils::castPoint($this->location),
             'heading'                => (int) data_get($this, 'heading', 0),
             'altitude'               => (int) data_get($this, 'altitude', 0),
             'speed'                  => (int) data_get($this, 'speed', 0),
@@ -245,7 +244,7 @@ class VehicleWithoutDriver extends FleetbaseResource
             'updated_at'                 => $this->updated_at,
             'created_at'                 => $this->created_at,
             // Location & telematics
-            'location'                   => data_get($this, 'location', new Point(0, 0)),
+            'location'                   => Utils::castPoint($this->location),
             'heading'                    => (int) data_get($this, 'heading', 0),
             'altitude'                   => (int) data_get($this, 'altitude', 0),
             'speed'                      => (int) data_get($this, 'speed', 0),

package/server/src/Models/Payload.php

@@ -245,6 +245,19 @@ class Payload extends Model
                 }
             }
 
+            // Handle entity photo upload
+            if (isset($attributes['photo'])) {
+                $path = 'uploads/' . session('company') . '/entities';
+                $file = app(\Fleetbase\Services\FileResolverService::class)->resolve($attributes['photo'], $path);
+
+                if ($file) {
+                    $attributes['photo_uuid'] = $file->uuid;
+                }
+
+                // Clean up raw photo data
+                unset($attributes['photo']);
+            }
+
             $entity = new Entity($attributes);
             $this->entities()->save($entity);
         }
@@ -286,6 +299,19 @@ class Payload extends Model
                 }
             }
 
+            // Handle entity photo upload
+            if (isset($attributes['photo'])) {
+                $path = 'uploads/' . session('company') . '/entities';
+                $file = app(\Fleetbase\Services\FileResolverService::class)->resolve($attributes['photo'], $path);
+
+                if ($file) {
+                    $attributes['photo_uuid'] = $file->uuid;
+                }
+
+                // Clean up raw photo data
+                unset($attributes['photo']);
+            }
+
             Entity::insertGetUuid($attributes, $this);
         }
 

package/server/src/Models/ServiceRate.php

@@ -12,7 +12,6 @@ use Fleetbase\Traits\HasPublicId;
 use Fleetbase\Traits\HasUuid;
 use Fleetbase\Traits\SendsWebhooks;
 use Fleetbase\Traits\TracksApiCredential;
-use Illuminate\Support\Facades\DB;
 
 class ServiceRate extends Model
 {
@@ -469,12 +468,11 @@ class ServiceRate extends Model
      */
     public static function getServicableForPlaces($places = [], $service = null, $currency = null, ?\Closure $queryCallback = null): array
     {
-        $reader                 = new GeoJSONReader();
-        $applicableServiceRates = [];
-        $serviceRatesQuery      = static::with(['zone', 'serviceArea', 'rateFees', 'parcelFees']);
+        $reader            = new GeoJSONReader();
+        $serviceRatesQuery = static::with(['zone', 'serviceArea', 'rateFees', 'parcelFees']);
 
         if ($currency) {
-            $serviceRatesQuery->where(DB::raw('lower(currency)'), strtolower($currency));
+            $serviceRatesQuery->whereRaw('lower(currency) = ?', [strtolower($currency)]);
         }
 
         if ($service) {
@@ -487,44 +485,115 @@ class ServiceRate extends Model
 
         $serviceRates = $serviceRatesQuery->get();
 
-        $waypoints = collect($places)->map(function ($place) {
-            $place = Place::createFromMixed($place);
+        $waypoints = collect($places)
+            ->map(function ($place) {
+                $place = Place::createFromMixed($place);
+
+                if (!$place instanceof Place) {
+                    return null;
+                }
 
-            if ($place instanceof Place) {
                 $point = $place->getLocationAsPoint();
 
-                // Conver to brick gis point
-                return \Brick\Geo\Point::fromText(sprintf('POINT (%F %F)', $point->getLng(), $point->getLat()), 4326);
+                // Brick point: X=lng, Y=lat (WKT order)
+                return \Brick\Geo\Point::fromText(
+                    sprintf('POINT (%F %F)', $point->getLng(), $point->getLat()),
+                    4326
+                );
+            })
+            ->filter()
+            ->values();
+
+        if ($waypoints->isEmpty()) {
+            return [];
+        }
+
+        /**
+         * Convert a casted spatial geometry (Zone::border / ServiceArea::border)
+         * into a Brick geometry using GeoJSONReader.
+         */
+        $toBrickGeometry = function ($spatialGeometry) use ($reader) {
+            if (!$spatialGeometry) {
+                return null;
+            }
+
+            // Most Fleetbase spatial casts/types implement toJson()
+            if (is_object($spatialGeometry) && method_exists($spatialGeometry, 'toJson')) {
+                $json = $spatialGeometry->toJson();
+                $json = is_string($json) ? trim($json) : null;
+
+                if ($json) {
+                    return $reader->read($json);
+                }
+
+                return null;
+            }
+
+            // Fallback if the cast ever returns array/object
+            if (is_array($spatialGeometry) || is_object($spatialGeometry)) {
+                $json = json_encode($spatialGeometry, JSON_UNESCAPED_UNICODE);
+                if ($json && $json !== 'null') {
+                    return $reader->read($json);
+                }
+            }
+
+            // Fallback if it’s a raw JSON string
+            if (is_string($spatialGeometry) && trim($spatialGeometry) !== '') {
+                return $reader->read($spatialGeometry);
+            }
+
+            return null;
+        };
+
+        /**
+         * Ensure ALL waypoints are inside the given Brick geometry.
+         */
+        $containsAllWaypoints = function ($brickGeometry) use ($waypoints): bool {
+            if (!$brickGeometry) {
+                return false;
+            }
+
+            foreach ($waypoints as $waypoint) {
+                if (!$brickGeometry->contains($waypoint)) {
+                    return false;
+                }
             }
-        });
+
+            return true;
+        };
+
+        $applicableServiceRates = [];
 
         foreach ($serviceRates as $serviceRate) {
+            // If a service area exists, all waypoints must be inside its border
             if ($serviceRate->hasServiceArea()) {
-                // make sure all waypoints fall within the service area
-                foreach ($serviceRate->serviceArea->border as $polygon) {
-                    $polygon = $reader->read($polygon->toJson());
-
-                    /** @var \Brick\Geo\Point $waypoint */
-                    foreach ($waypoints as $waypoint) {
-                        if (!$polygon->contains($waypoint)) {
-                            // waypoint outside of service area, not applicable to route
-                            continue;
-                        }
-                    }
+                $serviceAreaBorder = $serviceRate->serviceArea?->border;
+
+                $serviceAreaGeom = null;
+                try {
+                    $serviceAreaGeom = $toBrickGeometry($serviceAreaBorder);
+                } catch (\Throwable $e) {
+                    continue; // invalid geojson / geometry -> reject this rate
+                }
+
+                if (!$containsAllWaypoints($serviceAreaGeom)) {
+                    continue;
                 }
             }
 
+            // If a zone exists, all waypoints must be inside its border
             if ($serviceRate->hasZone()) {
-                // make sure all waypoints fall within the service area
-                foreach ($serviceRate->zone->border as $polygon) {
-                    $polygon = $reader->read($polygon->toJson());
+                $zoneBorder = $serviceRate->zone?->border;
 
-                    foreach ($waypoints as $waypoint) {
-                        if (!$polygon->contains($waypoint)) {
-                            // waypoint outside of zone, not applicable to route
-                            continue;
-                        }
-                    }
+                $zoneGeom = null;
+                try {
+                    $zoneGeom = $toBrickGeometry($zoneBorder);
+                } catch (\Throwable $e) {
+                    continue;
+                }
+
+                if (!$containsAllWaypoints($zoneGeom)) {
+                    continue;
                 }
             }
 
@@ -933,29 +1002,23 @@ class ServiceRate extends Model
      */
     public function findServiceRateFeeByDistance(int $totalDistance): ?ServiceRateFee
     {
-        $this->load('rateFees');
+        $this->loadMissing('rateFees');
 
-        $distanceInKms = round($totalDistance / 1000);
-        $distanceFee   = null;
+        // Convert meters to kilometers WITHOUT rounding up
+        $distanceInKm = $totalDistance / 1000;
 
-        foreach ($this->rateFees as $rateFee) {
-            $previousRateFee = $rateFee;
+        // Ensure predictable order
+        $rateFees = $this->rateFees->sortBy('distance');
 
-            if ($distanceInKms > $rateFee->distance) {
-                continue;
-            } elseif ($rateFee->distance > $distanceInKms) {
-                $distanceFee = $previousRateFee;
-            } else {
-                $distanceFee = $rateFee;
+        // Find the first tier that covers the distance
+        foreach ($rateFees as $rateFee) {
+            if ($distanceInKm <= $rateFee->distance) {
+                return $rateFee;
             }
         }
 
-        // if no distance fee use the last
-        if ($distanceFee === null) {
-            $distanceFee = $this->rateFees->sortByDesc('distance')->first();
-        }
-
-        return $distanceFee;
+        // If distance exceeds all tiers, use the largest tier
+        return $rateFees->last();
     }
 
     /**

package/server/src/Support/Utils.php

@@ -334,6 +334,24 @@ class Utils extends FleetbaseUtils
         return new Point((float) $latitude, (float) $longitude);
     }
 
+    /**
+     * Always return spatial point.
+     *
+     * @param [type] $mixed
+     *
+     * @return void
+     */
+    public static function castPoint($mixed)
+    {
+        try {
+            $point = static::getPointFromMixed($mixed);
+
+            return $point;
+        } catch (\Throwable $e) {
+            return new Point(0, 0);
+        }
+    }
+
     /**
      * Determines if the given coordinates strictly represent a Point object.
      * These will explude resolvable coordinates from records.