xoco70 / laravel-tournaments

database/migrations/2014_10_12_000010_add_users_fields_table.php

Spacing +3 added lines, -3 removed lines

@@ -13,7 +13,7 @@ 
      */
     public function up()
     {
-        Schema::table('users', function ($table) {
+        Schema::table('users', function($table) {
             $table->string('firstname')->default('firstname');
             $table->string('lastname')->default('lastname');
         });
@@ -28,12 +28,12 @@ 
     {
         DBHelpers::setFKCheckOff();
         if (Schema::hasColumn('users', 'firstname')) {
-            Schema::table('users', function (Blueprint $table) {
+            Schema::table('users', function(Blueprint $table) {
                 $table->dropColumn('firstname');
             });
         }
         if (Schema::hasColumn('users', 'lastname')) {
-            Schema::table('users', function (Blueprint $table) {
+            Schema::table('users', function(Blueprint $table) {
                 $table->dropColumn('lastname');
             });
         }

src/TreeGen/TreeGen.php

Spacing +3 added lines, -3 removed lines

@@ -72,7 +72,7 @@ 
     private function getMaxFightersByEntity($userGroups): int
     {
         return $userGroups
-            ->sortByDesc(function ($group) {
+            ->sortByDesc(function($group) {
                 return $group->count();
             })
             ->first()
@@ -113,7 +113,7 @@ 
     protected function getTreeSize($fighterCount, $groupSize)
     {
         $squareMultiplied = collect([1, 2, 4, 8, 16, 32, 64])
-            ->map(function ($item) use ($groupSize) {
+            ->map(function($item) use ($groupSize) {
                 return $item * $groupSize;
             }); // [4, 8, 16, 32, 64,...]
 
@@ -171,7 +171,7 @@ 
         $sizeGroupBy = count($byeGroup);
 
         $frequency = $sizeGroupBy != 0
-            ? (int) floor($sizeFighters / $sizeGroupBy)
+            ? (int)floor($sizeFighters / $sizeGroupBy)
             : -1;
 
         // Create Copy of $competitors

src/models/Competitor.php

Spacing +1 added lines, -1 removed lines

@@ -41,7 +41,7 @@
      */
     public function getFullName() //TODO Should remove get prefix
     {
-        return $this->defaultName() ?? $this->user->firstname.' '.$this->user->lastname;
+        return $this->defaultName() ?? $this->user->firstname . ' ' . $this->user->lastname;
     }
 
     /**

src/models/Championship.php

Spacing +3 added lines, -3 removed lines

@@ -28,11 +28,11 @@ 
     {
         parent::boot();
 
-        static::deleting(function ($championship) {
+        static::deleting(function($championship) {
             $championship->competitors()->delete();
             $championship->settings()->delete();
         });
-        static::restoring(function ($championship) {
+        static::restoring(function($championship) {
             $championship->competitors()->restore();
             $championship->settings()->restore();
         });
@@ -195,7 +195,7 @@ 
         $ageCategoryText = $this->category->getAgeString();
         $gradeText = $this->category->getGradeString();
 
-        return $teamText.' '.$genders[$this->category->gender].' '.$ageCategoryText.' '.$gradeText;
+        return $teamText . ' ' . $genders[$this->category->gender] . ' ' . $ageCategoryText . ' ' . $gradeText;
     }
 
     public function getSettings()

src/TreeController.php

Spacing +2 added lines, -2 removed lines

@@ -81,10 +81,10 @@
     {
         if ($isTeam) {
             $championship = Championship::find(2);
-            factory(Team::class, (int) $numFighters)->create(['championship_id' => $championship->id]);
+            factory(Team::class, (int)$numFighters)->create(['championship_id' => $championship->id]);
         } else {
             $championship = Championship::find(1);
-            $users = factory(User::class, (int) $numFighters)->create();
+            $users = factory(User::class, (int)$numFighters)->create();
             foreach ($users as $user) {
                 factory(Competitor::class)->create(
                     ['championship_id' => $championship->id,

Doc Comments +4 added lines, -1 removed lines

@@ -37,7 +37,7 @@ 
      *
      * @param Request $request
      *
-     * @return \Illuminate\Http\Response|string
+     * @return \Illuminate\Http\RedirectResponse
      */
     public function store(Request $request, $championshipId)
     {
@@ -137,6 +137,9 @@ 
         return back();
     }
 
+    /**
+     * @param integer $numFighter
+     */
     public function getWinnerId($fighters, $scores, $numFighter)
     {
         return $scores[$numFighter] != null ? $fighters[$numFighter] : null;

src/TournamentsServiceProvider.php

Spacing +8 added lines, -8 removed lines

@@ -16,18 +16,18 @@
      */
     public function boot(Router $router)
     {
-        $viewPath = __DIR__.'/../resources/views';
+        $viewPath = __DIR__ . '/../resources/views';
         $this->loadViewsFrom($viewPath, 'laravel-tournaments');
 
-        $this->loadMigrationsFrom(__DIR__.'/../database/migrations');
-        $this->loadTranslationsFrom(__DIR__.'/../translations', 'laravel-tournaments');
+        $this->loadMigrationsFrom(__DIR__ . '/../database/migrations');
+        $this->loadTranslationsFrom(__DIR__ . '/../translations', 'laravel-tournaments');
 
-        $this->publishes([__DIR__.'/../database/migrations' => $this->app->databasePath().'/migrations'], 'laravel-tournaments');
-        $this->publishes([__DIR__.'/../database/seeds' => $this->app->databasePath().'/seeds'], 'laravel-tournaments');
-        $this->publishes([__DIR__.'/../database/factories' => $this->app->databasePath().'/factories'], 'laravel-tournaments');
-        $this->publishes([__DIR__.'/../resources/assets' => public_path('vendor/laravel-tournaments')], 'laravel-tournaments');
+        $this->publishes([__DIR__ . '/../database/migrations' => $this->app->databasePath() . '/migrations'], 'laravel-tournaments');
+        $this->publishes([__DIR__ . '/../database/seeds' => $this->app->databasePath() . '/seeds'], 'laravel-tournaments');
+        $this->publishes([__DIR__ . '/../database/factories' => $this->app->databasePath() . '/factories'], 'laravel-tournaments');
+        $this->publishes([__DIR__ . '/../resources/assets' => public_path('vendor/laravel-tournaments')], 'laravel-tournaments');
 
-        $router->group(['prefix' => 'laravel-tournaments', 'middleware' => ['web']], function ($router) {
+        $router->group(['prefix' => 'laravel-tournaments', 'middleware' => ['web']], function($router) {
             $router->get('/', 'Xoco70\LaravelTournaments\TreeController@index')->name('tree.index');
             $router->post('/championships/{championship}/trees', 'Xoco70\LaravelTournaments\TreeController@store')->name('tree.store');
             $router->put('/championships/{championship}/trees', 'Xoco70\LaravelTournaments\TreeController@update')->name('tree.update');

src/TreeGen/PlayOffTreeGen.php

Doc Comments +4 added lines, -1 removed lines

@@ -30,7 +30,7 @@ 
      *
      * @param $fightersByEntity
      *
-     * @return mixed
+     * @return Collection
      */
     protected function chunkAndShuffle(Collection $fightersByEntity)
     {
@@ -97,6 +97,9 @@ 
         }
     }
 
+    /**
+     * @param integer $round
+     */
     protected function generateGroupsForRound(Collection $fightersByArea, $round)
     {
         $fightersId = $fightersByArea->pluck('id');

resources/views/partials/tree/brackets/playerList.blade.php

Spacing +3 added lines, -3 removed lines

@@ -1,10 +1,10 @@
 <?php
 $prefix = "singleElimination";
-if ($championship->hasPreliminary() && $roundNumber ==1){
+if ($championship->hasPreliminary() && $roundNumber == 1) {
     $prefix = "preliminary";
 }
-$className = $prefix."_select";
-$selectName = $prefix."_fighters[]";
+$className = $prefix . "_select";
+$selectName = $prefix . "_fighters[]";
 
 ?>
 <!-- r = round, m = match, f = fighter -->

src/models/Fight.php

Spacing +3 added lines, -3 removed lines

@@ -112,15 +112,15 @@
     {
         $isTeam = $this->group->championship->category->isTeam;
         if ($isTeam) {
-            $teamToUpdate = 'team'.$numFighter;
+            $teamToUpdate = 'team' . $numFighter;
 
             return optional($this->$teamToUpdate)->$attr;
         }
-        $competitorToUpdate = 'competitor'.$numFighter;
+        $competitorToUpdate = 'competitor' . $numFighter;
         if ($attr == 'name') {
             return $this->$competitorToUpdate == null
                 ? ''
-                : $this->$competitorToUpdate->user->firstname.' '.$this->$competitorToUpdate->user->lastname;
+                : $this->$competitorToUpdate->user->firstname . ' ' . $this->$competitorToUpdate->user->lastname;
         } elseif ($attr == 'short_id') {
             return optional($this->$competitorToUpdate)->short_id;
         }