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/controllers/PlayersController.php

Doc Comments -1 removed lines

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

Indentation +51 added lines, -51 removed lines

@@ -4,10 +4,10 @@ 
 
 class PlayersController extends \BaseController {
 
-    public function __construct(PlayerRepo $player)
-    {
-        $this->player = $player;
-    }
+	public function __construct(PlayerRepo $player)
+	{
+		$this->player = $player;
+	}
 
 	/**
 	 * Get Data for all players
@@ -19,58 +19,58 @@ 
 		return $this->player->all();
 	}
 
-    /**
-     * Store a newly created resource in storage.
-     *
-     * @return Response
-     */
-    public function store()
-    {
+	/**
+	 * Store a newly created resource in storage.
+	 *
+	 * @return Response
+	 */
+	public function store()
+	{
 		$this->player->create(Input::get('name'), Input::get('handicap'));
-    }
+	}
 
-    /**
-     * Display the specified resource.
-     *
-     * @param  int  $id
-     * @return Response
-     */
-    public function show($id)
-    {
-        return $this->player->find($id);
-    }
+	/**
+	 * Display the specified resource.
+	 *
+	 * @param  int  $id
+	 * @return Response
+	 */
+	public function show($id)
+	{
+		return $this->player->find($id);
+	}
 
-    /**
-     * 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()
+	{
 
-    }
+	}
 
-    /**
-     * 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

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 +95 added lines, -95 removed lines

@@ -18,103 +18,103 @@
  */
 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 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;
-
-        $ctpWinners = Ctp::where('match_id', '=', $match->id)->get();
-        $winningPlayers = $this->gameWinner($ctpWinners, $winningPlayers);
-
-        $netWinners = Netwinner::where('match_id', '=', $match->id)->get();
-        $winningPlayers = $this->gameWinner($netWinners, $winningPlayers);
-
-        $grossWinners = Grosswinner::where('match_id', '=', $match->id)->get();
-        $winningPlayers = $this->gameWinner($grossWinners, $winningPlayers);
-
-        $skinWinners = Skin::where('match_id', '=', $match->id)->get();
-        $winningPlayers = $this->gameWinner($skinWinners, $winningPlayers);
-
-        //Consolidate money for any multiple instances of a player
-        foreach ($winningPlayers as $key => $winningPlayer){
-            array_search($winningPlayer['player_id'], $winningPlayers);
-        }
-
-        // update pivot table with winners data
-        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');
-    }
-
-    /*
+	/**
+	 * 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
+
+		$winningPlayers = array();
+		$i = 1;
+
+		$ctpWinners = Ctp::where('match_id', '=', $match->id)->get();
+		$winningPlayers = $this->gameWinner($ctpWinners, $winningPlayers);
+
+		$netWinners = Netwinner::where('match_id', '=', $match->id)->get();
+		$winningPlayers = $this->gameWinner($netWinners, $winningPlayers);
+
+		$grossWinners = Grosswinner::where('match_id', '=', $match->id)->get();
+		$winningPlayers = $this->gameWinner($grossWinners, $winningPlayers);
+
+		$skinWinners = Skin::where('match_id', '=', $match->id)->get();
+		$winningPlayers = $this->gameWinner($skinWinners, $winningPlayers);
+
+		//Consolidate money for any multiple instances of a player
+		foreach ($winningPlayers as $key => $winningPlayer){
+			array_search($winningPlayer['player_id'], $winningPlayers);
+		}
+
+		// update pivot table with winners data
+		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');
+	}
+
+	/*
      * Takes array of
      * @param array $gameWinner
      * @param array $winningPlayers
      */
-    public function gameWinner($gameWinners, $winningPlayers)
-    {
-        foreach($gameWinners as $key => $gameWinner){
-            $playerExists = $this->recursive_array_search($gameWinner->player_id, $winningPlayers);
-            //if player is already there add money here to other money
-            if($playerExists){
-                $winningPlayers[$i]['player_id'] = $gameWinner->player_id;
-                $winningPlayers[$i]['money'] = ($gameWinner->money + $winningPlayers[$playerExists]['money']);
-                unset($winningPlayers[$playerExists]);
-            }
-            else {
-                $winningPlayers[$i]['player_id'] = $gameWinner->player_id;
-                $winningPlayers[$i]['money'] = $gameWinner->money;
-            }
-            $i++;
-        }
-        return $winningPlayers;
-    }
-
-    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 gameWinner($gameWinners, $winningPlayers)
+	{
+		foreach($gameWinners as $key => $gameWinner){
+			$playerExists = $this->recursive_array_search($gameWinner->player_id, $winningPlayers);
+			//if player is already there add money here to other money
+			if($playerExists){
+				$winningPlayers[$i]['player_id'] = $gameWinner->player_id;
+				$winningPlayers[$i]['money'] = ($gameWinner->money + $winningPlayers[$playerExists]['money']);
+				unset($winningPlayers[$playerExists]);
+			}
+			else {
+				$winningPlayers[$i]['player_id'] = $gameWinner->player_id;
+				$winningPlayers[$i]['money'] = $gameWinner->money;
+			}
+			$i++;
+		}
+		return $winningPlayers;
+	}
+
+	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 +9 added lines, -9 removed lines

@@ -26,7 +26,7 @@ 
      */
     public function __construct(RoundRepository $roundRepo)
     {
-        $this->roundRepo= $roundRepo;
+        $this->roundRepo = $roundRepo;
     } // End of __construct
 
     /**
@@ -57,12 +57,12 @@ 
         $winningPlayers = $this->gameWinner($skinWinners, $winningPlayers);
 
         //Consolidate money for any multiple instances of a player
-        foreach ($winningPlayers as $key => $winningPlayer){
+        foreach ($winningPlayers as $key => $winningPlayer) {
             array_search($winningPlayer['player_id'], $winningPlayers);
         }
 
         // update pivot table with winners data
-        foreach ($winningPlayers as $key => $player){
+        foreach ($winningPlayers as $key => $player) {
             $currentPlayer = Player::find($player['player_id']);
             $attributes = array(
                 "winnings" => $player['money']
@@ -91,10 +91,10 @@ 
      */
     public function gameWinner($gameWinners, $winningPlayers)
     {
-        foreach($gameWinners as $key => $gameWinner){
+        foreach ($gameWinners as $key => $gameWinner) {
             $playerExists = $this->recursive_array_search($gameWinner->player_id, $winningPlayers);
             //if player is already there add money here to other money
-            if($playerExists){
+            if ($playerExists) {
                 $winningPlayers[$i]['player_id'] = $gameWinner->player_id;
                 $winningPlayers[$i]['money'] = ($gameWinner->money + $winningPlayers[$playerExists]['money']);
                 unset($winningPlayers[$playerExists]);
@@ -108,10 +108,10 @@ 
         return $winningPlayers;
     }
 
-    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 +8 added lines, -9 removed lines

@@ -16,8 +16,7 @@ 
  *
  * @author          Michael Schmidt
  */
-class FinalizeHandler
-{
+class FinalizeHandler {
     /**
      * Create a new instance of the MatchHandler
      *
@@ -57,12 +56,12 @@ 
         $winningPlayers = $this->gameWinner($skinWinners, $winningPlayers);
 
         //Consolidate money for any multiple instances of a player
-        foreach ($winningPlayers as $key => $winningPlayer){
+        foreach ($winningPlayers as $key => $winningPlayer) {
             array_search($winningPlayer['player_id'], $winningPlayers);
         }
 
         // update pivot table with winners data
-        foreach ($winningPlayers as $key => $player){
+        foreach ($winningPlayers as $key => $player) {
             $currentPlayer = Player::find($player['player_id']);
             $attributes = array(
                 "winnings" => $player['money']
@@ -91,15 +90,14 @@ 
      */
     public function gameWinner($gameWinners, $winningPlayers)
     {
-        foreach($gameWinners as $key => $gameWinner){
+        foreach($gameWinners as $key => $gameWinner) {
             $playerExists = $this->recursive_array_search($gameWinner->player_id, $winningPlayers);
             //if player is already there add money here to other money
-            if($playerExists){
+            if($playerExists) {
                 $winningPlayers[$i]['player_id'] = $gameWinner->player_id;
                 $winningPlayers[$i]['money'] = ($gameWinner->money + $winningPlayers[$playerExists]['money']);
                 unset($winningPlayers[$playerExists]);
-            }
-            else {
+            } else {
                 $winningPlayers[$i]['player_id'] = $gameWinner->player_id;
                 $winningPlayers[$i]['money'] = $gameWinner->money;
             }
@@ -108,7 +106,8 @@ 
         return $winningPlayers;
     }
 
-    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/HoleScoreHandler.php

Doc Comments +1 added lines, -2 removed lines

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

Indentation +41 added lines, -41 removed lines

@@ -15,47 +15,47 @@
  */
 class HoleScoreHandler
 {
-    /**
-     * Create a new instance of the MatchHandler
-     *
-     * @param  GolfLeague\Storage\Round\RoundRepository $roundRepo
-     * @return void
-     */
-    public function __construct(RoundRepository $roundRepo, HoleScoreRepository $holescoreRepo)
-    {
-        $this->roundRepo= $roundRepo;
-        $this->holescoreRepo = $holescoreRepo;
-    } // End of __construct
+	/**
+	 * Create a new instance of the MatchHandler
+	 *
+	 * @param  GolfLeague\Storage\Round\RoundRepository $roundRepo
+	 * @return void
+	 */
+	public function __construct(RoundRepository $roundRepo, HoleScoreRepository $holescoreRepo)
+	{
+		$this->roundRepo= $roundRepo;
+		$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($holescore)
-    {
-        // Update score and esc in round table
-        $holescores = $this->holescoreRepo->getByRound($holescore->round_id);
-        $escTotal = 0;
-        foreach($holescores as $hole) {
-            $esc = new EquitableStrokeControl($hole);
-            $escTotal += $esc->calculate();
-        }
-        $round = $this->roundRepo->find($holescore->round_id);
-        $round->score = $holescores->sum('score');
-        $round->esc = $escTotal;
-        $this->roundRepo->update($round);
-    }
+	/**
+	 * Create an initial round for each player after a new match is created
+	 *
+	 * @param  Match $match
+	 * @return void
+	 */
+	public function handle($holescore)
+	{
+		// Update score and esc in round table
+		$holescores = $this->holescoreRepo->getByRound($holescore->round_id);
+		$escTotal = 0;
+		foreach($holescores as $hole) {
+			$esc = new EquitableStrokeControl($hole);
+			$escTotal += $esc->calculate();
+		}
+		$round = $this->roundRepo->find($holescore->round_id);
+		$round->score = $holescores->sum('score');
+		$round->esc = $escTotal;
+		$this->roundRepo->update($round);
+	}
 
-    /**
-     * Register the listeners for the subscriber.
-     *
-     * @param  Illuminate\Events\Dispatcher $events
-     * @return array
-     */
-    public function subscribe($events)
-    {
-        $events->listen('eloquent.updated: Holescore', 'GolfLeague\Handlers\HoleScoreHandler');
-    }
+	/**
+	 * Register the listeners for the subscriber.
+	 *
+	 * @param  Illuminate\Events\Dispatcher $events
+	 * @return array
+	 */
+	public function subscribe($events)
+	{
+		$events->listen('eloquent.updated: Holescore', 'GolfLeague\Handlers\HoleScoreHandler');
+	}
 }

Spacing +2 added lines, -2 removed lines

@@ -23,7 +23,7 @@ 
      */
     public function __construct(RoundRepository $roundRepo, HoleScoreRepository $holescoreRepo)
     {
-        $this->roundRepo= $roundRepo;
+        $this->roundRepo = $roundRepo;
         $this->holescoreRepo = $holescoreRepo;
     } // End of __construct
 
@@ -38,7 +38,7 @@ 
         // Update score and esc in round table
         $holescores = $this->holescoreRepo->getByRound($holescore->round_id);
         $escTotal = 0;
-        foreach($holescores as $hole) {
+        foreach ($holescores as $hole) {
             $esc = new EquitableStrokeControl($hole);
             $escTotal += $esc->calculate();
         }

Braces +1 added lines, -2 removed lines

@@ -13,8 +13,7 @@
  *
  * @author          Michael Schmidt
  */
-class HoleScoreHandler
-{
+class HoleScoreHandler {
     /**
      * Create a new instance of the MatchHandler
      *

app/library/Handlers/MatchHandler.php

Doc Comments +1 added lines, -1 removed lines

@@ -17,7 +17,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 +42 added lines, -42 removed lines

@@ -14,51 +14,51 @@
  */
 class MatchHandler
 {
-    /**
-     * 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 each player create an initial round
-        //var_dump($holescore);
-        $input = array(
-        'date' => $match['date'],
-        // use holescore->round_id to get all holescores for that round
-         'course_id' => $match['course'],
+	/**
+	 * Create an initial round for each player after a new match is created
+	 *
+	 * @param  Match $match
+	 * @return void
+	 */
+	public function handle($match)
+	{
+		//for each player create an initial round
+		//var_dump($holescore);
+		$input = array(
+		'date' => $match['date'],
+		// use holescore->round_id to get all holescores for that round
+		 'course_id' => $match['course'],
 
-         'match_id' => $match['match_id'],
-         'score' => 0,
-         'esc' =>0
-         );
-         foreach($match['player'] as $player){
-         $input['player_id'] = $player['player_id'];
-         $this->roundRepo->create($input);
+		 'match_id' => $match['match_id'],
+		 'score' => 0,
+		 'esc' =>0
+		 );
+		 foreach($match['player'] as $player){
+		 $input['player_id'] = $player['player_id'];
+		 $this->roundRepo->create($input);
  }
 
-    }
+	}
 
-    /**
-     * Register the listeners for the subscriber.
-     *
-     * @param  Illuminate\Events\Dispatcher $events
-     * @return array
-     */
-    public function subscribe($events)
-    {
-        $events->listen('match.create', 'GolfLeague\Handlers\MatchHandler');
-    }
+	/**
+	 * Register the listeners for the subscriber.
+	 *
+	 * @param  Illuminate\Events\Dispatcher $events
+	 * @return array
+	 */
+	public function subscribe($events)
+	{
+		$events->listen('match.create', 'GolfLeague\Handlers\MatchHandler');
+	}
 }

Spacing +2 added lines, -2 removed lines

@@ -22,7 +22,7 @@ 
      */
     public function __construct(RoundRepository $roundRepo)
     {
-        $this->roundRepo= $roundRepo;
+        $this->roundRepo = $roundRepo;
     } // End of __construct
 
     /**
@@ -44,7 +44,7 @@ 
          'score' => 0,
          'esc' =>0
          );
-         foreach($match['player'] as $player){
+         foreach ($match['player'] as $player) {
          $input['player_id'] = $player['player_id'];
          $this->roundRepo->create($input);
  }

Braces +2 added lines, -3 removed lines

@@ -12,8 +12,7 @@ 
  *
  * @author          Michael Schmidt
  */
-class MatchHandler
-{
+class MatchHandler {
     /**
      * Create a new instance of the MatchHandler
      *
@@ -44,7 +43,7 @@ 
          'score' => 0,
          'esc' =>0
          );
-         foreach($match['player'] as $player){
+         foreach($match['player'] as $player) {
          $input['player_id'] = $player['player_id'];
          $this->roundRepo->create($input);
  }

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;