Wikibase\Api\Service\LabelSetter

Complexity

Total Complexity

16

Size/Duplication

Total Lines	91
Duplicated Lines	100 %

Coupling/Cohesion

Components	1
Dependencies	4

Test Coverage

Coverage

0%

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	3	3	1
A	set()	13	13	1
B	throwExceptionsOnBadTarget()	8	8	8
A	getEntityIdentifierFromTarget()	7	7	3
A	getTargetParamsFromTarget()	12	12	3

<?php

namespace Wikibase\Api\Service;

use Mediawiki\DataModel\EditInfo;
use UnexpectedValueException;
use Wikibase\Api\WikibaseApi;
use Wikibase\DataModel\Entity\EntityId;
use Wikibase\DataModel\Entity\Item;
use Wikibase\DataModel\Entity\Property;
use Wikibase\DataModel\SiteLink;
use Wikibase\DataModel\Term\Term;

/**
 * @access private
 *
 * @author Addshore
 */
class LabelSetter {

This class seems to be duplicated in your project.

	/**
	 * @var WikibaseApi
	 */
	private $api;

	/**
	 * @param WikibaseApi $api
	 */
	public function __construct( WikibaseApi $api ) {
		$this->api = $api;
	}

	/**
	 * @since 0.2
	 *
	 * @param Term $label
	 * @param EntityId|Item|Property|SiteLink $target
	 * @param EditInfo|null $editInfo
	 *
	 * @return bool
	 */
	public function set( Term $label, $target, EditInfo $editInfo = null ) {
		$this->throwExceptionsOnBadTarget( $target );

		$params = $this->getTargetParamsFromTarget(
			$this->getEntityIdentifierFromTarget( $target )

It seems like $target defined by parameter $target on line 42 can also be of type object<Wikibase\DataModel\SiteLink>; however, Wikibase\Api\Service\LabelSetter::getEntityIdentifierFromTarget() does only seem to accept object<Wikibase\DataModel\Entity\EntityId>|object<Wikibase\DataModel\Entity\Item>|object<Wikibase\DataModel\Entity\Property>, maybe add an additional type check?

		);

		$params['language'] = $label->getLanguageCode();
		$params['value'] = $label->getText();

		$this->api->postRequest( 'wbsetlabel', $params, $editInfo );
		return true;
	}

	/**
	 * @param mixed $target
	 *
	 * @throws UnexpectedValueException
	 *
	 * @todo Fix duplicated code
	 */
	private function throwExceptionsOnBadTarget( $target ) {
		if ( !$target instanceof EntityId && !$target instanceof Item && !$target instanceof Property && !$target instanceof SiteLink ) {
			throw new UnexpectedValueException( '$target needs to be an EntityId, Item, Property or SiteLink' );
		}
		if ( ( $target instanceof Item || $target instanceof Property ) && $target->getId() === null ) {
			throw new UnexpectedValueException( '$target Entity object needs to have an Id set' );
		}
	}

	/**
	 * @param EntityId|Item|Property $target
	 *
	 * @throws UnexpectedValueException
	 * @return EntityId|SiteLink
	 *
	 * @todo Fix duplicated code
	 */
	private function getEntityIdentifierFromTarget( $target ) {
		if ( $target instanceof Item || $target instanceof Property ) {
			return $target->getId();
		} else {
			return $target;
		}
	}

	/**
	 * @param EntityId|SiteLink $target
	 *
	 * @throws UnexpectedValueException
	 * @return array
	 *
	 * @todo Fix duplicated code
	 */
	private function getTargetParamsFromTarget( $target ) {
		if ( $target instanceof EntityId ) {
			return [ 'id' => $target->getSerialization() ];
		} elseif ( $target instanceof SiteLink ) {
			return [
				'site' => $target->getSiteId(),
				'title' => $target->getPageName(),
			];
		} else {
			throw new UnexpectedValueException( '$target needs to be an EntityId or SiteLink' );
		}
	}

}