xoco70 / laravel-tournaments

database/factories/TournamentFactory.php

Spacing +1 added lines, -1 removed lines

@@ -4,7 +4,7 @@
 use Xoco70\LaravelTournaments\Models\Tournament;
 use Xoco70\LaravelTournaments\Models\Venue;
 
-$factory->define(Tournament::class, function (Faker\Generator $faker) {
+$factory->define(Tournament::class, function(Faker\Generator $faker) {
 
     $dateIni = $faker->dateTimeBetween('now', '+2 weeks')->format('Y-m-d');
     $venues = Venue::all()->pluck('id')->toArray();

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/Models/PreliminaryFight.php

Doc Comments +1 added lines, -1 removed lines

@@ -58,7 +58,7 @@
     /**
      * Save a Fight.
      *
-     * @param Collection $group
+     * @param FightersGroup $group
      * @param int $numGroup
      * @param int $order
      */

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 +1 added lines, -1 removed lines

@@ -301,7 +301,7 @@
     function getMaxFightersByEntity($userGroups): int
     {
         return $userGroups
-            ->sortByDesc(function ($group) {
+            ->sortByDesc(function($group) {
                 return $group->count();
             })
             ->first()

src/TreeGen/TreeGen.php

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);
@@ -84,7 +87,7 @@ 
      * Get the size the first round will have.
      *
      * @param $fighterCount
-     * @param $groupSize
+     * @param integer $groupSize
      *
      * @return int
      */
@@ -206,7 +209,7 @@ 
     /**
      * Logically build the tree ( attach a parent to every child for nestedSet Navigation )
      *
-     * @param $numFighters
+     * @param integer $numFighters
      */
     protected function addParentToChildren($numFighters)
     {

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
     }
 
     /**

src/TreeGen/CreateSingleEliminationTree.php

Spacing +1 added lines, -1 removed lines

@@ -28,7 +28,7 @@
 
     public function build()
     {
-        $fighters = $this->groupsByRound->first()->map(function ($item) {
+        $fighters = $this->groupsByRound->first()->map(function($item) {
             $fighters = $item->getFightersWithBye();
             $fighter1 = $fighters->get(0);
             $fighter2 = $fighters->get(1);

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');
     }
 
     /**