App\Parser\SeLogerParser

Complexity

Total Complexity

2

Size/Duplication

Total Lines	19
Duplicated Lines	0 %

Test Coverage

Coverage

80%

Importance

Changes	2
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	parseLocation()	0	9	2

<?php

namespace App\Parser;

use App\Enum\Provider;
use Symfony\Component\DomCrawler\Crawler;

class SeLogerParser extends AbstractParser
{
    protected const PROVIDER = Provider::SELOGER;

    protected const SELECTOR_AD_WRAPPER  = '.two-column > a[_category="Bloc annonce"]'; // Ignore "Recommendation" ads
    protected const SELECTOR_LOCATION    = '.contents tr:nth-child(2) a';

    /**
     * {@inheritDoc}
     */
    protected function parseLocation(Crawler $crawler): ?string
    {
        $description = parent::parseLocation($crawler);

Are you sure the assignment to $description is correct as parent::parseLocation($crawler) targeting App\Parser\AbstractParser::parseLocation() seems to always return null.

        if (1 === preg_match('/m²\s([-\w\s]+\s\(\d+\))/u', $description,$matches)) {

$description of type null is incompatible with the type string expected by parameter $subject of preg_match().

            return trim($matches[1]);
        }

        return null;
    }
}