WebSVN – LeadersLinked - Services – Diff – /module/LeadersLinked/src/Controller/MyProfilesController.php

         $request = $this->getRequest();
         $id = $this->params()->fromRoute('id');
-        if (!$id) {
-            $data = [
+        if (!$id) {
-                'success'   => false,
-                'data'   => 'ERROR_INVALID_PARAMETER'
-            ];
             // Use helper for consistency
-            return new JsonModel($data);
-        }
             return $this->_createSimpleErrorResponse('ERROR_INVALID_PARAMETER');
+        }
-        $userProfileMapper = UserProfileMapper::getInstance($this->adapter);
-        $userProfile = $userProfileMapper->fetchOneByUuid($id);
+        $userProfileMapper = UserProfileMapper::getInstance($this->adapter);
-        if (!$userProfile) {
+        $userProfile = $userProfileMapper->fetchOneByUuid($id);
-            $data = [
-                'success'   => false,
-                'data'   => 'ERROR_RECORD_NOT_FOUND'
-            ];
+        if (!$userProfile) {
+             // Use helper for consistency
-            return new JsonModel($data);
+            return $this->_createSimpleErrorResponse('ERROR_RECORD_NOT_FOUND');
-        }
+        }
-        if ($currentUser->id != $userProfile->user_id) {
-            $response = [
-                'success' => false,
-                'data' => 'ERROR_UNAUTHORIZED'
+        // Authorize: Check if current user owns the profile
+        if ($currentUser->id != $userProfile->user_id) {
-            ];
+             // Use helper for consistency
-            return new JsonModel($response);
+            return $this->_createSimpleErrorResponse('ERROR_UNAUTHORIZED');
         }
-        if (!$userProfile->public == UserProfile::PUBLIC_YES) {
-            $data = [
                 'success'   => false,
-                'data'   => 'ERROR_PUBLIC_PROFILE'
+        // Prevent deletion of the public profile
+        if ($userProfile->public == UserProfile::PUBLIC_YES) {
+            // Use helper for consistency
+            return $this->_createSimpleErrorResponse('ERROR_PUBLIC_PROFILE');
+        }
+        if ($request->isPost()) {
+            $storage = Storage::getInstance($this->config, $this->adapter);
+            $target_path = $storage->getPathUser();
+            $user_uuid = $currentUser->uuid; // Assuming user UUID matches profile owner's UUID for path
+            // Attempt to delete associated files first
+            $image_deleted = true;
+            if ($userProfile->image) {
+                $image_deleted = $storage->deleteFile($target_path, $user_uuid, $userProfile->image);
+                if (!$image_deleted) {
+                    $this->logger->err("Failed to delete profile image file: {$target_path}/{$user_uuid}/{$userProfile->image}", ['user_id' => $userProfile->user_id]);
+                    // Decide if this is a critical error. Proceeding with DB deletion for now.
+                }
+            }
+            $cover_deleted = true;
+            if ($userProfile->cover) {
+                $cover_deleted = $storage->deleteFile($target_path, $user_uuid, $userProfile->cover);
+                 if (!$cover_deleted) {
-            ];
+                    $this->logger->err("Failed to delete profile cover file: {$target_path}/{$user_uuid}/{$userProfile->cover}", ['user_id' => $userProfile->user_id]);
                     // Decide if this is a critical error. Proceeding with DB deletion for now.
             return new JsonModel($data);
+            }
             // Delete the database record
-        if ($request->isPost()) {
+            $result = $userProfileMapper->delete($userProfile);
-            $result = $userProfileMapper->delete($userProfile);
+            if ($result) {
-            if ($result) {
+                $this->logger->info('Se borro el perfil ' . ($userProfile->public == UserProfile::PUBLIC_YES ? 'público' : $userProfile->name), ['user_id' => $userProfile->user_id, 'ip' => Functions::getUserIP()]);
-                $this->logger->info('Se borro el perfil ' . ($userProfile->public == UserProfile::PUBLIC_YES ? 'público' : $userProfile->name), ['user_id' => $userProfile->user_id, 'ip' => Functions::getUserIP()]);
-                $data = [
-                    'success' => true,
+                // Success response
-                    'data' => 'LABEL_RECORD_DELETED'
-                ];
-            } else {
                 $data = [
-                $data = [
+                    'success' => true,
-                    'success'   => false,
-                    'data'      => $userProfileMapper->getError()
+                    'data' => 'LABEL_RECORD_DELETED'
                 ];
-                return new JsonModel($data);
-            }
-        } else {
+            } else {
-            $data = [
+                // Use helper for DB error
+                $data = $this->_createSimpleErrorResponse($userProfileMapper->getError() ?: 'ERROR_THERE_WAS_AN_ERROR');
                 'success' => false,
-                'data' => 'ERROR_METHOD_NOT_ALLOWED'
+        } else {
-            ];
+            // Use helper for method error
             $data = $this->_createSimpleErrorResponse('ERROR_METHOD_NOT_ALLOWED');
         $userProfileMapper = UserProfileMapper::getInstance($this->adapter);
         $userProfile = $userProfileMapper->fetchOneByUuid($user_profile_id);
-        if (!$userProfile) {
-            $response = [
+        if (!$userProfile) {
-                'success' => false,
-                'data' => 'ERROR_INVALID_PARAMETER'
-            ];
             // Use helper
+            return $this->_createSimpleErrorResponse('ERROR_INVALID_PARAMETER');
             return new JsonModel($response);
-        }
-        if ($currentUser->id != $userProfile->user_id) {
+        // Authorize: Check if current user owns the profile
-            $response = [
-                'success' => false,
-                'data' => 'ERROR_UNAUTHORIZED'
-            ];
+        if ($currentUser->id != $userProfile->user_id) {
-            return new JsonModel($response);
             // Use helper
             return $this->_createSimpleErrorResponse('ERROR_UNAUTHORIZED');
+        }
+        $request = $this->getRequest();
-        $request = $this->getRequest();
+        if ($request->isPost()) {
         if ($request->isPost()) {
             $storage = Storage::getInstance($this->config, $this->adapter);
+            $target_path = $storage->getPathUser();
-            $storage = Storage::getInstance($this->config, $this->adapter);
+            $user_uuid = $currentUser->uuid; // Assuming user UUID for path
-            $target_path = $storage->getPathUser();
             if ($operation == 'delete') {
-                $this->logger->info('Se borro el image del perfil ' . ($userProfile->public == UserProfile::PUBLIC_YES ? 'público' : $userProfile->name), ['user_id' => $userProfile->user_id, 'ip' => Functions::getUserIP()]);
                 $this->logger->info('Se borro el image del perfil ' . ($userProfile->public == UserProfile::PUBLIC_YES ? 'público' : $userProfile->name), ['user_id' => $userProfile->user_id, 'ip' => Functions::getUserIP()]);
                 if ($userProfile->image) {
+                if ($userProfile->image) {
-                    if (!$storage->deleteFile($target_path, $currentUser->uuid, $userProfile->image)) {
+                    // Attempt deletion
-                        return new JsonModel([
+                    if (!$storage->deleteFile($target_path, $user_uuid, $userProfile->image)) {
-                            'success'   => false,
-                            'data'   =>  'ERROR_THERE_WAS_AN_ERROR'
+                        // Use helper
-                        ]);
+                        return $this->_createSimpleErrorResponse('ERROR_THERE_WAS_AN_ERROR');
-                    }
+                    }
+                }
                 $userProfile->image = '';
-                if (!$userProfileMapper->updateImage($userProfile)) {
+                // Clear image field and update DB
-                    return new JsonModel([
+                $userProfile->image = '';
-                        'success'   => false,
+                if (!$userProfileMapper->updateImage($userProfile)) {
-                        'data'   =>  'ERROR_THERE_WAS_AN_ERROR'
+                    // Use helper
-                    ]);
-                }
+                    return $this->_createSimpleErrorResponse('ERROR_THERE_WAS_AN_ERROR');
-            } else {
-                $form = new ImageForm($this->config);
+                }
-                $data     = array_merge($request->getPost()->toArray(), $request->getFiles()->toArray());
+            } else { // Handle upload
+                $form = new ImageForm($this->config);
-                $form->setData($data);
                 $data = array_merge($request->getPost()->toArray(), $request->getFiles()->toArray());
-                if ($form->isValid()) {
+                $form->setData($data);
+                if ($form->isValid()) {
-                    $storage->setFiles($request->getFiles()->toArray());
                     $storage->setFiles($request->getFiles()->toArray());
-                    if (!$storage->setCurrentFilename('error')) {
+                    // *** FIX: Use correct input field name 'image' ***
+                    if (!$storage->setCurrentFilename('image')) {
-                        return new JsonModel([
+                        // Use helper
-                            'success'   => false,
+                        return $this->_createSimpleErrorResponse('ERROR_UPLOAD_FILE');
-                            'data'   =>  'ERROR_UPLOAD_FILE'
-                        ]);
+                    }
-                    }
                     // Prepare for resize and save
+                    list($target_width_str, $target_height_str) = explode('x', $this->config['leaderslinked.image_sizes.user_size']);
                     $target_width = (int)$target_width_str;
-                    list($target_width, $target_height) = explode('x', $this->config['leaderslinked.image_sizes.user_size']);
+                    $target_height = (int)$target_height_str;
-                    $source_filename    = $storage->getTmpFilename();
-                    $filename           = 'user-profile-' . uniqid() . '.png';
-                    $target_filename    = $storage->composePathToFilename(Storage::TYPE_USER, $currentUser->uuid, $filename);
+                    $source_filename    = $storage->getTmpFilename();
                     $filename           = 'user-profile-' . uniqid() . '.png';
-                    if (!$storage->uploadImageResize($source_filename, $target_filename, $target_width, $target_height)) {
+                    $target_filename    = $storage->composePathToFilename(Storage::TYPE_USER, $user_uuid, $filename);
-                        return new JsonModel([
+                    // Upload, resize, save
-                            'success'   => false,
+                    if (!$storage->uploadImageResize($source_filename, $target_filename, $target_width, $target_height)) {
-                            'data'   =>  'ERROR_THERE_WAS_AN_ERROR'
-                        ]);
+                        // Use helper
-                    }
+                        return $this->_createSimpleErrorResponse('ERROR_THERE_WAS_AN_ERROR');
                     if ($userProfile->image) {
-                        if (!$storage->deleteFile($target_path, $currentUser->uuid, $userProfile->image)) {
+                    // Delete old image if exists
-                            return new JsonModel([
+                    if ($userProfile->image) {
-                                'success'   => false,
+                        if (!$storage->deleteFile($target_path, $user_uuid, $userProfile->image)) {
-                                'data'   =>  'ERROR_THERE_WAS_AN_ERROR'
+                            // Use helper
-                            ]);
-                        }
+                            return $this->_createSimpleErrorResponse('ERROR_THERE_WAS_AN_ERROR');
-                    }
+                        }
+                    }
-                    $userProfile->image = $filename;
-                    if (!$userProfileMapper->updateImage($userProfile)) {
-                        return new JsonModel([
+                    // Update profile model and DB
+                    $userProfile->image = $filename;
-                            'success'   => false,
+                    if (!$userProfileMapper->updateImage($userProfile)) {
-                            'data'   =>  'ERROR_THERE_WAS_AN_ERROR'
+                        // Use helper
-                        ]);
-                    }
+                        return $this->_createSimpleErrorResponse('ERROR_THERE_WAS_AN_ERROR');
+                    }
-                    if ($userProfile->public == UserProfile::PUBLIC_YES) {
-                        $filename    = 'user-' . uniqid() . '.png';
+                    // Special handling: If this is the PUBLIC profile, also update the main user image
+                    if ($userProfile->public == UserProfile::PUBLIC_YES) {
-                        $source_filename = $target_filename;
-                        $target_filename    = $storage->composePathToFilename(Storage::TYPE_USER, $currentUser->uuid, $filename);
                         $main_user_filename = 'user-' . uniqid() . '.png'; // Use different name for safety?
+                        // $source_filename_for_copy = $target_filename; // The newly saved profile image
+                        $target_filename_main_user = $storage->composePathToFilename(Storage::TYPE_USER, $user_uuid, $main_user_filename);
+                        // Copy the *newly processed* profile image to be the main user image
-                        if (!$storage->copyFile($source_filename, $target_filename)) {
+                        if (!$storage->copyFile($target_filename, $target_filename_main_user)) { // Source is the *new* profile image
-                            return new JsonModel([
+                            // Use helper
-                                'success'   => false,
+                             return $this->_createSimpleErrorResponse('ERROR_THERE_WAS_AN_ERROR');
-                                'data'   =>  'ERROR_THERE_WAS_AN_ERROR'
+                        }
-                            ]);
-                        }
-                        if ($currentUser->image) {
-                            if (!$storage->deleteFile($target_path, $currentUser->uuid, $currentUser->image)) {
-                                return new JsonModel([
-                                    'success'   => false,
-                                    'data'   =>  'ERROR_THERE_WAS_AN_ERROR'
-                                ]);
                         // Delete the old main user image if it exists
-                        }
+                        if ($currentUser->image) {
+                            if (!$storage->deleteFile($target_path, $user_uuid, $currentUser->image)) {
+                                 // Use helper
                                 return $this->_createSimpleErrorResponse('ERROR_THERE_WAS_AN_ERROR');
-                        $currentUser->image = $filename;
+                            }
                         $userMapper = UserMapper::getInstance($this->adapter);
                         $userMapper->updateImage($currentUser);
                         // Update current user model and DB
                         $currentUser->image = $main_user_filename;
-                    $this->logger->info('Se actualizo el image del perfil ' . ($userProfile->public == UserProfile::PUBLIC_YES ? 'público' : $userProfile->name), ['user_id' => $userProfile->user_id, 'ip' => Functions::getUserIP()]);
+                        $userMapper = UserMapper::getInstance($this->adapter);
-                } else {
-                    $messages = [];
+                        if (!$userMapper->updateImage($currentUser)) {
-                    $form_messages = (array) $form->getMessages();
+                             // Use helper
-                    foreach ($form_messages  as $fieldname => $field_messages) {
+                             return $this->_createSimpleErrorResponse('ERROR_THERE_WAS_AN_ERROR');
-                        $messages[$fieldname] = array_values($field_messages);
-                    }
+                        }
+                    }
-                    return new JsonModel([
-                        'success'   => false,
-                        'data'   => $messages
-                    ]);
-                }
                     $this->logger->info('Se actualizo el image del perfil ' . ($userProfile->public == UserProfile::PUBLIC_YES ? 'público' : $userProfile->name), ['user_id' => $userProfile->user_id, 'ip' => Functions::getUserIP()]);
                 } else {
-            $storage = Storage::getInstance($this->config, $this->adapter);
+                    // Use helper for form errors
                     return $this->_createFormErrorResponse($form);
                     // Set files in storage and select the 'cover' file by its input name
                     $storage->setFiles($request->getFiles()->toArray());
                     if (!$storage->setCurrentFilename('cover')) { // Ensure 'cover' is the correct file input name from CoverForm
                         return $this->_createSimpleErrorResponse('ERROR_UPLOAD_FILE');
+                    }
                     // Prepare for image resize and save: Get target dimensions from config
                     // Example config value: '1200x400'
                     list($target_width_str, $target_height_str) = explode('x', $this->config['leaderslinked.image_sizes.user_cover_size']);
                     $target_width = (int)$target_width_str;
                     $target_height = (int)$target_height_str;

Proyectos de Subversion LeadersLinked - Services

(root)/module/LeadersLinked/src/Controller/MyProfilesController.php – Rev 551 → 553