From cacc5426fca9f333547d4db582f52a5a6362d54b Mon Sep 17 00:00:00 2001
From: kshitige <mkshitige@gmail.com>
Date: Tue, 15 Apr 2025 19:12:46 +0530
Subject: [PATCH] deleteAsset changes

---
 .../APIS/AdminApi/AssetadmintController.php   | 57 +++++++++++--------
 1 file changed, 34 insertions(+), 23 deletions(-)

diff --git a/app/Http/Controllers/APIS/AdminApi/AssetadmintController.php b/app/Http/Controllers/APIS/AdminApi/AssetadmintController.php
index dc8d7b1..a23a7f5 100644
--- a/app/Http/Controllers/APIS/AdminApi/AssetadmintController.php
+++ b/app/Http/Controllers/APIS/AdminApi/AssetadmintController.php
@@ -162,34 +162,45 @@ class AssetadmintController extends Controller
             return jsonResponseWithErrorMessage('Asset ID is required', 400);
         }
 
-        $response = $this->adminService->deleteAsset(['assetId' => $assetId]);
-        Log::info("Response: " . json_encode($response));
+        try {
+            $response = $this->adminService->deleteAsset(['assetId' => $assetId]);
+            Log::info("Asset delete API response", ['response' => $response]);
 
-        if (!is_array($response)) {
-            Log::error("Unexpected API response format.", ['response' => $response]);
-            return jsonResponseWithErrorMessage('Unexpected API response format', 500);
+            if (!is_array($response)) {
+                Log::error("Unexpected API response format", ['response' => $response]);
+                return jsonResponseWithErrorMessage('Unexpected API response format', 500);
+            }
+
+            if (isset($response['status']) && $response['status'] === 400) {
+                Log::error("API failed to delete asset", ['message' => $response['message']]);
+                return jsonResponseWithErrorMessage($response['message'], 400, $response);
+            }
+
+            if (empty($response)) {
+                Log::error("API response is empty while deleting asset");
+                return jsonResponseWithErrorMessage('Failed to delete asset', 400);
+            }
+
+            // Delete from local database
+            $asset = Asset::find($assetId);
+            if ($asset) {
+                $asset->delete();
+            } else {
+                Log::warning("Asset not found in DB", ['assetId' => $assetId]);
+            }
+
+            return jsonResponseWithSuccessMessage('Asset deleted successfully', ['api_response' => $response]);
+        } catch (\Exception $e) {
+            Log::error("Exception while deleting asset", [
+                'message' => $e->getMessage(),
+                'trace' => $e->getTraceAsString(),
+            ]);
+            return jsonResponseWithErrorMessage('An error occurred while deleting the asset', 500);
         }
-
-        if (isset($response['status']) && $response['status'] === 400) {
-            Log::error("Failed to delete asset: " . $response['message']);
-            return jsonResponseWithErrorMessage($response['message'], 400, $response);
-        }
-
-        if (empty($response)) {
-            Log::error("API Data is empty, cannot delete asset.");
-            return jsonResponseWithErrorMessage('Failed to delete asset', 400);
-        }
-
-        // Delete from local database
-        $asset = Asset::where('id', $assetId)->first();
-        if ($asset) {
-            $asset->delete();
-        }
-
-        return jsonResponseWithSuccessMessage('Asset deleted successfully', ['api_response' => $response]);
     }
 
 
+
     public function assignAssetToUser(Request $request)
     {