Yajra\CMS\Http\Controllers\ProfileController

Complexity

Total Complexity

7

Size/Duplication

Total Lines	67
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	4

Importance

Changes	2
Bugs	0	Features	2

4 Methods

Rating	Name	Duplication	Size	Complexity
A	edit()	0	4	1
A	update()	0	21	4
A	getFilename()	0	6	1
A	removeAvatar()	0	8	1

<?php

namespace Yajra\CMS\Http\Controllers;

use Yajra\CMS\Http\Requests\ProfileFormRequest;
use App\Http\Requests;

class ProfileController extends Controller
{
    /**
     * Show the form for editing the specified resource.
     *
     * @return \Illuminate\Http\Response
     */
    public function edit()
    {
        return view('administrator.profile.edit');
    }

    /**
     * Update the specified resource in storage.
     *
     * @param  \Yajra\CMS\Http\Requests\ProfileFormRequest $request
     * @return \Illuminate\Http\Response
     * @throws \Laracasts\Presenter\Exceptions\PresenterException
     */
    public function update(ProfileFormRequest $request)
    {
        $profile = $request->user();
        $profile->fill($request->only(['first_name', 'last_name', 'email']));

        if ($request->get('password')) {
            $profile->password = bcrypt($request->get('password'));
        }

        if ((null !== $request->file('avatar')) && $request->file('avatar')->isValid()) {
            $filename = $this->getFilename($request);
            $request->file('avatar')->move('img/avatar', $filename);
            $profile->avatar = url('img/avatar/' . $filename);
        }

        $profile->save();

        flash()->success(trans('cms::profile.alert.success'));

It seems like trans('cms::profile.alert.success') targeting trans() can also be of type object<Symfony\Component\Translation\TranslatorInterface>; however, Laracasts\Flash\FlashNotifier::success() does only seem to accept string, maybe add an additional type check?

        return redirect()->route('administrator.profile.edit');
    }

    /**
     * @param \Yajra\CMS\Http\Requests\ProfileFormRequest $request
     * @return string
     * @throws \Laracasts\Presenter\Exceptions\PresenterException
     */
    protected function getFilename(ProfileFormRequest $request)
    {
        $fileName = $request->user()->id . '.' . $request->file('avatar')->getClientOriginalExtension();

        return $fileName;
    }

    /**
     * Remove current user's avatar.
     *
     * @return \Illuminate\Http\RedirectResponse
     */
    public function removeAvatar()
    {
        $profile = auth()->user();

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

        $profile->avatar = '';
        $profile->save();

        return redirect()->route('administrator.profile.edit');
    }
}