SESP\PropertyAnnotators\ApprovedStatusPropertyAnnotator

Complexity

Total Complexity

11

Size/Duplication

Total Lines	72
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Test Coverage

Coverage

78.26%

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	3	1
A	setApprovedStatus()	0	3	1
A	isAnnotatorFor()	0	3	1
B	addAnnotation()	0	27	8

<?php

namespace SESP\PropertyAnnotators;

use ApprovedRevs;
use SMW\DIProperty;
use SMW\SemanticData;
use SMWDIString as DIString;
use SESP\PropertyAnnotator;
use SESP\AppFactory;
use LogReader;
use Title;

/**
 * @private
 * @ingroup SESP
 *
 * @license GNU GPL v2+
 */
class ApprovedStatusPropertyAnnotator implements PropertyAnnotator {

	/**
	 * Predefined property ID
	 */
	const PROP_ID = '___APPROVEDSTATUS';

	/**
	 * @var AppFactory
	 */
	private $appFactory;

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

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

	/**
	 * @since 2.0
	 *
	 * @param string $approvedStatus
	 */
	public function setApprovedStatus( $approvedStatus ) {
		$this->approvedStatus = $approvedStatus;

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

	}

	/**
	 * {@inheritDoc}
	 */
	public function isAnnotatorFor( DIProperty $property ) {
		return $property->getKey() === self::PROP_ID;
	}

	/**
	 * {@inheritDoc}
	 */
	public function addAnnotation( DIProperty $property, SemanticData $semanticData ) {

		if ( $this->approvedStatus === null && class_exists( 'ApprovedRevs' ) ) {
			$title = $semanticData->getSubject()->getTitle();
			if ( ApprovedRevs::pageIsApprovable( $title ) ) {
				$revId = ApprovedRevs::getApprovedRevID( $title );
				if ( $revId ) {
					if ( $title->getLatestRevID( Title::GAID_FOR_UPDATE ) === $revId  ) {
						$this->approvedStatus = "approved";

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

					} else {
						$this->approvedStatus = "pending";

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

					}
				} else {
					$this->approvedStatus = "unapproved";

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

				}
			}
		}

		if ( is_string( $this->approvedStatus ) && $this->approvedStatus !== '' ) {
			$semanticData->addPropertyObjectValue(
				$property,
				new DIString( $this->approvedStatus )
			);
		} else {
			$semanticData->removeProperty( $property );
		}
	}

}