App\Parser\LeBonCoinParser

Complexity

Total Complexity

3

Size/Duplication

Total Lines	24
Duplicated Lines	0 %

Test Coverage

Coverage

57.14%

Importance

Changes	1
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	parsePhoto()	0	12	3

<?php

namespace App\Parser;

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

class LeBonCoinParser extends AbstractParser
{
    protected const PROVIDER = Provider::LEBONCOIN;

    protected const SELECTOR_AD_WRAPPER  = 'table[style*="border: solid 1px #e6ebef"]';
    protected const SELECTOR_TITLE       = 'td:nth-child(2) > a > span:first-of-type';
    protected const SELECTOR_LOCATION    = 'td:nth-child(2) > a > div > span';
    protected const SELECTOR_PHOTO       = 'td:nth-child(1) div';

    /**
     * {@inheritDoc}
     */
    protected function parsePhoto(Crawler $crawler): ?string
    {
        try {
            $backgroundImage = $crawler->filter(static::SELECTOR_PHOTO)->attr('style');

            if (1 === preg_match('/background-image: url\((.*)\)/', $backgroundImage, $matches)) {

It seems like $backgroundImage can also be of type null; however, parameter $subject of preg_match() does only seem to accept string, maybe add an additional type check?

                return $matches[1];
            }

            throw new ParseException('Error while parsing the photo: no property "background-image" found');
        } catch (Exception $e) {
            throw new ParseException('Error while parsing the photo: ' . $e->getMessage());
        }
    }
}