xoco70 / laravel-tournaments

src/TreeGen/SingleEliminationCompetitorTreeGen.php

Unused Use Statements -2 removed lines

@@ -3,8 +3,6 @@
 namespace Xoco70\LaravelTournaments\TreeGen;
 
 use Illuminate\Support\Collection;
-use Xoco70\LaravelTournaments\Exceptions\DuplicatedException;
-use Xoco70\LaravelTournaments\Exceptions\DuplicatedFighterException;
 use Xoco70\LaravelTournaments\Models\Competitor;
 use Xoco70\LaravelTournaments\Models\Fight;
 use Xoco70\LaravelTournaments\Models\FighterGroupCompetitor;

Spacing +2 added lines, -2 removed lines

@@ -83,8 +83,8 @@
     protected function updateGroupFighters(FightersGroup $group, Fight $fight, $oldCompetitors)
     {
 
-        for ($num = 0; $num <=1;$num++){
-            $c1or2 = "c".($num+1);
+        for ($num = 0; $num <= 1; $num++) {
+            $c1or2 = "c" . ($num + 1);
             $fgc = FighterGroupCompetitor::where('fighters_group_id', $group->id)
                 ->where('competitor_id', $oldCompetitors[$num])
                 ->first();

src/TreeGen/TreeGen.php

Doc Comments +14 added lines, -8 removed lines

@@ -34,8 +34,14 @@ 
 
     abstract protected function generateGroupsForRound(Collection $fightersByArea, $round);
 
+    /**
+     * @return Collection
+     */
     abstract protected function getByeGroup($fighters);
 
+    /**
+     * @param integer $fighterId
+     */
     abstract protected function getFighter($fighterId);
 
     abstract protected function getFighters();
@@ -114,7 +120,7 @@ 
     /**
      * Get the biggest entity group.
      *
-     * @param $userGroups
+     * @param Collection $userGroups
      *
      * @return int
      */
@@ -182,7 +188,7 @@ 
     /**
      * Repart BYE in the tree,.
      *
-     * @param $fighterGroups
+     * @param Collection $fighterGroups
      * @param int $max
      *
      * @return Collection
@@ -342,7 +348,7 @@ 
     /**
      * Attach a parent to every child for nestedSet Navigation.
      *
-     * @param $numFightersElim
+     * @param integer $numFightersElim
      */
     protected function addParentToChildren($numFightersElim)
     {
@@ -366,8 +372,8 @@ 
 
     /**
      * @param Collection $fighters
-     * @param $frequency
-     * @param $sizeGroupBy
+     * @param integer $frequency
+     * @param integer $sizeGroupBy
      * @param $bye
      *
      * @return Collection
@@ -392,8 +398,8 @@ 
     /**
      * @param $frequency
      * @param $sizeGroupBy
-     * @param $count
-     * @param $byeCount
+     * @param integer $count
+     * @param integer $byeCount
      *
      * @return bool
      */
@@ -511,7 +517,7 @@ 
 
     /**
      * @param $fighters
-     * @param $numFighter
+     * @param integer $numFighter
      * @param $fight
      * @param $winners
      * @return int

Unused Use Statements -1 removed lines

@@ -7,7 +7,6 @@
 use Illuminate\Http\Request;
 use Illuminate\Support\Facades\DB;
 use Xoco70\LaravelTournaments\Contracts\TreeGenerable;
-use Xoco70\LaravelTournaments\Exceptions\DuplicatedException;
 use Xoco70\LaravelTournaments\Exceptions\DuplicatedFighterException;
 use Xoco70\LaravelTournaments\Exceptions\TreeGenerationException;
 use Xoco70\LaravelTournaments\Models\Championship;

src/TreeGen/SingleEliminationTeamTreeGen.php

Spacing +2 added lines, -2 removed lines

@@ -82,8 +82,8 @@
     protected function updateGroupFighters(FightersGroup $group, Fight $fight, $oldTeams)
     {
 
-        for ($num = 0; $num <=1;$num++){
-            $c1or2 = "c".($num+1);
+        for ($num = 0; $num <= 1; $num++) {
+            $c1or2 = "c" . ($num + 1);
             $fgc = FighterGroupTeam::where('fighters_group_id', $group->id)
                 ->where('team_id', $oldTeams[$num])
                 ->first();

resources/views/partials/tree/brackets/playerList.blade.php

Spacing +2 added lines, -2 removed lines

@@ -1,9 +1,9 @@
 <?php
 $prefix = "singleElimination";
-if ($championship->hasPreliminary() && $roundNumber ==1){
+if ($championship->hasPreliminary() && $roundNumber == 1) {
     $prefix = "preliminary";
 }
-$className = $prefix."_select";
+$className = $prefix . "_select";
 
 ?>
 <!-- r = round, m = match, f = fighter -->