Bouss / boussimmo

src/Parser/SuperimmoNeufParser.php

<?php

namespace App\Parser;

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

class SuperimmoNeufParser extends AbstractParser
{
    protected const PROVIDER = Provider::SUPERIMMO_NEUF;

    protected const SELECTOR_AD_WRAPPER  = 'td[style="width: 540px;"]';
    protected const SELECTOR_LOCATION    = 'table:nth-child(2) tr:nth-child(2) span';
    protected const SELECTOR_TITLE       = 'table:nth-child(2) tr:nth-child(1) span';
    protected const SELECTOR_DESCRIPTION = 'table:nth-child(2) tr:nth-child(5) span';

    /**
     * {@inheritDoc}
     */
    protected function parsePhoto(Crawler $crawler): ?string
    {
        $photo = parent::parsePhoto($crawler);
        $photo = substr($photo, strpos($photo, '#'));

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

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

        return str_replace('wide', 'biggest', $photo);
    }
}