Xetaravel\Http\Controllers\SecurityController

Complexity

Total Complexity

3

Size/Duplication

Total Lines	37
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	index()	0	20	2

<?php
namespace Xetaravel\Http\Controllers;

use Illuminate\Http\Request;
use Illuminate\Support\Facades\Auth;
use Illuminate\View\View;
use Propa\BrowscapPHP\Facades\Browscap;
use Xetaravel\Models\Session;

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

        $this->breadcrumbs->addCrumb('Security', route('users.security.index'));
    }

    /**
     * Show the security index page.
     *
     * @return \Illuminate\View\View
     */
    public function index(Request $request): View
    {
        $records = Session::expires()->where('user_id', Auth::id())->get();

        $sessions = [];

        foreach ($records as $record) {
            $infos = Browscap::getBrowser($record->user_agent);

The property user_agent does not exist on Xetaravel\Models\Session. Since you implemented __get, consider adding a @property annotation.

            $record->infos = $infos;

The property infos does not exist on Xetaravel\Models\Session. Since you implemented __set, consider adding a @property annotation.

            array_push($sessions, $record);
        }

        $sessionId = $request->session()->getId();

        return view('security.index', [
            'sessions' => $sessions,
            'sessionId' => $sessionId,
            'breadcrumbs' => $this->breadcrumbs
        ]);
    }
}