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()

Indentation +78 added lines, -78 removed lines

@@ -2,96 +2,96 @@
 
 class CoursesController extends \BaseController {
 
-    public function __construct(Course $course)
-    {
-        $this->course = $course;
-    }
+	public function __construct(Course $course)
+	{
+		$this->course = $course;
+	}
 
-    /**
-     * Show the form for creating a new resource.
-     *
-     * @return Response
-     */
-    public function index()
-    {
-        $data = $this->course->all();
-        return $data;
-    }
+	/**
+	 * Show the form for creating a new resource.
+	 *
+	 * @return Response
+	 */
+	public function index()
+	{
+		$data = $this->course->all();
+		return $data;
+	}
 
-    /**
-     * Store a newly created resource in storage.
-     *
-     * @return Response
-     */
-    public function store()
-    {
-        //
-    }
+	/**
+	 * Store a newly created resource in storage.
+	 *
+	 * @return Response
+	 */
+	public function store()
+	{
+		//
+	}
 
-    /**
-     * Display the specified resource.
-     *
-     * @param  int  $id
-     * @return Response
-     */
-    public function show($id)
-    {
-        return Course::find($id)->get();
-    }
+	/**
+	 * Display the specified resource.
+	 *
+	 * @param  int  $id
+	 * @return Response
+	 */
+	public function show($id)
+	{
+		return Course::find($id)->get();
+	}
 
-    /**
-     * Show the form for editing the specified resource.
-     *
-     * @param  int  $id
-     * @return Response
-     */
-    public function edit()
-    {
+	/**
+	 * Show the form for editing the specified resource.
+	 *
+	 * @param  int  $id
+	 * @return Response
+	 */
+	public function edit()
+	{
 		$operation = Input::get('oper');
-        switch ($operation) {
-        case "add":
-            $this->course->name = Input::get('name');
+		switch ($operation) {
+		case "add":
+			$this->course->name = Input::get('name');
 			$this->course->par = Input::get('par');
 			$this->course->rating = Input::get('rating');
 			$this->course->slope = Input::get('slope');
-            $this->course->save();
-            break;
-        case "edit":
-            $id = Input::get('id');
-            $this->course = $this->course->find($id);
-            $this->course->name = Input::get('name');
+			$this->course->save();
+			break;
+		case "edit":
+			$id = Input::get('id');
+			$this->course = $this->course->find($id);
+			$this->course->name = Input::get('name');
 			$this->course->par = Input::get('par');
 			$this->course->rating = Input::get('rating');
 			$this->course->slope = Input::get('slope');
-            $this->course->save();
-            break;
-        case "del":
-            $id = Input::get('id');
-            $this->course = $this->course->destroy($id);
-            break;
-        }
-    }
+			$this->course->save();
+			break;
+		case "del":
+			$id = Input::get('id');
+			$this->course = $this->course->destroy($id);
+			break;
+		}
+	}
 
-    /**
-     * Update the specified resource in storage.
-     *
-     * @param  int  $id
-     * @return Response
-     */
-    public function update($id)
-    {
-        //
-    }
+	/**
+	 * Update the specified resource in storage.
+	 *
+	 * @param  int  $id
+	 * @return Response
+	 */
+	public function update($id)
+	{
+		//
+	}
 
-    /**
-     * Remove the specified resource from storage.
-     *
-     * @param  int  $id
-     * @return Response
-     */
-    public function destroy($id)
-    {
-        //
-    }
+	/**
+	 * Remove the specified resource from storage.
+	 *
+	 * @param  int  $id
+	 * @return Response
+	 */
+	public function destroy($id)
+	{
+		//
+	}
 
 }
\ No newline at end of file

Switch Indentation +20 added lines, -20 removed lines

@@ -49,26 +49,26 @@
     {
 		$operation = Input::get('oper');
         switch ($operation) {
-        case "add":
-            $this->course->name = Input::get('name');
-			$this->course->par = Input::get('par');
-			$this->course->rating = Input::get('rating');
-			$this->course->slope = Input::get('slope');
-            $this->course->save();
-            break;
-        case "edit":
-            $id = Input::get('id');
-            $this->course = $this->course->find($id);
-            $this->course->name = Input::get('name');
-			$this->course->par = Input::get('par');
-			$this->course->rating = Input::get('rating');
-			$this->course->slope = Input::get('slope');
-            $this->course->save();
-            break;
-        case "del":
-            $id = Input::get('id');
-            $this->course = $this->course->destroy($id);
-            break;
+        	case "add":
+            	$this->course->name = Input::get('name');
+				$this->course->par = Input::get('par');
+				$this->course->rating = Input::get('rating');
+				$this->course->slope = Input::get('slope');
+            	$this->course->save();
+            	break;
+        	case "edit":
+            	$id = Input::get('id');
+            	$this->course = $this->course->find($id);
+            	$this->course->name = Input::get('name');
+				$this->course->par = Input::get('par');
+				$this->course->rating = Input::get('rating');
+				$this->course->slope = Input::get('slope');
+            	$this->course->save();
+            	break;
+        	case "del":
+            	$id = Input::get('id');
+            	$this->course = $this->course->destroy($id);
+            	break;
         }
     }
 

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()

Indentation +3 added lines, -3 removed lines

@@ -3,10 +3,10 @@ 
 class HolesController extends \BaseController {
 
 	public function __construct(Course $course, Hole $hole)
-    {
+	{
 		$this->course = $course;
 		$this->hole = $hole;
-    }
+	}
 	/**
 	 * Display a listing of the resource.
 	 *
@@ -15,7 +15,7 @@ 
 	public function index()
 	{
 		$data = $this->hole->all();
-        return $data;
+		return $data;
 	}
 
 	/**

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()
 	{

Indentation +8 added lines, -8 removed lines

@@ -6,11 +6,11 @@ 
 class LiveLeaderboardController extends \BaseController {
 
 	public function __construct(LeaderboardService $leaderboard)
-    {
-        $this->leaderboard = $leaderboard;
-    }
+	{
+		$this->leaderboard = $leaderboard;
+	}
 
-    /**
+	/**
 	 * Display a listing of the resource.
 	 *
 	 * @return Response
@@ -51,12 +51,12 @@ 
 	 */
 	public function show($id)
 	{
-        $match = Input::get('match');
-        $type = $id;
+		$match = Input::get('match');
+		$type = $id;
 
-        //return net or gross leaderboard for live scoring
+		//return net or gross leaderboard for live scoring
 		$data['data'] = $this->leaderboard->calculate($match,$type);
-        return $data;
+		return $data;
 
 	}
 

Spacing +1 added lines, -1 removed lines

@@ -55,7 +55,7 @@
         $type = $id;
 
         //return net or gross leaderboard for live scoring
-		$data['data'] = $this->leaderboard->calculate($match,$type);
+		$data['data'] = $this->leaderboard->calculate($match, $type);
         return $data;
 
 	}

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)

Indentation +114 added lines, -114 removed lines

@@ -18,130 +18,130 @@
  */
 class FinalizeHandler
 {
-    /**
-     * Create a new instance of the MatchHandler
-     *
-     * @param  GolfLeague\Storage\Round\RoundRepository $roundRepo
-     * @return void
-     */
-    public function __construct(RoundRepository $roundRepo)
-    {
-        $this->roundRepo= $roundRepo;
-    } // End of __construct
+	/**
+	 * Create a new instance of the MatchHandler
+	 *
+	 * @param  GolfLeague\Storage\Round\RoundRepository $roundRepo
+	 * @return void
+	 */
+	public function __construct(RoundRepository $roundRepo)
+	{
+		$this->roundRepo= $roundRepo;
+	} // End of __construct
 
-    /**
-     * Create an initial round for each player after a new match is created
-     *
-     * @param  Match $match
-     * @return void
-     */
-    public function handle($match)
-    {
-        //for the match get ctps, netwinner, grosswinner, and skins
-        //calculate winnings
-        //store in the pivot table match_player for the given player
+	/**
+	 * Create an initial round for each player after a new match is created
+	 *
+	 * @param  Match $match
+	 * @return void
+	 */
+	public function handle($match)
+	{
+		//for the match get ctps, netwinner, grosswinner, and skins
+		//calculate winnings
+		//store in the pivot table match_player for the given player
 
-        $winningPlayers = array();
-        $i = 1;
+		$winningPlayers = array();
+		$i = 1;
 
-        $ctpWinners = Ctp::where('match_id', '=', $match->id)->get();
-        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){
-                $winningPlayers[$i]['player_id'] = $ctpWinner->player_id;
-                $winningPlayers[$i]['money'] = ($ctpWinner->money + $winningPlayers[$playerExists]['money']);
-                unset($winningPlayers[$playerExists]);
-            }
-            else {
-                $winningPlayers[$i]['player_id'] = $ctpWinner->player_id;
-                $winningPlayers[$i]['money'] = $ctpWinner->money;
-            }
-            $i++;
-        }
+		$ctpWinners = Ctp::where('match_id', '=', $match->id)->get();
+		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){
+				$winningPlayers[$i]['player_id'] = $ctpWinner->player_id;
+				$winningPlayers[$i]['money'] = ($ctpWinner->money + $winningPlayers[$playerExists]['money']);
+				unset($winningPlayers[$playerExists]);
+			}
+			else {
+				$winningPlayers[$i]['player_id'] = $ctpWinner->player_id;
+				$winningPlayers[$i]['money'] = $ctpWinner->money;
+			}
+			$i++;
+		}
 
-        $netWinners = Netwinner::where('match_id', '=', $match->id)->get();
-        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){
-                $winningPlayers[$i]['player_id'] = $netWinner->player_id;
-                $winningPlayers[$i]['money'] = ($netWinner->money + $winningPlayers[$playerExists]['money']);
-                unset($winningPlayers[$playerExists]);
-            }
-            else{
-                $winningPlayers[$i]['player_id'] = $netWinner->player_id;
-                $winningPlayers[$i]['money'] = $netWinner->money;
-            }
-            $i++;
-        }
+		$netWinners = Netwinner::where('match_id', '=', $match->id)->get();
+		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){
+				$winningPlayers[$i]['player_id'] = $netWinner->player_id;
+				$winningPlayers[$i]['money'] = ($netWinner->money + $winningPlayers[$playerExists]['money']);
+				unset($winningPlayers[$playerExists]);
+			}
+			else{
+				$winningPlayers[$i]['player_id'] = $netWinner->player_id;
+				$winningPlayers[$i]['money'] = $netWinner->money;
+			}
+			$i++;
+		}
 
-        $grossWinners = Grosswinner::where('match_id', '=', $match->id)->get();
-        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){
-                $winningPlayers[$i]['player_id'] = $grossWinner->player_id;
-                $winningPlayers[$i]['money'] = ($grossWinner->money + $winningPlayers[$playerExists]['money']);
-                unset($winningPlayers[$playerExists]);
-            }
-            else{
-                $winningPlayers[$i]['player_id'] = $grossWinner->player_id;
-                $winningPlayers[$i]['money'] = $grossWinner->money;
-            }
-            $i++;
-        }
+		$grossWinners = Grosswinner::where('match_id', '=', $match->id)->get();
+		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){
+				$winningPlayers[$i]['player_id'] = $grossWinner->player_id;
+				$winningPlayers[$i]['money'] = ($grossWinner->money + $winningPlayers[$playerExists]['money']);
+				unset($winningPlayers[$playerExists]);
+			}
+			else{
+				$winningPlayers[$i]['player_id'] = $grossWinner->player_id;
+				$winningPlayers[$i]['money'] = $grossWinner->money;
+			}
+			$i++;
+		}
 
-        $skinWinners = Skin::where('match_id', '=', $match->id)->get();
-        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){
-                $winningPlayers[$i]['player_id'] = $skinWinner->player_id;
-                $winningPlayers[$i]['money'] = ($skinWinner->money + $winningPlayers[$playerExists]['money']);
-                unset($winningPlayers[$playerExists]);
-            }
-            else{
-                $winningPlayers[$i]['player_id'] = $skinWinner->player_id;
-                $winningPlayers[$i]['money'] = $skinWinner->money;
-            }
-            $i++;
-        }
+		$skinWinners = Skin::where('match_id', '=', $match->id)->get();
+		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){
+				$winningPlayers[$i]['player_id'] = $skinWinner->player_id;
+				$winningPlayers[$i]['money'] = ($skinWinner->money + $winningPlayers[$playerExists]['money']);
+				unset($winningPlayers[$playerExists]);
+			}
+			else{
+				$winningPlayers[$i]['player_id'] = $skinWinner->player_id;
+				$winningPlayers[$i]['money'] = $skinWinner->money;
+			}
+			$i++;
+		}
 
-        //Consolidate any multiple instances of a player
-        foreach ($winningPlayers as $key => $winningPlayer){
-            array_search($winningPlayer['player_id'], $winningPlayers);
-        }
+		//Consolidate any multiple instances of a player
+		foreach ($winningPlayers as $key => $winningPlayer){
+			array_search($winningPlayer['player_id'], $winningPlayers);
+		}
 
-        foreach ($winningPlayers as $key => $player){
-            $currentPlayer = Player::find($player['player_id']);
-            $attributes = array(
-                "winnings" => $player['money']
-            );
-            $currentPlayer->matches()->updateExistingPivot($match->id, $attributes); //save match_player pivot data
-        }// End foreach
+		foreach ($winningPlayers as $key => $player){
+			$currentPlayer = Player::find($player['player_id']);
+			$attributes = array(
+				"winnings" => $player['money']
+			);
+			$currentPlayer->matches()->updateExistingPivot($match->id, $attributes); //save match_player pivot data
+		}// End foreach
 
 
-    }
+	}
 
-    /**
-     * Register the listeners for the subscriber.
-     *
-     * @param  Illuminate\Events\Dispatcher $events
-     * @return array
-     */
-    public function subscribe($events)
-    {
-        $events->listen('match.finalize', 'GolfLeague\Handlers\FinalizeHandler');
-    }
+	/**
+	 * Register the listeners for the subscriber.
+	 *
+	 * @param  Illuminate\Events\Dispatcher $events
+	 * @return array
+	 */
+	public function subscribe($events)
+	{
+		$events->listen('match.finalize', 'GolfLeague\Handlers\FinalizeHandler');
+	}
 
-    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)) {
-                return $current_key;
-            }
-        }
-        return false;
-    }
+	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)) {
+				return $current_key;
+			}
+		}
+		return false;
+	}
 }

Spacing +18 added lines, -18 removed lines

@@ -26,7 +26,7 @@ 
      */
     public function __construct(RoundRepository $roundRepo)
     {
-        $this->roundRepo= $roundRepo;
+        $this->roundRepo = $roundRepo;
     } // End of __construct
 
     /**
@@ -45,10 +45,10 @@ 
         $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]);
@@ -61,15 +61,15 @@ 
         }
 
         $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 +77,15 @@ 
         }
 
         $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 +93,15 @@ 
         }
 
         $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 +109,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,10 +135,10 @@ 
         $events->listen('match.finalize', 'GolfLeague\Handlers\FinalizeHandler');
     }
 
-    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)) {
+    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)) {
                 return $current_key;
             }
         }

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)

Indentation +43 added lines, -43 removed lines

@@ -14,47 +14,47 @@
  */
 class RoundHandler
 {
-    /**
-     * Create a new instance of the MatchHandler
-     *
-     * @param  GolfLeague\Storage\Round\RoundRepository $roundRepo
-     * @return void
-     */
-    public function __construct(HoleScoreRepository $holescoreRepo)
-    {
-        $this->holescoreRepo= $holescoreRepo;
-    } // End of __construct
-
-    /**
-     * Create an initial round for each player after a new match is created
-     *
-     * @param  Match $match
-     * @return void
-     */
-    public function handle($round)
-    {
-        $holes = Hole::where('course_id', '=', $round['course_id'])->get();
-        foreach($holes as $hole){
-            $input = array(
-                'score' => null,
-                'hole_id' => $hole->id,
-                'round_id' => $round['id']
-            );
-            $this->holescoreRepo->create($input);
-        }
-
-
-
-    }
-
-    /**
-     * Register the listeners for the subscriber.
-     *
-     * @param  Illuminate\Events\Dispatcher $events
-     * @return array
-     */
-    public function subscribe($events)
-    {
-        $events->listen('eloquent.created: Round', 'GolfLeague\Handlers\RoundHandler');
-    }
+	/**
+	 * Create a new instance of the MatchHandler
+	 *
+	 * @param  GolfLeague\Storage\Round\RoundRepository $roundRepo
+	 * @return void
+	 */
+	public function __construct(HoleScoreRepository $holescoreRepo)
+	{
+		$this->holescoreRepo= $holescoreRepo;
+	} // End of __construct
+
+	/**
+	 * Create an initial round for each player after a new match is created
+	 *
+	 * @param  Match $match
+	 * @return void
+	 */
+	public function handle($round)
+	{
+		$holes = Hole::where('course_id', '=', $round['course_id'])->get();
+		foreach($holes as $hole){
+			$input = array(
+				'score' => null,
+				'hole_id' => $hole->id,
+				'round_id' => $round['id']
+			);
+			$this->holescoreRepo->create($input);
+		}
+
+
+
+	}
+
+	/**
+	 * Register the listeners for the subscriber.
+	 *
+	 * @param  Illuminate\Events\Dispatcher $events
+	 * @return array
+	 */
+	public function subscribe($events)
+	{
+		$events->listen('eloquent.created: Round', 'GolfLeague\Handlers\RoundHandler');
+	}
 }

Spacing +2 added lines, -2 removed lines

@@ -22,7 +22,7 @@ 
      */
     public function __construct(HoleScoreRepository $holescoreRepo)
     {
-        $this->holescoreRepo= $holescoreRepo;
+        $this->holescoreRepo = $holescoreRepo;
     } // End of __construct
 
     /**
@@ -34,7 +34,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,

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)

Indentation +34 added lines, -34 removed lines

@@ -9,41 +9,41 @@
 */
 class MatchRoundService
 {
-    // Containing our pokemonRepository to make all our database calls to
-    protected $matchRepo;
+	// Containing our pokemonRepository to make all our database calls to
+	protected $matchRepo;
 
-    /**
-    * Loads our $matchRepo
-    *
-    * @param MatchRepository $matchRepo
-    * @return MatchService
-    */
-    public function __construct(MatchRoundRepository $matchRoundRepo, RoundRepository $roundRepo, MatchRepository $match)
-    {
-        $this->matchRoundRepo = $matchRoundRepo;
-        $this->roundRepo = $roundRepo;
-        $this->match = $match;
-    }
+	/**
+	 * Loads our $matchRepo
+	 *
+	 * @param MatchRepository $matchRepo
+	 * @return MatchService
+	 */
+	public function __construct(MatchRoundRepository $matchRoundRepo, RoundRepository $roundRepo, MatchRepository $match)
+	{
+		$this->matchRoundRepo = $matchRoundRepo;
+		$this->roundRepo = $roundRepo;
+		$this->match = $match;
+	}
 
-    public function all()
-    {
-        return $this->matchRoundRepo->all();
-    }
-    /**
-    * Method to get match data from input Match id
-    *
-    * @param mixed $matchdata
-    * @return JSON object
-    */
-    public function getByMatch($matchId)
-    {
-        return $this->matchRoundRepo->getByMatch($matchId);
-    }
+	public function all()
+	{
+		return $this->matchRoundRepo->all();
+	}
+	/**
+	 * Method to get match data from input Match id
+	 *
+	 * @param mixed $matchdata
+	 * @return JSON object
+	 */
+	public function getByMatch($matchId)
+	{
+		return $this->matchRoundRepo->getByMatch($matchId);
+	}
 
-    public function getByMatchAndGroup($matchdata)
-    {
-        //Retrieve players in group
-        $players = $this->matchRoundRepo->matchGroup($matchdata['match_id'], $matchdata['group']);
-        return $players;
-    }
+	public function getByMatchAndGroup($matchdata)
+	{
+		//Retrieve players in group
+		$players = $this->matchRoundRepo->matchGroup($matchdata['match_id'], $matchdata['group']);
+		return $players;
+	}
 }

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()

Unused Use Statements -2 removed lines

@@ -7,8 +7,6 @@
 use \Netwinner;
 use \Skin;
 
-use Illuminate\Events\Dispatcher;
-
 /**
 * MoneyService, containing all useful methods for business logic for money relating to a match
 */

Indentation +20 added lines, -20 removed lines

@@ -15,28 +15,28 @@
 class MoneyService
 {
 
-    // Containing our matchRepository to make all our database calls
-    protected $matchRepo;
+	// Containing our matchRepository to make all our database calls
+	protected $matchRepo;
 
-    /**
-    * Loads our $matchRepo
-    *
-    * @param MatchRepository $matchRepo
-    * @return MatchService
-    */
-    public function __construct()
-    {
+	/**
+	 * Loads our $matchRepo
+	 *
+	 * @param MatchRepository $matchRepo
+	 * @return MatchService
+	 */
+	public function __construct()
+	{
 
-    }
+	}
 
-    public function totalMatchMoney($matchId)
-    {
-        //get player id and money for ctp
-        $results['ctps'] =  Ctp::with('player')->where('match_id', '=', $matchId)->get();
-        $results['skins'] = Skin::with('player', 'hole')->where('match_id', '=', $matchId)->get();
-        $results['netwinner'] = Netwinner::with('player')->where('match_id', '=', $matchId)->get();
-        $results['grosswinner'] = Grosswinner::with('player')->where('match_id', '=', $matchId)->get();
-        $results['moneylist'] = Match::with('players')->where('id', '=', $matchId)->get();
+	public function totalMatchMoney($matchId)
+	{
+		//get player id and money for ctp
+		$results['ctps'] =  Ctp::with('player')->where('match_id', '=', $matchId)->get();
+		$results['skins'] = Skin::with('player', 'hole')->where('match_id', '=', $matchId)->get();
+		$results['netwinner'] = Netwinner::with('player')->where('match_id', '=', $matchId)->get();
+		$results['grosswinner'] = Grosswinner::with('player')->where('match_id', '=', $matchId)->get();
+		$results['moneylist'] = Match::with('players')->where('id', '=', $matchId)->get();
 		return $results;
-    }
+	}
 }

Spacing +1 added lines, -1 removed lines

@@ -32,7 +32,7 @@
     public function totalMatchMoney($matchId)
     {
         //get player id and money for ctp
-        $results['ctps'] =  Ctp::with('player')->where('match_id', '=', $matchId)->get();
+        $results['ctps'] = Ctp::with('player')->where('match_id', '=', $matchId)->get();
         $results['skins'] = Skin::with('player', 'hole')->where('match_id', '=', $matchId)->get();
         $results['netwinner'] = Netwinner::with('player')->where('match_id', '=', $matchId)->get();
         $results['grosswinner'] = Grosswinner::with('player')->where('match_id', '=', $matchId)->get();

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/Hole.php

Spacing +1 added lines, -1 removed lines

@@ -14,6 +14,6 @@
 	
 	public function courses()
     {
-        return $this->hasManyThrough('Course','Holescore');
+        return $this->hasManyThrough('Course', 'Holescore');
     }
 }

Indentation +11 added lines, -11 removed lines

@@ -2,20 +2,20 @@
 
 class Hole extends Eloquent 
 {
-    protected $fillable = ['course_id', 'number', 'par', 'yards', 'handicap'];
+	protected $fillable = ['course_id', 'number', 'par', 'yards', 'handicap'];
 
-    public function course()
-    {
-        return $this->belongsTo('Course');
-    }
+	public function course()
+	{
+		return $this->belongsTo('Course');
+	}
 	
 	public function holescores()
-    {
-        return $this->hasMany('Holescore');
-    }
+	{
+		return $this->hasMany('Holescore');
+	}
 	
 	public function courses()
-    {
-        return $this->hasManyThrough('Course','Holescore');
-    }
+	{
+		return $this->hasManyThrough('Course','Holescore');
+	}
 }

Braces +3 added lines, -4 removed lines

@@ -1,7 +1,6 @@ 
 <?php
 
-class Hole extends Eloquent 
-{
+class Hole extends Eloquent {
     protected $fillable = ['course_id', 'number', 'par', 'yards', 'handicap'];
 
     public function course()
@@ -10,12 +9,12 @@ 
     }
 	
 	public function holescores()
-    {
+	{
         return $this->hasMany('Holescore');
     }
 	
 	public function courses()
-    {
+	{
         return $this->hasManyThrough('Course','Holescore');
     }
 }

app/models/Single.php

Indentation +7 added lines, -7 removed lines

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

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