GolfLeague\Storage\MatchRound\EloquentMatchRoundRepository

Complexity

Total Complexity

8

Size/Duplication

Total Lines	53
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	0

Importance

Changes	7
Bugs	0	Features	0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	all()	0	4	1
A	getByMatch()	0	4	1
A	getMatchPlayerHandicap()	0	8	2
A	create()	0	4	1
A	matchGroup()	0	9	2
A	getMatchData()	0	5	1

<?php namespace GolfLeague\Storage\MatchRound;

use \Match as Match;
use \Round as Round;
use \Player as Player;

class EloquentMatchRoundRepository implements MatchRoundRepository
{
  /*Return Score collections that include:
   * Player Name
   * Date
   * Total
   * Course
   * Multideminsional Array of Hole Numbers and scores
   **/

   //Retrieve all rounds associated with a match with holescores
    public function all()
    {
        return Round::with('player', 'holescores')->whereNotNull('match_id')->get();
    }

    // Retrieve Rounds of a match with holescores
    // Return data is sorted by the order of the holes of the round
    public function getByMatch($matchid)
    {
		return Round::with('player', 'holescores')->where('match_id', '=', $matchid)->get();
    }

    public function create($matchdata)
    {

    }

	public function matchGroup($matchid, $group)
	{
        $players = Match::find($matchid)->players()->where('match_player.group','=',$group)->get();
		foreach($players as $key => $player) {
			$rounds = Round::with('holescores', 'course.holes')->where('player_id', '=', $player->id)->where('match_id', '=', $matchid)->get();
            $players[$key]['round'] = $rounds;
		}
		return $players;
	}

    public function getMatchData($matchid)
    {
        $match = Match::find($matchid)->players()->where('match_player.match_id','=', $matchid)->get();
		return $match;
    }

	public function getMatchPlayerHandicap($matchid, $playerId)
	{
		$players = Player::find($playerId)->matches()->where('match_player.match_id','=', $matchid)->get();
		foreach($players as $player){
			$handicap = $player['pivot']['handicap'];
		}
		return $handicap;

The variable $handicap does not seem to be defined for all execution paths leading up to this point.

	}
}