xoco70 / kendozone

app/AuthenticateUser.php

Doc Comments +1 added lines, -1 removed lines

@@ -27,7 +27,7 @@
 
     /**
      * @param $request
-     * @param $listener
+     * @param Http\Controllers\Auth\LoginController $listener
      * @param $provider
      * @return \Illuminate\Http\RedirectResponse|\Illuminate\Routing\Redirector|\Symfony\Component\HttpFoundation\RedirectResponse
      */

app/Category.php

Doc Comments +2 added lines, -2 removed lines

@@ -85,7 +85,7 @@ 
     }
 
     /**
-     * @param $ageCategoryText
+     * @param string $ageCategoryText
      * @return string
      */
     protected function hasAgeMinAndMax($ageCategoryText): string
@@ -123,7 +123,7 @@ 
 
     /**
      * @param $grades
-     * @param $gradeText
+     * @param string $gradeText
      * @return string
      */
     protected function hasGradeMinAndMax($grades, $gradeText): string

app/Console/Commands/LangPull.php

Doc Comments +3 added lines, -3 removed lines

@@ -61,8 +61,8 @@ 
     }
 
     /**
-     * @param $newfile
-     * @param $savepath
+     * @param string $newfile
+     * @param string $savepath
      */
     protected function unzipTranslations($newfile, $savepath)
     {
@@ -81,7 +81,7 @@ 
 
     /**
      * Delete a folder and his content
-     * @param $dir
+     * @param string $dir
      */
     private function rmdir_recursive($dir)
     {

Braces +8 added lines, -3 removed lines

@@ -86,9 +86,14 @@
     private function rmdir_recursive($dir)
     {
         foreach (scandir($dir) as $file) {
-            if ('.' === $file || '..' === $file) continue;
-            if (is_dir("$dir/$file")) $this->rmdir_recursive("$dir/$file");
-            else unlink("$dir/$file");
+            if ('.' === $file || '..' === $file) {
+                continue;
+            }
+            if (is_dir("$dir/$file")) {
+                $this->rmdir_recursive("$dir/$file");
+            } else {
+                unlink("$dir/$file");
+            }
         }
         rmdir($dir);
     }

app/FightersGroup.php

Doc Comments +1 added lines, -1 removed lines

@@ -6,7 +6,7 @@
 {
     /**
      * Get tournament with a lot of stuff Inside - Should Change name
-     * @param $request
+     * @param \Illuminate\Http\Request $request
      * @return \Illuminate\Database\Eloquent\Builder
      */
     public static function getTournament($request)

Spacing +5 added lines, -5 removed lines

@@ -14,27 +14,27 @@
         $tournament = null;
         if (FightersGroup::hasTournamentInRequest($request)) {
             $tournamentSlug = $request->tournament;
-            $tournament = Tournament::with(['championships' => function ($query) use ($request) {
+            $tournament = Tournament::with(['championships' => function($query) use ($request) {
                 $query->with([
                     'settings',
                     'category',
                     'users',
-                    'fightersGroups' => function ($query) {
+                    'fightersGroups' => function($query) {
                         return $query->with('teams', 'competitors', 'fights');
                     }]);
             }])->withCount('competitors', 'teams')
                 ->where('slug', $tournamentSlug)->firstOrFail();
         } elseif (FightersGroup::hasChampionshipInRequest($request)) {
-            $tournament = Tournament::whereHas('championships', function ($query) use ($request) {
+            $tournament = Tournament::whereHas('championships', function($query) use ($request) {
                 return $query->where('id', $request->championshipId);
             })
-                ->with(['championships' => function ($query) use ($request) {
+                ->with(['championships' => function($query) use ($request) {
                     $query->where('id', '=', $request->championshipId)
                         ->with([
                             'settings',
                             'category',
                             'users',
-                            'fightersGroups' => function ($query) {
+                            'fightersGroups' => function($query) {
                                 return $query->with('teams', 'competitors', 'fights');
                             }]);
                 }])

app/Http/Controllers/AssociationController.php

Doc Comments +1 added lines, -1 removed lines

@@ -124,7 +124,7 @@
     /**
      * Update the specified resource in storage.
      *
-     * @param AssociationRequest|Request $request
+     * @param AssociationRequest $request
      * @param  int $id
      * @return Response|RedirectResponse
      * @throws AuthorizationException

Braces +6 added lines, -4 removed lines

@@ -62,8 +62,9 @@ 
     public function store(AssociationRequest $request)
     {
         try {
-            if ($request->president_id == 0)
-                $request->merge(['president_id' => null]);
+            if ($request->president_id == 0) {
+                            $request->merge(['president_id' => null]);
+            }
 
             $association = Association::create($request->all());
             if (Request::ajax()) {
@@ -137,8 +138,9 @@ 
             throw new AuthorizationException();
         }
         try {
-            if ($request->president_id == 0)
-                $request->merge(['president_id' => null]);
+            if ($request->president_id == 0) {
+                            $request->merge(['president_id' => null]);
+            }
 
             $association->update($request->all());
             $msg = trans('msg.association_edit_successful', ['name' => $association->name]);

app/Http/Controllers/Auth/LoginController.php

Doc Comments +1 added lines

@@ -67,6 +67,7 @@
     }
 
     /**
+     * @param string $folder
      * @return int
      */
     protected function getLangWordsCount($folder): int

app/Http/Controllers/Auth/RegisterController.php

Doc Comments +2 added lines, -2 removed lines

@@ -89,7 +89,7 @@ 
      * Handle a registration request for the application.
      *
      * @param AuthRequest $request
-     * @return bool
+     * @return \Illuminate\Http\RedirectResponse
      */
     public function register(AuthRequest $request)
     {
@@ -121,7 +121,7 @@ 
      * Confirm a user's email address.
      *
      * @param  string $token
-     * @return mixed
+     * @return \Illuminate\Http\RedirectResponse
      */
     public function confirmEmail($token)
     {

app/Http/Controllers/ChampionshipController.php

Doc Comments +1 added lines, -1 removed lines

@@ -64,7 +64,7 @@
      *
      * @param Tournament $tournament
      * @param ChampionshipRequest $request
-     * @return \Illuminate\Http\Response
+     * @return \Illuminate\Http\RedirectResponse
      */
     public function store(Tournament $tournament, ChampionshipRequest $request)
     {

Braces +9 added lines, -3 removed lines

@@ -38,9 +38,15 @@
         $grades = Grade::getAllPlucked();
         $invite = Invite::getInviteFromToken($token);
 
-        if (is_null($invite)) throw new InvitationNeededException();
-        if ($invite->hasExpired()) throw new InvitationExpiredException();
-        if ($invite->active != 1) throw new InvitationNotActiveException();
+        if (is_null($invite)) {
+            throw new InvitationNeededException();
+        }
+        if ($invite->hasExpired()) {
+            throw new InvitationExpiredException();
+        }
+        if ($invite->active != 1) {
+            throw new InvitationNotActiveException();
+        }
 
         // Check if user is already registered
         $user = User::where('email', $invite->email)->first();

app/Http/Controllers/ClubController.php

Doc Comments +2 added lines, -2 removed lines

@@ -150,9 +150,9 @@
     /**
      * Update the specified resource in storage.
      *
-     * @param ClubRequest|Request $request
+     * @param ClubRequest $request
      * @param  int $id
-     * @return \Illuminate\Http\Response
+     * @return RedirectResponse
      * @throws AuthorizationException
      */
     public function update(ClubRequest $request, $id)

Braces +15 added lines, -5 removed lines

@@ -88,9 +88,15 @@ 
     {
 
         try {
-            if ($request->president_id == 0) $request->merge(['president_id' => null]);
-            if ($request->federation_id == 0) $request->merge(['federation_id' => null]);
-            if ($request->association_id == 0) $request->merge(['association_id' => null]);
+            if ($request->president_id == 0) {
+                $request->merge(['president_id' => null]);
+            }
+            if ($request->federation_id == 0) {
+                $request->merge(['federation_id' => null]);
+            }
+            if ($request->association_id == 0) {
+                $request->merge(['association_id' => null]);
+            }
             $club = Club::create($request->all());
 
             if (Request::ajax()) {
@@ -162,8 +168,12 @@ 
         $this->authorize('update', [$club, Auth::user()]);
 
         try {
-            if ($request->president_id == 0) $request->merge(['president_id' => null]);
-            if ($request->association_id == 0) $request->merge(['association_id' => null]);
+            if ($request->president_id == 0) {
+                $request->merge(['president_id' => null]);
+            }
+            if ($request->association_id == 0) {
+                $request->merge(['association_id' => null]);
+            }
 
             $club->update($request->all());
             $msg = trans('msg.club_edit_successful', ['name' => $club->name]);