App\Http\Controllers\MoviesController

Complexity

Total Complexity

18

Size/Duplication

Total Lines	212
Duplicated Lines	22.64 %

Coupling/Cohesion

Components	0
Dependencies	6

Importance

Changes	6
Bugs	0	Features	5

9 Methods

Rating	Name	Duplication	Size	Complexity
A	index()	0	9	1
A	create()	0	14	1
A	read()	0	4	1
A	edit()	0	14	1
A	activate()	17	17	2
A	cover()	20	19	2
A	delete()	0	11	2
A	like()	0	19	2
B	store()	0	54	6

<?php
// chemin relatif ou se trouve la classe
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 MainController
 * @package App\Http\Controllers
 * Sufficé par le mot clef Controller
 * et doit hérité de la super classe Controller
 */
class MoviesController extends Controller{


    /**
     * Page Acceuil
     */
    public function index(){

        $movies = Movies::all();

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

    /**
     * Page Create
     */
    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 Read
     */
    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.

        // find() permet de retourner un Objet Movie
        // depuis son id
        $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');
    }



    /**
     * Suppresion
     */
    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
     * @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", []);

        if ($action == "like") {

            $likes[$id] = $movie->id;

        }else{
            unset($likes[$id]);
        }
        Session::put("likes", $likes);
        Session::flash('success', "Le film {$movie->title} a bien été liké");

        return Redirect::route('movies_index');

    }



    /**
     * Store database
     */
    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');

    }


}