App\Http\Controllers\ProfileController

Complexity

Total Complexity

4

Size/Duplication

Total Lines	55
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	4

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	updateProfile()	0	21	2
A	updateSecurity()	0	9	1

<?php

namespace App\Http\Controllers;

use App\User;
use Faker\Provider\Image;
use Illuminate\Http\Request;

use App\Http\Requests;
use Illuminate\Support\Facades\Input;

class ProfileController extends Controller
{
    /**
     * ProfileController constructor.
     */
    public function __construct()
    {
        $this->middleware('auth');
        $this->middleware('lang');
    }

    /**
     * Update the profile information.
     *
     * @param  Requests\ProfileValidator $input
     * @return \Illuminate\Http\RedirectResponse
     */
    public function updateProfile(Requests\ProfileValidator $input)
    {
        // User information
        User::find(auth()->user()->id)->update($input->except(['_token', 'avatar']));

The method user() does not seem to exist on object<Illuminate\Contracts\Auth\Factory>.

        // User image information.
        if (Input::file()) {
            $image    = Input::file('avatar');
            $filename = time() . '.' . $image->getClientOriginalExtension();
            $path     = public_path('profilepics/' . $filename);
            
            Image::make($image->getRealPath())->resize(80, 80)->save($path);

The method make() does not seem to exist on object<Faker\Provider\Image>.

            $avatar         = User::find(auth()->user()->id);

The method user() does not seem to exist on object<Illuminate\Contracts\Auth\Factory>.

            $avatar->avatar = $filename;
            $avatar->save();
        }

        session()->flash('message', 'Your profile information has been updated.');
        return redirect()->back(302);
    }

    /**
     * Update your password.
     *
     * @param  Requests\PasswordValidation $input
     * @return \Illuminate\Http\RedirectResponse
     */
    public function updateSecurity(Requests\PasswordValidation $input)
    {
        $data['password'] = bcrypt($input->password);

$data was never initialized. Although not strictly required by PHP, it is generally a good practice to add $data = array(); before regardless.

        User::find(auth()->user()->id)->update($data);

The method user() does not seem to exist on object<Illuminate\Contracts\Auth\Factory>.

        session()->flash('message', 'Your password has been updated.');

        return redirect()->back(302);
    }
}