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

         return new JsonModel($data);
+    }
+    /**
-    /**
+     * Handles changing the user profile's cover image.
-     * Cambio de la imagen de fondo superior (cover) del perfil
+     * Supports uploading a new cover or deleting the existing one.
-     * @return \Laminas\View\Model\JsonModel
+     * @return \\Laminas\\View\\Model\\JsonModel
      */
+    public function coverAction()
     public function coverAction()
         // Get current user
+        $currentUserPlugin = $this->plugin('currentUserPlugin');
-        $currentUserPlugin = $this->plugin('currentUserPlugin');
+        $currentUser = $currentUserPlugin->getUser();
-        $currentUser = $currentUserPlugin->getUser();
+        // Get parameters from route
         $user_profile_id    = $this->params()->fromRoute('id');
-        $user_profile_id    = $this->params()->fromRoute('id');
-        $operation          = $this->params()->fromRoute('operation');
+        $operation          = $this->params()->fromRoute('operation'); // Expected: 'upload' or 'delete'
-        $userProfileMapper = UserProfileMapper::getInstance($this->adapter);
-        $userProfile = $userProfileMapper->fetchOneByUuid($user_profile_id);
-        if (!$userProfile) {
-            $response = [
+        // Fetch user profile
+        $userProfileMapper = UserProfileMapper::getInstance($this->adapter);
-                'success' => false,
+        $userProfile = $userProfileMapper->fetchOneByUuid($user_profile_id);
-                'data' => 'ERROR_INVALID_PARAMETER'
-            ];
         // Validate profile existence
-            return new JsonModel($response);
-        }
+        if (!$userProfile) {
-        if ($currentUser->id != $userProfile->user_id) {
-            $response = [
+            return $this->_createSimpleErrorResponse('ERROR_INVALID_PARAMETER');
-                'success' => false,
-                'data' => 'ERROR_UNAUTHORIZED'
             ];
-            return new JsonModel($response);
+        // Authorize current user against the profile: only the profile owner can modify it
-        }
+        if ($currentUser->id != $userProfile->user_id) {
+            return $this->_createSimpleErrorResponse('ERROR_UNAUTHORIZED');
+        }
+        $request = $this->getRequest();
         if ($request->isPost()) {
-        $request = $this->getRequest();
-        if ($request->isPost()) {
             $storage = Storage::getInstance($this->config, $this->adapter);
-            $target_path = $storage->getPathUser();
             $target_path = $storage->getPathUser(); // Base path for user-specific files
             if ($operation == 'delete') {
-                if ($userProfile->cover) {
+            // Handle cover deletion operation
-                    if (!$storage->deleteFile($target_path, $currentUser->uuid, $userProfile->cover)) {
+            if ($operation == 'delete') {
+                if ($userProfile->cover) {
+                    // Attempt to delete the existing cover file from storage
+                    if (!$storage->deleteFile($target_path, $currentUser->uuid, $userProfile->cover)) {
-                        return new JsonModel([
+                        return $this->_createSimpleErrorResponse('ERROR_THERE_WAS_AN_ERROR');
+                    }
+                }
-                            'success'   => false,
+                // Log the deletion action
+                $this->logger->info('Se borro el cover del perfil ' . ($userProfile->public == UserProfile::PUBLIC_YES ? 'público' : $userProfile->name), ['user_id' => $userProfile->user_id, 'ip' => Functions::getUserIP()]);
-                            'data'   =>  'ERROR_THERE_WAS_AN_ERROR'
+                $userProfile->cover = ''; // Clear the cover field in the profile model
-                        ]);
+                // Persist the cleared cover information to the database
-                    }
-                }
-                $this->logger->info('Se borro el cover del perfil ' . ($userProfile->public == UserProfile::PUBLIC_YES ? 'público' : $userProfile->name), ['user_id' => $userProfile->user_id, 'ip' => Functions::getUserIP()]);
+                if (!$userProfileMapper->updateCover($userProfile)) {
-                $userProfile->cover = '';
+                    // Handle error if database update fails for deletion
-            } else {
+                    return $this->_createSimpleErrorResponse('ERROR_THERE_WAS_AN_ERROR');
                 $form = new CoverForm($this->config);
-                $data     = array_merge($request->getPost()->toArray(), $request->getFiles()->toArray());
+            } else { // Handle cover upload operation
-                $form->setData($data);
+                $form = new CoverForm($this->config);
-                if ($form->isValid()) {
+                // Merge POST data and FILES data for the form
+                $data = array_merge($request->getPost()->toArray(), $request->getFiles()->toArray());
+                $form->setData($data);
+                if ($form->isValid()) {
+                    // Set files in storage and select the 'cover' file by its input name
+                    $storage->setFiles($request->getFiles()->toArray());
-                    $storage->setFiles($request->getFiles()->toArray());
-                    if (!$storage->setCurrentFilename('cover')) {
-                        return new JsonModel([
-                            'success'   => false,
-                            'data'   =>  'ERROR_UPLOAD_FILE'
-                        ]);
+                    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
-                    //echo '$target_path = ' . $target_path . ' cover =  ' . $userProfile->cover;
+                    // Example config value: '1200x400'
-                    //exit;
                     list($target_width_str, $target_height_str) = explode('x', $this->config['leaderslinked.image_sizes.user_cover_size']);
+                    $target_width = (int)$target_width_str;
-                    list($target_width, $target_height) = explode('x', $this->config['leaderslinked.image_sizes.user_cover_size']);
+                    $target_height = (int)$target_height_str;
                     $filename           = 'user-cover-' . uniqid() . '.png';
-                    $source_filename    = $storage->getTmpFilename();
+                    $filename           = 'user-cover-' . uniqid() . '.png'; // Generate unique filename for the new cover
-                    $target_filename    = $storage->composePathToFilename(Storage::TYPE_USER, $currentUser->id, $filename);
+                    $source_filename    = $storage->getTmpFilename(); // Temporary path of the uploaded file
+                    $target_filename    = $storage->composePathToFilename(Storage::TYPE_USER, $currentUser->id, $filename);
-                    if (!$storage->uploadImageResize($source_filename, $target_filename, $target_width, $target_height)) {
                         return new JsonModel([
-                            'success'   => false,
+                    // Upload, resize, and save the image
+                    if (!$storage->uploadImageResize($source_filename, $target_filename, $target_width, $target_height)) {
-                            'data'   =>  'ERROR_THERE_WAS_AN_ERROR'
+                        return $this->_createSimpleErrorResponse('ERROR_THERE_WAS_AN_ERROR');
+                    }
                         ]);
-                    }
-                    if ($userProfile->cover) {
+                    // If an old cover exists, delete it from storage
-                        if (!$storage->deleteFile($target_path, $currentUser->uuid, $userProfile->cover)) {
-                            return new JsonModel([
+                    if ($userProfile->cover) {
-                                'success'   => false,
+                        if (!$storage->deleteFile($target_path, $currentUser->uuid, $userProfile->cover)) {
-                                'data'   =>  'ERROR_THERE_WAS_AN_ERROR'
+                            // Decide if this is a critical error. Original code returns error.
-                            ]);
+                            // Consider logging this and proceeding if deletion of old file is non-critical.
-                        }
-                    }
-                    $userProfile->cover = $filename;
+                            return $this->_createSimpleErrorResponse('ERROR_THERE_WAS_AN_ERROR');
-                    if (!$userProfileMapper->updateCover($userProfile)) {
-                        return new JsonModel([
-                            'success'   => false,
+                        }
-                            'data'   =>  'ERROR_THERE_WAS_AN_ERROR'
-                        ]);
-                    }
+                    }
-                    $this->logger->info('Se actualizo el cover del perfil ' . ($userProfile->public == UserProfile::PUBLIC_YES ? 'público' : $userProfile->name), ['user_id' => $userProfile->user_id, 'ip' => Functions::getUserIP()]);
-                } else {
+                    // Update profile model with new cover filename
+                    $userProfile->cover = $filename;
+                    // Persist the new cover information to the database
-                    $messages = [];
+                    if (!$userProfileMapper->updateCover($userProfile)) {
-                    $form_messages = (array) $form->getMessages();
+                        return $this->_createSimpleErrorResponse('ERROR_THERE_WAS_AN_ERROR');
                     foreach ($form_messages  as $fieldname => $field_messages) {
-                        $messages[$fieldname] = array_values($field_messages);
-                    return new JsonModel([
-                        'success'   => false,
-                        'data'   => $messages
-                    ]);
+                    // Log successful update
-                }
-            }
-            $storage = Storage::getInstance($this->config, $this->adapter);
+                    $this->logger->info('Se actualizo el cover del perfil ' . ($userProfile->public == UserProfile::PUBLIC_YES ? 'público' : $userProfile->name), ['user_id' => $userProfile->user_id, 'ip' => Functions::getUserIP()]);
+                } else {
                     // Form is invalid, return validation messages
-            return new JsonModel([
+                    return $this->_createFormErrorResponse($form);
                 'success'   => true,
-                'data' => $storage->getUserProfileCover($currentUser, $userProfile)
+            } // End of upload/delete specific logic
         ];
+        return new JsonModel($data);
+    }
+    // Helper methods for creating standardized JSON responses
+    // These can be used across various actions in this controller
+    /**
+     * Creates a standardized JSON error response with a simple message.
+     * @param string $errorMessageKey Identifier for the error message (e.g., a translation key or constant).
+     * @return JsonModel
+     */
+    private function _createSimpleErrorResponse(string $errorMessageKey): JsonModel
+    {
+        return new JsonModel([
+            'success' => false,
+            'data'    => $errorMessageKey
+        ]);
+    }
+    /**
+     * Creates a JSON response for form validation errors.
+     * Extracts messages from the form and structures them for the client.
+     * @param \\Laminas\\Form\\FormInterface $form The form instance containing validation messages.
+     * @return JsonModel
+     */
+    private function _createFormErrorResponse(\\Laminas\\Form\\FormInterface $form): JsonModel
+    {
+        $messages = [];
+        $form_messages = (array) $form->getMessages(); // Get all error messages from the form
+        foreach ($form_messages as $fieldname => $field_messages_for_field) {
+            // Ensure messages for each field are in a simple array format for the frontend
+            $messages[$fieldname] = array_values((array)$field_messages_for_field);
+        }
+        return new JsonModel([
+            'success' => false,
+            'data'    => $messages
-        return new JsonModel($data);
+        ]);

Proyectos de Subversion LeadersLinked - Services

(root)/module/LeadersLinked/src/Controller/MyProfilesController.php – Rev 549 → 550