App\Http\Controllers\MoviesController

Complexity

Total Complexity

18

Size/Duplication

Total Lines	227
Duplicated Lines	21.15 %

Coupling/Cohesion

Components	0
Dependencies	6

Importance

Changes	19
Bugs	4	Features	14

9 Methods

Rating	Name	Duplication	Size	Complexity
A	read()	0	4	1
A	index()	0	9	1
A	activate()	16	16	2
A	cover()	18	16	2
A	delete()	0	11	2
B	like()	0	27	2
A	create()	0	13	1
A	edit()	0	15	1
B	store()	0	54	6

<?php

namespace App\Http\Controllers;

use App\Http\Models\Actors;
use App\Http\Models\Categories;
use App\Http\Models\Directors;
use App\Http\Models\Movies;
use App\Http\Requests\MoviesRequest;
use Illuminate\Http\Request;
use Illuminate\Support\Facades\DB;
use Illuminate\Support\Facades\Redirect;
use Illuminate\Support\Facades\Session;

/**
 * Class MoviesController.
 */
class MoviesController extends Controller
{
    /**
     * Page Index.
     *
     * @return \Illuminate\Contracts\View\Factory|\Illuminate\View\View
     */
    public function index()
    {
        $movies = Movies::all();

        return view('Movies/index', [
                'movies' => $movies,
            ]
        );
    }

    /**
     * Page create.
     *
     * @return \Illuminate\Contracts\View\Factory|\Illuminate\View\View
     */
    public function create()
    {
        $categories = Categories::all();
        $actors = Actors::all();
        $directors = Directors::all();

        return view('Movies/create',
        [
            'categories' => $categories,
            'actors'     => $actors,
            'directors'  => $directors,
        ]);
    }

    /**
     * Page Read.
     */
    public function read($id)

The parameter $id is not used and could be removed.

    {
        return view('Movies/read');
    }

    /**
     * Page edit.
     *
     * @param $id
     *
     * @return \Illuminate\Contracts\View\Factory|\Illuminate\View\View
     */
    public function edit($id)
    {
        $categories = Categories::all();
        $actors = Actors::all();
        $directors = Directors::all();

        $movie = Movies::find($id);

        return view('Movies/edit', [
            'movie'      => $movie,
            'categories' => $categories,
            'actors'     => $actors,
            'directors'  => $directors,
        ]);
    }

    /**
     * Action qui va me permettre d'activer un film.
     *
     * @param $id
     */
    public function activate($id)

This method seems to be duplicated in your project.

    {
        $movie = Movies::find($id);

        if ($movie->visible == 0) {
            $movie->visible = 1;
            Session::flash('success', "Le film {$movie->title} a bien été activé");
        } else {
            $movie->visible = 0;
            Session::flash('success', "Le film {$movie->title} a bien été desactivé");
        }

        $movie->save();

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

    /**
     * Action qui va me permettre de metter en avant un film.
     *
     * @param $id
     */
    public function cover($id)

This method seems to be duplicated in your project.

    {
        $movie = Movies::find($id);

        if ($movie->cover == 0) {
            $movie->cover = 1;
            Session::flash('success', "Le film {$movie->title} a bien été mis en avant");
        } else {
            $movie->cover = 0;
            Session::flash('danger', "Le film {$movie->title} a bien été retiré de l'avant");
        }

        $movie->save();

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

    /**
     * Action of suppression.
     *
     * @param $id
     *
     * @return mixed
     */
    public function delete($id)
    {
        $movie = Movies::find($id);

        if ($movie) {
            Session::flash('success', "Le film {$movie->title} a bien été supprimé");
            $movie->delete();
        }

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

    /**
     * Fonction de like des films, enregistré en session
     * Session : mécanisme de stockage temporelle
     * BDD: mécanisme de stockage atemporelle.
     *
     * @param Request $request

There is no parameter named $request. Was it maybe removed?

     */
    public function like($id, $action)
    {
        $movie = Movies::find($id);

        $likes = session('likes', []);

        // si l'action est "like"
        if ($action == 'like') {

            // J'ajoute mon movie dans le tableaux des likes en session
            $likes[$id] = $movie->id;
            Session::flash('danger', "Le film {$movie->title} a bien été liké");
        } else {

            // je supprime le like dans le tableaux des likes
            // unset() supprimer un element dans un tableau en PHP
            unset($likes[$id]);

            Session::flash('success', "Le film {$movie->title} a bien été disliké");
        }

        //j'enregistre en session mon nouveau tableaux des likes
        Session::put('likes', $likes);

        // une redirection avec message flash
        return Redirect::route('movies_index');
    }

    /**
     * Storing in database.
     *
     * @param MoviesRequest $request
     *
     * @return mixed
     */
    public function store(MoviesRequest $request)
    {
        $dateoutput = \DateTime::createFromFormat('d/m/Y', $request->date_release);

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

        $movie = new Movies();
        $movie->type = $request->type;

The property type does not exist on object<App\Http\Models\Movies>. Since you implemented __set, maybe consider adding a @property annotation.

The property type does not seem to exist in App\Http\Requests\MoviesRequest.

        $movie->title = $request->title;

The property title does not exist on object<App\Http\Models\Movies>. Since you implemented __set, maybe consider adding a @property annotation.

The property title does not seem to exist in App\Http\Requests\MoviesRequest.

        $movie->synopsis = $request->synopsis;

The property synopsis does not exist on object<App\Http\Models\Movies>. Since you implemented __set, maybe consider adding a @property annotation.

The property synopsis does not seem to exist in App\Http\Requests\MoviesRequest.

        $movie->description = $request->description;

The property description does not exist on object<App\Http\Models\Movies>. Since you implemented __set, maybe consider adding a @property annotation.

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

        $movie->trailer = $request->trailer;

The property trailer does not exist on object<App\Http\Models\Movies>. Since you implemented __set, maybe consider adding a @property annotation.

The property trailer does not seem to exist in App\Http\Requests\MoviesRequest.

        $movie->date_release = $dateoutput;

The property date_release does not exist on object<App\Http\Models\Movies>. Since you implemented __set, maybe consider adding a @property annotation.

        $movie->visible = $request->visible;

The property $visible is declared protected in Illuminate\Database\Eloquent\Model. Since you implemented __set(), maybe consider adding a @property or @property-write annotation. This makes it easier for IDEs to provide auto-completion.

The property visible does not seem to exist in App\Http\Requests\MoviesRequest.

        $movie->cover = $request->cover;

The property cover does not exist on object<App\Http\Models\Movies>. Since you implemented __set, maybe consider adding a @property annotation.

The property cover does not seem to exist in App\Http\Requests\MoviesRequest.

        $movie->languages = $request->lang;

The property languages does not exist on object<App\Http\Models\Movies>. Since you implemented __set, maybe consider adding a @property annotation.

The property lang does not seem to exist in App\Http\Requests\MoviesRequest.

        $movie->categories_id = $request->categories_id;

The property categories_id does not exist on object<App\Http\Models\Movies>. Since you implemented __set, maybe consider adding a @property annotation.

The property categories_id does not seem to exist in App\Http\Requests\MoviesRequest.

        $movie->note_presse = $request->note_presse;

The property note_presse does not exist on object<App\Http\Models\Movies>. Since you implemented __set, maybe consider adding a @property annotation.

The property note_presse does not seem to exist in App\Http\Requests\MoviesRequest.

        $movie->distributeur = $request->distributeur;

The property distributeur does not exist on object<App\Http\Models\Movies>. Since you implemented __set, maybe consider adding a @property annotation.

The property distributeur does not seem to exist in App\Http\Requests\MoviesRequest.

        $filename = '';


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

        $movie->image = asset('uploads/movies/'.$filename);

The property image does not exist on object<App\Http\Models\Movies>. Since you implemented __set, maybe consider adding a @property annotation.

        $movie->save();

        $actors = $request->actors;

The property actors does not seem to exist in App\Http\Requests\MoviesRequest.

        if (isset($actors)) {
            foreach ($actors as $actor) {
                DB::table('actors_movies')
                    ->insert([
                        ['movies_id' => $movie->id, 'actors_id' => $actor],

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

                    ]);
            }
        }

        $directors = $request->directors;

The property directors does not seem to exist in App\Http\Requests\MoviesRequest.

        if (isset($directors)) {
            foreach ($directors as $director) {
                DB::table('directors_movies')
                    ->insert([
                        ['movies_id' => $movie->id, 'directors_id' => $director],

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

                    ]);
            }
        }

        Session::flash('success', "Le film {$movie->title} a été enregistré");

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

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