ApprovedStatusPropertyAnnotator::addAnnotation() - Code Metrics - Inspection of "Add ability to track information from ApprovedRevi..." - SemanticMediaWiki/SemanticExtraSpecialProperties - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#95)

by Mark A.

created 2017-12-30 00:44 UTC

ApprovedStatusPropertyAnnotator::addAnnotation() B

↳ Parent: ApprovedStatusPropertyAnnotator

Complexity

Conditions	5
Paths	4

Size

Total Lines	18
Code Lines	11

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
c	1
b	0
f	0
dl	0
loc	18
rs	8.8571
cc	5
eloc	11
nc	4
nop	2

<?php

namespace SESP\PropertyAnnotators;

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

/**
 * @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;

	}

	/**
	 * {@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' ) ) {
			$logReader = new LogReader(
				$semanticData->getSubject()->getTitle(), 'approval'
			);
			$this->approvedStatus = $logReader->getStatus();
		}

		if ( is_string( $this->approvedStatus ) && $this->approvedStatus !== "" ) {
			$semanticData->addPropertyObjectValue(
				$property, new DIString( $this->approvedStatus )

			);
		} else {
			$semanticData->removeProperty( $property );
		}
	}
}


1			<?php
2
3			namespace SESP\PropertyAnnotators;
4
5			use SMW\DIProperty;
6			use SMW\SemanticData;
7			use SMWDIString as DIString;
8			use SESP\PropertyAnnotator;
9			use SESP\AppFactory;
10			use LogReader;
11
12			/**
13			* @private
14			* @ingroup SESP
15			*
16			* @license GNU GPL v2+
17			*/
18			class ApprovedStatusPropertyAnnotator implements PropertyAnnotator {
19
20			/**
21			* Predefined property ID
22			*/
23			const PROP_ID = '___APPROVEDSTATUS';
24
25			/**
26			* @var AppFactory
27			*/
28			private $appFactory;
29
30			/**
31			* @var Integer\|null
32			*/
33			private $approvedStatus;
34
35			/**
36			* @param AppFactory $appFactory
37			*/
38			public function __construct( AppFactory $appFactory ) {
39			$this->appFactory = $appFactory;
40			}
41
42			/**
43			* @since 2.0
44			*
45			* @param string $approvedStatus
46			*/
47			public function setApprovedStatus( $approvedStatus ) {
48			$this->approvedStatus = $approvedStatus;
			0 ignored issues – show Documentation Bug introduced 2017-12-30 00:52 UTC by Report Bug Copy Issue Report It seems like `$approvedStatus` of type `string` is incompatible with the declared type `integer\|null` of property `$approvedStatus`. Our type inference engine has found an assignment to a property that is incompatible with the declared type of that property. Either this assignment is in error or the assigned type should be added to the documentation/type hint for that property.. Loading history...
49			}
50
51			/**
52			* {@inheritDoc}
53			*/
54			public function isAnnotatorFor( DIProperty $property ) {
55			return $property->getKey() === self::PROP_ID;
56			}
57
58			/**
59			* {@inheritDoc}
60			*/
61			public function addAnnotation(
62			DIProperty $property, SemanticData $semanticData
63			) {
64			if ( $this->approvedStatus === null && class_exists( 'ApprovedRevs' ) ) {
65			$logReader = new LogReader(
66			$semanticData->getSubject()->getTitle(), 'approval'
67			);
68			$this->approvedStatus = $logReader->getStatus();
69			}
70
71			if ( is_string( $this->approvedStatus ) && $this->approvedStatus !== "" ) {
72			$semanticData->addPropertyObjectValue(
73			$property, new DIString( $this->approvedStatus )
			0 ignored issues – show Deprecated Code introduced 2017-12-30 00:52 UTC by Report Bug Copy Issue Report The class `SMWDIString` has been deprecated with message: Will be removed after SMW 1.9; use SMWDIBlob instead This class, trait or interface has been deprecated. The supplier of the file has supplied an explanatory message. The explanatory message should give you some clue as to whether and when the type will be removed from the class and what other constant to use instead. Loading history...
74			);
75			} else {
76			$semanticData->removeProperty( $property );
77			}
78			}
79			}
80

SemanticMediaWiki / SemanticExtraSpecialProperties

Pull Request — master (#95)

ApprovedStatusPropertyAnnotator::addAnnotation() B

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like