How do I delete comment replies along with a comment?

posted 1 year ago

I have made a blogging application in Laravel 8. I am currently working on giving any authenticated user to delete his/her comments.

Deleting comments works fine but, since comments have replies, I run into a problem with deleting replies along with a comment.

In comments-list.blade.php, I have the delete comment form:

@if ($comment->user->id == Auth::user()->id)
    <form class="commentDeleteForm" onsubmit="return confirm('Delete this comment?')" method="post" action="{{ route('comment.delete', $comment->id) }}" novalidate>
      @csrf
      <button type="submit" class="comment-delete-btn">
        <i class="fa fa-trash"></i> 
        Delete
      </button>
    </form>
@endif

In the controller, I had:

public function delete_comment($id) {
    $comment = Comment::find($id);

    if($comment->user_id === auth()->user()->id) {
      $comment->delete();
      return redirect()->back()->with('success', 'The comment was deleted');
    }
}

But, in order to delete replies too, I changed $comment->delete() to $comment->delete()->with('replies').

The goal

This change causes a failure to delete the targeted comment, with the error:

Call to a member function with() on bool

Questions

What am I doing wrong?
What is the most reliable way to fix this problem?

Last updated by @ajax30 1 year ago.

faisal

replied 1 year ago

Solution

@ajax30 You are deleting the comment first and then you are trying to delete replies of comment which you have already deleted.

Here is the updated delete comment function function

public function delete_comment($id) {
    $comment = Comment::find($id);

    if($comment->user_id === auth()->user()->id) {
      $comment->replies()->delete();
      $comment->delete();
      return redirect()->back()->with('success', 'The comment was deleted');
    }
}

I assume you have already created a hasMany relationship of replies in the Comment modal.

Thanks!

Solution selected by @ajax30

ajax30

replied 1 year ago

Please try answering this question too. Thanks!

(Link removed)

Last updated by @tvbeek 1 year ago.

tvbeek

replied 1 year ago

moderator

@ajax30 people can answer other questions if they have the time and know the answer. It is not wanted to ask them from one thread to answer another one.

Last updated by @tvbeek 1 year ago.