HansOtt\Holiday\Calendar\Belgium\EasterMonday

Complexity

Total Complexity

2

Size/Duplication

Total Lines	17
Duplicated Lines	100 %

Coupling/Cohesion

Components	0
Dependencies	1

Test Coverage

Coverage

100%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	9	9	1
A	isOfficial()	4	4	1

<?php

namespace HansOtt\Holiday\Calendar\Belgium;

use HansOtt\Holiday\Calendar\Year;
use HansOtt\Holiday\HolidayAbstract;

final class EasterMonday extends HolidayAbstract

This class seems to be duplicated in your project.

{
    public function __construct(Year $year)
    {
        $startsAt = \HansOtt\Holiday\getEasterMonday($year);

        parent::__construct(
            $startsAt,
            \HansOtt\Holiday\getEndOfDay($startsAt)

It seems like \HansOtt\Holiday\getEndOfDay($startsAt) targeting HansOtt\Holiday\getEndOfDay() can also be of type false; however, HansOtt\Holiday\HolidayAbstract::__construct() does only seem to accept object<DateTimeImmutable>, did you maybe forget to handle an error condition?

        );
    }

    public function isOfficial()
    {
        return true;
    }
}