BigBad / golfleague

app/controllers/CoursesController.php

Doc Comments -1 removed lines

@@ -42,7 +42,6 @@
     /**
      * Show the form for editing the specified resource.
      *
-     * @param  int  $id
      * @return Response
      */
     public function edit()

app/controllers/HolesController.php

Doc Comments -1 removed lines

@@ -41,7 +41,6 @@
 	/**
 	 * Display the specified resource.
 	 *
-	 * @param  int  $course_id
 	 * @return Response
 	 */
 	public function show()

Braces +1 added lines, -1 removed lines

@@ -3,7 +3,7 @@
 class HolesController extends \BaseController {
 
 	public function __construct(Course $course, Hole $hole)
-    {
+	{
 		$this->course = $course;
 		$this->hole = $hole;
     }

app/controllers/LiveLeaderboardController.php

Doc Comments +1 added lines, -1 removed lines

@@ -13,7 +13,7 @@
     /**
 	 * Display a listing of the resource.
 	 *
-	 * @return Response
+	 * @return string
 	 */
 	public function index()
 	{

Braces +1 added lines, -1 removed lines

@@ -6,7 +6,7 @@
 class LiveLeaderboardController extends \BaseController {
 
 	public function __construct(LeaderboardService $leaderboard)
-    {
+	{
         $this->leaderboard = $leaderboard;
     }
 

app/library/Handlers/FinalizeHandler.php

Doc Comments +1 added lines, -1 removed lines

@@ -21,7 +21,7 @@
     /**
      * Create a new instance of the MatchHandler
      *
-     * @param  GolfLeague\Storage\Round\RoundRepository $roundRepo
+     * @param  RoundRepository $roundRepo
      * @return void
      */
     public function __construct(RoundRepository $roundRepo)

Braces +17 added lines, -21 removed lines

@@ -16,8 +16,7 @@ 
  *
  * @author          Michael Schmidt
  */
-class FinalizeHandler
-{
+class FinalizeHandler {
     /**
      * Create a new instance of the MatchHandler
      *
@@ -45,15 +44,14 @@ 
         $i = 1;
 
         $ctpWinners = Ctp::where('match_id', '=', $match->id)->get();
-        foreach($ctpWinners as $key => $ctpWinner){
+        foreach($ctpWinners as $key => $ctpWinner) {
             $playerExists = $this->recursive_array_search($ctpWinner->player_id, $winningPlayers);
             //if player is already there add money here to other money
-            if($playerExists){
+            if($playerExists) {
                 $winningPlayers[$i]['player_id'] = $ctpWinner->player_id;
                 $winningPlayers[$i]['money'] = ($ctpWinner->money + $winningPlayers[$playerExists]['money']);
                 unset($winningPlayers[$playerExists]);
-            }
-            else {
+            } else {
                 $winningPlayers[$i]['player_id'] = $ctpWinner->player_id;
                 $winningPlayers[$i]['money'] = $ctpWinner->money;
             }
@@ -61,15 +59,14 @@ 
         }
 
         $netWinners = Netwinner::where('match_id', '=', $match->id)->get();
-        foreach($netWinners as $key => $netWinner){
+        foreach($netWinners as $key => $netWinner) {
             $playerExists = $this->recursive_array_search($netWinner->player_id, $winningPlayers);
             //if player is already there add money here to other money
-            if($playerExists){
+            if($playerExists) {
                 $winningPlayers[$i]['player_id'] = $netWinner->player_id;
                 $winningPlayers[$i]['money'] = ($netWinner->money + $winningPlayers[$playerExists]['money']);
                 unset($winningPlayers[$playerExists]);
-            }
-            else{
+            } else {
                 $winningPlayers[$i]['player_id'] = $netWinner->player_id;
                 $winningPlayers[$i]['money'] = $netWinner->money;
             }
@@ -77,15 +74,14 @@ 
         }
 
         $grossWinners = Grosswinner::where('match_id', '=', $match->id)->get();
-        foreach($grossWinners as $key => $grossWinner){
+        foreach($grossWinners as $key => $grossWinner) {
             $playerExists = $this->recursive_array_search($grossWinner->player_id, $winningPlayers);
             //if player is already there add money here to other money
-            if($playerExists){
+            if($playerExists) {
                 $winningPlayers[$i]['player_id'] = $grossWinner->player_id;
                 $winningPlayers[$i]['money'] = ($grossWinner->money + $winningPlayers[$playerExists]['money']);
                 unset($winningPlayers[$playerExists]);
-            }
-            else{
+            } else {
                 $winningPlayers[$i]['player_id'] = $grossWinner->player_id;
                 $winningPlayers[$i]['money'] = $grossWinner->money;
             }
@@ -93,15 +89,14 @@ 
         }
 
         $skinWinners = Skin::where('match_id', '=', $match->id)->get();
-        foreach($skinWinners as $key => $skinWinner){
+        foreach($skinWinners as $key => $skinWinner) {
             $playerExists = $this->recursive_array_search($skinWinner->player_id, $winningPlayers);
             //if player is already there add money here to other money
-            if($playerExists){
+            if($playerExists) {
                 $winningPlayers[$i]['player_id'] = $skinWinner->player_id;
                 $winningPlayers[$i]['money'] = ($skinWinner->money + $winningPlayers[$playerExists]['money']);
                 unset($winningPlayers[$playerExists]);
-            }
-            else{
+            } else {
                 $winningPlayers[$i]['player_id'] = $skinWinner->player_id;
                 $winningPlayers[$i]['money'] = $skinWinner->money;
             }
@@ -109,11 +104,11 @@ 
         }
 
         //Consolidate any multiple instances of a player
-        foreach ($winningPlayers as $key => $winningPlayer){
+        foreach ($winningPlayers as $key => $winningPlayer) {
             array_search($winningPlayer['player_id'], $winningPlayers);
         }
 
-        foreach ($winningPlayers as $key => $player){
+        foreach ($winningPlayers as $key => $player) {
             $currentPlayer = Player::find($player['player_id']);
             $attributes = array(
                 "winnings" => $player['money']
@@ -135,7 +130,8 @@ 
         $events->listen('match.finalize', 'GolfLeague\Handlers\FinalizeHandler');
     }
 
-    public function recursive_array_search($needle,$haystack) {
+    public function recursive_array_search($needle,$haystack)
+    {
         foreach($haystack as $key=>$value) {
             $current_key=$key;
             if($needle===$value || (is_array($value) && $this->recursive_array_search($needle,$value) !== false)) {

app/library/Handlers/RoundHandler.php

Doc Comments -2 removed lines

@@ -17,7 +17,6 @@ 
     /**
      * Create a new instance of the MatchHandler
      *
-     * @param  GolfLeague\Storage\Round\RoundRepository $roundRepo
      * @return void
      */
     public function __construct(HoleScoreRepository $holescoreRepo)
@@ -28,7 +27,6 @@ 
     /**
      * Create an initial round for each player after a new match is created
      *
-     * @param  Match $match
      * @return void
      */
     public function handle($round)

Braces +2 added lines, -3 removed lines

@@ -12,8 +12,7 @@ 
  *
  * @author          Michael Schmidt
  */
-class RoundHandler
-{
+class RoundHandler {
     /**
      * Create a new instance of the MatchHandler
      *
@@ -34,7 +33,7 @@ 
     public function handle($round)
     {
         $holes = Hole::where('course_id', '=', $round['course_id'])->get();
-        foreach($holes as $hole){
+        foreach($holes as $hole) {
             $input = array(
                 'score' => null,
                 'hole_id' => $hole->id,

app/library/Services/MatchRoundService.php

Doc Comments +2 added lines, -2 removed lines

@@ -15,7 +15,7 @@ 
     /**
     * Loads our $matchRepo
     *
-    * @param MatchRepository $matchRepo
+    * @param MatchRepository $matchRoundRepo
     * @return MatchService
     */
     public function __construct(MatchRoundRepository $matchRoundRepo, RoundRepository $roundRepo, MatchRepository $match)
@@ -32,7 +32,7 @@ 
     /**
     * Method to get match data from input Match id
     *
-    * @param mixed $matchdata
+    * @param mixed $matchId
     * @return JSON object
     */
     public function getByMatch($matchId)

Braces +1 added lines, -2 removed lines

@@ -7,8 +7,7 @@
 /**
 * Our MatchService, containing all useful methods for business logic around Matches
 */
-class MatchRoundService
-{
+class MatchRoundService {
     // Containing our pokemonRepository to make all our database calls to
     protected $matchRepo;
 

app/library/Services/MoneyService.php

Doc Comments -1 removed lines

@@ -21,7 +21,6 @@
     /**
     * Loads our $matchRepo
     *
-    * @param MatchRepository $matchRepo
     * @return MatchService
     */
     public function __construct()

Braces +1 added lines, -2 removed lines

@@ -12,8 +12,7 @@
 /**
 * MoneyService, containing all useful methods for business logic for money relating to a match
 */
-class MoneyService
-{
+class MoneyService {
 
     // Containing our matchRepository to make all our database calls
     protected $matchRepo;

app/models/Single.php

Braces +2 added lines, -3 removed lines

@@ -1,14 +1,13 @@
 <?php
 
-class Single extends Eloquent
-{
+class Single extends Eloquent {
     public function match()
     {
         return $this->belongsTo('Match');
     }
 
 	public function players()
-    {
+	{
         return $this->hasMany('Player');
     }
 }
\ No newline at end of file

app/models/Game.php

Braces +1 added lines, -1 removed lines

@@ -3,7 +3,7 @@
 class Game extends Eloquent {
 	
 	public function winners()
-    {
+	{
         return $this->hasMany('Winner');
     }
 }
\ No newline at end of file