xoco70 / laravel-tournaments

src/Models/DirectEliminationFight.php

Unused Use Statements -3 removed lines

@@ -2,9 +2,6 @@
 
 namespace Xoco70\KendoTournaments\Models;
 
-use Illuminate\Database\Eloquent\Model;
-use Illuminate\Support\Collection;
-
 class DirectEliminationFight extends Fight
 {
     public function __construct(Fight $fight = null)

Spacing +6 added lines, -6 removed lines

@@ -9,12 +9,12 @@
 {
     public function __construct(Fight $fight = null)
     {
-        if ($fight!=null){
-            $this->id= $fight->id;
-            $this->short_id= $fight->short_id;
-            $this->fighters_group_id= $fight->fighters_group_id;
-            $this->c1= $fight->c1;
-            $this->c2= $fight->c2;
+        if ($fight != null) {
+            $this->id = $fight->id;
+            $this->short_id = $fight->short_id;
+            $this->fighters_group_id = $fight->fighters_group_id;
+            $this->c1 = $fight->c1;
+            $this->c2 = $fight->c2;
         }
     }
     /**

src/Models/PreliminaryFight.php

Spacing +6 added lines, -6 removed lines

@@ -9,12 +9,12 @@
 
     public function __construct(Fight $fight = null)
     {
-        if ($fight!=null){
-            $this->id= $fight->id;
-            $this->short_id= $fight->short_id;
-            $this->fighters_group_id= $fight->fighters_group_id;
-            $this->c1= $fight->c1;
-            $this->c2= $fight->c2;
+        if ($fight != null) {
+            $this->id = $fight->id;
+            $this->short_id = $fight->short_id;
+            $this->fighters_group_id = $fight->fighters_group_id;
+            $this->c1 = $fight->c1;
+            $this->c2 = $fight->c2;
         }
     }
 

src/models/FightersGroup.php

Spacing +1 added lines, -1 removed lines

@@ -180,7 +180,7 @@
     public function hasDeterminedParent()
     {
         // There is more than 1 fight, should be Preliminary
-        if (sizeof($this->fighters()) > 1){
+        if (sizeof($this->fighters()) > 1) {
             return false;
         }
         foreach ($this->children as $child) {

Braces +3 added lines, -1 removed lines

@@ -184,7 +184,9 @@
             return false;
         }
         foreach ($this->children as $child) {
-            if (sizeof($child->fighters()) > 1) return false;
+            if (sizeof($child->fighters()) > 1) {
+                return false;
+            }
         }
         return true;
 

src/TreeGen/DirectEliminationTreeGen.php

Unused Use Statements -1 removed lines

@@ -6,7 +6,6 @@
 use Illuminate\Support\Facades\App;
 use Xoco70\KendoTournaments\Models\Championship;
 use Xoco70\KendoTournaments\Models\DirectEliminationFight;
-use Xoco70\KendoTournaments\Models\Fight;
 
 abstract class DirectEliminationTreeGen extends TreeGen
 {

src/TreeGen/CreateDirectEliminationTree.php

Spacing +1 added lines, -1 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);

src/models/Fight.php

Braces +12 added lines, -4 removed lines

@@ -44,7 +44,9 @@ 
      */
     protected static function getFightersWithByes(FightersGroup $group)
     {
-        if ($group == null) return null;
+        if ($group == null) {
+            return null;
+        }
         $fighters = $group->getFightersWithBye();
         $fighterType = $group->getFighterType();
         if (sizeof($fighters) == 0) {
@@ -127,8 +129,12 @@ 
      */
     public function shouldBeInFightList()
     {
-        if ($this->belongsToFirstRound() && $this->dontHave2Fighters()) return false;
-        if ($this->has2Fighters()) return true;
+        if ($this->belongsToFirstRound() && $this->dontHave2Fighters()) {
+            return false;
+        }
+        if ($this->has2Fighters()) {
+            return true;
+        }
         // We aint in the first round, and there is 1 or 0 competitor
         // We check children, and see :
         // if there is 2  - 2 fighters -> undetermine, we cannot add it to fight list
@@ -157,7 +163,9 @@ 
     private function belongsToFirstRound()
     {
         $firstRoundFights = $this->group->championship->firstRoundFights->pluck('id')->toArray();
-        if (in_array($this->id, $firstRoundFights)) return true;
+        if (in_array($this->id, $firstRoundFights)) {
+            return true;
+        }
         return false;
     }
 

database/factories/ChampionshipFactory.php

Spacing +1 added lines, -1 removed lines

@@ -5,7 +5,7 @@
 use Xoco70\KendoTournaments\Models\Championship;
 use Xoco70\KendoTournaments\Models\Tournament;
 
-$factory->define(Championship::class, function (Faker\Generator $faker) {
+$factory->define(Championship::class, function(Faker\Generator $faker) {
     $tournaments = Tournament::all()->pluck('id')->toArray();
     $categories = Cat::all()->pluck('id')->toArray();
 

database/factories/CategoryFactory.php

Spacing +1 added lines, -1 removed lines

@@ -3,7 +3,7 @@
 
 use Xoco70\KendoTournaments\Models\Category as Cat;
 
-$factory->define(Cat::class, function (Faker\Generator $faker) {
+$factory->define(Cat::class, function(Faker\Generator $faker) {
     $name = ['categories.junior',
         'categories.junior_team',
         'categories.men_single',

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');
             $router->put('/championships/{championship}/trees', 'Xoco70\KendoTournaments\TreeController@update')->name('tree.update');