ApprovedStatusPropertyAnnotator::addAnnotation() - Code Metrics - Inspection of "Fix styling issues, refs #98" - SemanticMediaWiki/SemanticExtraSpecialProperties - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 1fe2bc...e619ac )

by mw

created 2018-02-03 13:37 UTC

ApprovedStatusPropertyAnnotator::addAnnotation() B

↳ Parent: ApprovedStatusPropertyAnnotator

Complexity

Conditions	5
Paths	4

Size

Total Lines	20
Code Lines	11

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	10
CRAP Score	5.9256

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
c	1
b	0
f	0
dl	0
loc	20
ccs	10
cts	15
cp	0.6667
rs	8.8571
cc	5
eloc	11
nc	4
nop	2
crap	5.9256

<?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 = $this->appFactory->newDatabaseLogReader(
				$semanticData->getSubject()->getTitle(), 'approval'
			);

			$this->approvedStatus = $logReader->getStatusOfLogEntry();

		}

		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	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		/**
59		* {@inheritDoc}
60		*/
61	2	public function addAnnotation( DIProperty $property, SemanticData $semanticData ) {
62
63	2	if ( $this->approvedStatus === null && class_exists( 'ApprovedRevs' ) ) {
64
65		$logReader = $this->appFactory->newDatabaseLogReader(
66		$semanticData->getSubject()->getTitle(), 'approval'
67		);
68
69		$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...
70		}
71
72	2	if ( is_string( $this->approvedStatus ) && $this->approvedStatus !== '' ) {
73	1	$semanticData->addPropertyObjectValue(
74	1	$property,
75	1	new DIString( $this->approvedStatus )
		0 ignored issues – show Deprecated Code introduced 2018-02-03 13:39 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...
76	1	);
77	1	} else {
78	1	$semanticData->removeProperty( $property );
79		}
80	2	}
81
82		}
83

SemanticMediaWiki / SemanticExtraSpecialProperties

Push — master ( 1fe2bc...e619ac )

ApprovedStatusPropertyAnnotator::addAnnotation() B

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like