From 7c2aed2d580cf4898421a8f511b32209aec2f5cd Mon Sep 17 00:00:00 2001
From: Faris Armoush <farmoush@uba.com.jo>
Date: Tue, 8 Jul 2025 12:20:10 +0300
Subject: [PATCH] Refactor RemoteUpdateSpaceService: Improved error handling in
 updateSpace method by checking API response success before returning the
 updated space. This enhances robustness and ensures proper error propagation
 for failed updates.

---
 .../data/services/remote_update_space_service.dart  | 13 +++++++------
 1 file changed, 7 insertions(+), 6 deletions(-)

diff --git a/lib/pages/space_management_v2/modules/update_space/data/services/remote_update_space_service.dart b/lib/pages/space_management_v2/modules/update_space/data/services/remote_update_space_service.dart
index 452a7375..b595e2b9 100644
--- a/lib/pages/space_management_v2/modules/update_space/data/services/remote_update_space_service.dart
+++ b/lib/pages/space_management_v2/modules/update_space/data/services/remote_update_space_service.dart
@@ -17,19 +17,20 @@ class RemoteUpdateSpaceService implements UpdateSpaceService {
   Future<SpaceDetailsModel> updateSpace(UpdateSpaceParam param) async {
     try {
       final path = await _makeUrl(param);
-      final response = await _httpService.put(
+      await _httpService.put(
         path: path,
         body: param.space.toJson(),
         expectedResponseModel: (data) {
           final response = data as Map<String, dynamic>;
-          final space = SpaceDetailsModel.fromJson(
-            response['data'] as Map<String, dynamic>,
-          );
-          return space;
+          final isSuccess = response['success'] as bool;
+          if (!isSuccess) {
+            throw APIException(response['error'] as String);
+          }
+          return isSuccess;
         },
       );
 
-      return response;
+      return param.space;
     } on DioException catch (e) {
       final message = e.response?.data as Map<String, dynamic>?;
       final error = message?['error'] as Map<String, dynamic>?;