Symfomany / laravelcinema

app/Http/Controllers/AdministratorsController.php

Unused Use Statements -1 removed lines

@@ -4,7 +4,6 @@
 
 use App\Http\Models\Administrators;
 use App\Http\Requests\AdministratorsRequest;
-use Illuminate\Http\Request;
 use Illuminate\Support\Facades\Auth;
 use Illuminate\Support\Facades\Redirect;
 use Illuminate\Support\Facades\Session;

Indentation +1 added lines, -1 removed lines

@@ -90,7 +90,7 @@
 
         $administrator->save();
 
-       // Auth::login($administrator); => Autologin
+        // Auth::login($administrator); => Autologin
         Session::flash('success', "L'administrateur {$administrator->email} a bien été crée");
         return Redirect::route('administrators_index');
 

Spacing +15 added lines, -15 removed lines

@@ -12,15 +12,15 @@ 
 /**
  * Class AdministratorsController
  */
-class AdministratorsController extends Controller{
+class AdministratorsController extends Controller {
 
     /**
      * Page de liste des administrateurs
      */
-    public function index(){
+    public function index() {
         $administrators = Administrators::all();
 
-        return view('Administrators/index',[
+        return view('Administrators/index', [
             'administrators' => $administrators
         ]);
     }
@@ -31,12 +31,12 @@ 
      * @param $id
      * @return mixed
      */
-    public function remove($id){
+    public function remove($id) {
 
         // To load an administrator
         $administrator = Administrators::find($id);
 
-        if($administrator){
+        if ($administrator) {
             Session::flash('success', "L'administrateur {$administrator->email} a bien été supprimé");
             $administrator->delete();
         }
@@ -51,7 +51,7 @@ 
      * @param $id
      * @return mixed
      */
-    public function create(){
+    public function create() {
 
         return view('Administrators/create');
     }
@@ -59,12 +59,12 @@ 
      * To store an administrators
      * ARgument $id est facultatif: null est sa valeur par défaut
      */
-    public function store(AdministratorsRequest $request, $id = null){
+    public function store(AdministratorsRequest $request, $id = null) {
 
         // creation: créer un nouvel administrateur
-        if($id == null){
+        if ($id == null) {
             $administrator = new Administrators();
-        }else{
+        } else {
             // edition: récupérer un administrateur
             $administrator = Administrators::find($id);
         }
@@ -76,22 +76,22 @@ 
 
         // si l'utilisateur modifie
         // son mot de passe en mode edition
-        if(!empty($request->password)){
+        if (!empty($request->password)) {
             $administrator->password = bcrypt($request->password); //if en mode edit
         }
 
-        $administrator->super_admin =  $request->super_admin;
+        $administrator->super_admin = $request->super_admin;
         $administrator->expiration_date = new \DateTime('+1 year');
         $administrator->active = true;
 
         //upload
         $filename = "";
-        if($request->hasFile('image')) {
+        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
+            $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->photo = asset("uploads/administrators/".$filename);
         }
 
         $administrator->save();
@@ -110,7 +110,7 @@ 
      * @param $id
      * @return mixed
      */
-    public function edit($id){
+    public function edit($id) {
 
         $administrator = Administrators::find($id);
 

Braces +1 added lines, -1 removed lines

@@ -35,7 +35,7 @@
                 'password' => 'required|confirmed|min:6',
                 'image' => 'required|image',
             ];
-        }else{
+        } else{
             return [
                 'firstname' => 'required|max:255',
                 'lastname' => 'required|max:255',

app/Http/Controllers/MoviesController.php

Doc Comments -1 removed lines

@@ -148,7 +148,6 @@
 
     /**
      * Fonction de like des films, enregistré en session
-     * @param Request $request
      */
     public function like($id, $action)
     {

Braces +3 added lines, -5 removed lines

@@ -92,8 +92,7 @@ 
             $movie->visible = 1;
             Session::flash("success","Le film {$movie->title} a bien été activé");
 
-        }
-        else{
+        } else{
             $movie->visible = 0;
             Session::flash("success","Le film {$movie->title} a bien été desactivé");
         }
@@ -117,8 +116,7 @@ 
             $movie->cover = 1;
             Session::flash("success","Le film {$movie->title} a bien été mis en avant");
 
-        }
-        else{
+        } else{
             $movie->cover = 0;
             Session::flash("danger","Le film {$movie->title} a bien été retiré de l'avant");
         }
@@ -160,7 +158,7 @@ 
 
             $likes[$id] = $movie->id;
 
-        }else{
+        } else{
             unset($likes[$id]);
         }
         Session::put("likes", $likes);

Spacing +29 added lines, -29 removed lines

@@ -20,17 +20,17 @@ 
  * Sufficé par le mot clef Controller
  * et doit hérité de la super classe Controller
  */
-class MoviesController extends Controller{
+class MoviesController extends Controller {
 
 
     /**
      * Page Acceuil
      */
-    public function index(){
+    public function index() {
 
         $movies = Movies::all();
 
-        return view('Movies/index',[
+        return view('Movies/index', [
                 'movies' => $movies
             ]
         );
@@ -39,7 +39,7 @@ 
     /**
      * Page Create
      */
-    public function create(){
+    public function create() {
 
         $categories = Categories::all();
         $actors = Actors::all();
@@ -58,14 +58,14 @@ 
     /**
      * Page Read
      */
-    public function read($id){
+    public function read($id) {
 
         return view('Movies/read');
     }
     /**
      * Page Read
      */
-    public function edit($id){
+    public function edit($id) {
 
         $categories = Categories::all();
         $actors = Actors::all();
@@ -85,17 +85,17 @@ 
      * Action qui va me permettre d'activer un film
      * @param $id
      */
-    public function activate($id){
+    public function activate($id) {
         $movie = Movies::find($id);
 
-        if($movie->visible == 0){
+        if ($movie->visible == 0) {
             $movie->visible = 1;
-            Session::flash("success","Le film {$movie->title} a bien été activé");
+            Session::flash("success", "Le film {$movie->title} a bien été activé");
 
         }
-        else{
+        else {
             $movie->visible = 0;
-            Session::flash("success","Le film {$movie->title} a bien été desactivé");
+            Session::flash("success", "Le film {$movie->title} a bien été desactivé");
         }
 
         $movie->save();
@@ -108,19 +108,19 @@ 
      * Action qui va me permettre de metter en avant un film
      * @param $id
      */
-    public function cover($id){
+    public function cover($id) {
         // find() permet de retourner un Objet Movie
         // depuis son id
         $movie = Movies::find($id);
 
-        if($movie->cover == 0){
+        if ($movie->cover == 0) {
             $movie->cover = 1;
-            Session::flash("success","Le film {$movie->title} a bien été mis en avant");
+            Session::flash("success", "Le film {$movie->title} a bien été mis en avant");
 
         }
-        else{
+        else {
             $movie->cover = 0;
-            Session::flash("danger","Le film {$movie->title} a bien été retiré de l'avant");
+            Session::flash("danger", "Le film {$movie->title} a bien été retiré de l'avant");
         }
 
         $movie->save();
@@ -133,11 +133,11 @@ 
     /**
      * Suppresion
      */
-    public function delete($id){
+    public function delete($id) {
 
         $movie = Movies::find($id);
 
-        if($movie){
+        if ($movie) {
             Session::flash('success', "Le film {$movie->title} a bien été supprimé");
             $movie->delete();
         }
@@ -160,20 +160,20 @@ 
         // je recupère ma variable likes en session
         // et je fixe un tableau par défaut
         // si j'ai rien en sessions likes
-        $likes = session("likes", []);
+        $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;
+            $likes[ $id ] = $movie->id;
             Session::flash('danger', "Le film {$movie->title} a bien été liké");
 
-        }else{
+        } else {
 
             // je supprime le like dans le tableaux des likes
             // unset() supprimer un element dans un tableau en PHP
-            unset($likes[$id]);
+            unset($likes[ $id ]);
 
             Session::flash('success', "Le film {$movie->title} a bien été disliké");
         }
@@ -192,9 +192,9 @@ 
     /**
      * Store database
      */
-    public function store(MoviesRequest $request){
+    public function store(MoviesRequest $request) {
 
-        $dateoutput =  \DateTime::createFromFormat('d/m/Y',$request->date_release);
+        $dateoutput = \DateTime::createFromFormat('d/m/Y', $request->date_release);
         $movie = new Movies();
         $movie->type = $request->type;
         $movie->title = $request->title;
@@ -211,14 +211,14 @@ 
 
         $filename = "";
 
-        if($request->hasFile('image')) {
+        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
+            $destinationPath = public_path().'/uploads/movies'; // Indique où stocker le fichier
             $file->move($destinationPath, $filename); // Déplace le fichier
         }
 
-        $movie->image = asset("uploads/movies/". $filename);
+        $movie->image = asset("uploads/movies/".$filename);
         $movie->save();
 
         $actors = $request->actors;
@@ -226,7 +226,7 @@ 
             foreach ($actors as $actor) {
                 DB::table('actors_movies')
                     ->insert([
-                        ['movies_id' => $movie->id, 'actors_id' => $actor]
+                        [ 'movies_id' => $movie->id, 'actors_id' => $actor ]
                     ]);
             }
         }
@@ -236,7 +236,7 @@ 
             foreach ($directors as $director) {
                 DB::table('directors_movies')
                     ->insert([
-                        ['movies_id' => $movie->id, 'directors_id' => $director]
+                        [ 'movies_id' => $movie->id, 'directors_id' => $director ]
                     ]);
             }
         }

app/Http/Requests/AdministratorsRequest.php

Unused Use Statements -4 removed lines

@@ -2,11 +2,7 @@
 
 namespace App\Http\Requests;
 
-use App\Http\Models\Administrators;
 use Illuminate\Foundation\Http\FormRequest;
-use Illuminate\Support\Facades\Auth;
-
-use Illuminate\Support\Facades\Request;
 
 /**
  * Class AdministratorsRequest

Spacing +3 added lines, -3 removed lines

@@ -28,7 +28,7 @@ 
         $id = $this->route('id');
 
         // Création d'administrateur
-        if($id == null) {
+        if ($id == null) {
             return [
                 'firstname' => 'required|max:255',
                 'lastname' => 'required|max:255',
@@ -39,12 +39,12 @@ 
             ];
         }
         // Edition administrateur
-        else{
+        else {
             return [
                 'firstname' => 'required|max:255',
                 'lastname' => 'required|max:255',
                 'description' => 'required|min:10',
-                'email' => 'required|email|max:255|unique:administrators,email,' . $id,
+                'email' => 'required|email|max:255|unique:administrators,email,'.$id,
                 'password' => 'confirmed|min:6',
                 'image' => 'image',
             ];

app/Policies/SuperAdminPolicy.php

Spacing +4 added lines, -4 removed lines

@@ -20,9 +20,9 @@ 
      */
     public function isSuperAdmin(Administrators $administrator)
     {
-        if($administrator->super_admin == true){
+        if ($administrator->super_admin == true) {
             return true;
-        }else{
+        } else {
             return false;
         }
     }
@@ -36,9 +36,9 @@ 
     {
 
         // si j'ai expiration au niveau des date d'expiratio n
-        if($administrator->expiration_date > date('Y-m-d')){
+        if ($administrator->expiration_date > date('Y-m-d')) {
             return true;
-        }else{
+        } else {
             return false;
         }
     }

Braces +2 added lines, -2 removed lines

@@ -22,7 +22,7 @@ 
     {
         if($administrator->super_admin == true){
             return true;
-        }else{
+        } else{
             return false;
         }
     }
@@ -38,7 +38,7 @@ 
         // si j'ai expiration au niveau des date d'expiratio n
         if($administrator->expiration_date > date('Y-m-d')){
             return true;
-        }else{
+        } else{
             return false;
         }
     }

app/Http/Models/Actors.php

Indentation +2 added lines, -2 removed lines

@@ -36,7 +36,7 @@ 
 
         //1ere methode: Utilisation de MYSQL
         // marche mais n'est peu souple
-         /*
+            /*
           * $results = DB::select('
             SELECT ROUND(AVG(TIMESTAMPDIFF(YEAR,dob, NOW()))) as age
             FROM actors
@@ -59,7 +59,7 @@ 
 
         // MAITRISE
 
-          $results = DB::table('actors')
+            $results = DB::table('actors')
             ->select(DB::raw('ROUND(AVG(TIMESTAMPDIFF(YEAR,dob, NOW()))) as age'))
             ->first();
 

Spacing +3 added lines, -3 removed lines

@@ -11,7 +11,7 @@ 
  * de ma table movies
  * Hérite de ma super classe Model
  */
-class Actors extends Model{
+class Actors extends Model {
 
     /**
      * Décrit le nom de la table
@@ -30,7 +30,7 @@ 
      * Get nb actors by city
      * @return mixed
      */
-    public function getNbActorsByCity(){
+    public function getNbActorsByCity() {
 
         $result = DB::table('actors')
                     ->select(DB::raw("COUNT(id) as nb"), "city")
@@ -48,7 +48,7 @@ 
      * 1er mode de Laravel pour construire mes requetes
      * Cela me permet de conserver une syntaxe pure en Mysql
      */
-    public function getAvgActors(){
+    public function getAvgActors() {
 
         //1ere methode: Utilisation de MYSQL
         // marche mais n'est peu souple

app/Http/Models/Cinema.php

Spacing +1 added lines, -1 removed lines

@@ -11,7 +11,7 @@
  * de ma table movies
  * Hérite de ma super classe Model
  */
-class Cinema extends Model{
+class Cinema extends Model {
 
     /**
      * Décrit le nom de la table

app/Http/Models/Comments.php

Spacing +2 added lines, -2 removed lines

@@ -11,7 +11,7 @@ 
  * de ma table movies
  * Hérite de ma super classe Model
  */
-class Comments extends Model{
+class Comments extends Model {
 
     /**
      * Décrit le nom de la table
@@ -23,7 +23,7 @@ 
     /**
      *  Return the average of notes
      */
-    public function getAvgNote(){
+    public function getAvgNote() {
         $results = DB::table('comments')
             ->select(DB::raw('ROUND(AVG(note)) as avgnote'))
             ->first();

app/Http/Models/Sessions.php

Spacing +6 added lines, -6 removed lines

@@ -11,7 +11,7 @@ 
  * de ma table movies
  * Hérite de ma super classe Model
  */
-class Sessions extends Model{
+class Sessions extends Model {
 
 
     /**
@@ -24,7 +24,7 @@ 
     /**
      * Retourne la moyenne des heure de sessions
      */
-    public function getAvgHourDate(){
+    public function getAvgHourDate() {
         $result = DB::table('sessions')
             ->select(DB::raw('ROUND(AVG(HOUR(date_session))) as avghour'))
             ->first();
@@ -42,7 +42,7 @@ 
         WHERE date_session > NOW( )
         LIMIT 0 , 15
      */
-    public function getNextSession(){
+    public function getNextSession() {
 
         /* 1ere methode
         $result = DB::select("SELECT sessions.date_session AS DATE, movies.title AS mtitle, cinema.title AS ctitle
@@ -55,10 +55,10 @@ 
 
         $result = DB::table('sessions')
                     ->select("sessions.date_session", "movies.title AS mtitle", "cinema.title AS mcinema")
-                    ->join("movies", "sessions.movies_id",'=', "movies.id")
-                    ->join("cinema", "sessions.cinema_id",'=', "cinema.id")
+                    ->join("movies", "sessions.movies_id", '=', "movies.id")
+                    ->join("cinema", "sessions.cinema_id", '=', "cinema.id")
 
-                    ->where("date_session",  ">", DB::raw("NOW()"))
+                    ->where("date_session", ">", DB::raw("NOW()"))
                     ->orderBy('date_session', "ASC")
                     ->take(15)
                     ->get();

app/Http/Models/User.php

Spacing +4 added lines, -4 removed lines

@@ -11,7 +11,7 @@ 
  * de ma table movies
  * Hérite de ma super classe Model
  */
-class User extends Model{
+class User extends Model {
 
 
     /**
@@ -23,10 +23,10 @@ 
     /**
      *  Retourne les 24 derniers utilisateurs
      */
-    public function getLastUsers(){
+    public function getLastUsers() {
 
         // retourne le resultat de ma requete SELECT * FROM movies
-        $result =  DB::table('user')
+        $result = DB::table('user')
             ->orderBy('created_at', 'DESC')
             ->take(24)
             ->get();
@@ -41,7 +41,7 @@ 
     /**
      *  Retourne tous les films
      */
-    public function getAllMovies(){
+    public function getAllMovies() {
 
         // retourne le resultat de ma requete SELECT * FROM movies
         return DB::table('user')->get();