GolfLeague\Handlers\MatchHandler

Complexity

Total Complexity

5

Size/Duplication

Total Lines	52
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	0

Importance

Changes	2
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	handle()	0	21	3
A	subscribe()	0	4	1

<?php

namespace GolfLeague\Handlers;

use GolfLeague\Storage\Round\RoundRepository;


/**
 * MatchHandler Connection Class
 *
 * This class subscribes to events in related to match creation
 *
 * @author          Michael Schmidt
 */
class MatchHandler
{
    /**
     * Create a new instance of the MatchHandler
     *
     * @param  GolfLeague\Storage\Round\RoundRepository $roundRepo
     * @return void

Adding a @return annotation to constructors is generally not recommended as a constructor does not have a meaningful return value.

     */
    public function __construct(RoundRepository $roundRepo)
    {
        $this->roundRepo= $roundRepo;

The property roundRepo does not exist. Did you maybe forget to declare it?

    } // 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
        $input = array(
        'date' => $match['date'],
         'course_id' => $match['course'],

         'match_id' => $match['match_id'],
         'score' => 0,
         'esc' => 0
         );
         foreach($match['player'] as $player){
             $input['player_id'] = $player['player_id'];
             //If team match then add team id to $input array
             if(isset($player['team'])){
                 $input['team_id'] = $player['team'];
             }
             $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');
    }
}