App\Domain\WikiOptimizer\Handlers\AuthorLinkHandler::handle()

Complexity

Conditions	4
Paths	3

Size

Total Lines	17
Code Lines	12

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

<?php
    public function handle()
    {
        $auteurParams = ['auteur1', 'auteur2', 'auteur2', 'titre'];
        foreach ($auteurParams as $auteurParam) {
            if ($this->ouvrage->hasParamValue($auteurParam)
                && $this->ouvrage->hasParamValue('lien '.$auteurParam)
            ) {
                $this->ouvrage->setParam(
                    $auteurParam,
                    WikiTextUtil::wikilink(
                        $this->ouvrage->getParam($auteurParam),

It seems like $this->ouvrage->getParam($auteurParam) can also be of type null; however, parameter $label of App\Domain\Utils\WikiTextUtil::wikilink() does only seem to accept string, maybe add an additional type check?

                        $this->ouvrage->getParam('lien '.$auteurParam)
                    )
                );
                $this->ouvrage->unsetParam('lien '.$auteurParam);
                $this->optiStatus->addSummaryLog('±lien '.$auteurParam);
                $this->optiStatus->setNotCosmetic(true);
            }
}