Fix Location model decimal cast mismatch causing API 500 errors

app/Models/Location.php

@@ -48,13 +48,13 @@ class Location extends Model
     private $location_ignore_regex = '/\(.*?\)/';
 
     /**
-     * @return array{lat: 'float', lng: 'float', fixed_coordinates: 'bool'}
+     * @return array{lat: 'decimal:8', lng: 'decimal:8', fixed_coordinates: 'bool'}
      */
     protected function casts(): array
     {
         return [
-            'lat' => 'float',
-            'lng' => 'float',
+            'lat' => 'decimal:8',
+            'lng' => 'decimal:8',
             'fixed_coordinates' => 'bool',
         ];
     }
@@ -77,7 +77,7 @@ public function hasCoordinates()
      */
     public function coordinatesValid()
     {
-        return $this->lat && $this->lng &&
+        return $this->lat != 0 && $this->lng != 0 &&
             abs($this->lat) <= 90 && abs($this->lng) <= 180;
     }
 

tests/Feature/ApiLocationTest.php

@@ -0,0 +1,148 @@
+<?php
+
+/**
+ * ApiLocationTest.php
+ *
+ * Test for Location API endpoints
+ *
+ * This program is free software: you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation, either version 3 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program.  If not, see <https://www.gnu.org/licenses/>.
+ *
+ * @link       https://www.librenms.org
+ *
+ * @copyright  2025 LibreNMS
+ * @author     LibreNMS Contributors
+ */
+
+namespace Tests\Feature;
+
+use App\Models\ApiToken;
+use App\Models\Location;
+use App\Models\User;
+use Illuminate\Foundation\Testing\DatabaseTransactions;
+use LibreNMS\Tests\DBTestCase;
+
+final class ApiLocationTest extends DBTestCase
+{
+    use DatabaseTransactions;
+
+    public function testEditLocationWithDecimalCoordinates(): void
+    {
+        /** @var User $user */
+        $user = User::factory()->admin()->create();
+        $token = ApiToken::generateToken($user);
+
+        /** @var Location $location */
+        $location = Location::factory()->create([
+            'location' => 'Test Location',
+            'lat' => '51.50354111',
+            'lng' => '-0.12766972',
+        ]);
+
+        // Update the location with new coordinates
+        $newLat = '40.71277800';
+        $newLng = '-74.00597200';
+
+        $response = $this->json(
+            'PATCH',
+            "/api/v0/locations/{$location->id}",
+            [
+                'lat' => $newLat,
+                'lng' => $newLng,
+            ],
+            ['X-Auth-Token' => $token->token_hash]
+        );
+
+        $response->assertStatus(201)
+            ->assertJson([
+                'status' => 'ok',
+                'message' => 'Location updated successfully',
+            ]);
+
+        // Verify the database was updated correctly
+        $location->refresh();
+        $this->assertEquals($newLat, $location->lat);
+        $this->assertEquals($newLng, $location->lng);
+    }
+
+    public function testEditLocationByName(): void
+    {
+        /** @var User $user */
+        $user = User::factory()->admin()->create();
+        $token = ApiToken::generateToken($user);
+
+        /** @var Location $location */
+        $location = Location::factory()->create([
+            'location' => 'Test Location Name',
+            'lat' => '51.50354111',
+            'lng' => '-0.12766972',
+        ]);
+
+        // Update the location by name with new coordinates
+        $newLat = '48.85661400';
+        $newLng = '2.35222190';
+
+        $response = $this->json(
+            'PATCH',
+            '/api/v0/locations/' . urlencode($location->location),
+            [
+                'lat' => $newLat,
+                'lng' => $newLng,
+            ],
+            ['X-Auth-Token' => $token->token_hash]
+        );
+
+        $response->assertStatus(201)
+            ->assertJson([
+                'status' => 'ok',
+                'message' => 'Location updated successfully',
+            ]);
+
+        // Verify the database was updated correctly
+        $location->refresh();
+        $this->assertEquals($newLat, $location->lat);
+        $this->assertEquals($newLng, $location->lng);
+    }
+
+    public function testEditLocationWithNullCoordinates(): void
+    {
+        /** @var User $user */
+        $user = User::factory()->admin()->create();
+        $token = ApiToken::generateToken($user);
+
+        /** @var Location $location */
+        $location = Location::factory()->create([
+            'location' => 'Test Location Null',
+            'lat' => '51.50354111',
+            'lng' => '-0.12766972',
+        ]);
+
+        // Update the location with null coordinates
+        $response = $this->json(
+            'PATCH',
+            "/api/v0/locations/{$location->id}",
+            [
+                'lat' => null,
+                'lng' => null,
+            ],
+            ['X-Auth-Token' => $token->token_hash]
+        );
+
+        $response->assertStatus(201);
+
+        // Verify the database was updated correctly
+        $location->refresh();
+        $this->assertNull($location->lat);
+        $this->assertNull($location->lng);
+    }
+}