GolfLeague\Handlers\RoundHandler

Complexity

Total Complexity

4

Size/Duplication

Total Lines	46
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	0

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	handle()	0	15	2
A	subscribe()	0	4	1

<?php

namespace GolfLeague\Handlers;

use GolfLeague\Storage\HoleScore\HoleScoreRepository;
use \Hole;

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

There is no parameter named $roundRepo. Was it maybe removed?

     * @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(HoleScoreRepository $holescoreRepo)
    {
        $this->holescoreRepo= $holescoreRepo;

The property holescoreRepo 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

There is no parameter named $match. Was it maybe removed?

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