Callingallpapers\Subcommands\Sessionize\Parser\Location::parse()

Complexity

Conditions	7
Paths	5

Size

Total Lines

28

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    public function parse(DOMDocument $dom, DOMXPath $xpath) : string

The parameter $dom is not used and could be removed.

    {
        // This expression does not work. It looks like the reason is the array-notation...
        //$locations = $xpath->query('//div[contains(text()[2], "location")]/following-sibling::h2/span');
        $locationMarker = $xpath->query("//div[contains(., 'location')]");
        if (! $locationMarker || $locationMarker->length == 0) {
            throw new \InvalidArgumentException('The Event does not seem to have a locationMarker');
        }

        $locations = $xpath->query('//h2/span', $locationMarker->item(0)->parentNode);

        if (! $locations || $locations->length == 0) {
            throw new \InvalidArgumentException('The Event does not seem to have a location');
        }
        $location = [];
        foreach ($locations as $item) {
            $location = trim($item->textContent);
            if ($location === '') {
                continue;
            }

            return $location;

            $location[] = $location;

$location[] = $location; does not seem to be reachable.

        }

        return implode(', ', array_unique($location));
    }
}