ThemeComposer::compose() - Code Metrics - jeremykenedy/laravel-material-design - Measure and Improve Code Quality continuously with Scrutinizer

ThemeComposer::compose() A
last analyzed 2021-01-06 01:06 UTC

↳ Parent: ThemeComposer

Complexity

Conditions	4
Paths	4

Size

Total Lines	17
Code Lines	8

Duplication

Lines	0
Ratio	0 %

Importance

Changes	2
Bugs	1	Features	0

Metric	Value
cc	4
eloc	8
c	2
b	1
f	0
nc	4
nop	1
dl	0
loc	17
rs	10

<?php

namespace App\Http\ViewComposers;

use App\Models\Theme;
use Illuminate\Support\Facades\Auth;
use Illuminate\View\View;

class ThemeComposer
{
    protected $user;
    protected $theme;

    public function __construct()
    {
        $this->user = Auth::user();
    }

    /**
     * Bind data to the view.
     *
     * @param View $view
     *
     * @return void
     */
    public function compose(View $view)
    {
        $theme = null;

        if (Auth::check()) {
            $user = $this->user;

            if ($user->profile) {

                $theme = Theme::find($user->profile->theme_id);

                if ($theme->status == 0) {
                    $theme = Theme::find(1);
                }
            }
        }

        $view->with('theme', $theme);
    }
}


1			<?php
2
3			namespace App\Http\ViewComposers;
4
5			use App\Models\Theme;
6			use Illuminate\Support\Facades\Auth;
7			use Illuminate\View\View;
8
9			class ThemeComposer
10			{
11			protected $user;
12			protected $theme;
13
14			public function __construct()
15			{
16			$this->user = Auth::user();
17			}
18
19			/**
20			* Bind data to the view.
21			*
22			* @param View $view
23			*
24			* @return void
25			*/
26			public function compose(View $view)
27			{
28			$theme = null;
29
30			if (Auth::check()) {
31			$user = $this->user;
32
33			if ($user->profile) {
			0 ignored issues – show Bug introduced 2018-06-13 07:49 UTC by Report Bug Copy Issue Report Accessing `profile` on the interface `Illuminate\Contracts\Auth\Authenticatable` suggest that you code against a concrete implementation. How about adding an `instanceof` check? Loading history...
34			$theme = Theme::find($user->profile->theme_id);
35
36			if ($theme->status == 0) {
37			$theme = Theme::find(1);
38			}
39			}
40			}
41
42			$view->with('theme', $theme);
43			}
44			}
45

jeremykenedy / laravel-material-design

ThemeComposer::compose() A last analyzed 2021-01-06 01:06 UTC

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like

ThemeComposer::compose() A
last analyzed 2021-01-06 01:06 UTC