UserController::show() - Code Metrics - Inspection of "WIP" - XetaIO/Xetaravel - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — 5.0.0 ( f10804...44b42e )

by Fèvre

created 2025-04-08 09:20 UTC

UserController::show() A

↳ Parent: UserController

Complexity

Conditions	4
Paths	4

Size

Total Lines	60
Code Lines	44

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	4
eloc	44
c	0
b	0
f	0
nc	4
nop	1
dl	0
loc	60
rs	9.216

How to fix Long Method

<?php

declare(strict_types=1);

namespace Xetaravel\Http\Controllers;

use Illuminate\Http\RedirectResponse;
use Illuminate\Http\Request;
use Illuminate\Support\Facades\Auth;
use Illuminate\Support\Facades\Hash;
use Illuminate\Support\Facades\Route;
use Illuminate\Support\Str;
use Illuminate\View\View;
use Xetaravel\Models\Badge;
use Xetaravel\Models\Repositories\UserRepository;
use Xetaravel\Models\User;
use Xetaravel\Models\Validators\UserValidator;
use Xetaravel\Utility\UserUtility;

class UserController extends Controller
{
    /**
     * Constructor
     */
    public function __construct()
    {
        parent::__construct();

        $action = Route::getFacadeRoot()->current()->getActionMethod();

        if ($action === 'show') {
            $this->breadcrumbs->addCrumb('<i class="fa-regular fa-id-card mr-2"></i> Profile', route('page.index'));
        }
    }

    /**
     * Show the user profile page.
     *
     * @param string $slug The slug of the user.
     *
     * @return RedirectResponse|View
     */
    public function show(string $slug)
    {
        $user = User::with('articles', 'comments')
            ->where('slug', Str::lower($slug))
            ->first();

        if (is_null($user)) {
            return redirect()
                ->route('page.index')
                ->/** @scrutinizer ignore-call */ route('page.index')
                ->error('This user does not exist or has been deleted !');
        }
        $articles = $user->articles()
            ->latest()
            ->take(config('xetaravel.pagination.user.articles_profile_page'))
            ->get();

        $comments = $user->comments()
            ->with('article')
            ->latest()
            ->take(config('xetaravel.pagination.user.comments_profile_page'))
            ->get();

        $discussPosts = $user->discussPosts()
            ->join('discuss_conversations', 'discuss_posts.conversation_id', '=', 'discuss_conversations.id')
            ->where('discuss_conversations.first_post_id', '!=', 'discuss_posts.id')
            ->select(
                'discuss_posts.*',
                'discuss_conversations.first_post_id AS conversation_first_post_id',
                'discuss_conversations.title AS conversation_title',
                'discuss_conversations.slug AS conversation_slug',
                'discuss_conversations.id AS conversation_id'
            )
            ->orderBy('discuss_posts.created_at', 'DESC')
            ->take(config('xetaravel.pagination.user.posts_profile_page'))
            ->get();

        $breadcrumbs = $this->breadcrumbs->addCrumb(
            e($user->username),
            $user->profile_url

        );

        $badges = Badge::all();

        $articles = collect($articles);

        $activities = $articles->merge($comments)->merge($discussPosts)->sortBy('created_at', SORT_NATURAL, true);

        $level = UserUtility::getLevel($user->experiences_total);

        if ($level['maxLevel'] === true) {
            $level['currentProgression'] = 100;
        } elseif ($level['matchExactXPLevel'] === true) {
            $level['currentProgression'] = 0;
        } else {
            $level['currentProgression'] = ($level['currentUserExperience'] / $level['nextLevelExperience']) * 100;
        }

        return view(
            'user.show',
            compact('user', 'activities', 'articles', 'comments', 'breadcrumbs', 'level', 'badges')
        );
    }

    /**
     * Handle the delete request for the user.
     *
     * @param Request $request
     *
     * @return RedirectResponse
     */
    public function delete(Request $request): RedirectResponse
    {
        $user = Auth::user();

        if (!Hash::check($request->input('password'), $user->password)) {

            return redirect()
                ->route('user.setting.index')
                ->error('Your Password does not match !');
        }
        Auth::logout();

        if ($user->delete()) {
            return redirect()
                ->route('page.index')
                ->success('Your Account has been deleted successfully !');
        }

        return redirect()
            ->route('page.index')
            ->error('An error occurred while deleting your account !');
    }
}


XetaIO / Xetaravel

Push — 5.0.0 ( f10804...44b42e )

UserController::show() A

Complexity

Size

Duplication

Importance

How to fix Long Method

Long Method

Duplication Side-by-Side

Filter issues like