Xetaravel\Http\Controllers\Discuss\PostController

Complexity

Total Complexity

5

Size/Duplication

Total Lines	74
Duplicated Lines	0 %

Importance

Changes	2
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	show()	0	24	2
A	solved()	0	29	3

<?php

declare(strict_types=1);

namespace Xetaravel\Http\Controllers\Discuss;

use Illuminate\Http\RedirectResponse;
use Illuminate\Http\Request;
use Illuminate\Support\Facades\Auth;
use Xetaravel\Events\Discuss\PostWasSolvedEvent;
use Xetaravel\Models\DiscussConversation;
use Xetaravel\Models\DiscussPost;

class PostController extends Controller
{
    /**
     * Redirect an user to a conversation, page and post.
     *
     * @param Request $request
     * @param int $id The ID of the post.
     *
     * @return RedirectResponse
     */
    public function show(Request $request, int $id): RedirectResponse
    {
        $post = DiscussPost::findOrFail($id);

        $postsBefore = DiscussPost::where([
            ['conversation_id', $post->conversation_id],

The property conversation_id does not seem to exist on Illuminate\Database\Eloquent\PendingHasThroughRelationship.

            ['created_at', '<', $post->created_at]

The property created_at does not seem to exist on Illuminate\Database\Eloquent\PendingHasThroughRelationship.

        ])->count();

        $postsPerPage = config('xetaravel.pagination.discuss.post_per_page');

        $page = floor($postsBefore / $postsPerPage) + 1;
        $page = ($page > 1) ? $page : 1;

        $request->session()->keep(['primary', 'error', 'warning', 'success', 'info']);

        return redirect()
            ->route(

The method route() does not exist on Illuminate\Routing\Redirector.

                'discuss.conversation.show',
                [
                    'slug' => $post->conversation->slug,

The property conversation does not seem to exist on Illuminate\Database\Eloquent\PendingHasThroughRelationship.

                    'id' => $post->conversation->id,
                    'page' => $page,
                    '#post-' . $post->getKey()
                ]
            );
    }

    /**
     * Mark as solved.
     *
     * @param int $id
     *
     * @return RedirectResponse
     */
    public function solved(int $id): RedirectResponse
    {
        $post = DiscussPost::findOrFail($id);

        $this->authorize('solved', $post->conversation);

The property conversation does not seem to exist on Illuminate\Database\Eloquent\PendingHasThroughRelationship.

        if ($post->getKey() === $post->conversation->solved_post_id) {
            return back()
                ->with('danger', 'This post is already the solved post !');

The method with() does not exist on Illuminate\Http\RedirectResponse.

        }

        if (!is_null($post->conversation->solved_post_id)) {

The condition is_null($post->conversation->solved_post_id) is always false.

            return back()
                ->with('danger', 'This conversation has already a solved post !');
        }
        $conversation = DiscussConversation::findOrFail($post->conversation_id);

        $conversation->solved_post_id = $post->getKey();
        $conversation->is_solved = true;
        $conversation->save();

        $post->is_solved = true;
        $post->save();

        event(new PostWasSolvedEvent(Auth::user(), $post));

        return redirect()
            ->route('discuss.conversation.show', ['slug' => $conversation->slug, 'id' => $conversation->getKey()])
            ->with('success', 'This reply as been marked as solved !');
    }
}