fix: do not crash on not loaded device attributes

The device attributes when not loaded were not `nil`, but rather an `Ash`
internal representation of it. When trying to query for them the server crashed,
making it impossible to read any device.

This PR removes the code handling this case and makes use of ash internal cache
system.

Signed-off-by: Luca Zaninotto <luca.zaninotto@secomind.com>

Author: lusergit

backend/lib/edgehog/geolocation/providers/ipbase.ex

@@ -1,7 +1,6 @@
-#
 # This file is part of Edgehog.
 #
-# Copyright 2021 - 2025 SECO Mind Srl
+# Copyright 2021-2026 SECO Mind Srl
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
 # you may not use this file except in compliance with the License.
@@ -16,7 +15,6 @@
 # limitations under the License.
 #
 # SPDX-License-Identifier: Apache-2.0
-#
 
 defmodule Edgehog.Geolocation.Providers.IPBase do
   @moduledoc false
@@ -27,8 +25,6 @@ defmodule Edgehog.Geolocation.Providers.IPBase do
   alias Edgehog.EdgehogTeslaClient
   alias Edgehog.Geolocation.Position
 
-  @device_status_attributes [:last_connection, :last_disconnection, :last_seen_ip]
-
   @impl Edgehog.Geolocation.GeolocationProvider
   def geolocate(%Device{} = device) do
     with {:ok, device_status} <- fetch_device_status(device) do
@@ -49,25 +45,12 @@ defmodule Edgehog.Geolocation.Providers.IPBase do
   end
 
   defp fetch_device_status(%Device{} = device) do
-    if uninitialized_device_status?(device) do
-      with {:ok, device_data} <- Ash.load(device, :device_status),
-           :ok <- validate_device_status_exists(device_data.device_status) do
-        {:ok, device_data.device_status}
-      end
-    else
-      {:ok,
-       %{
-         last_seen_ip: device.last_seen_ip,
-         last_connection: device.last_connection,
-         last_disconnection: device.last_disconnection
-       }}
+    with {:ok, device_data} <- Ash.load(device, :device_status),
+         :ok <- validate_device_status_exists(device_data.device_status) do
+      {:ok, device_data.device_status}
     end
   end
 
-  defp uninitialized_device_status?(device) do
-    Enum.any?(@device_status_attributes, &(Map.get(device, &1) == nil))
-  end
-
   defp validate_device_status_exists(nil), do: {:error, :device_status_not_found}
   defp validate_device_status_exists(_), do: :ok