Wikibase\DataModel\Services\Diff\FingerprintPatcher

Complexity

Total Complexity

6

Size/Duplication

Total Lines	53
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	5

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	patchFingerprint()	0	26	5

<?php

namespace Wikibase\DataModel\Services\Diff;

use Diff\DiffOp\Diff\Diff;
use Diff\Patcher\PatcherException;
use Wikibase\DataModel\Services\Diff\Internal\AliasGroupListPatcher;
use Wikibase\DataModel\Term\Fingerprint;

/**
 * @since 1.0
 *
 * @license GPL-2.0-or-later
 * @author Jeroen De Dauw < jeroendedauw@gmail.com >
 * @author Thiemo Kreuz
 */
class FingerprintPatcher {

	/**
	 * @var TermListPatcher
	 */
	private $termListPatcher;

	/**
	 * @var AliasGroupListPatcher
	 */
	private $aliasGroupListPatcher;

	public function __construct() {
		$this->termListPatcher = new TermListPatcher();
		$this->aliasGroupListPatcher = new AliasGroupListPatcher();
	}

	/**
	 * @since 1.0
	 *
	 * @param Fingerprint $fingerprint
	 * @param Diff $patch typically an {@link EntityDiff}
	 *
	 * @throws PatcherException
	 */
	public function patchFingerprint( Fingerprint $fingerprint, Diff $patch ) {
		if ( !$patch->isAssociative() ) {

The expression $patch->isAssociative() of type boolean|null is loosely compared to false; this is ambiguous if the boolean can be false. You might want to explicitly use !== null instead.

			throw new PatcherException( '$patch must be associative' );
		}

		if ( isset( $patch['label'] ) ) {
			$this->termListPatcher->patchTermList(
				$fingerprint->getLabels(),
				$patch['label']
			);
		}

		if ( isset( $patch['description'] ) ) {
			$this->termListPatcher->patchTermList(
				$fingerprint->getDescriptions(),
				$patch['description']
			);
		}

		if ( isset( $patch['aliases'] ) ) {
			$this->aliasGroupListPatcher->patchAliasGroupList(
				$fingerprint->getAliasGroups(),
				$patch['aliases']
			);
		}
	}

}