Andreshg112\HolidaysPhp\Providers\Colombia\CalendarioHispanohablanteCom::holidays()

Complexity

Conditions	5
Paths	5

Size

Total Lines	55
Code Lines	27

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	1	Features	0

<?php
    public function holidays(int $year = null): ?array
    {
        $year = $year ?? date('Y');

        $baseUrl = $this->baseUrl();

        $url = "{$baseUrl}/{$year}/calendario-colombia-{$year}.html";

        try {
            $html = file_get_contents($url);
        } catch (\Throwable $th) {
            throw HolidaysPhpException::notFound();
        }

        $page = new HtmlPage($html);

        $pList = $page->filter('.group-three > p');

        if ($pList->count() === 0) {
            throw HolidaysPhpException::unrecognizedStructure();
        }

        $holidays = [];

        $country = $this->country();

        /** @var \DOMElement */
        foreach ($pList as $pItem) {
            $pChildren = $pItem->childNodes;

            // It must contain 2 nodes, else, it must be something different than a date
            if ($pChildren->count() !== 2) {
                continue;
            }

            // The text is like "Viernes 1 de Enero", so it has to be separated
            [$day, $spanishDate] = explode(' ', trim($pChildren->item(0)->textContent), 2);

            // The ending : has to be removed
            $spanishDate = rtrim($spanishDate, ':');

            Date::setLocale($this->getLanguage());

            // The date looks this way "1 de Enero 2021"
            $date = Date::createFromFormat('j \d\e F Y', "{$spanishDate} {$year}");

            $holidays[] = new Holiday(
                $country,
                $date->setTime(0, 0),

It seems like $date->setTime(0, 0) can also be of type false; however, parameter $date of Andreshg112\HolidaysPhp\Holiday::__construct() does only seem to accept Jenssegers\Date\Date, maybe add an additional type check?

                trim($pChildren->item(1)->textContent), // title
                $this->getLanguage()
            );
        }

        return $holidays;
    }