xoco70 / laravel-tournaments

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

Doc Comments +13 added lines, -7 removed lines

@@ -22,8 +22,14 @@ 
     abstract protected function generateFights();
 
 
+    /**
+     * @param string|null $fighterToUpdate
+     */
     abstract protected function addFighterToGroup(FightersGroup $group, $fighter, $fighterToUpdate);
 
+    /**
+     * @return Collection
+     */
     abstract protected function getByeGroup($fighters);
 
     abstract protected function getNumRounds($fightersCount);
@@ -61,7 +67,7 @@ 
     /**
      * Get the biggest entity group.
      *
-     * @param $userGroups
+     * @param Collection $userGroups
      *
      * @return int
      */
@@ -130,7 +136,7 @@ 
     /**
      * Repart BYE in the tree,.
      *
-     * @param $fighterGroups
+     * @param Collection $fighterGroups
      * @param int $max
      *
      * @return Collection
@@ -288,7 +294,7 @@ 
     /**
      * Attach a parent to every child for nestedSet Navigation.
      *
-     * @param $numFightersElim
+     * @param integer $numFightersElim
      */
     protected function addParentToChildren($numFightersElim)
     {
@@ -312,8 +318,8 @@ 
 
     /**
      * @param Collection $fighters
-     * @param $frequency
-     * @param $sizeGroupBy
+     * @param integer $frequency
+     * @param integer $sizeGroupBy
      * @param $bye
      *
      * @return Collection
@@ -338,8 +344,8 @@ 
     /**
      * @param $frequency
      * @param $sizeGroupBy
-     * @param $count
-     * @param $byeCount
+     * @param integer $count
+     * @param integer $byeCount
      *
      * @return bool
      */

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

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

Spacing +2 added lines, -2 removed lines

@@ -56,13 +56,13 @@
     protected static function boot()
     {
         parent::boot();
-        static::deleting(function ($tournament) {
+        static::deleting(function($tournament) {
             foreach ($tournament->championships as $ct) {
                 $ct->delete();
             }
             $tournament->invites()->delete();
         });
-        static::restoring(function ($tournament) {
+        static::restoring(function($tournament) {
             foreach ($tournament->championships()->withTrashed()->get() as $ct) {
                 $ct->restore();
             }

src/TreeGen/SingleEliminationTreeGen.php

Doc Comments +2 added lines, -2 removed lines

@@ -57,7 +57,7 @@ 
     /**
      * Create empty groups after round 1.
      *
-     * @param $numFighters
+     * @param integer $numFighters
      */
     protected function pushEmptyGroupsToTree($numFighters)
     {
@@ -155,7 +155,7 @@ 
 
     /**
      * @param Collection $usersByArea
-     * @param $round
+     * @param integer $round
      */
     public function generateGroupsForRound(Collection $usersByArea, $round)
     {

src/TreeGen/CreateSingleEliminationTree.php

Spacing +5 added lines, -5 removed lines

@@ -26,7 +26,7 @@ 
         $this->groupsByRound = $groupsByRound;
         $this->hasPreliminary = $hasPreliminary;
 
-        $this->firstRoundName = $groupsByRound->first()->map(function ($item) use ($championship) {
+        $this->firstRoundName = $groupsByRound->first()->map(function($item) use ($championship) {
             $fighters = $item->getFightersWithBye();
             $fighter1 = $fighters->get(0);
             $fighter2 = $fighters->get(1);
@@ -140,7 +140,7 @@ 
             //The minus 3 is to ignore the final, semi final and quarter final rounds
 
             for ($i = 0; $i < $noRounds - 3; $i++) {
-                $tempRounds[] = 'Last '.$noTeamsInFirstRound;
+                $tempRounds[] = 'Last ' . $noTeamsInFirstRound;
                 $noTeamsInFirstRound /= 2;
             }
 
@@ -162,7 +162,7 @@ 
         foreach ($roundTitles as $key => $roundTitle) {
             $left = $key * ($this->matchWrapperWidth + $this->roundSpacing - 1);
 
-            echo '<div class="round-title" style="left: '.$left.'px;">'.$roundTitle.'</div>';
+            echo '<div class="round-title" style="left: ' . $left . 'px;">' . $roundTitle . '</div>';
         }
         echo '</div>';
     }
@@ -175,7 +175,7 @@ 
     public function getPlayerList($selected)
     {
         $html = '<select>
-                <option'.($selected == '' ? ' selected' : '').'></option>';
+                <option'.($selected == '' ? ' selected' : '') . '></option>';
 
         foreach ($this->championship->fighters as $fighter) {
             $html = $this->addOptionToSelect($selected, $fighter, $html);
@@ -235,7 +235,7 @@ 
     {
         if ($fighter != null) {
             $select = $selected != null && $selected->id == $fighter->id ? ' selected' : '';
-            $html .= '<option'.$select
+            $html .= '<option' . $select
                 .' value='
                 .($fighter->id ?? '')
                 .'>'

database/factories/ChampionshipSettingsFactory.php

Spacing +1 added lines, -1 removed lines

@@ -4,7 +4,7 @@
 use Xoco70\LaravelTournaments\Models\Championship;
 use Xoco70\LaravelTournaments\Models\ChampionshipSettings;
 
-$factory->define(ChampionshipSettings::class, function (Faker\Generator $faker) use ($factory) {
+$factory->define(ChampionshipSettings::class, function(Faker\Generator $faker) use ($factory) {
     $tcs = Championship::all()->pluck('id')->toArray();
 
     $fightingAreas = [1, 2, 4, 8];