Fix remove image

EmanElhelaly11 · EmanElhelaly11 · commit 743b5d933fbd · 2023-09-09T02:36:37.000+02:00
diff --git a/app/Http/Controllers/Api/Home/CommentController.php b/app/Http/Controllers/Api/Home/CommentController.php
@@ -6,7 +6,6 @@
 use Illuminate\Http\Request;
 use App\Models\Comment;
 use App\Models\Post;
-use App\Models\User;
 use Illuminate\Support\Facades\Auth;
 use App\Http\Requests\CommentRequest;
 use App\Traits\ApiTrait;
@@ -16,68 +15,115 @@ class CommentController extends Controller
     use ApiTrait;
 
     public function showAllComments(Post $post)
-{
-    $comments = $post->comments->map(function ($comment) {
-        $user = $comment->user;
-        return [
-            'id' => $comment->id,
-            'content' => $comment->content,
-            'post_id' => $comment->post_id,
-            'user_id' => $comment->user_id,
-            'user_name' => $user->name, 
-            'user_imageUrl' => $user->imageUrl, 
-            'deleted_at' => $comment->deleted_at,
-            'created_at' => $comment->created_at,
-            'updated_at' => $comment->updated_at,
-        ];
-    });
-
-    return $this->data(compact('comments'));
-}
-
+    {
+        try {
+            $comments = $post->comments->map(function ($comment) {
+                $user = $comment->user;
+                return [
+                    'id' => $comment->id,
+                    'content' => $comment->content,
+                    'post_id' => $comment->post_id,
+                    'user_id' => $comment->user_id,
+                    'user_name' => $user->name, 
+                    'user_imageUrl' => $user->imageUrl, 
+                    'deleted_at' => $comment->deleted_at,
+                    'created_at' => $comment->created_at,
+                    'updated_at' => $comment->updated_at,
+                ];
+            });
+
+            return $this->data(compact('comments'));
+        } catch (\Exception $e) {
+            return $this->errorMessage([], 'An error occurred while fetching comments', 500);
+        }
+    }
 
     public function createComment(CommentRequest $request, Post $post)
     {
-        // Create a new comment
-        $comment = new Comment();
-        $comment->content = $request->input('content');
-        $comment->user_id = Auth::id(); // Set the user ID from the authenticated user
-        $comment->post_id = $post->id; // Set the post ID
-        $comment->save();
-
-        // Update the comments_count for the post
-        $post->increment('comments_count');
-
-        return $this->successMessage('Comment created successfully', 201);
+        try {
+            // Create a new comment
+            $comment = new Comment();
+            $comment->content = $request->input('content');
+            $comment->user_id = Auth::id(); // Set the user ID from the authenticated user
+            $comment->post_id = $post->id; // Set the post ID
+            $comment->save();
+
+            // Update the comments_count for the post
+            $post->increment('comments_count');
+
+            // Fetch all comments for the post
+            $comments = $post->comments->map(function ($comment) {
+                $user = $comment->user;
+                return [
+                    'id' => $comment->id,
+                    'content' => $comment->content,
+                    'post_id' => $comment->post_id,
+                    'user_id' => $comment->user_id,
+                    'user_name' => $user->name, 
+                    'user_imageUrl' => $user->imageUrl, 
+                    'deleted_at' => $comment->deleted_at,
+                    'created_at' => $comment->created_at,
+                    'updated_at' => $comment->updated_at,
+                ];
+            });
+
+            return $this->data(compact('comments'), 'Comment created successfully', 201);
+        } catch (\Exception $e) {
+            return $this->errorMessage([], 'An error occurred while creating the comment', 500);
+        }
     }
 
     public function editComment(CommentRequest $request, Post $post, Comment $comment)
     {
-        // Check if the comment belongs to the post
-        if ($comment->post_id !== $post->id) {
-            return $this->errorMessage([], 'Comment not found for the specified post', 404);
+        try {
+            // Check if the comment belongs to the post
+            if ($comment->post_id !== $post->id) {
+                return $this->errorMessage([], 'Comment not found for the specified post', 404);
+            }
+
+            // Update the comment content
+            $comment->content = $request->input('content');
+            $comment->save();
+
+            // Fetch all comments for the post
+            $comments = $post->comments->map(function ($comment) {
+                $user = $comment->user;
+                return [
+                    'id' => $comment->id,
+                    'content' => $comment->content,
+                    'post_id' => $comment->post_id,
+                    'user_id' => $comment->user_id,
+                    'user_name' => $user->name, 
+                    'user_imageUrl' => $user->imageUrl, 
+                    'deleted_at' => $comment->deleted_at,
+                    'created_at' => $comment->created_at,
+                    'updated_at' => $comment->updated_at,
+                ];
+            });
+
+            return $this->data(compact('comments'), 'Comment updated successfully', 200);
+        } catch (\Exception $e) {
+            return $this->errorMessage([], 'An error occurred while updating the comment', 500);
         }
-
-        // Update the comment content
-        $comment->content = $request->input('content');
-        $comment->save();
-
-        return $this->successMessage('Comment updated successfully', 200);
     }
 
     public function deleteComment(Post $post, Comment $comment)
     {
-        // Check if the comment belongs to the post
-        if ($comment->post_id !== $post->id) {
-            return $this->errorMessage([], 'Comment not found for the specified post', 404);
-        }
+        try {
+            // Check if the comment belongs to the post
+            if ($comment->post_id !== $post->id) {
+                return $this->errorMessage([], 'Comment not found for the specified post', 404);
+            }
 
-        // Decrement the comments_count for the post
-        $post->decrement('comments_count');
+            // Decrement the comments_count for the post
+            $post->decrement('comments_count');
 
-        // Delete the comment
-        $comment->delete();
+            // Delete the comment
+            $comment->delete();
 
-        return $this->successMessage('Comment deleted successfully', 200);
+            return $this->successMessage('Comment deleted successfully', 200);
+        } catch (\Exception $e) {
+            return $this->errorMessage([], 'An error occurred while deleting the comment', 500);
+        }
     }
 }
diff --git a/app/Http/Controllers/Api/Home/LikesController.php b/app/Http/Controllers/Api/Home/LikesController.php
@@ -41,21 +41,19 @@ public function likePost(Post $post)
     }
 
     public function getLikesForPost(Post $post)
-{
-    // Get all likes for the post
-    $likes = Like::where('post_id', $post->id)->get();
-
-    // Transform the likes data to include user_name and user_imageUrl
-    $likeData = $likes->map(function ($like) {
-        $data = $like->toArray();
-        $data['user_name'] = $like->user->name; // Change 'name' to the actual column name in your users table
-        $data['user_imageUrl'] = $like->user->imageUrl; // Change 'imageUrl' to the actual column name in your users table
-        unset($data['user']); // Remove the user relationship to avoid redundancy
-        return $data;
-    });
-
-    return $this->data(compact('likeData'));
+    {
+        // Get all likes for the post
+        $likes = Like::where('post_id', $post->id)->get();
+
+        // Transform the likes data to include user_name and user_imageUrl
+        $likeData = $likes->map(function ($like) {
+            $data = $like->toArray();
+            $data['user_name'] = $like->user->name; 
+            $data['user_imageUrl'] = $like->user->imageUrl; 
+            unset($data['user']); 
+            return $data;
+        });
+
+        return $this->data(compact('likeData'));
+    }
 }
-
-    
-}
diff --git a/app/Http/Controllers/Api/Home/PostController.php b/app/Http/Controllers/Api/Home/PostController.php
@@ -18,15 +18,14 @@ class PostController extends Controller
 
     public function getPosts()
     {
-        // Retrieve all posts with the user relationship and order them by 'created_at' in descending order
         $posts = Post::with('user')->orderBy('created_at', 'desc')->get();
     
         // Transform the posts data to include user_name and user_imageUrl
         $postData = $posts->map(function ($post) {
             $data = $post->toArray();
-            $data['user_name'] = $post->user->name; // Change 'name' to the actual column name in your users table
-            $data['user_imageUrl'] = $post->user->imageUrl; // Change 'imageUrl' to the actual column name in your users table
-            unset($data['user']); // Remove the user relationship to avoid redundancy
+            $data['user_name'] = $post->user->name;
+            $data['user_imageUrl'] = $post->user->imageUrl; 
+            unset($data['user']); 
             return $data;
         });
     
@@ -42,14 +41,12 @@ public function showPost($id)
            return $this->errorMessage([], 'Post not found', 404);
        }
    
-       // Convert the Post model and user model to an array
        $postData = $post->toArray();
    
        // Replace the user_id with user's name and imageUrl
-       $postData['user_name'] = $post->user->name; // Change 'name' to the actual column name in your users table
-       $postData['user_imageUrl'] = $post->user->imageUrl; // Change 'imageUrl' to the actual column name in your users table
+       $postData['user_name'] = $post->user->name; 
+       $postData['user_imageUrl'] = $post->user->imageUrl; 
    
-       // Unset the user relationship to avoid redundancy
        unset($postData['user']);
    
        return $this->data(['post' => $postData], 'Post retrieved successfully', 200);
@@ -58,7 +55,6 @@ public function showPost($id)
 
    public function createPost(PostRequest $request)
    {        
-       // Check if either 'content' or 'file_path' is provided
        if (!$request->filled('content') && !$request->hasFile('file_path')) {
            return $this->errorMessage([], 'Either content or an image must be provided', 422);
        }
@@ -81,71 +77,69 @@ public function createPost(PostRequest $request)
    
        $post->save();
    
-       // Fetch the post again to include user_name, user_imageUrl, and other data
        $post = Post::with('user')->find($post->id);
        
        $postData = $post->toArray();
        // Replace the user_id with user's name and imageUrl
-       $postData['user_name'] = $post->user->name; // Change 'name' to the actual column name in your users table
-       $postData['user_imageUrl'] = $post->user->imageUrl; // Change 'imageUrl' to the actual column name in your users table
-       unset($postData['user']); // Remove the user relationship to avoid redundancy
+       $postData['user_name'] = $post->user->name;
+       $postData['user_imageUrl'] = $post->user->imageUrl;
+       unset($postData['user']); 
    
        return $this->data(['post' => $postData], 'Post created successfully', 201);
    }
+ 
    
    public function editPost(PostRequest $request, $id)
    {
        $post = Post::find($id);
+
        if (!$post) {
            return $this->errorMessage([], 'Post not found', 404);
        }
-   
+
        if ($post->user_id !== Auth::id()) {
            return $this->errorMessage([], 'You are not authorized to edit this post', 403);
        }
-   
+
        // Check if there are any changes to update
-       if (!$request->filled('content') && !$request->hasFile('file_path')) {
+       if (!$request->filled('content') && !$request->hasFile('file_path') && !$request->filled('remove_image')) {
            return $this->errorMessage([], 'No changes to update', 422);
        }
-   
+
        // Update the post content if provided
        if ($request->filled('content')) {
            $post->content = $request->input('content');
        }
-   
-       // Handle image upload, if provided
+
+       // Handle image upload or removal, if provided
        if ($request->hasFile('file_path')) {
            $image = $request->file('file_path');
            $imagePath = $this->upload($image, 'posts');
-           $post->image_path = "images/posts/$imagePath"; 
-       } elseif (empty($request->input('file_path'))) {
-           // No new image provided, check if 'file_path' input is empty
-           // If it's empty, delete the existing image
+           $post->image_path = "images/posts/$imagePath";
+       } elseif ($request->filled('remove_image')) {
+           // Check if the 'remove_image' input is provided and set to true
+           // If 'remove_image' is true, delete the existing image and set image_path to null
            $this->delete($post->image_path);
-           $post->image_path = null; // Set image_path to null in the database
+           $post->image_path = null;
        }
-   
+
        $post->save();
-   
-       // Fetch the post again to include user_name, user_imageUrl, and other data
+
        $post = Post::with('user')->find($post->id);
-   
        $postData = $post->toArray();
+
        // Replace the user_id with user's name and imageUrl
-       $postData['user_name'] = $post->user->name; // Change 'name' to the actual column name in your users table
-       $postData['user_imageUrl'] = $post->user->imageUrl; // Change 'imageUrl' to the actual column name in your users table
-       unset($postData['user']); // Remove the user relationship to avoid redundancy
-   
-       if ($request->filled('content') || $request->hasFile('file_path')) {
+       $postData['user_name'] = $post->user->name; 
+       $postData['user_imageUrl'] = $post->user->imageUrl; 
+       unset($postData['user']); 
+
+       if ($request->filled('content') || $request->hasFile('file_path') || $request->filled('remove_image')) {
            return $this->data(['post' => $postData], 'Post updated successfully', 200);
        } else {
            return $this->errorMessage([], 'No changes to update', 422);
        }
    }
    
-   
-
   
     public function deletePost($id)
     {
@@ -167,9 +161,9 @@ public function deletePost($id)
         return $this->successMessage('Post deleted successfully', 200);
     }
 
+    
     public function getUserPosts($userId)
     {
-        // Find the user by their ID
         $user = User::find($userId);
     
         if (!$user) {
@@ -182,15 +176,14 @@ public function getUserPosts($userId)
         // Transform the posts data to include user_name and user_imageUrl
         $postData = $posts->map(function ($post) {
             $data = $post->toArray();
-            $data['user_name'] = $post->user->name; // Change 'name' to the actual column name in your users table
-            $data['user_imageUrl'] = $post->user->imageUrl; // Change 'imageUrl' to the actual column name in your users table
-            unset($data['user']); // Remove the user relationship to avoid redundancy
+            $data['user_name'] = $post->user->name; 
+            $data['user_imageUrl'] = $post->user->imageUrl; 
+            unset($data['user']); 
             return $data;
         });
     
         return $this->data(compact('postData'));
     }
 
 
-
 }
