App\Http\Controllers\AdministratorsController

Complexity

Total Complexity

9

Size/Duplication

Total Lines	111
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	3

Importance

Changes	6
Bugs	2	Features	4

5 Methods

Rating	Name	Duplication	Size	Complexity
A	create()	0	4	1
A	index()	0	8	1
A	remove()	0	13	2
B	store()	0	43	4
A	edit()	0	8	1

<?php

namespace App\Http\Controllers;

use App\Http\Models\Administrators;
use App\Http\Requests\AdministratorsRequest;
use Illuminate\Support\Facades\Redirect;
use Illuminate\Support\Facades\Session;

/**
 * Class AdministratorsController.
 */
class AdministratorsController extends Controller
{
    /**
     * Page de liste des administrateurs.
     */
    public function index()
    {
        $administrators = Administrators::all();

        return view('Administrators/index', [
            'administrators' => $administrators,
        ]);
    }

    /**
     * To remove an administrators.
     *
     * @param $id
     *
     * @return mixed
     */
    public function remove($id)
    {

        // To load an administrator
        $administrator = Administrators::find($id);

        if ($administrator) {
            Session::flash('success', "L'administrateur {$administrator->email} a bien été supprimé");
            $administrator->delete();
        }

        return Redirect::route('administrators_index');
    }

    /**
     * To remove an administrators.
     *
     * @param $id
     *
     * @return mixed
     */
    public function create()
    {
        return view('Administrators/create');
    }

    /**
     * To store an administrators
     * ARgument $id est facultatif: null est sa valeur par défaut.
     */
    public function store(AdministratorsRequest $request, $id = null)
    {

        // creation: créer un nouvel administrateur
        if ($id === null) {
            $administrator = new Administrators();
        } else {
            // edition: récupérer un administrateur
            $administrator = Administrators::find($id);
        }

        $administrator->firstname = $request->firstname;

The property firstname does not seem to exist in App\Http\Requests\AdministratorsRequest.

        $administrator->lastname = $request->lastname;

The property lastname does not seem to exist in App\Http\Requests\AdministratorsRequest.

        $administrator->email = $request->email;

The property email does not seem to exist in App\Http\Requests\AdministratorsRequest.

        $administrator->description = $request->description;

The property description does not seem to exist in App\Http\Requests\AdministratorsRequest.

        // si l'utilisateur modifie
        // son mot de passe en mode edition
        if (!empty($request->password)) {

The property password does not exist on object<App\Http\Requests\AdministratorsRequest>. Since you implemented __get, maybe consider adding a @property annotation.

            $administrator->password = bcrypt($request->password); //if en mode edit

The property password does not exist on object<App\Http\Requests\AdministratorsRequest>. Since you implemented __get, maybe consider adding a @property annotation.

        }

        $administrator->super_admin = $request->super_admin;

The property super_admin does not seem to exist in App\Http\Requests\AdministratorsRequest.

        $administrator->expiration_date = new \DateTime('+1 year');
        $administrator->active = true;

        //upload
        $filename = '';

$filename is not used, you could remove the assignment.

        if ($request->hasFile('image')) {
            $file = $request->file('image');
            $filename = $file->getClientOriginalName(); // Récupère le nom original du fichier
            $destinationPath = public_path().'/uploads/administrators'; // Indique où stocker le fichier
            $file->move($destinationPath, $filename); // Déplace le fichier
            $administrator->photo = asset('uploads/administrators/'.$filename);
        }

        $administrator->save();

       // Auth::login($administrator); => Autologin

50% of this comment could be valid code. Did you maybe forget this after debugging?

        Session::flash('success', "L'administrateur {$administrator->email} a bien été crée");

        return Redirect::route('administrators_index');
    }

    /**
     * To edit an administrator.
     *
     * @param $id
     *
     * @return mixed
     */
    public function edit($id)
    {
        $administrator = Administrators::find($id);

        return view('Administrators/edit', [
            'administrator' => $administrator,
        ]);
    }
}