Wikibase\DataModel\Services\Lookup\LegacyAdapterPropertyLookup

Complexity

Total Complexity

3

Size/Duplication

Total Lines	19
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	getPropertyForId()	0	5	2
A	__construct()	0	2	1

<?php

namespace Wikibase\DataModel\Services\Lookup;

use Wikibase\DataModel\Entity\Property;
use Wikibase\DataModel\Entity\PropertyId;

/**
 * PropertyLookup implementation providing a migration path away from
 * the EntityLookup interface.
 *
 * @license GPL-2.0-or-later
 * @author Jeroen De Dauw < jeroendedauw@gmail.com >
 */
class LegacyAdapterPropertyLookup implements PropertyLookup {

	private EntityLookup $lookup;

	public function __construct( EntityLookup $lookup ) {
		$this->lookup = $lookup;
	}

	/**
	 * @param PropertyId $propertyId
	 *
	 * @return Property|null
	 * @throws PropertyLookupException
	 */
	public function getPropertyForId( PropertyId $propertyId ) {
		try {
			return $this->lookup->getEntity( $propertyId );

The expression return $this->lookup->getEntity($propertyId) also could return the type Wikibase\DataModel\Entity\EntityDocument which includes types incompatible with the return type mandated by Wikibase\DataModel\Services\Lookup\PropertyLookup::getPropertyForId() of Wikibase\DataModel\Entity\Property|null. Consider adding a type-check to rule them out.

		} catch ( EntityLookupException $ex ) {
			throw new PropertyLookupException( $propertyId, $ex->getMessage(), $ex );
		}
	}

}