SMW\ApprovedRevs\PropertyAnnotators\ApprovedByPropertyAnnotator

Complexity

Total Complexity

9

Size/Duplication

Total Lines	74
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	4

Test Coverage

Coverage

84.62%

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	3	1
A	setApprovedBy()	0	3	1
A	newDIProperty()	0	3	1
A	addAnnotation()	0	17	3
A	newDIWikiPage()	0	12	3

<?php

namespace SMW\ApprovedRevs\PropertyAnnotators;

use SMW\DIProperty;
use SMW\SemanticData;
use SMW\DIWikiPage;
use SMW\ApprovedRevs\DatabaseLogReader;
use SMW\ApprovedRevs\PropertyRegistry;
use User;
use Title;

/**
 * @private
 *
 * @license GNU GPL v2+
 */
class ApprovedByPropertyAnnotator {

	/**
	 * @var DatabaseLogReader
	 */
	private $databaseLogReader;

	/**
	 * @var Integer|null
	 */
	private $approvedBy;

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

	/**
	 * @since 1.0
	 *
	 * @param string $approvedBy
	 */
	public function setApprovedBy( $approvedBy ) {
		$this->approvedBy = $approvedBy;

It seems like $approvedBy of type string is incompatible with the declared type integer|null of property $approvedBy.

	}

	/**
	 * @since 1.0
	 *
	 * @return DIProperty
	 */
	public function newDIProperty() {
		return new DIProperty( PropertyRegistry::SAR_PROP_APPROVED_BY );
	}

	/**
	 * @since 1.0
	 *
	 * @param SemanticData $semanticData
	 */
	public function addAnnotation( SemanticData $semanticData ) {

		if ( $this->approvedBy === null ) {
			$this->approvedBy = $this->databaseLogReader->getUserForLogEntry(

It seems like $this->databaseLogReader->getUserForLogEntry($semanticData->getSubject()->getTitle(), 'approval') of type object<User> is incompatible with the declared type integer|null of property $approvedBy.

				$semanticData->getSubject()->getTitle(),
				'approval'
			);
		}

		$property = $this->newDIProperty();
		$dataItem = $this->newDIWikiPage();
		$semanticData->removeProperty( $property );

		if ( $dataItem ) {
			$semanticData->addPropertyObjectValue( $property, $dataItem );
		}
	}

	private function newDIWikiPage() {

		if ( !$this->approvedBy instanceof User ) {

The class User does not exist. Is this class maybe located in a folder that is not analyzed, or in a newer version of your dependencies than listed in your composer.lock/composer.json?

			return;
		}

		$userPage = $this->approvedBy->getUserPage();

		if ( $userPage instanceof Title ) {

The class Title does not exist. Is this class maybe located in a folder that is not analyzed, or in a newer version of your dependencies than listed in your composer.lock/composer.json?

			return DIWikiPage::newFromTitle( $userPage );
		}
	}

}