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/models/FightersGroup.php

Braces +3 added lines, -1 removed lines

@@ -206,7 +206,9 @@
             foreach ($fightsByRound as $numFight => $fight) {
 
                 $parentGroup = $fight->group->parent;
-                if ($parentGroup == null) break;
+                if ($parentGroup == null) {
+                    break;
+                }
                 $parentFight = $parentGroup->fights->get(0);
 
                 // IN this $fight, if it is the first child or the second child --> c1 or c2 in parent

src/TreeGen/TreeGen.php

Doc Comments +9 added lines, -8 removed lines

@@ -49,7 +49,7 @@ 
 
     /**
      * Get the biggest entity group
-     * @param $userGroups
+     * @param Collection $userGroups
      *
      * @return int
      */
@@ -68,6 +68,7 @@ 
      * Get Competitor's list ordered by entities
      * Countries for Internation Tournament, State for a National Tournament, etc.
      *
+     * @param Collection $fighters
      * @return Collection
      */
     private function getFightersByEntity($fighters): Collection
@@ -153,7 +154,7 @@ 
 
     /**
      * Repart BYE in the tree,
-     * @param $fighterGroups
+     * @param Collection $fighterGroups
      * @param int $max
      *
      * @return Collection
@@ -243,9 +244,9 @@ 
 
     /**
      * @param $fighters
-     * @param $area
+     * @param integer $area
      * @param integer $order
-     * @param $round
+     * @param integer $round
      * @return FightersGroup
      */
     public function saveGroupAndSync($fighters, $area, $order, $round, $parent, $shuffle)
@@ -284,7 +285,7 @@ 
 
     /**
      * @param $area
-     * @param $order
+     * @param integer $order
      * @param $round
      * @param $parent
      * @return FightersGroup
@@ -324,7 +325,7 @@ 
     }
 
     /**
-     * @param $fighters
+     * @param Collection $fighters
      * @param Collection $fighterGroups
      * @return Collection
      */
@@ -409,8 +410,8 @@ 
 
     /**
      * Chunk Fighters into groups for fighting, and optionnaly shuffle
-     * @param $round
-     * @param $shuffle
+     * @param integer $round
+     * @param integer $shuffle
      * @param $fightersByEntity
      * @return mixed
      */

Spacing +1 added lines, -1 removed lines

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