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

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

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;
 
 	}

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)

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;
             }
         }

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)

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,

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)

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

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

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');
     }
 }

app/models/Player.php

Spacing +1 added lines, -1 removed lines

@@ -10,7 +10,7 @@
 
 	public function holescores()
 	{
-		return $this->hasManyThrough('Holescore','Round');
+		return $this->hasManyThrough('Holescore', 'Round');
 	}
 
 	public function skins()