Arcanesoft\Auth\Http\Controllers\Admin\ProfileController

Complexity

Total Complexity

7

Size/Duplication

Total Lines	113
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Test Coverage

Coverage

0%

Importance

Changes

0

7 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	7	1
A	index()	0	9	1
A	edit()	0	4	1
A	update()	0	4	1
A	updatePassword()	0	8	1
A	getAuthenticatedUser()	0	4	1
A	transNotification()	0	10	1

<?php namespace Arcanesoft\Auth\Http\Controllers\Admin;

use Arcanesoft\Auth\Http\Requests\Admin\Profile\UpdatePasswordRequest;
use Arcanesoft\Contracts\Auth\Models\User;
use Arcanesoft\Core\Http\Controllers\AdminController;
use Arcanesoft\Core\Traits\Notifyable;
use Log;

/**
 * Class     ProfileController
 *
 * @package  Arcanesoft\Auth\Http\Controllers\Admin
 * @author   ARCANEDEV <arcanedev.maroc@gmail.com>
 *
 * @todo: Adding the authorization checks
 */
class ProfileController extends AdminController
{
    /* -----------------------------------------------------------------
     |  Traits
     | -----------------------------------------------------------------
     */

    use Notifyable;

    /* -----------------------------------------------------------------
     |  Properties
     | -----------------------------------------------------------------
     */

    /**
     * The authenticated user.
     *
     * @var \Arcanesoft\Contracts\Auth\Models\User
     */
    protected $user;

    /**
     * The view namespace.
     *
     * @var string
     */
    protected $viewNamespace = 'auth';

    /* -----------------------------------------------------------------
     |  Constructor
     | -----------------------------------------------------------------
     */

    /**
     * ProfileController constructor.
     */
    public function __construct()
    {
        parent::__construct();

        $this->addBreadcrumbRoute('Profile', 'admin::auth.profile.index');
        $this->setCurrentPage('auth-profile');
    }

    /* -----------------------------------------------------------------
     |  Main Methods
     | -----------------------------------------------------------------
     */

    public function index()
    {
        $user = $this->getAuthenticatedUser();

        $this->setTitle("Profile - {$user->full_name}");

Accessing full_name on the interface Arcanesoft\Contracts\Auth\Models\User suggest that you code against a concrete implementation. How about adding an instanceof check?

        $this->addBreadcrumbRoute($user->full_name, 'admin::auth.profile.index');

Accessing full_name on the interface Arcanesoft\Contracts\Auth\Models\User suggest that you code against a concrete implementation. How about adding an instanceof check?

        return $this->view('admin.profile.index', compact('user'));
    }

    public function edit()
    {
        // TODO: complete the implementation
    }

    public function update()
    {
        // TODO: complete the implementation
    }

    public function updatePassword(UpdatePasswordRequest $request, User $user)
    {
        $user->update($request->only(['password']));

The method update() does not seem to exist on object<Arcanesoft\Contracts\Auth\Models\User>.

        $this->transNotification('password-updated', [], $user->toArray());

The method toArray() does not seem to exist on object<Arcanesoft\Contracts\Auth\Models\User>.

        return redirect()->route('admin::auth.profile.index');
    }

    /* -----------------------------------------------------------------
     |  Other methods
     | -----------------------------------------------------------------
     */

    /**
     * Get the authenticated user.
     *
     * @return \Arcanesoft\Contracts\Auth\Models\User
     */
    private function getAuthenticatedUser()
    {
        return auth()->user();
    }

    /**
     * Notify with translation.
     *
     * @param  string  $action
     * @param  array   $replace
     * @param  array   $context
     *
     * @return string
     */
    protected function transNotification($action, array $replace = [], array $context = [])
    {
        $title   = trans("auth::profile.messages.{$action}.title");
        $message = trans("auth::profile.messages.{$action}.message", $replace);

        Log::info($message, $context);
        $this->notifySuccess($message, $title);

        return $message;
    }
}