GolfLeague\Storage\Team\EloquentTeamRepository

Complexity

Total Complexity

6

Size/Duplication

Total Lines	36
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	0

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	all()	0	4	1
A	findById()	0	4	1
A	findByYear()	0	4	1
A	create()	0	4	1
A	update()	0	6	1

<?php namespace GolfLeague\Storage\Team;

use \Team as Team;

class EloquentTeamRepository implements TeamRepository
{
    private $team;

    public function __construct(Team $team)
    {
        $this->team = $team;
    }

    public function all()
    {
        return $this->team->all();
    }
    public function findById($id)
    {
        return $this->team->where('id', '=', $id)->get();
    }

    public function findByYear($year)
    {
        return $this->team->whereYear('created_at', '=', $year)->get();
    }

    public function create($input)
    {
        $this->team->create($input);
    }

    public function update($team)
    {
        $updateTeam = $this->team->where('id', '=', $id)->get();

The variable $id does not exist. Did you forget to declare it?

$updateTeam is not used, you could remove the assignment.

        $updateTeam = $team;
        $updateTeam->save();
    }

}