Passed
Push — master ( f2e9a4...5d9f44 )
by Tomáš
01:51
created
src/TournamentGenerator/TeamFilter.php 2 patches
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -43,11 +43,11 @@
 block discarded – undo
43 43
 
44 44
 	private $groups = [];
45 45
 
46
-	function __construct(string $what = 'points', string $how = '>', $val = 0, $groups = []){
46
+	function __construct(string $what = 'points', string $how = '>', $val = 0, $groups = []) {
47 47
 		if (in_array(strtolower($what), ['points', 'score', 'wins', 'draws', 'losses', 'second', 'third', 'team', 'notprogressed', 'progressed'])) $this->what = strtolower($what);
48 48
 		if (in_array($how, ['>', '<', '>=', '<=', '=', '!='])) $this->how = $how;
49 49
 		if ((gettype($val) === 'integer' && strtolower($what) !== 'team') || ($val instanceof Team && strtolower($what) === 'team')) $this->val = $val;
50
-		$this->groups = array_map(function($a) { return $a->id; }, array_filter($groups, function($a) {return ($a instanceof Group);}));
50
+		$this->groups = array_map(function($a) { return $a->id; }, array_filter($groups, function($a) {return ($a instanceof Group); }));
51 51
 	}
52 52
 	public function __toString() {
53 53
 		return 'Filter: '.$this->what.' '.($this->what !== 'notprogressed' && $this->what !== 'progressed' ? $this->how.' '.$this->val : '');
Please login to merge, or discard this patch.
Braces   +37 added lines, -13 removed lines patch added patch discarded remove patch
@@ -44,9 +44,15 @@  discard block
 block discarded – undo
44 44
 	private $groups = [];
45 45
 
46 46
 	function __construct(string $what = 'points', string $how = '>', $val = 0, $groups = []){
47
-		if (in_array(strtolower($what), ['points', 'score', 'wins', 'draws', 'losses', 'second', 'third', 'team', 'notprogressed', 'progressed'])) $this->what = strtolower($what);
48
-		if (in_array($how, ['>', '<', '>=', '<=', '=', '!='])) $this->how = $how;
49
-		if ((gettype($val) === 'integer' && strtolower($what) !== 'team') || ($val instanceof Team && strtolower($what) === 'team')) $this->val = $val;
47
+		if (in_array(strtolower($what), ['points', 'score', 'wins', 'draws', 'losses', 'second', 'third', 'team', 'notprogressed', 'progressed'])) {
48
+			$this->what = strtolower($what);
49
+		}
50
+		if (in_array($how, ['>', '<', '>=', '<=', '=', '!='])) {
51
+			$this->how = $how;
52
+		}
53
+		if ((gettype($val) === 'integer' && strtolower($what) !== 'team') || ($val instanceof Team && strtolower($what) === 'team')) {
54
+			$this->val = $val;
55
+		}
50 56
 		$this->groups = array_map(function($a) { return $a->id; }, array_filter($groups, function($a) {return ($a instanceof Group);}));
51 57
 	}
52 58
 	public function __toString() {
@@ -54,11 +60,17 @@  discard block
 block discarded – undo
54 60
 	}
55 61
 
56 62
 	public function validate(Team $team, $groupsId, string $operation = 'sum', Group $from = null) {
57
-		if (count($this->groups) > 0) $groupsId = array_unique(array_merge($this->groups, (gettype($groupsId) === 'array' ? $groupsId : [$groupsId])), SORT_REGULAR);
63
+		if (count($this->groups) > 0) {
64
+			$groupsId = array_unique(array_merge($this->groups, (gettype($groupsId) === 'array' ? $groupsId : [$groupsId])), SORT_REGULAR);
65
+		}
58 66
 
59
-		if ($this->what == 'team') return ($this->how === '!=' ? !$this->validateTeam($team) : $this->validateTeam($team));
60
-		elseif ($this->what == 'notprogressed') return !$this->validateProgressed($team, $from);
61
-		elseif ($this->what == 'progressed') return $this->validateProgressed($team, $from);
67
+		if ($this->what == 'team') {
68
+			return ($this->how === '!=' ? !$this->validateTeam($team) : $this->validateTeam($team));
69
+		} elseif ($this->what == 'notprogressed') {
70
+			return !$this->validateProgressed($team, $from);
71
+		} elseif ($this->what == 'progressed') {
72
+			return $this->validateProgressed($team, $from);
73
+		}
62 74
 
63 75
 		return $this->validateCalc($team, $groupsId, $operation);
64 76
 	}
@@ -67,13 +79,19 @@  discard block
 block discarded – undo
67 79
 		return $this->val === $team;
68 80
 	}
69 81
 	private function validateProgressed(Team $team, Group $from = null) {
70
-		if ($from === null) throw new \Exception('Group $from was not defined.');
82
+		if ($from === null) {
83
+			throw new \Exception('Group $from was not defined.');
84
+		}
71 85
 		return $from->isProgressed($team);
72 86
 	}
73 87
 	private function validateCalc(Team $team, $groupsId, string $operation = 'sum') {
74
-		if (gettype($groupsId) === 'array' && !in_array(strtolower($operation), ['sum', 'avg', 'max', 'min'])) throw new \Exception('Unknown operation of '.$operation.'. Only "sum", "avg", "min", "max" possible.');
88
+		if (gettype($groupsId) === 'array' && !in_array(strtolower($operation), ['sum', 'avg', 'max', 'min'])) {
89
+			throw new \Exception('Unknown operation of '.$operation.'. Only "sum", "avg", "min", "max" possible.');
90
+		}
75 91
 		$comp = 0;
76
-		if (gettype($groupsId) === 'string') $groupsId = [$groupsId];
92
+		if (gettype($groupsId) === 'string') {
93
+			$groupsId = [$groupsId];
94
+		}
77 95
 		switch (strtolower($operation)) {
78 96
 			case 'sum': $comp = $this->calcSum($team, $groupsId); break;
79 97
 			case 'avg': $comp = $this->calcSum($team, $groupsId)/count($groupsId); break;
@@ -95,21 +113,27 @@  discard block
 block discarded – undo
95 113
 	private function calcSum(Team $team, $groupsId) {
96 114
 		$sum = 0;
97 115
 		foreach ($groupsId as $id) {
98
-			if (isset($team->groupResults[$id])) $sum += $team->groupResults[$id][$this->what];
116
+			if (isset($team->groupResults[$id])) {
117
+				$sum += $team->groupResults[$id][$this->what];
118
+			}
99 119
 		}
100 120
 		return $sum;
101 121
 	}
102 122
 	private function calcMax(Team $team, $groupsId) {
103 123
 		$max = null;
104 124
 		foreach ($groupsId as $id) {
105
-			if (isset($team->groupResults[$id]) && ($team->groupResults[$id][$this->what] > $max || $max === null)) $max = $team->groupResults[$id][$this->what];
125
+			if (isset($team->groupResults[$id]) && ($team->groupResults[$id][$this->what] > $max || $max === null)) {
126
+				$max = $team->groupResults[$id][$this->what];
127
+			}
106 128
 		}
107 129
 		return $max;
108 130
 	}
109 131
 	private function calcMin(Team $team, $groupsId) {
110 132
 		$min = null;
111 133
 		foreach ($groupsId as $id) {
112
-			if (isset($team->groupResults[$id]) && ($team->groupResults[$id][$this->what] < $min || $min === null)) $min = $team->groupResults[$id][$this->what];
134
+			if (isset($team->groupResults[$id]) && ($team->groupResults[$id][$this->what] < $min || $min === null)) {
135
+				$min = $team->groupResults[$id][$this->what];
136
+			}
113 137
 		}
114 138
 		return $min;
115 139
 	}
Please login to merge, or discard this patch.
src/TournamentGenerator/Progression.php 1 patch
Braces   +18 added lines, -7 removed lines patch added patch discarded remove patch
@@ -27,18 +27,26 @@  discard block
 block discarded – undo
27 27
 
28 28
 	public function addFilter(...$filters) {
29 29
 		foreach ($filters as $filter) {
30
-			if (!$filter instanceof TeamFilter) throw new \Exception('Trying to add filter which is not an instance of TeamFilter.');
30
+			if (!$filter instanceof TeamFilter) {
31
+				throw new \Exception('Trying to add filter which is not an instance of TeamFilter.');
32
+			}
31 33
 			$this->filters[] = $filter;
32 34
 		}
33 35
 		return $this;
34 36
 	}
35 37
 
36 38
 	public function progress(bool $blank = false) {
37
-		if ($blank) $teams = $this->from->isPlayed() ? $this->from->sortTeams($this->filters) : $this->from->simulate($this->filters);
38
-		else $teams = $this->from->sortTeams($this->filters);
39
+		if ($blank) {
40
+			$teams = $this->from->isPlayed() ? $this->from->sortTeams($this->filters) : $this->from->simulate($this->filters);
41
+		} else {
42
+			$teams = $this->from->sortTeams($this->filters);
43
+		}
39 44
 
40
-		if (count($this->filters) === 0 || $this->len !== null || $this->start !== 0) $next = array_splice($teams, $this->start, ($this->len === null ? count($teams) : $this->len));
41
-		else $next = $teams;
45
+		if (count($this->filters) === 0 || $this->len !== null || $this->start !== 0) {
46
+			$next = array_splice($teams, $this->start, ($this->len === null ? count($teams) : $this->len));
47
+		} else {
48
+			$next = $teams;
49
+		}
42 50
 
43 51
 		$i = 1;
44 52
 
@@ -46,12 +54,15 @@  discard block
 block discarded – undo
46 54
 			if ($blank) {
47 55
 				$this->to->addTeam(new BlankTeam($this.' - '.$i, $team));
48 56
 				$i++;
57
+			} else {
58
+				$team->sumPoints += $this->from->progressPoints;
49 59
 			}
50
-			else $team->sumPoints += $this->from->progressPoints;
51 60
 		}
52 61
 
53 62
 		$this->from->addProgressed($next);
54
-		if (!$blank) $this->to->addTeam($next);
63
+		if (!$blank) {
64
+			$this->to->addTeam($next);
65
+		}
55 66
 		return $this;
56 67
 	}
57 68
 
Please login to merge, or discard this patch.
src/TournamentGenerator/Utilis/Simulator.php 1 patch
Braces   +12 added lines, -4 removed lines patch added patch discarded remove patch
@@ -18,7 +18,9 @@  discard block
 block discarded – undo
18 18
 			$game->setResults($results);
19 19
 		}
20 20
 		$return = $group->sortTeams($filters);
21
-		if (!$reset) return $return;
21
+		if (!$reset) {
22
+			return $return;
23
+		}
22 24
 		foreach ($group->getGames() as $game) {
23 25
 			$game->resetResults();
24 26
 		}
@@ -27,14 +29,18 @@  discard block
 block discarded – undo
27 29
 
28 30
 	public static function simulateRound(\TournamentGenerator\Round $round) {
29 31
 		foreach ($round->getGroups() as $group) {
30
-			if ($group->isPlayed()) continue;
32
+			if ($group->isPlayed()) {
33
+				continue;
34
+			}
31 35
 			$group->simulate([], false);
32 36
 		}
33 37
 		return true;
34 38
 	}
35 39
 
36 40
 	public static function simulateTournament(\TournamentGenerator\Tournament $tournament) {
37
-		if (count($tournament->getCategories()) === 0 && count($tournament->getRounds()) === 0) throw new \Exception('There are no rounds or categories to simulate games from.');
41
+		if (count($tournament->getCategories()) === 0 && count($tournament->getRounds()) === 0) {
42
+			throw new \Exception('There are no rounds or categories to simulate games from.');
43
+		}
38 44
 
39 45
 		$games = [];
40 46
 
@@ -55,7 +61,9 @@  discard block
 block discarded – undo
55 61
 
56 62
 	public static function simulateTournamentReal(\TournamentGenerator\Tournament $tournament) {
57 63
 		$games = [];
58
-		if (count($tournament->getCategories()) === 0 && count($tournament->getRounds()) === 0) throw new \Exception('There are no rounds or categories to simulate games from.');
64
+		if (count($tournament->getCategories()) === 0 && count($tournament->getRounds()) === 0) {
65
+			throw new \Exception('There are no rounds or categories to simulate games from.');
66
+		}
59 67
 
60 68
 		foreach ($tournament->getCategories() as $category) {
61 69
 			$games = array_merge($games, $category->genGamesSimulate());
Please login to merge, or discard this patch.
src/TournamentGenerator/Utilis/Sorter/Teams.php 2 patches
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -18,13 +18,13 @@  discard block
 block discarded – undo
18 18
 					if ($a->groupResults[$group->id]["points"] === $b->groupResults[$group->id]["points"]) return ($a->groupResults[$group->id]["score"] > $b->groupResults[$group->id]["score"] ? -1 : 1);
19 19
 					return ($a->groupResults[$group->id]["points"] > $b->groupResults[$group->id]["points"] ? -1 : 1);
20 20
 				});
21
-				break;}
21
+				break; }
22 22
 			case \TournamentGenerator\Constants::SCORE:{
23 23
 				uasort($teams, function($a, $b) use ($group) {
24 24
 					if ($a->groupResults[$group->id]["score"] === $b->groupResults[$group->id]["score"]) return 0;
25 25
 					return ($a->groupResults[$group->id]["score"] > $b->groupResults[$group->id]["score"] ? -1 : 1);
26 26
 				});
27
-				break;}
27
+				break; }
28 28
 		}
29 29
 
30 30
 		return $teams;
@@ -41,13 +41,13 @@  discard block
 block discarded – undo
41 41
 					if ($a->sumPoints($groupsIds) === $b->sumPoints($groupsIds)) return ($a->sumScore($groupsIds) > $b->sumScore($groupsIds) ? -1 : 1);
42 42
 					return ($a->sumPoints($groupsIds) > $b->sumPoints($groupsIds) ? -1 : 1);
43 43
 				});
44
-				break;}
44
+				break; }
45 45
 			case \TournamentGenerator\Constants::SCORE:{
46 46
 				uasort($teams, function($a, $b) use ($groupsIds) {
47 47
 					if ($a->sumScore($groupsIds) === $b->sumScore($groupsIds)) return 0;
48 48
 					return ($a->sumScore($groupsIds) > $b->sumScore($groupsIds) ? -1 : 1);
49 49
 				});
50
-				break;}
50
+				break; }
51 51
 		}
52 52
 
53 53
 		return $teams;
Please login to merge, or discard this patch.
Braces   +24 added lines, -8 removed lines patch added patch discarded remove patch
@@ -9,19 +9,27 @@  discard block
 block discarded – undo
9 9
 {
10 10
 
11 11
 	public static function sortGroup(array &$teams, \TournamentGenerator\Group $group, string $ordering = \TournamentGenerator\Constants::POINTS) {
12
-		if (!in_array($ordering, \TournamentGenerator\Constants::OrderingTypes)) throw new \Exception('Unknown ordering type `'.$ordering.'`');
12
+		if (!in_array($ordering, \TournamentGenerator\Constants::OrderingTypes)) {
13
+			throw new \Exception('Unknown ordering type `'.$ordering.'`');
14
+		}
13 15
 
14 16
 		switch ($ordering) {
15 17
 			case \TournamentGenerator\Constants::POINTS:{
16 18
 				uasort($teams, function($a, $b) use ($group) {
17
-					if ($a->groupResults[$group->id]["points"] === $b->groupResults[$group->id]["points"] && $a->groupResults[$group->id]["score"] === $b->groupResults[$group->id]["score"]) return 0;
18
-					if ($a->groupResults[$group->id]["points"] === $b->groupResults[$group->id]["points"]) return ($a->groupResults[$group->id]["score"] > $b->groupResults[$group->id]["score"] ? -1 : 1);
19
+					if ($a->groupResults[$group->id]["points"] === $b->groupResults[$group->id]["points"] && $a->groupResults[$group->id]["score"] === $b->groupResults[$group->id]["score"]) {
20
+						return 0;
21
+					}
22
+					if ($a->groupResults[$group->id]["points"] === $b->groupResults[$group->id]["points"]) {
23
+						return ($a->groupResults[$group->id]["score"] > $b->groupResults[$group->id]["score"] ? -1 : 1);
24
+					}
19 25
 					return ($a->groupResults[$group->id]["points"] > $b->groupResults[$group->id]["points"] ? -1 : 1);
20 26
 				});
21 27
 				break;}
22 28
 			case \TournamentGenerator\Constants::SCORE:{
23 29
 				uasort($teams, function($a, $b) use ($group) {
24
-					if ($a->groupResults[$group->id]["score"] === $b->groupResults[$group->id]["score"]) return 0;
30
+					if ($a->groupResults[$group->id]["score"] === $b->groupResults[$group->id]["score"]) {
31
+						return 0;
32
+					}
25 33
 					return ($a->groupResults[$group->id]["score"] > $b->groupResults[$group->id]["score"] ? -1 : 1);
26 34
 				});
27 35
 				break;}
@@ -30,21 +38,29 @@  discard block
 block discarded – undo
30 38
 		return $teams;
31 39
 	}
32 40
 	public static function sortRound(array &$teams, \TournamentGenerator\Round $round, string $ordering = \TournamentGenerator\Constants::POINTS) {
33
-		if (!in_array($ordering, \TournamentGenerator\Constants::OrderingTypes)) throw new \Exception('Unknown ordering type `'.$ordering.'`');
41
+		if (!in_array($ordering, \TournamentGenerator\Constants::OrderingTypes)) {
42
+			throw new \Exception('Unknown ordering type `'.$ordering.'`');
43
+		}
34 44
 
35 45
 		$groupsIds = $round->getGroupsIds();
36 46
 
37 47
 		switch ($ordering) {
38 48
 			case \TournamentGenerator\Constants::POINTS:{
39 49
 				uasort($teams, function($a, $b) use ($groupsIds) {
40
-					if ($a->sumPoints($groupsIds) === $b->sumPoints($groupsIds) && $a->sumScore($groupsIds) === $b->sumScore($groupsIds)) return 0;
41
-					if ($a->sumPoints($groupsIds) === $b->sumPoints($groupsIds)) return ($a->sumScore($groupsIds) > $b->sumScore($groupsIds) ? -1 : 1);
50
+					if ($a->sumPoints($groupsIds) === $b->sumPoints($groupsIds) && $a->sumScore($groupsIds) === $b->sumScore($groupsIds)) {
51
+						return 0;
52
+					}
53
+					if ($a->sumPoints($groupsIds) === $b->sumPoints($groupsIds)) {
54
+						return ($a->sumScore($groupsIds) > $b->sumScore($groupsIds) ? -1 : 1);
55
+					}
42 56
 					return ($a->sumPoints($groupsIds) > $b->sumPoints($groupsIds) ? -1 : 1);
43 57
 				});
44 58
 				break;}
45 59
 			case \TournamentGenerator\Constants::SCORE:{
46 60
 				uasort($teams, function($a, $b) use ($groupsIds) {
47
-					if ($a->sumScore($groupsIds) === $b->sumScore($groupsIds)) return 0;
61
+					if ($a->sumScore($groupsIds) === $b->sumScore($groupsIds)) {
62
+						return 0;
63
+					}
48 64
 					return ($a->sumScore($groupsIds) > $b->sumScore($groupsIds) ? -1 : 1);
49 65
 				});
50 66
 				break;}
Please login to merge, or discard this patch.
src/TournamentGenerator/Utilis/Sorter/Games.php 2 patches
Spacing   +10 added lines, -10 removed lines patch added patch discarded remove patch
@@ -69,7 +69,7 @@  discard block
 block discarded – undo
69 69
 
70 70
 			// CYCLE 6
71 71
 			// FIRST AVAILABLE GAME
72
-			$this->moveCalculatedGames(array_shift($games),$teams);
72
+			$this->moveCalculatedGames(array_shift($games), $teams);
73 73
 		}
74 74
 
75 75
 		return $this->games;
@@ -79,8 +79,8 @@  discard block
 block discarded – undo
79 79
 	private function cycle1(array &$games, array &$teams) {
80 80
 		$found = false;
81 81
 		foreach ($games as $key => $game) {
82
-			if ($this->orderCheckTeamsVal($game, $teams, [4,5,6,7])) {
83
-				$this->moveCalculatedGames($game,$teams);
82
+			if ($this->orderCheckTeamsVal($game, $teams, [4, 5, 6, 7])) {
83
+				$this->moveCalculatedGames($game, $teams);
84 84
 				unset($games[$key]);
85 85
 				$found = true;
86 86
 				break;
@@ -92,8 +92,8 @@  discard block
 block discarded – undo
92 92
 	private function cycle2(array &$games, array &$teams) {
93 93
 		$found = false;
94 94
 		foreach ($games as $key => $game) {
95
-			if ($this->orderCheckTeamsVal($game, $teams, [6,7])) {
96
-				$this->moveCalculatedGames($game,$teams);
95
+			if ($this->orderCheckTeamsVal($game, $teams, [6, 7])) {
96
+				$this->moveCalculatedGames($game, $teams);
97 97
 				unset($games[$key]);
98 98
 				$found = true;
99 99
 				break;
@@ -106,8 +106,8 @@  discard block
 block discarded – undo
106 106
 	private function cycle3(array &$games, array &$teams) {
107 107
 		$found = false;
108 108
 		foreach ($games as $key => $game) {
109
-			if ($this->orderCheckTeamsVal($game, $teams, [7], [1,2,3])) {
110
-				$this->moveCalculatedGames($game,$teams);
109
+			if ($this->orderCheckTeamsVal($game, $teams, [7], [1, 2, 3])) {
110
+				$this->moveCalculatedGames($game, $teams);
111 111
 				unset($games[$key]);
112 112
 				$found = true;
113 113
 				break;
@@ -120,7 +120,7 @@  discard block
 block discarded – undo
120 120
 		$found = false;
121 121
 		foreach ($games as $key => $game) {
122 122
 			if ($this->orderCheckTeamsVal($game, $teams, [7])) {
123
-				$this->moveCalculatedGames($game,$teams);
123
+				$this->moveCalculatedGames($game, $teams);
124 124
 				unset($games[$key]);
125 125
 				$found = true;
126 126
 				break;
@@ -132,8 +132,8 @@  discard block
 block discarded – undo
132 132
 	private function cycle5(array &$games, array &$teams) {
133 133
 		$found = false;
134 134
 		foreach ($games as $key => $game) {
135
-			if ($this->orderCheckTeamsVal($game, $teams, [], [1,2,3])) {
136
-				$this->moveCalculatedGames($game,$teams);
135
+			if ($this->orderCheckTeamsVal($game, $teams, [], [1, 2, 3])) {
136
+				$this->moveCalculatedGames($game, $teams);
137 137
 				unset($games[$key]);
138 138
 				$found = true;
139 139
 				break;
Please login to merge, or discard this patch.
Braces   +27 added lines, -9 removed lines patch added patch discarded remove patch
@@ -36,7 +36,9 @@  discard block
 block discarded – undo
36 36
 
37 37
 		$games = $this->group->getGames();
38 38
 
39
-		if (count($games) <= 4) return $games;
39
+		if (count($games) <= 4) {
40
+			return $games;
41
+		}
40 42
 
41 43
 		$this->games = [];
42 44
 
@@ -48,24 +50,34 @@  discard block
 block discarded – undo
48 50
 		while (count($games) > 0) {
49 51
 			// CYCLE 1
50 52
 			// TEAM WHICH DIDN'T PLAY IN LAST GAME (< 4)
51
-			if ($this->cycle1($games, $teams)) continue;
53
+			if ($this->cycle1($games, $teams)) {
54
+				continue;
55
+			}
52 56
 
53 57
 			// CYCLE 2
54 58
 			// NOT TEAM WHICH PLAYED IN LAST TWO GAMES (NOT 6 or 7)
55
-			if ($this->cycle2($games, $teams)) continue;
59
+			if ($this->cycle2($games, $teams)) {
60
+				continue;
61
+			}
56 62
 
57 63
 			// CYCLE 3
58 64
 			// NOT TEAM WHICH PLAYED IN LAST THREE GAMES (NOT 7)
59 65
 			// TEAMS THAT DIDN'T PLAY IN LAST GAME WILL PLAY THIS GAME (< 4)
60
-			if ($this->cycle3($games, $teams)) continue;
66
+			if ($this->cycle3($games, $teams)) {
67
+				continue;
68
+			}
61 69
 
62 70
 			// CYCLE 4
63 71
 			// NOT TEAM WHICH PLAYED IN LAST THREE GAMES (NOT 7)
64
-			if ($this->cycle4($games, $teams)) continue;
72
+			if ($this->cycle4($games, $teams)) {
73
+				continue;
74
+			}
65 75
 
66 76
 			// CYCLE 5
67 77
 			// TEAMS THAT DIDN'T PLAY IN LAST GAME WILL PLAY THIS GAME (< 4)
68
-			if ($this->cycle5($games, $teams)) continue;
78
+			if ($this->cycle5($games, $teams)) {
79
+				continue;
80
+			}
69 81
 
70 82
 			// CYCLE 6
71 83
 			// FIRST AVAILABLE GAME
@@ -174,11 +186,17 @@  discard block
 block discarded – undo
174 186
 		$requiredTeams = array_filter($teams, function($a) use ($required) { return in_array($a, $required); });
175 187
 
176 188
 		foreach ($game->getTeamsIds() as $tid) {
177
-			if (in_array($teams[$tid], $checkVals)) return false;
178
-			if (isset($requiredTeams[$tid])) unset($requiredTeams[$tid]);
189
+			if (in_array($teams[$tid], $checkVals)) {
190
+				return false;
191
+			}
192
+			if (isset($requiredTeams[$tid])) {
193
+				unset($requiredTeams[$tid]);
194
+			}
179 195
 		}
180 196
 
181
-		if (count($requiredTeams) > 0) return false;
197
+		if (count($requiredTeams) > 0) {
198
+			return false;
199
+		}
182 200
 
183 201
 		return true;
184 202
 
Please login to merge, or discard this patch.
src/TournamentGenerator/Utilis/Generator.php 2 patches
Spacing   +8 added lines, -8 removed lines patch added patch discarded remove patch
@@ -18,11 +18,11 @@  discard block
 block discarded – undo
18 18
 		$this->group = $group;
19 19
 	}
20 20
 
21
-	public function allowSkip(){
21
+	public function allowSkip() {
22 22
 		$this->allowSkip = true;
23 23
 		return $this;
24 24
 	}
25
-	public function disallowSkip(){
25
+	public function disallowSkip() {
26 26
 		$this->allowSkip = false;
27 27
 		return $this;
28 28
 	}
@@ -45,7 +45,7 @@  discard block
 block discarded – undo
45 45
 	}
46 46
 
47 47
 	public function setInGame(int $inGame) {
48
-		if ($inGame < 2 ||  $inGame > 4) throw new \Exception('Expected 2,3 or 4 as inGame '.$inGame.' given');
48
+		if ($inGame < 2 || $inGame > 4) throw new \Exception('Expected 2,3 or 4 as inGame '.$inGame.' given');
49 49
 		$this->inGame = $inGame;
50 50
 		return $this;
51 51
 	}
@@ -120,11 +120,11 @@  discard block
 block discarded – undo
120 120
 		$discard = [];
121 121
 		shuffle($teams);
122 122
 		$count = count($teams);
123
-		while (count($teams) % $this->inGame !== 0) { $discard[] = array_shift($teams); }
123
+		while (count($teams)%$this->inGame !== 0) { $discard[] = array_shift($teams); }
124 124
 
125 125
 		while (count($teams) > 0) {
126 126
 			$tInGame = [];
127
-			for ($i=0; $i < $this->inGame; $i++) { $tInGame[] = array_shift($teams); }
127
+			for ($i = 0; $i < $this->inGame; $i++) { $tInGame[] = array_shift($teams); }
128 128
 			$this->group->game($tInGame);
129 129
 		}
130 130
 
@@ -165,11 +165,11 @@  discard block
 block discarded – undo
165 165
 	public static function circle_genGames2(array $teams = [], \tournamentGenerator\Group $group) {
166 166
 		$bracket = []; // ARRAY OF GAMES
167 167
 
168
-		if (count($teams) % 2 != 0) $teams[] = \TournamentGenerator\Constants::DUMMY_TEAM; // IF NOT EVEN NUMBER OF TEAMS, ADD DUMMY
168
+		if (count($teams)%2 != 0) $teams[] = \TournamentGenerator\Constants::DUMMY_TEAM; // IF NOT EVEN NUMBER OF TEAMS, ADD DUMMY
169 169
 
170 170
 		shuffle($teams); // SHUFFLE TEAMS FOR MORE RANDOMNESS
171 171
 
172
-		for ($i=0; $i < count($teams)-1; $i++) {
172
+		for ($i = 0; $i < count($teams)-1; $i++) {
173 173
 			$bracket = array_merge($bracket, Generator::circle_saveBracket($teams, $group)); // SAVE CURRENT ROUND
174 174
 
175 175
 			$teams = Generator::circle_rotateBracket($teams); // ROTATE TEAMS IN BRACKET
@@ -183,7 +183,7 @@  discard block
 block discarded – undo
183 183
 
184 184
 		$bracket = [];
185 185
 
186
-		for ($i=0; $i < count($teams)/2; $i++) { // GO THROUGH HALF OF THE TEAMS
186
+		for ($i = 0; $i < count($teams)/2; $i++) { // GO THROUGH HALF OF THE TEAMS
187 187
 
188 188
 			$home = $teams[$i];
189 189
 			$reverse = array_reverse($teams);
Please login to merge, or discard this patch.
Braces   +37 added lines, -12 removed lines patch added patch discarded remove patch
@@ -36,8 +36,11 @@  discard block
 block discarded – undo
36 36
 
37 37
 
38 38
 	public function setType(/** @scrutinizer ignore-all */ string $type = \TournamentGenerator\Constants::ROUND_ROBIN) {
39
-		if (in_array($type, \TournamentGenerator\Constants::GroupTypes)) $this->type = $type;
40
-		else throw new \Exception('Unknown group type: '.$type);
39
+		if (in_array($type, \TournamentGenerator\Constants::GroupTypes)) {
40
+			$this->type = $type;
41
+		} else {
42
+			throw new \Exception('Unknown group type: '.$type);
43
+		}
41 44
 		return $this;
42 45
 	}
43 46
 	public function getType() {
@@ -45,7 +48,9 @@  discard block
 block discarded – undo
45 48
 	}
46 49
 
47 50
 	public function setInGame(int $inGame) {
48
-		if ($inGame < 2 ||  $inGame > 4) throw new \Exception('Expected 2,3 or 4 as inGame '.$inGame.' given');
51
+		if ($inGame < 2 ||  $inGame > 4) {
52
+			throw new \Exception('Expected 2,3 or 4 as inGame '.$inGame.' given');
53
+		}
49 54
 		$this->inGame = $inGame;
50 55
 		return $this;
51 56
 	}
@@ -54,7 +59,9 @@  discard block
 block discarded – undo
54 59
 	}
55 60
 
56 61
 	public function setMaxSize(int $maxSize) {
57
-		if ($maxSize < 2) throw new \Exception('Max group size has to be at least 2, '.$maxSize.' given');
62
+		if ($maxSize < 2) {
63
+			throw new \Exception('Max group size has to be at least 2, '.$maxSize.' given');
64
+		}
58 65
 		$this->maxSize = $maxSize;
59 66
 		return $this;
60 67
 	}
@@ -78,7 +85,9 @@  discard block
 block discarded – undo
78 85
 	}
79 86
 	private function r_rGames(array $teams = []) {
80 87
 		$games = [];
81
-		if (count($teams) === 0) $teams = $this->group->getTeams();
88
+		if (count($teams) === 0) {
89
+			$teams = $this->group->getTeams();
90
+		}
82 91
 		switch ($this->inGame) {
83 92
 			case 2:
84 93
 				$games = Generator::circle_genGames2($teams, $this->group);
@@ -98,7 +107,9 @@  discard block
 block discarded – undo
98 107
 			$lockedTeam = array_shift($teamsB);
99 108
 			$gamesTemp = Generator::circle_genGames2($teamsB, $this->group);
100 109
 			foreach ($gamesTemp as $game) {
101
-				if (isset($lockedTeam1)) $game->addTeam($lockedTeam1);
110
+				if (isset($lockedTeam1)) {
111
+					$game->addTeam($lockedTeam1);
112
+				}
102 113
 				$game->addTeam($lockedTeam);
103 114
 			}
104 115
 			$games = array_merge($games, $gamesTemp);
@@ -116,7 +127,9 @@  discard block
 block discarded – undo
116 127
 		return $games;
117 128
 	}
118 129
 	private function two_twoGames(array $teams = []) {
119
-		if (count($teams) === 0) $teams = $this->group->getTeams();
130
+		if (count($teams) === 0) {
131
+			$teams = $this->group->getTeams();
132
+		}
120 133
 		$discard = [];
121 134
 		shuffle($teams);
122 135
 		$count = count($teams);
@@ -128,13 +141,17 @@  discard block
 block discarded – undo
128 141
 			$this->group->game($tInGame);
129 142
 		}
130 143
 
131
-		if (count($discard) > 0 && !$this->allowSkip) throw new \Exception('Couldn\'t make games with all teams. Expected k*'.$this->inGame.' teams '.$count.' teams given - discarting '.count($discard).' teams ('.implode(', ', $discard).') in group '.$this->group.' - allow skip '.($this->allowSkip ? 'True' : 'False'));
144
+		if (count($discard) > 0 && !$this->allowSkip) {
145
+			throw new \Exception('Couldn\'t make games with all teams. Expected k*'.$this->inGame.' teams '.$count.' teams given - discarting '.count($discard).' teams ('.implode(', ', $discard).') in group '.$this->group.' - allow skip '.($this->allowSkip ? 'True' : 'False'));
146
+		}
132 147
 
133 148
 		return $this;
134 149
 	}
135 150
 	private function cond_splitGames(array $teams = []) {
136 151
 		$games = [];
137
-		if (count($teams) === 0) $teams = $this->group->getTeams();
152
+		if (count($teams) === 0) {
153
+			$teams = $this->group->getTeams();
154
+		}
138 155
 
139 156
 		if (count($teams) > $this->maxSize) {
140 157
 			$groups = array_chunk($teams, /** @scrutinizer ignore-type */ ceil(count($teams)/ceil(count($teams)/$this->maxSize))); // SPLIT TEAMS INTO GROUP OF MAXIMUM SIZE OF $this->maxSize
@@ -144,7 +161,9 @@  discard block
 block discarded – undo
144 161
 			while ($g > 0) {
145 162
 				foreach ($games as $key => $group) {
146 163
 					$this->group->addGame(array_shift($games[$key]));
147
-					if (count($games[$key]) === 0) unset($games[$key]);
164
+					if (count($games[$key]) === 0) {
165
+						unset($games[$key]);
166
+					}
148 167
 					$g--;
149 168
 				}
150 169
 			}
@@ -165,7 +184,10 @@  discard block
 block discarded – undo
165 184
 	public static function circle_genGames2(array $teams = [], \tournamentGenerator\Group $group) {
166 185
 		$bracket = []; // ARRAY OF GAMES
167 186
 
168
-		if (count($teams) % 2 != 0) $teams[] = \TournamentGenerator\Constants::DUMMY_TEAM; // IF NOT EVEN NUMBER OF TEAMS, ADD DUMMY
187
+		if (count($teams) % 2 != 0) {
188
+			$teams[] = \TournamentGenerator\Constants::DUMMY_TEAM;
189
+		}
190
+		// IF NOT EVEN NUMBER OF TEAMS, ADD DUMMY
169 191
 
170 192
 		shuffle($teams); // SHUFFLE TEAMS FOR MORE RANDOMNESS
171 193
 
@@ -189,7 +211,10 @@  discard block
 block discarded – undo
189 211
 			$reverse = array_reverse($teams);
190 212
 			$away = $reverse[$i];
191 213
 
192
-			if (($home == \TournamentGenerator\Constants::DUMMY_TEAM || $away == \TournamentGenerator\Constants::DUMMY_TEAM)) continue; // SKIP WHEN DUMMY_TEAM IS PRESENT
214
+			if (($home == \TournamentGenerator\Constants::DUMMY_TEAM || $away == \TournamentGenerator\Constants::DUMMY_TEAM)) {
215
+				continue;
216
+			}
217
+			// SKIP WHEN DUMMY_TEAM IS PRESENT
193 218
 
194 219
 			$bracket[] = new \TournamentGenerator\Game([$home, $away], $group);
195 220
 
Please login to merge, or discard this patch.
src/TournamentGenerator/Round.php 2 patches
Braces   +20 added lines, -9 removed lines patch added patch discarded remove patch
@@ -25,8 +25,11 @@  discard block
 block discarded – undo
25 25
 
26 26
 	public function addGroup(Group ...$groups){
27 27
 		foreach ($groups as $group) {
28
-			if ($group instanceof Group) $this->groups[] = $group;
29
-			else throw new \Exception('Trying to add group which is not an instance of Group class.');
28
+			if ($group instanceof Group) {
29
+				$this->groups[] = $group;
30
+			} else {
31
+				throw new \Exception('Trying to add group which is not an instance of Group class.');
32
+			}
30 33
 		}
31 34
 		return $this;
32 35
 	}
@@ -77,7 +80,9 @@  discard block
 block discarded – undo
77 80
 	}
78 81
 	public function isPlayed(){
79 82
 		foreach ($this->groups as $group) {
80
-			if (!$group->isPlayed()) return false;
83
+			if (!$group->isPlayed()) {
84
+				return false;
85
+			}
81 86
 		}
82 87
 		return true;
83 88
 	}
@@ -86,10 +91,11 @@  discard block
 block discarded – undo
86 91
 		foreach ($teams as $team) {
87 92
 			if ($team instanceof Team)  {
88 93
 				$this->teams[] = $team;
89
-			}
90
-			elseif (gettype($team) === 'array') {
94
+			} elseif (gettype($team) === 'array') {
91 95
 				foreach ($team as $team2) {
92
-					if ($team2 instanceof Team) $this->teams[] = $team2;
96
+					if ($team2 instanceof Team) {
97
+						$this->teams[] = $team2;
98
+					}
93 99
 					$team2->groupResults[$this->id] = [
94 100
 						'group' => $this,
95 101
 						'points' => 0,
@@ -101,8 +107,9 @@  discard block
 block discarded – undo
101 107
 						'third'  => 0
102 108
 					];
103 109
 				}
110
+			} else {
111
+				throw new \Exception('Trying to add team which is not an instance of Team class');
104 112
 			}
105
-			else throw new \Exception('Trying to add team which is not an instance of Team class');
106 113
 		}
107 114
 		return $this;
108 115
 	}
@@ -131,7 +138,9 @@  discard block
 block discarded – undo
131 138
 
132 139
 	public function splitTeams(...$groups) {
133 140
 
134
-		if (count($groups) === 0) $groups = $this->getGroups();
141
+		if (count($groups) === 0) {
142
+			$groups = $this->getGroups();
143
+		}
135 144
 
136 145
 		foreach ($groups as $key => $value) {
137 146
 			if (gettype($value) === 'array') {
@@ -146,7 +155,9 @@  discard block
 block discarded – undo
146 155
 		while (count($teams) > 0) {
147 156
 			foreach ($groups as $group) {
148 157
 				if ($group instanceof Group) {
149
-					if (count($teams) > 0) $group->addTeam(array_shift($teams));
158
+					if (count($teams) > 0) {
159
+						$group->addTeam(array_shift($teams));
160
+					}
150 161
 				}
151 162
 			}
152 163
 		}
Please login to merge, or discard this patch.
Spacing   +10 added lines, -10 removed lines patch added patch discarded remove patch
@@ -23,7 +23,7 @@  discard block
 block discarded – undo
23 23
 		return $this->name;
24 24
 	}
25 25
 
26
-	public function addGroup(Group ...$groups){
26
+	public function addGroup(Group ...$groups) {
27 27
 		foreach ($groups as $group) {
28 28
 			if ($group instanceof Group) $this->groups[] = $group;
29 29
 			else throw new \Exception('Trying to add group which is not an instance of Group class.');
@@ -35,7 +35,7 @@  discard block
 block discarded – undo
35 35
 		$this->groups[] = $g->setSkip($this->allowSkip);
36 36
 		return $g;
37 37
 	}
38
-	public function getGroups(){
38
+	public function getGroups() {
39 39
 		$this->orderGroups();
40 40
 		return $this->groups;
41 41
 	}
@@ -44,16 +44,16 @@  discard block
 block discarded – undo
44 44
 		return array_map(function($a) { return $a->id; }, $this->groups);
45 45
 	}
46 46
 	public function orderGroups() {
47
-		usort($this->groups, function($a, $b){
48
-			return $a->getOrder() - $b->getOrder();
47
+		usort($this->groups, function($a, $b) {
48
+			return $a->getOrder()-$b->getOrder();
49 49
 		});
50 50
 	}
51 51
 
52
-	public function allowSkip(){
52
+	public function allowSkip() {
53 53
 		$this->allowSkip = true;
54 54
 		return $this;
55 55
 	}
56
-	public function disallowSkip(){
56
+	public function disallowSkip() {
57 57
 		$this->allowSkip = false;
58 58
 		return $this;
59 59
 	}
@@ -65,7 +65,7 @@  discard block
 block discarded – undo
65 65
 		return $this->allowSkip;
66 66
 	}
67 67
 
68
-	public function genGames(){
68
+	public function genGames() {
69 69
 		foreach ($this->groups as $group) {
70 70
 			$group->genGames();
71 71
 			$this->games = array_merge($this->games, $group->orderGames());
@@ -75,7 +75,7 @@  discard block
 block discarded – undo
75 75
 	public function getGames() {
76 76
 		return $this->games;
77 77
 	}
78
-	public function isPlayed(){
78
+	public function isPlayed() {
79 79
 		foreach ($this->groups as $group) {
80 80
 			if (!$group->isPlayed()) return false;
81 81
 		}
@@ -84,7 +84,7 @@  discard block
 block discarded – undo
84 84
 
85 85
 	public function addTeam(...$teams) {
86 86
 		foreach ($teams as $team) {
87
-			if ($team instanceof Team)  {
87
+			if ($team instanceof Team) {
88 88
 				$this->teams[] = $team;
89 89
 			}
90 90
 			elseif (gettype($team) === 'array') {
@@ -153,7 +153,7 @@  discard block
 block discarded – undo
153 153
 		return $this;
154 154
 	}
155 155
 
156
-	public function progress(bool $blank = false){
156
+	public function progress(bool $blank = false) {
157 157
 		foreach ($this->groups as $group) {
158 158
 			$group->progress($blank);
159 159
 		}
Please login to merge, or discard this patch.
src/TournamentGenerator/Category.php 2 patches
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -21,7 +21,7 @@  discard block
 block discarded – undo
21 21
 		$this->name = $name;
22 22
 	}
23 23
 
24
-	public function addRound(Round ...$rounds){
24
+	public function addRound(Round ...$rounds) {
25 25
 		foreach ($rounds as $round) {
26 26
 			if ($round instanceof Round) $this->rounds[] = $round;
27 27
 			else throw new \Exception('Trying to add round which is not an instance of Round class.');
@@ -33,15 +33,15 @@  discard block
 block discarded – undo
33 33
 		$this->rounds[] = $r->setSkip($this->allowSkip);
34 34
 		return $r;
35 35
 	}
36
-	public function getRounds(){
36
+	public function getRounds() {
37 37
 		return $this->rounds;
38 38
 	}
39 39
 
40
-	public function allowSkip(){
40
+	public function allowSkip() {
41 41
 		$this->allowSkip = true;
42 42
 		return $this;
43 43
 	}
44
-	public function disallowSkip(){
44
+	public function disallowSkip() {
45 45
 		$this->allowSkip = false;
46 46
 		return $this;
47 47
 	}
@@ -55,7 +55,7 @@  discard block
 block discarded – undo
55 55
 
56 56
 	public function addTeam(...$teams) {
57 57
 		foreach ($teams as $team) {
58
-			if ($team instanceof Team)  {
58
+			if ($team instanceof Team) {
59 59
 				$this->teams[] = $team;
60 60
 			}
61 61
 			elseif (gettype($team) === 'array') {
Please login to merge, or discard this patch.
Braces   +20 added lines, -9 removed lines patch added patch discarded remove patch
@@ -21,8 +21,11 @@  discard block
 block discarded – undo
21 21
 
22 22
 	public function addRound(Round ...$rounds){
23 23
 		foreach ($rounds as $round) {
24
-			if ($round instanceof Round) $this->rounds[] = $round;
25
-			else throw new \Exception('Trying to add round which is not an instance of Round class.');
24
+			if ($round instanceof Round) {
25
+				$this->rounds[] = $round;
26
+			} else {
27
+				throw new \Exception('Trying to add round which is not an instance of Round class.');
28
+			}
26 29
 		}
27 30
 		return $this;
28 31
 	}
@@ -55,13 +58,15 @@  discard block
 block discarded – undo
55 58
 		foreach ($teams as $team) {
56 59
 			if ($team instanceof Team)  {
57 60
 				$this->teams[] = $team;
58
-			}
59
-			elseif (gettype($team) === 'array') {
61
+			} elseif (gettype($team) === 'array') {
60 62
 				foreach ($team as $team2) {
61
-					if ($team2 instanceof Team) $this->teams[] = $team2;
63
+					if ($team2 instanceof Team) {
64
+						$this->teams[] = $team2;
65
+					}
62 66
 				}
67
+			} else {
68
+				throw new \Exception('Trying to add team which is not an instance of Team class');
63 69
 			}
64
-			else throw new \Exception('Trying to add team which is not an instance of Team class');
65 70
 		}
66 71
 		return $this;
67 72
 	}
@@ -71,7 +76,9 @@  discard block
 block discarded – undo
71 76
 		return $t;
72 77
 	}
73 78
 	public function getTeams() {
74
-		if (count($this->teams) > 0) return $this->teams;
79
+		if (count($this->teams) > 0) {
80
+			return $this->teams;
81
+		}
75 82
 		$teams = [];
76 83
 		foreach ($this->rounds as $round) {
77 84
 			$teams = array_merge($teams, $round->getTeams());
@@ -90,7 +97,9 @@  discard block
 block discarded – undo
90 97
 
91 98
 	public function splitTeams(...$rounds) {
92 99
 
93
-		if (count($rounds) === 0) $rounds = $this->getRounds();
100
+		if (count($rounds) === 0) {
101
+			$rounds = $this->getRounds();
102
+		}
94 103
 
95 104
 		$teams = $this->getTeams();
96 105
 		shuffle($teams);
@@ -110,7 +119,9 @@  discard block
 block discarded – undo
110 119
 
111 120
 	public function genGamesSimulate() {
112 121
 		$games = [];
113
-		if (count($this->rounds) <= 0) throw new \Exception('There are no rounds to simulate games from.');
122
+		if (count($this->rounds) <= 0) {
123
+			throw new \Exception('There are no rounds to simulate games from.');
124
+		}
114 125
 		foreach ($this->rounds as $round) {
115 126
 			$games = array_merge($games, $round->genGames());
116 127
 			$round->simulate()->progress(true)->resetGames();
Please login to merge, or discard this patch.
src/TournamentGenerator/Team.php 3 patches
Indentation   +14 added lines, -14 removed lines patch added patch discarded remove patch
@@ -16,20 +16,20 @@
 block discarded – undo
16 16
 	public $sumScore = 0;
17 17
 
18 18
 	/**
19
-	* ARRAY WITH GROUPS AND IT'S RESULTS
20
-	* array (
21
-	* * groupId => array (
22
-	* * * "group"  => Group, # GROUP OBJECT
23
-	* * * "points" => int 0, # NUMBER OF POINTS AQUIRED
24
-	* * * "score"  => int 0, # SUM OF SCORE AQUIRED
25
-	* * * "wins"   => int 0, # NUMBER OF WINS
26
-	* * * "draws"  => int 0, # NUMBER OF DRAWS
27
-	* * * "losses" => int 0, # NUMBER OF LOSSES
28
-	* * * "second" => int 0, # NUMBER OF TIMES BEING SECOND (ONLY FOR INGAME OPTION OF 3 OR 4)
29
-	* * * "third"  => int 0  # NUMBER OF TIMES BEING THIRD  (ONLY FOR INGAME OPTION OF 4)
30
-	* * )
31
-	*)
32
-	*/
19
+	 * ARRAY WITH GROUPS AND IT'S RESULTS
20
+	 * array (
21
+	 * * groupId => array (
22
+	 * * * "group"  => Group, # GROUP OBJECT
23
+	 * * * "points" => int 0, # NUMBER OF POINTS AQUIRED
24
+	 * * * "score"  => int 0, # SUM OF SCORE AQUIRED
25
+	 * * * "wins"   => int 0, # NUMBER OF WINS
26
+	 * * * "draws"  => int 0, # NUMBER OF DRAWS
27
+	 * * * "losses" => int 0, # NUMBER OF LOSSES
28
+	 * * * "second" => int 0, # NUMBER OF TIMES BEING SECOND (ONLY FOR INGAME OPTION OF 3 OR 4)
29
+	 * * * "third"  => int 0  # NUMBER OF TIMES BEING THIRD  (ONLY FOR INGAME OPTION OF 4)
30
+	 * * )
31
+	 *)
32
+	 */
33 33
 	public $groupResults = [];
34 34
 
35 35
 	function __construct(string $name = 'team', $id = null) {
Please login to merge, or discard this patch.
Spacing   +10 added lines, -10 removed lines patch added patch discarded remove patch
@@ -59,70 +59,70 @@
 block discarded – undo
59 59
 		return $this;
60 60
 	}
61 61
 
62
-	public function addWin(string $groupId = ''){
62
+	public function addWin(string $groupId = '') {
63 63
 		if (!isset($this->groupResults[$groupId])) throw new \Exception('Group '.$groupId.' is not set for this team ('.$this->name.')');
64 64
 		$this->groupResults[$groupId]['points'] += $this->groupResults[$groupId]['group']->winPoints;
65 65
 		$this->sumPoints += $this->groupResults[$groupId]['group']->winPoints;
66 66
 		$this->groupResults[$groupId]['wins']++;
67 67
 		return $this;
68 68
 	}
69
-	public function removeWin(string $groupId = ''){
69
+	public function removeWin(string $groupId = '') {
70 70
 		if (!isset($this->groupResults[$groupId])) throw new \Exception('Group '.$groupId.' is not set for this team ('.$this->name.')');
71 71
 		$this->groupResults[$groupId]['points'] -= $this->groupResults[$groupId]['group']->winPoints;
72 72
 		$this->sumPoints -= $this->groupResults[$groupId]['group']->winPoints;
73 73
 		$this->groupResults[$groupId]['wins']--;
74 74
 		return $this;
75 75
 	}
76
-	public function addDraw(string $groupId = ''){
76
+	public function addDraw(string $groupId = '') {
77 77
 		if (!isset($this->groupResults[$groupId])) throw new \Exception('Group '.$groupId.' is not set for this team ('.$this->name.')');
78 78
 		$this->groupResults[$groupId]['points'] += $this->groupResults[$groupId]['group']->drawPoints;
79 79
 		$this->sumPoints += $this->groupResults[$groupId]['group']->drawPoints;
80 80
 		$this->groupResults[$groupId]['draws']++;
81 81
 		return $this;
82 82
 	}
83
-	public function removeDraw(string $groupId = ''){
83
+	public function removeDraw(string $groupId = '') {
84 84
 		if (!isset($this->groupResults[$groupId])) throw new \Exception('Group '.$groupId.' is not set for this team ('.$this->name.')');
85 85
 		$this->groupResults[$groupId]['points'] -= $this->groupResults[$groupId]['group']->drawPointsPoints;
86 86
 		$this->sumPoints -= $this->groupResults[$groupId]['group']->drawPoints;
87 87
 		$this->groupResults[$groupId]['draws']--;
88 88
 		return $this;
89 89
 	}
90
-	public function addLoss(string $groupId = ''){
90
+	public function addLoss(string $groupId = '') {
91 91
 		if (!isset($this->groupResults[$groupId])) throw new \Exception('Group '.$groupId.' is not set for this team ('.$this->name.')');
92 92
 		$this->groupResults[$groupId]['points'] += $this->groupResults[$groupId]['group']->lostPoints;
93 93
 		$this->sumPoints += $this->groupResults[$groupId]['group']->lostPoints;
94 94
 		$this->groupResults[$groupId]['losses']++;
95 95
 		return $this;
96 96
 	}
97
-	public function removeLoss(string $groupId = ''){
97
+	public function removeLoss(string $groupId = '') {
98 98
 		if (!isset($this->groupResults[$groupId])) throw new \Exception('Group '.$groupId.' is not set for this team ('.$this->name.')');
99 99
 		$this->groupResults[$groupId]['points'] -= $this->groupResults[$groupId]['group']->lostPoints;
100 100
 		$this->sumPoints -= $this->groupResults[$groupId]['group']->lostPoints;
101 101
 		$this->groupResults[$groupId]['losses']--;
102 102
 		return $this;
103 103
 	}
104
-	public function addSecond(string $groupId = ''){
104
+	public function addSecond(string $groupId = '') {
105 105
 		if (!isset($this->groupResults[$groupId])) throw new \Exception('Group '.$groupId.' is not set for this team ('.$this->name.')');
106 106
 		$this->groupResults[$groupId]['points'] += $this->groupResults[$groupId]['group']->secondPoints;
107 107
 		$this->sumPoints += $this->groupResults[$groupId]['group']->secondPoints;
108 108
 		$this->groupResults[$groupId]['second']++;
109 109
 		return $this;
110 110
 	}
111
-	public function removeSecond(string $groupId = ''){
111
+	public function removeSecond(string $groupId = '') {
112 112
 		if (!isset($this->groupResults[$groupId])) throw new \Exception('Group '.$groupId.' is not set for this team ('.$this->name.')');
113 113
 		$this->groupResults[$groupId]['points'] -= $this->groupResults[$groupId]['group']->secondPoints;
114 114
 		$this->sumPoints -= $this->groupResults[$groupId]['group']->secondPoints;
115 115
 		$this->groupResults[$groupId]['second']--;
116 116
 		return $this;
117 117
 	}
118
-	public function addThird(string $groupId = ''){
118
+	public function addThird(string $groupId = '') {
119 119
 		if (!isset($this->groupResults[$groupId])) throw new \Exception('Group '.$groupId.' is not set for this team ('.$this->name.')');
120 120
 		$this->groupResults[$groupId]['points'] += $this->groupResults[$groupId]['group']->thirdPoints;
121 121
 		$this->sumPoints += $this->groupResults[$groupId]['group']->thirdPoints;
122 122
 		$this->groupResults[$groupId]['third']++;
123 123
 		return $this;
124 124
 	}
125
-	public function removeThird(string $groupId = ''){
125
+	public function removeThird(string $groupId = '') {
126 126
 		if (!isset($this->groupResults[$groupId])) throw new \Exception('Group '.$groupId.' is not set for this team ('.$this->name.')');
127 127
 		$this->groupResults[$groupId]['points'] -= $this->groupResults[$groupId]['group']->thirdPoints;
128 128
 		$this->sumPoints -= $this->groupResults[$groupId]['group']->thirdPoints;
Please login to merge, or discard this patch.
Braces   +51 added lines, -17 removed lines patch added patch discarded remove patch
@@ -44,104 +44,138 @@
 block discarded – undo
44 44
 	}
45 45
 
46 46
 	public function addGameWith(Team $team, Group $group) {
47
-		if (!isset($this->gamesWith[$group->id][$team->id])) $this->gamesWith[$group->id][$team->id] = 0;
47
+		if (!isset($this->gamesWith[$group->id][$team->id])) {
48
+			$this->gamesWith[$group->id][$team->id] = 0;
49
+		}
48 50
 		$this->gamesWith[$group->id][$team->id]++;
49 51
 		return $this;
50 52
 	}
51 53
 	public function addGroup(Group $group) {
52
-		if (!isset($this->games[$group->id])) $this->games[$group->id] = [];
54
+		if (!isset($this->games[$group->id])) {
55
+			$this->games[$group->id] = [];
56
+		}
53 57
 		return $this;
54 58
 	}
55 59
 	public function addGame(Game $game) {
56 60
 		$group = $game->getGroup();
57
-		if (!isset($this->games[$group->id])) $this->games[$group->id] = [];
61
+		if (!isset($this->games[$group->id])) {
62
+			$this->games[$group->id] = [];
63
+		}
58 64
 		$this->games[$group->id][] = $game;
59 65
 		return $this;
60 66
 	}
61 67
 
62 68
 	public function addWin(string $groupId = ''){
63
-		if (!isset($this->groupResults[$groupId])) throw new \Exception('Group '.$groupId.' is not set for this team ('.$this->name.')');
69
+		if (!isset($this->groupResults[$groupId])) {
70
+			throw new \Exception('Group '.$groupId.' is not set for this team ('.$this->name.')');
71
+		}
64 72
 		$this->groupResults[$groupId]['points'] += $this->groupResults[$groupId]['group']->winPoints;
65 73
 		$this->sumPoints += $this->groupResults[$groupId]['group']->winPoints;
66 74
 		$this->groupResults[$groupId]['wins']++;
67 75
 		return $this;
68 76
 	}
69 77
 	public function removeWin(string $groupId = ''){
70
-		if (!isset($this->groupResults[$groupId])) throw new \Exception('Group '.$groupId.' is not set for this team ('.$this->name.')');
78
+		if (!isset($this->groupResults[$groupId])) {
79
+			throw new \Exception('Group '.$groupId.' is not set for this team ('.$this->name.')');
80
+		}
71 81
 		$this->groupResults[$groupId]['points'] -= $this->groupResults[$groupId]['group']->winPoints;
72 82
 		$this->sumPoints -= $this->groupResults[$groupId]['group']->winPoints;
73 83
 		$this->groupResults[$groupId]['wins']--;
74 84
 		return $this;
75 85
 	}
76 86
 	public function addDraw(string $groupId = ''){
77
-		if (!isset($this->groupResults[$groupId])) throw new \Exception('Group '.$groupId.' is not set for this team ('.$this->name.')');
87
+		if (!isset($this->groupResults[$groupId])) {
88
+			throw new \Exception('Group '.$groupId.' is not set for this team ('.$this->name.')');
89
+		}
78 90
 		$this->groupResults[$groupId]['points'] += $this->groupResults[$groupId]['group']->drawPoints;
79 91
 		$this->sumPoints += $this->groupResults[$groupId]['group']->drawPoints;
80 92
 		$this->groupResults[$groupId]['draws']++;
81 93
 		return $this;
82 94
 	}
83 95
 	public function removeDraw(string $groupId = ''){
84
-		if (!isset($this->groupResults[$groupId])) throw new \Exception('Group '.$groupId.' is not set for this team ('.$this->name.')');
96
+		if (!isset($this->groupResults[$groupId])) {
97
+			throw new \Exception('Group '.$groupId.' is not set for this team ('.$this->name.')');
98
+		}
85 99
 		$this->groupResults[$groupId]['points'] -= $this->groupResults[$groupId]['group']->drawPointsPoints;
86 100
 		$this->sumPoints -= $this->groupResults[$groupId]['group']->drawPoints;
87 101
 		$this->groupResults[$groupId]['draws']--;
88 102
 		return $this;
89 103
 	}
90 104
 	public function addLoss(string $groupId = ''){
91
-		if (!isset($this->groupResults[$groupId])) throw new \Exception('Group '.$groupId.' is not set for this team ('.$this->name.')');
105
+		if (!isset($this->groupResults[$groupId])) {
106
+			throw new \Exception('Group '.$groupId.' is not set for this team ('.$this->name.')');
107
+		}
92 108
 		$this->groupResults[$groupId]['points'] += $this->groupResults[$groupId]['group']->lostPoints;
93 109
 		$this->sumPoints += $this->groupResults[$groupId]['group']->lostPoints;
94 110
 		$this->groupResults[$groupId]['losses']++;
95 111
 		return $this;
96 112
 	}
97 113
 	public function removeLoss(string $groupId = ''){
98
-		if (!isset($this->groupResults[$groupId])) throw new \Exception('Group '.$groupId.' is not set for this team ('.$this->name.')');
114
+		if (!isset($this->groupResults[$groupId])) {
115
+			throw new \Exception('Group '.$groupId.' is not set for this team ('.$this->name.')');
116
+		}
99 117
 		$this->groupResults[$groupId]['points'] -= $this->groupResults[$groupId]['group']->lostPoints;
100 118
 		$this->sumPoints -= $this->groupResults[$groupId]['group']->lostPoints;
101 119
 		$this->groupResults[$groupId]['losses']--;
102 120
 		return $this;
103 121
 	}
104 122
 	public function addSecond(string $groupId = ''){
105
-		if (!isset($this->groupResults[$groupId])) throw new \Exception('Group '.$groupId.' is not set for this team ('.$this->name.')');
123
+		if (!isset($this->groupResults[$groupId])) {
124
+			throw new \Exception('Group '.$groupId.' is not set for this team ('.$this->name.')');
125
+		}
106 126
 		$this->groupResults[$groupId]['points'] += $this->groupResults[$groupId]['group']->secondPoints;
107 127
 		$this->sumPoints += $this->groupResults[$groupId]['group']->secondPoints;
108 128
 		$this->groupResults[$groupId]['second']++;
109 129
 		return $this;
110 130
 	}
111 131
 	public function removeSecond(string $groupId = ''){
112
-		if (!isset($this->groupResults[$groupId])) throw new \Exception('Group '.$groupId.' is not set for this team ('.$this->name.')');
132
+		if (!isset($this->groupResults[$groupId])) {
133
+			throw new \Exception('Group '.$groupId.' is not set for this team ('.$this->name.')');
134
+		}
113 135
 		$this->groupResults[$groupId]['points'] -= $this->groupResults[$groupId]['group']->secondPoints;
114 136
 		$this->sumPoints -= $this->groupResults[$groupId]['group']->secondPoints;
115 137
 		$this->groupResults[$groupId]['second']--;
116 138
 		return $this;
117 139
 	}
118 140
 	public function addThird(string $groupId = ''){
119
-		if (!isset($this->groupResults[$groupId])) throw new \Exception('Group '.$groupId.' is not set for this team ('.$this->name.')');
141
+		if (!isset($this->groupResults[$groupId])) {
142
+			throw new \Exception('Group '.$groupId.' is not set for this team ('.$this->name.')');
143
+		}
120 144
 		$this->groupResults[$groupId]['points'] += $this->groupResults[$groupId]['group']->thirdPoints;
121 145
 		$this->sumPoints += $this->groupResults[$groupId]['group']->thirdPoints;
122 146
 		$this->groupResults[$groupId]['third']++;
123 147
 		return $this;
124 148
 	}
125 149
 	public function removeThird(string $groupId = ''){
126
-		if (!isset($this->groupResults[$groupId])) throw new \Exception('Group '.$groupId.' is not set for this team ('.$this->name.')');
150
+		if (!isset($this->groupResults[$groupId])) {
151
+			throw new \Exception('Group '.$groupId.' is not set for this team ('.$this->name.')');
152
+		}
127 153
 		$this->groupResults[$groupId]['points'] -= $this->groupResults[$groupId]['group']->thirdPoints;
128 154
 		$this->sumPoints -= $this->groupResults[$groupId]['group']->thirdPoints;
129 155
 		$this->groupResults[$groupId]['third']--;
130 156
 		return $this;
131 157
 	}
132 158
 	public function sumPoints(array $groupIds = []) {
133
-		if (count($groupIds) === 0) return $this->sumPoints;
159
+		if (count($groupIds) === 0) {
160
+			return $this->sumPoints;
161
+		}
134 162
 		$sum = 0;
135 163
 		foreach ($groupIds as $gid) {
136
-			if (isset($this->groupResults[$gid])) $sum += $this->groupResults[$gid]['points'];
164
+			if (isset($this->groupResults[$gid])) {
165
+				$sum += $this->groupResults[$gid]['points'];
166
+			}
137 167
 		}
138 168
 		return $sum;
139 169
 	}
140 170
 	public function sumScore(array $groupIds = []) {
141
-		if (count($groupIds) === 0) return $this->sumScore;
171
+		if (count($groupIds) === 0) {
172
+			return $this->sumScore;
173
+		}
142 174
 		$sum = 0;
143 175
 		foreach ($groupIds as $gid) {
144
-			if (isset($this->groupResults[$gid])) $sum += $this->groupResults[$gid]['score'];
176
+			if (isset($this->groupResults[$gid])) {
177
+				$sum += $this->groupResults[$gid]['score'];
178
+			}
145 179
 		}
146 180
 		return $sum;
147 181
 	}
Please login to merge, or discard this patch.