xoco70 / laravel-tournaments

src/models/Competitor.php

Braces +12 added lines, -4 removed lines

@@ -86,15 +86,23 @@
 
     public function getName()
     {
-        if ($this == null) return "BYE";
-        if ($this->user == null) return "BYE";
+        if ($this == null) {
+            return "BYE";
+        }
+        if ($this->user == null) {
+            return "BYE";
+        }
         return $this->user->name;
     }
 
     public function getFullName()
     {
-        if ($this == null) return "BYE";
-        if ($this->user == null) return "BYE";
+        if ($this == null) {
+            return "BYE";
+        }
+        if ($this->user == null) {
+            return "BYE";
+        }
         return $this->user->firstname . " " . $this->user->lastname;
     }
 

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, 'kendo-tournaments');
 
-        $this->loadMigrationsFrom(__DIR__.'/../database/migrations');
-        $this->loadTranslationsFrom(__DIR__.'/../translations', 'kendo-tournaments');
+        $this->loadMigrationsFrom(__DIR__ . '/../database/migrations');
+        $this->loadTranslationsFrom(__DIR__ . '/../translations', 'kendo-tournaments');
 
-        $this->publishes([__DIR__.'/../database/migrations' => $this->app->databasePath().'/migrations'], 'migrations');
-        $this->publishes([__DIR__.'/../database/seeds' => $this->app->databasePath().'/seeds'], 'seeds');
-        $this->publishes([__DIR__.'/../database/factories' => $this->app->databasePath().'/factories'], 'seeds');
-        $this->publishes([__DIR__.'/../resources/assets' => public_path('vendor/kendo-tournaments')], 'assets');
+        $this->publishes([__DIR__ . '/../database/migrations' => $this->app->databasePath() . '/migrations'], 'migrations');
+        $this->publishes([__DIR__ . '/../database/seeds' => $this->app->databasePath() . '/seeds'], 'seeds');
+        $this->publishes([__DIR__ . '/../database/factories' => $this->app->databasePath() . '/factories'], 'seeds');
+        $this->publishes([__DIR__ . '/../resources/assets' => public_path('vendor/kendo-tournaments')], 'assets');
 
-        $router->group(['prefix' => 'kendo-tournaments', 'middleware' => ['web']], function ($router) {
+        $router->group(['prefix' => 'kendo-tournaments', 'middleware' => ['web']], function($router) {
             $router->get('/', 'Xoco70\KendoTournaments\TreeController@index')->name('tree.index');
             $router->post('/championships/{championship}/trees', 'Xoco70\KendoTournaments\TreeController@store')->name('tree.store');
         });

database/migrations/2016_11_23_125509_create_Fight_table.php

Spacing +1 added lines, -1 removed lines

@@ -13,7 +13,7 @@
      */
     public function up()
     {
-        Schema::create('fight', function (Blueprint $table) {
+        Schema::create('fight', function(Blueprint $table) {
             $table->increments('id');
             $table->integer('short_id')->unsigned()->nullable();
             $table->integer('fighters_group_id')->unsigned()->index();

database/migrations/2015_12_03_230346_create_Category_table.php

Spacing +3 added lines, -3 removed lines

@@ -12,7 +12,7 @@ 
      */
     public function up()
     {
-        Schema::create('category', function (Blueprint $table) {
+        Schema::create('category', function(Blueprint $table) {
             $table->increments('id');
             $table->string('name');
             $table->string('gender')->nullable();
@@ -28,7 +28,7 @@ 
             $table->engine = 'InnoDB';
         });
 
-        Schema::create('championship', function (Blueprint $table) {
+        Schema::create('championship', function(Blueprint $table) {
             $table->increments('id');
             $table->integer('tournament_id')->unsigned()->index();
             $table->integer('category_id')->unsigned()->index();
@@ -50,7 +50,7 @@ 
             $table->engine = 'InnoDB';
         });
 
-        Schema::create('competitor', function (Blueprint $table) {
+        Schema::create('competitor', function(Blueprint $table) {
             $table->increments('id');
             $table->integer('short_id')->unsigned()->nullable();
             $table->integer('championship_id')->unsigned()->index();

database/migrations/2016_10_24_223706_create_FightersGroup_table.php

Spacing +1 added lines, -1 removed lines

@@ -13,7 +13,7 @@
      */
     public function up()
     {
-        Schema::create('fighters_groups', function (Blueprint $table) {
+        Schema::create('fighters_groups', function(Blueprint $table) {
             $table->increments('id');
             $table->tinyInteger('short_id')->unsigned()->nullable();
             $table->integer('championship_id')->unsigned()->index();

resources/views/partials/tree/preliminary.blade.php

Indentation +1 added lines, -1 removed lines

@@ -1,5 +1,5 @@
 <?php
-  $settings = $championship->getSettings();
+    $settings = $championship->getSettings();
 ?>
 <div align="center">
     @if (Request::is('championships/'.$championship->id.'/pdf'))

src/TreeGen/TreeGen.php

Doc Comments +11 added lines, -6 removed lines

@@ -226,8 +226,10 @@ 
     }
 
     /**
-     * @param $usersByArea
-     * @param $area
+     * @param Collection $usersByArea
+     * @param integer $area
+     * @param integer $round
+     * @param integer $shuffle
      *
      */
     public function generateGroupsForRound($usersByArea, $area, $round, $shuffle)
@@ -247,7 +249,7 @@ 
     /**
      * @param $fighters
      * @param $area
-     * @param $order
+     * @param integer $order
      * @param $round
      * @return FightersGroup
      */
@@ -312,6 +314,9 @@ 
             : new Competitor();
     }
 
+    /**
+     * @param integer $groupSize
+     */
     public function createByeGroup($groupSize): Collection
     {
         $byeFighter = $this->createByeFighter();
@@ -324,7 +329,7 @@ 
 
     /**
      * @param $fighters
-     * @param $fighterGroups
+     * @param Collection $fighterGroups
      * @return Collection
      */
     public function adjustFightersGroupWithByes($fighters, $fighterGroups): Collection
@@ -359,7 +364,7 @@ 
     /**
      * Get the next group on the right ( parent ), final round being the ancestor
      * @param $matchNumber
-     * @param $previousRound
+     * @param Collection $previousRound
      * @return mixed
      */
     private function getParentGroup($matchNumber, $previousRound)
@@ -371,7 +376,7 @@ 
 
     /**
      * Save Groups with their parent info
-     * @param $numRounds
+     * @param integer $numRounds
      * @param $numFightersEliminatory
      */
     private function pushGroups($numRounds, $numFightersEliminatory, $shuffle = true)

Spacing +2 added lines, -2 removed lines

@@ -130,7 +130,7 @@ 
     private function getTreeSize($fighterCount, $groupSize)
     {
         $square = collect([1, 2, 4, 8, 16, 32, 64]);
-        $squareMultiplied = $square->map(function ($item, $key) use ($groupSize) {
+        $squareMultiplied = $square->map(function($item, $key) use ($groupSize) {
             return $item * $groupSize;
         });
 
@@ -352,7 +352,7 @@ 
     private function getPreviousRound($currentRound)
     {
         $previousRound = $this->championship->groupsByRound($currentRound + 1)->get();
-        dump("previous". $previousRound->pluck('id'));
+        dump("previous" . $previousRound->pluck('id'));
         return $previousRound;
     }
 

Braces +9 added lines, -3 removed lines

@@ -255,7 +255,9 @@ 
     {
 
         $fighters = $fighters->pluck('id');
-        if ($shuffle) $fighters->shuffle();
+        if ($shuffle) {
+            $fighters->shuffle();
+        }
         $group = $this->saveGroup($area, $order, $round, $parent);
 
         // Add all competitors to Pivot Table
@@ -417,10 +419,14 @@ 
     {
         if ($this->championship->hasPreliminary()) {
             $fightersGroup = $fightersByEntity->chunk($this->settings->preliminaryGroupSize);
-            if ($shuffle) $fightersGroup->shuffle();
+            if ($shuffle) {
+                $fightersGroup->shuffle();
+            }
         } elseif ($this->championship->isDirectEliminationType() || $round > 1) {
             $fightersGroup = $fightersByEntity->chunk(2);
-            if ($shuffle) $fightersGroup->shuffle();
+            if ($shuffle) {
+                $fightersGroup->shuffle();
+            }
         } else { // Round Robin
             $fightersGroup = $fightersByEntity->chunk($fightersByEntity->count());
         }

database/factories/CompetitorFactory.php

Spacing +1 added lines, -1 removed lines

@@ -4,7 +4,7 @@
 use Xoco70\KendoTournaments\Models\Championship;
 use Xoco70\KendoTournaments\Models\Competitor;
 
-$factory->define(Competitor::class, function (Faker\Generator $faker) {
+$factory->define(Competitor::class, function(Faker\Generator $faker) {
     $tcs = Championship::all()->pluck('id')->toArray();
     $users = User::all()->pluck('id')->toArray();
     $championshipId = $faker->randomElement($tcs);

src/TreeController.php

Spacing +1 added lines, -1 removed lines

@@ -51,7 +51,7 @@
         DB::table('fighters_group_competitor')->delete();
         DB::table('fighters_group_team')->delete();
         DB::table('competitor')->delete();
-        DB::table('users')->where('id','<>',1)->delete();
+        DB::table('users')->where('id', '<>', 1)->delete();
 
         $championship = Championship::with('teams', 'users', 'category', 'settings')->find($championship->id);
         $numFighters = $request->numFighters;