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 +14 added lines, -14 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');
     }
@@ -60,12 +60,12 @@ 
      * @param $id
      * @return mixed
      */
-    public function store(AdministratorsRequest $request, $id = null){
+    public function store(AdministratorsRequest $request, $id = null) {
 
         //creation
-        if($id == null){
+        if ($id == null) {
             $administrator = new Administrators();
-        }else{
+        } else {
             $administrator = Administrators::find($id);
         }
 
@@ -74,18 +74,18 @@ 
         $administrator->email = $request->email;
         $administrator->description = $request->description;
         $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();
@@ -104,7 +104,7 @@ 
      * @param $id
      * @return mixed
      */
-    public function edit($id){
+    public function edit($id) {
 
         $administrator = Administrators::find($id);
 

Braces +1 added lines, -1 removed lines

@@ -65,7 +65,7 @@
         //creation
         if($id == null){
             $administrator = new Administrators();
-        }else{
+        } else{
             $administrator = Administrators::find($id);
         }
 

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)
     {

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();
         }
@@ -154,14 +154,14 @@ 
     {
         $movie = Movies::find($id);
 
-        $likes = session("likes", []);
+        $likes = session("likes", [ ]);
 
         if ($action == "like") {
 
-            $likes[$id] = $movie->id;
+            $likes[ $id ] = $movie->id;
 
-        }else{
-            unset($likes[$id]);
+        } else {
+            unset($likes[ $id ]);
         }
         Session::put("likes", $likes);
         Session::flash('success', "Le film {$movie->title} a bien été liké");
@@ -175,9 +175,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;
@@ -194,14 +194,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;
@@ -209,7 +209,7 @@ 
             foreach ($actors as $actor) {
                 DB::table('actors_movies')
                     ->insert([
-                        ['movies_id' => $movie->id, 'actors_id' => $actor]
+                        [ 'movies_id' => $movie->id, 'actors_id' => $actor ]
                     ]);
             }
         }
@@ -219,7 +219,7 @@ 
             foreach ($directors as $director) {
                 DB::table('directors_movies')
                     ->insert([
-                        ['movies_id' => $movie->id, 'directors_id' => $director]
+                        [ 'movies_id' => $movie->id, 'directors_id' => $director ]
                     ]);
             }
         }

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);

app/Http/Models/Administrators.php

Unused Use Statements -3 removed lines

@@ -3,12 +3,9 @@
 namespace App\Http\Models;
 
 use Illuminate\Database\Eloquent\Model;
-
-
 use Illuminate\Auth\Authenticatable;
 use Illuminate\Foundation\Auth\Access\Authorizable;
 use Illuminate\Auth\Passwords\CanResetPassword;
-
 use Illuminate\Contracts\Auth\Authenticatable as AuthenticatableContract;
 use Illuminate\Contracts\Auth\Access\Authorizable as AuthorizableContract;
 use Illuminate\Contracts\Auth\CanResetPassword as CanResetPasswordContract;

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

@@ -26,7 +26,7 @@ 
         // get id since router
         $id = $this->route('id');
 
-        if($id == null) {
+        if ($id == null) {
             return [
                 'firstname' => 'required|max:255',
                 'lastname' => 'required|max:255',
@@ -35,12 +35,12 @@ 
                 'password' => 'required|confirmed|min:6',
                 'image' => 'required|image',
             ];
-        }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',
             ];

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/routes.php

Spacing +12 added lines, -12 removed lines

@@ -25,7 +25,7 @@ 
 /**
  * BackOffice
  */
-Route::group(['prefix' => 'admin', 'middleware' => 'auth'], function () {
+Route::group([ 'prefix' => 'admin', 'middleware' => 'auth' ], function() {
 
     /**
      * Pages Dashboard
@@ -40,9 +40,9 @@ 
     /**
      * COMMENTAIRES
      */
-    Route::group(['prefix' => 'comments'], function () {
-        Route::get('/index', ['uses' => 'CommentsController@index','as' => 'comments.index']);
-        Route::post('{id}/update', ['uses' => 'CommentsController@update','as' => 'comments.update']);
+    Route::group([ 'prefix' => 'comments' ], function() {
+        Route::get('/index', [ 'uses' => 'CommentsController@index', 'as' => 'comments.index' ]);
+        Route::post('{id}/update', [ 'uses' => 'CommentsController@update', 'as' => 'comments.update' ]);
 
     });
 
@@ -50,7 +50,7 @@ 
     /**
      * CRUD de Movies
      */
-    Route::group(['prefix' => 'movies'], function () {
+    Route::group([ 'prefix' => 'movies' ], function() {
 
         /**
          * Page index: liste des films
@@ -136,7 +136,7 @@ 
 
 
 // CRUD de categories
-    Route::group(['prefix' => 'categories'], function () {
+    Route::group([ 'prefix' => 'categories' ], function() {
 
         Route::get('/index', [
             'as' => 'categories_index',
@@ -187,7 +187,7 @@ 
 
 
 // CRUD de actors
-    Route::group(['prefix' => 'actors'], function () {
+    Route::group([ 'prefix' => 'actors' ], function() {
 
         Route::get('/index', [
             'as' => 'actors_index',
@@ -215,7 +215,7 @@ 
 
 
 // CRUD de directors
-    Route::group(['prefix' => 'directors'], function () {
+    Route::group([ 'prefix' => 'directors' ], function() {
 
         Route::get('/index', [
             'as' => 'directors_delete',
@@ -243,7 +243,7 @@ 
 
 
     // CRUD de administrators
-    Route::group(['prefix' => "administrators",    "middleware" => "authorisation"], function () {
+    Route::group([ 'prefix' => "administrators", "middleware" => "authorisation" ], function() {
 
         Route::get('/index', [
             'as' => 'administrators_index',
@@ -295,7 +295,7 @@ 
 /**
  * Page FAQ
  */
-Route::get('/faq', function () {
+Route::get('/faq', function() {
 
     return view('Pages/faq');
 });
@@ -304,7 +304,7 @@ 
 /**
  * Page about
  */
-Route::get('/about', function () {
+Route::get('/about', function() {
 
     // retourne le nom de la vue
     return view('Pages/about');
@@ -314,7 +314,7 @@ 
 /**
  * Pages concept
  */
-Route::get('/concept', function () {
+Route::get('/concept', function() {
 
     // retourne le nom de la vue
     return view('Pages/concept');

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;
         }
     }