App\Domain\ExternLink\SummaryExternTrait

Complexity

Total Complexity

5

Size/Duplication

Total Lines	29
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	addSummaryLog()	0	4	1
A	tagAndLog()	0	13	4

<?php
/*
 * This file is part of dispositif/wikibot application (@github)
 * 2019-2023 © Philippe M./Irønie  <dispositif@gmail.com>
 * For the full copyright and MIT license information, view the license file.
 */

declare(strict_types=1);

namespace App\Domain\ExternLink;

use App\Domain\Models\Summary;
use Exception;

trait SummaryExternTrait
{
    /**
     * todo move
     *
     * @param array $mapData
     *
     * @throws Exception
     */
    protected function tagAndLog(array $mapData)
    {
        $this->log->debug('mapData', $mapData);
        $this->summary->citationNumber = $this->summary->citationNumber ?? 0;
        $this->summary->citationNumber++;

        if (!isset($this->summary->memo['sites'])
            || !in_array($this->externalPage->getPrettyDomainName(), $this->summary->memo['sites'])
        ) {
            $this->summary->memo['sites'][] = $this->externalPage->getPrettyDomainName(); // ???
        }
        if (isset($mapData['accès url'])) {
            $this->log->debug('accès 🔒 ' . $mapData['accès url']);
        }
    }

    protected function addSummaryLog(array $mapData, Summary $summary)
    {
        $this->summary = $summary;

The property summary does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

        $this->summaryLog[] = $mapData['site'] ?? $mapData['périodique'] ?? '?';

The property summaryLog does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

    }
}