ApprovedStatusPropertyAnnotator::addAnnotation() - Code Metrics - Inspection of "Add additional properties for AR" - SemanticMediaWiki/SemanticExtraSpecialProperties - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#98)

by mw

created 2018-02-03 13:07 UTC

ApprovedStatusPropertyAnnotator::addAnnotation() B

↳ Parent: ApprovedStatusPropertyAnnotator

Complexity

Conditions	5
Paths	4

Size

Total Lines	18
Code Lines	11

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	9
CRAP Score	6.1384

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
c	1
b	0
f	0
dl	0
loc	18
ccs	9
cts	14
cp	0.6429
rs	8.8571
cc	5
eloc	11
nc	4
nop	2
crap	6.1384

<?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;
	}

	public function getDataItem() {
		return new DIString( $this->approvedStatus );

	}

	/**
	 * {@inheritDoc}
	 */
	public function addAnnotation(
		DIProperty $property, SemanticData $semanticData
	) {
		if ( $this->approvedStatus === null && class_exists( 'ApprovedRevs' ) ) {
			$logReader = $this->appFactory->newDatabaseLogReader(
				$semanticData->getSubject()->getTitle(), 'approval'
			);
			$this->approvedStatus = $logReader->getStatusOfLogEntry();

		}

		if ( is_string( $this->approvedStatus ) && $this->approvedStatus !== "" ) {
			$semanticData->addPropertyObjectValue(
				$property, $this->getDataItem()
			);
		} 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	4	public function __construct( AppFactory $appFactory ) {
39	4	$this->appFactory = $appFactory;
40	4	}
41
42		/**
43		* @since 2.0
44		*
45		* @param string $approvedStatus
46		*/
47	2	public function setApprovedStatus( $approvedStatus ) {
48	2	$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	2	}
50
51		/**
52		* {@inheritDoc}
53		*/
54	1	public function isAnnotatorFor( DIProperty $property ) {
55	1	return $property->getKey() === self::PROP_ID;
56		}
57
58	1	public function getDataItem() {
59	1	return new DIString( $this->approvedStatus );
		0 ignored issues – show Deprecated Code introduced 2018-01-07 04:49 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...
60		}
61
62		/**
63		* {@inheritDoc}
64		*/
65	2	public function addAnnotation(
66		DIProperty $property, SemanticData $semanticData
67		) {
68	2	if ( $this->approvedStatus === null && class_exists( 'ApprovedRevs' ) ) {
69		$logReader = $this->appFactory->newDatabaseLogReader(
70		$semanticData->getSubject()->getTitle(), 'approval'
71		);
72		$this->approvedStatus = $logReader->getStatusOfLogEntry();
		0 ignored issues – show Documentation Bug introduced 2018-01-07 04:49 UTC by Report Bug Copy Issue Report It seems like `$logReader->getStatusOfLogEntry()` 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...
73		}
74
75	2	if ( is_string( $this->approvedStatus ) && $this->approvedStatus !== "" ) {
76	1	$semanticData->addPropertyObjectValue(
77	1	$property, $this->getDataItem()
78	1	);
79	1	} else {
80	1	$semanticData->removeProperty( $property );
81		}
82	2	}
83		}
84

SemanticMediaWiki / SemanticExtraSpecialProperties

Pull Request — master (#98)

ApprovedStatusPropertyAnnotator::addAnnotation() B

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like