xoco70 / laravel-tournaments

src/Models/Team.php

Doc Comments +3 added lines

@@ -31,6 +31,9 @@
         return $this->name;
     }
 
+    /**
+     * @param string $attr
+     */
     public function hasAttribute($attr)
     {
         return array_key_exists($attr, $this->attributes);

database/migrations/2014_10_12_000001_alter_lt_users_table.php

Spacing +3 added lines, -3 removed lines

@@ -14,7 +14,7 @@ 
     public function up()
     {
         if (!Schema::hasTable('users')) {
-            Schema::create('users', function (Blueprint $table) {
+            Schema::create('users', function(Blueprint $table) {
                 $table->increments('id');
                 $table->string('name');
                 $table->string('firstname')->default('firstname');
@@ -24,7 +24,7 @@ 
                 $table->timestamps();
             });
         } else {
-            Schema::table('users', function (Blueprint $table) {
+            Schema::table('users', function(Blueprint $table) {
                 if (!Schema::hasColumn('users', 'name')) {
                     $table->string('name')->default('name');
                 }
@@ -35,7 +35,7 @@ 
                     $table->string('lastname')->default('lastname');
                 }
                 if (!Schema::hasColumn('users', 'email')) {
-                    $table->string('email')->default('user_'.rand(100000, 999999).'@kendozone.com')->unique();
+                    $table->string('email')->default('user_' . rand(100000, 999999) . '@kendozone.com')->unique();
                 }
 
                 if (!Schema::hasColumn('users', 'password')) {

src/TreeGen/PlayOffTreeGen.php

Doc Comments +4 added lines, -1 removed lines

@@ -29,7 +29,7 @@ 
      *
      * @param $fightersByEntity
      *
-     * @return mixed
+     * @return Collection
      */
     protected function chunk(Collection $fightersByEntity)
     {
@@ -103,6 +103,9 @@ 
         $this->generateGroupsForRound($fighters, 1);
     }
 
+    /**
+     * @param integer $round
+     */
     protected function generateGroupsForRound(Collection $fightersByArea, $round)
     {
         $fightersId = $fightersByArea->pluck('id');

src/TreeGen/SingleEliminationTreeGen.php

Doc Comments +8 added lines, -7 removed lines

@@ -59,7 +59,7 @@ 
     /**
      * Create empty groups after round 1.
      *
-     * @param $numFighters
+     * @param integer $numFighters
      */
     protected function pushEmptyGroupsToTree($numFighters)
     {
@@ -162,7 +162,7 @@ 
 
     /**
      * @param Collection $usersByArea
-     * @param $round
+     * @param integer $round
      */
     public function generateGroupsForRound(Collection $usersByArea, $round)
     {
@@ -249,10 +249,11 @@ 
     }
 
     /**
-     * @param $frequency
+     * @param integer $frequency
      * @param $groupSize
-     * @param $count
-     * @param $byeCount
+     * @param integer $count
+     * @param integer $byeCount
+     * @param integer $numByeTotal
      *
      * @return bool
      */
@@ -293,7 +294,7 @@ 
     /**
      * Get the biggest entity group.
      *
-     * @param $userGroups
+     * @param Collection $userGroups
      *
      * @return int
      */
@@ -311,7 +312,7 @@ 
     /**
      * Repart BYE in the tree,.
      *
-     * @param $fighterGroups
+     * @param Collection $fighterGroups
      * @param int $max
      *
      * @return Collection

Spacing +2 added lines, -2 removed lines

@@ -226,7 +226,7 @@ 
         $bye = $this->createByeFighter();
         $groupSize = $this->firstRoundGroupSize();
         $frequency = $groupSize != 0
-            ? (int) floor(count($fighters) / $groupSize / $groupSize)
+            ? (int)floor(count($fighters) / $groupSize / $groupSize)
             : -1;
         if ($frequency < $groupSize) {
             $frequency = $groupSize;
@@ -305,7 +305,7 @@ 
     private function getMaxFightersByEntity($userGroups): int
     {
         return $userGroups
-            ->sortByDesc(function ($group) {
+            ->sortByDesc(function($group) {
                 return $group->count();
             })
             ->first()

src/TreeGen/TreeGen.php

Spacing +5 added lines, -5 removed lines

@@ -91,7 +91,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,...]
 
@@ -197,10 +197,10 @@ 
     protected function getFightersByArea()
     {
         $areas = $this->settings->fightingAreas;
-        $fighters = $this->getFighters();   // Get Competitor or Team Objects
-        $fighterByEntity = $this->getFightersByEntity($fighters);   // Chunk it by entities (Fede, Assoc, Club,...)
-        $fightersWithBye = $this->adjustFightersGroupWithByes($fighters, $fighterByEntity);     // Fill with Byes
-        return $fightersWithBye->chunk(count($fightersWithBye) / $areas);   // Chunk user by areas
+        $fighters = $this->getFighters(); // Get Competitor or Team Objects
+        $fighterByEntity = $this->getFightersByEntity($fighters); // Chunk it by entities (Fede, Assoc, Club,...)
+        $fightersWithBye = $this->adjustFightersGroupWithByes($fighters, $fighterByEntity); // Fill with Byes
+        return $fightersWithBye->chunk(count($fightersWithBye) / $areas); // Chunk user by areas
     }
 
     /**

Doc Comments +5 added lines, -2 removed lines

@@ -21,6 +21,9 @@ 
 
     abstract protected function generateFights();
 
+    /**
+     * @param string|null $fighterToUpdate
+     */
     abstract protected function addFighterToGroup(FightersGroup $group, $fighter, $fighterToUpdate);
 
     abstract protected function getByeGroup($fighters);
@@ -82,7 +85,7 @@ 
      * Get the size the first round will have.
      *
      * @param $fighterCount
-     * @param $groupSize
+     * @param integer $groupSize
      *
      * @return int
      */
@@ -200,7 +203,7 @@ 
     /**
      * Logically build the tree ( attach a parent to every child for nestedSet Navigation ).
      *
-     * @param $numFighters
+     * @param integer $numFighters
      */
     protected function addParentToChildren($numFighters)
     {

src/TournamentsServiceProvider.php

Doc Comments -1 removed lines

@@ -10,7 +10,6 @@
     /**
      * Bootstrap the application services.
      *
-     * @param Router $router
      *
      * @return void
      */

Spacing +7 added lines, -7 removed lines

@@ -16,16 +16,16 @@
      */
     public function boot()
     {
-        $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/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__.'/../config/laravel-tournaments.php' => config_path('laravel-tournaments.php')], '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__ . '/../config/laravel-tournaments.php' => config_path('laravel-tournaments.php')], 'laravel-tournaments');
     }
 
     /**

database/factories/UserFactory.php

Spacing +1 added lines, -1 removed lines

@@ -1,7 +1,7 @@
 <?php
 
 
-$factory->define(config('laravel-tournaments.user.model'), function (Faker\Generator $faker) {
+$factory->define(config('laravel-tournaments.user.model'), function(Faker\Generator $faker) {
     return [
         'name'               => $faker->name,
         'email'              => $faker->unique()->safeEmail,

database/factories/CompetitorFactory.php

Spacing +1 added lines, -1 removed lines

@@ -3,7 +3,7 @@
 use Xoco70\LaravelTournaments\Models\Championship;
 use Xoco70\LaravelTournaments\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 = config('laravel-tournaments.user.model')::all()->pluck('id')->toArray();
     $championshipId = $faker->randomElement($tcs);

database/migrations/2014_11_01_171759_create_lt_Tournament_table.php

Spacing +1 added lines, -1 removed lines

@@ -9,7 +9,7 @@
 {
     public function up()
     {
-        Schema::create('tournament', function (Blueprint $table) {
+        Schema::create('tournament', function(Blueprint $table) {
             $table->increments('id');
             //TODO Added ->nullable() to solve FK issue with Sqlite :(
             $table->bigInteger('user_id')->unsigned()->nullable()->index();