Scheduler\Application\Service\GetShiftsInTimePeriod

Complexity

Total Complexity

7

Size/Duplication

Total Lines	49
Duplicated Lines	12.24 %

Coupling/Cohesion

Components	0
Dependencies	3

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
B	__invoke()	6	35	6

<?php

namespace Scheduler\Application\Service;

use DateTime;
use Aura\Payload\Payload;
use Scheduler\Domain\Model\Shift\ShiftMapper;
use Scheduler\Domain\Model\User\User;

class GetShiftsInTimePeriod
{
    const INVALID_DATE_MESSAGE = "must be a valid string representation of a date";

    private $payload;
    private $shiftMapper;

    public function __construct(ShiftMapper $shiftMapper)
    {
        $this->payload = new Payload();
        $this->shiftMapper = $shiftMapper;
    }

    public function __invoke(User $user, $start, $end)
    {
        if (! $user->isAuthenticated()) {
            return $this->payload->setStatus(Payload::NOT_AUTHENTICATED);
        }

        if ($user->getRole() !== "manager") {
            return $this->payload->setStatus(Payload::NOT_AUTHORIZED);
        }

        $invalid = [];
        try {
            $start = new DateTime($start);
        } catch (\Exception $e) {
            $invalid["start"] = self::INVALID_DATE_MESSAGE;
        }

        try {
            $end = new DateTime($end);
        } catch (\Exception $e) {
            $invalid["end"] = self::INVALID_DATE_MESSAGE;
        }

        if (count($invalid)) {

This code seems to be duplicated across your project.

            return $this->payload
                            ->setStatus(Payload::NOT_VALID)
                            ->setInput([$start, $end])
                            ->setMessages($invalid);
        }

        $shifts = $this->shiftMapper->findShiftsInTimePeriod($start, $end);

        return $this->payload->setStatus(Payload::SUCCESS)
                             ->setOutput($shifts);
    }
}