ExtraPropertyAnnotator - Code Metrics - Inspection of "Allows to add page length to pages" - SemanticMediaWiki/SemanticExtraSpecialProperties - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — 1.x (#76)

by mw

created 2017-06-04 04:39 UTC

ExtraPropertyAnnotator D

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	379
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	10

Test Coverage

Coverage

46.4%

Importance

Changes

Metric	Value
dl	0
loc	379
ccs	103
cts	222
cp	0.464
rs	4.8717
c	0
b	0
f	0
wmc	80
lcom	1
cbo	10

25 Methods

Rating	Name	Size	Complexity
A	__construct()	5	1
B	addAnnotation()	15	6
A	getSemanticData()	3	1
A	getWikiPage()	8	2
B	addPropertyValues()	30	5
A	hasRegisteredPropertyId()	4	2
A	isUserPage()	3	1
A	isFilePage()	3	1
A	makeFirstAuthorDataItem()	7	2
A	makeNumberOfPageViewsDataItem()	13	3
A	getPageViewCount()	11	3
B	addPropertyValuesForPageContributors()	20	5
A	makePageIdDataItem()	7	3
A	makeRevisionIdDataItem()	7	3
A	getPageRevisionsForId()	12	2
A	makeNumberOfRevisionsDataItem()	9	3
A	makeNumberOfTalkPageRevisionsDataItem()	9	3
A	addPropertyValuesForMIMEAndMediaType()	21	2
A	addPropertyValuesForSubPages()	12	2
A	addPropertyValuesForExifData()	5	2
A	addPropertyValuesForShortUrl()	8	2
A	makeUserRegistrationDataItem()	23	3
A	makeUserEditCountDataItem()	14	4
C	createDataItemById()	55	16
A	makePageLengthDataItem()	7	3

How to fix Complexity

<?php

namespace SESP\Annotator;

use SESP\PropertyRegistry;
use SESP\AppFactory;

use SMW\SemanticData;
use SMW\DIProperty;
use SMW\DIWikiPage;
use SMW\Store;

use SMWDataItem as DataItem;
use SMWDIBlob as DIBlob;
use SMWDIBoolean as DIBoolean;
use SMWDITime as DITime;
use SMWDINumber as DINumber;

use WikiPage;
use User;
use RuntimeException;

/**
 * @ingroup SESP
 *
 * @license GNU GPL v2+
 * @since 1.0
 *
 * @author mwjames
 * @author rotsee
 */
class ExtraPropertyAnnotator {

	/**
	 * @var SemanticData
	 */
	protected $semanticData = null;

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


	protected $configuration = null;
	private $dbConnection = null;
	private $page = null;

	/**
	 * @since 1.0
	 *
	 * @param SemanticData $semanticData
	 * @param Factory $factory
/**
 * @param array $germany
 * @param array $ireland
 */
function finale($germany, $island) {
    return "2:1";
}
	 * @param array $configuration
	 */
	public function __construct( SemanticData $semanticData, AppFactory $appFactory, array $configuration ) {
		$this->semanticData = $semanticData;
		$this->appFactory = $appFactory;
		$this->configuration = $configuration;
	}

	/**
	 * @since 1.0
	 *
	 * @return boolean
	 * @throws RuntimeException
	 *
	 * @return boolean
	 */
	public function addAnnotation() {

		$subject = $this->semanticData->getSubject();

		if ( $subject === null || $subject->getTitle() === null || $subject->getTitle()->isSpecialPage() ) {
			return false;
		}

		if ( isset( $this->configuration['sespSpecialProperties'] ) &&
			is_array( $this->configuration['sespSpecialProperties'] ) ) {
			return $this->addPropertyValues();
		}

		throw new RuntimeException( "Expected a 'sespSpecialProperties' configuration array" );
	}

	/**
	 * @since 1.0
	 *
	 * @return SemanticData
	 */
	public function getSemanticData() {
		return $this->semanticData;
	}

	/**
	 * @since 1.0
	 *
	 * @return WikiPage
	 */
	public function getWikiPage() {

		if ( $this->page === null ) {
			$this->page = $this->appFactory->newWikiPage( $this->semanticData->getSubject()->getTitle() ); //$this->loadRegisteredObject( 'WikiPage' );
/** @return stdClass|null */
function mayReturnNull() { }

function doesNotAcceptNull(stdClass $x) { }

// With potential error.
function withoutCheck() {
    $x = mayReturnNull();
    doesNotAcceptNull($x); // Potential error here.
}

// Safe - Alternative 1
function withCheck1() {
    $x = mayReturnNull();
    if ( ! $x instanceof stdClass) {
        throw new \LogicException('$x must be defined.');
    }
    doesNotAcceptNull($x);
}

// Safe - Alternative 2
function withCheck2() {
    $x = mayReturnNull();
    if ($x instanceof stdClass) {
        doesNotAcceptNull($x);
    }
}
		}

		return $this->page;
	}

	protected function addPropertyValues() {

		$cachedProperties = array();

		foreach ( $this->configuration['sespSpecialProperties'] as $externalId ) {

			$propertyId = PropertyRegistry::getInstance()->getPropertyId( $externalId );

			if ( $this->hasRegisteredPropertyId( $propertyId, $cachedProperties ) ) {
				continue;
			}

			$propertyDI = new DIProperty( $propertyId );

			if ( $this->getSemanticData()->getPropertyValues( $propertyDI ) !== array() ) {
				$cachedProperties[ $propertyId ] = true;
				continue;
			}

			$dataItem = $this->createDataItemById( $externalId, $propertyDI );

			if ( $dataItem instanceof DataItem ) {
				$cachedProperties[ $propertyId ] = true;
				$this->getSemanticData()->addPropertyObjectValue( $propertyDI, $dataItem );
			}

		}

		return true;
	}

	protected function hasRegisteredPropertyId( $propertyId, $cachedProperties ) {
		return ( DIProperty::getPredefinedPropertyTypeId( $propertyId ) === '' ) ||

			array_key_exists( $propertyId, $cachedProperties );
	}

	protected function createDataItemById( $externalId, $property ) {

		$dataItem = null;

		// _REVID was incorrect in the original SESP because getId returns the
		// page id not the revision Id

		switch ( $externalId ) {
			case '_CUSER' :
				$dataItem = $this->makeFirstAuthorDataItem();
				break;
			case '_VIEWS' :
				$dataItem = $this->makeNumberOfPageViewsDataItem();
				break;
			case '_USERREG' :
				$dataItem = $this->makeUserRegistrationDataItem();
				break;
			case '_USEREDITCNT' :
				$dataItem = $this->makeUserEditCountDataItem();
				break;
			case '_PAGEID' :
				$dataItem = $this->makePageIdDataItem();
				break;
			case '_PAGELGTH' :
				$dataItem = $this->makePageLengthDataItem();
				break;
			case '_REVID' :
				$dataItem = $this->makeRevisionIdDataItem();
				break;
			case '_NREV' :
				$dataItem = $this->makeNumberOfRevisionsDataItem();
				break;
			case '_NTREV' :
				$dataItem = $this->makeNumberOfTalkPageRevisionsDataItem();
				break;
			case '_EUSER' :
				$this->addPropertyValuesForPageContributors( $property );
				break;
			case '_SUBP' :
				$this->addPropertyValuesForSubPages( $property );
				break;
			case '_MEDIATYPE' :
			case '_MIMETYPE' :
				$this->addPropertyValuesForMIMEAndMediaType();
				break;
			case '_EXIFDATA' :
				$this->addPropertyValuesForExifData();
				break;
			case '_SHORTURL' :
				$this->addPropertyValuesForShortUrl();
				break;
		}

		return $dataItem;
	}

	private function isUserPage() {
		return $this->getWikiPage()->getTitle()->inNamespace( NS_USER );
	}

	private function isFilePage() {
		return $this->getWikiPage()->getTitle()->inNamespace( NS_FILE );
	}

	private function makeFirstAuthorDataItem() {
		$creator = $this->getWikiPage()->getCreator();

		if ( $creator ) {
			return DIWikiPage::newFromTitle( $creator->getUserPage() );
		}
	}

	private function makeNumberOfPageViewsDataItem() {
		if ( $this->configuration['wgDisableCounters'] ) {
			return null;
		}

		$count = $this->getPageViewCount();

		if ( !is_numeric( $count ) ) {
			return null;
		}

		return new DINumber( $count );
	}

	private function getPageViewCount() {
		if ( class_exists( '\HitCounters\HitCounters' ) ) {
			return \HitCounters\HitCounters::getCount( $this->getWikiPage()->getTitle() );
		}

		if ( method_exists( $this->getWikiPage(), 'getCount' ) ) {
			return $this->getWikiPage()->getCount();
		}

		return null;
	}

	private function addPropertyValuesForPageContributors( DIProperty $property ) {

		$user = User::newFromId( $this->getWikiPage()->getUser() );
		$authors = $this->getWikiPage()->getContributors();

		while ( $user ) {
			if ( !( in_array( 'bot', $user->getRights() ) &&
				$this->configuration['wgSESPExcludeBots'] ) &&
				!$user->isAnon() ) { //no anonymous users (hidden users are not returned)

				$this->getSemanticData()->addPropertyObjectValue(
					$property,
					DIWikiPage::newFromTitle( $user->getUserPage() )
				);
			}

			$user = $authors->current();
			$authors->next();
		}
	}

	private function makePageIdDataItem() {
		$pageID = $this->getWikiPage()->getId();

		if ( is_integer( $pageID ) && $pageID > 0 ) {
			return new DINumber( $pageID );
		}
	}

	private function makePageLengthDataItem() {
		$pageLen = $this->getWikiPage()->getTitle()->getLength();

		if ( is_integer( $pageLen ) && $pageLen > 0 ) {
			return new DINumber( $pageLen );
		}
	}

	private function makeRevisionIdDataItem() {
		$revID = $this->getWikiPage()->getLatest();

		if ( is_integer( $revID ) && $revID > 0 ) {
			return new DINumber( $revID );
		}
	}

	private function getPageRevisionsForId( $pageId ) {

		if ( $this->dbConnection === null ) {
			$this->dbConnection = $this->appFactory->newDatabaseConnection(); //( 'DBConnection', 'DatabaseBase' );
		}

		return $this->dbConnection->estimateRowCount(
			"revision",
			"*",
			array( "rev_page" => $pageId )
		);
	}

	private function makeNumberOfRevisionsDataItem() {
		$numberOfPageRevisions = $this->getPageRevisionsForId(
			$this->getWikiPage()->getTitle()->getArticleID()
		);

		if ( $this->getWikiPage()->getTitle()->exists() && $numberOfPageRevisions > 0 ) {
			return new DINumber( $numberOfPageRevisions );
		}
	}

	private function makeNumberOfTalkPageRevisionsDataItem() {
		$numberOfTalkPageRevisions = $this->getPageRevisionsForId(
			$this->getWikiPage()->getTitle()->getTalkPage()->getArticleID()
		);

		if ( $this->getWikiPage()->getTitle()->getTalkPage()->exists() && $numberOfTalkPageRevisions > 0 ) {
			return new DINumber( $numberOfTalkPageRevisions );
		}
	}

	private function addPropertyValuesForMIMEAndMediaType(){

		if ( $this->isFilePage() ) {

			$file = $this->getWikiPage()->getFile();
			$mimetype = $file->getMimeType();
			$mediaType = \MimeMagic::singleton()->findMediaType( $mimetype );
			list( $mimetypemajor, $mimetypeminor ) = $file->splitMime( $mimetype );
<?php

function returnThreeValues() {
    return array('a', 'b', 'c');
}

list($a, $b, $c) = returnThreeValues();

print $a . " - " . $c;

			$this->getSemanticData()->addPropertyObjectValue(
				new DIProperty( PropertyRegistry::getInstance()->getPropertyId( '_MIMETYPE' ) ),
				new DIBlob( $mimetypeminor )
			);

			$this->getSemanticData()->addPropertyObjectValue(
				new DIProperty( PropertyRegistry::getInstance()->getPropertyId( '_MEDIATYPE' ) ),
				new DIBlob( $mediaType )
			);
		}

	}

	private function addPropertyValuesForSubPages( DIProperty $property ) {

		//-1 = no limit. Returns TitleArray object
		$subpages = $this->getWikiPage()->getTitle()->getSubpages ( -1 );

		foreach ( $subpages as $title ) {
			$this->getSemanticData()->addPropertyObjectValue(
				$property,
				DIWikiPage::newFromTitle( $title )
			);
		}
	}

	private function addPropertyValuesForExifData() {
		if ( $this->isFilePage() ) {
			$this->appFactory->newExifDataAnnotator( $this->getSemanticData(), $this->getWikiPage()->getFile() )->addAnnotation();
		}
	}

	private function addPropertyValuesForShortUrl() {

		$shortUrlAnnotator = $this->appFactory->newShortUrlAnnotator( $this->getSemanticData() );

		if ( $shortUrlAnnotator->canUseShortUrl() ) {
			$shortUrlAnnotator->addAnnotation();
		}
	}

	private function makeUserRegistrationDataItem() {

		if ( !$this->isUserPage() ) {
			return null;
		}

		$user = $this->appFactory->newUserFromTitle( $this->getWikiPage()->getTitle() );

		if ( $user instanceof User ) {


			$timestamp = wfTimestamp( TS_ISO_8601, $user->getRegistration() );
			$date = new \DateTime( $timestamp );

			return new DITime(
				DITime::CM_GREGORIAN,
				$date->format('Y'),
				$date->format('m'),
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
				$date->format('d'),
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
				$date->format('H'),
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
				$date->format('i')
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
			);
		}
	}

	private function makeUserEditCountDataItem() {

		if ( !$this->isUserPage() ) {
			return;
		}

		$user = $this->appFactory->newUserFromTitle( $this->getWikiPage()->getTitle() );

		$count = $user instanceof User ? $user->getEditCount() : false;


		if ( $count ) {
			return new DINumber( $count );
		}
	}

}


1		<?php
2
3		namespace SESP\Annotator;
4
5		use SESP\PropertyRegistry;
6		use SESP\AppFactory;
7
8		use SMW\SemanticData;
9		use SMW\DIProperty;
10		use SMW\DIWikiPage;
11		use SMW\Store;
12
13		use SMWDataItem as DataItem;
14		use SMWDIBlob as DIBlob;
15		use SMWDIBoolean as DIBoolean;
16		use SMWDITime as DITime;
17		use SMWDINumber as DINumber;
18
19		use WikiPage;
20		use User;
21		use RuntimeException;
22
23		/**
24		* @ingroup SESP
25		*
26		* @license GNU GPL v2+
27		* @since 1.0
28		*
29		* @author mwjames
30		* @author rotsee
31		*/
32		class ExtraPropertyAnnotator {
33
34		/**
35		* @var SemanticData
36		*/
37		protected $semanticData = null;
38
39		/**
40		* @var AppFactory
41		*/
42		private $appFactory = null;
43
44
45		protected $configuration = null;
46		private $dbConnection = null;
47		private $page = null;
48
49		/**
50		* @since 1.0
51		*
52		* @param SemanticData $semanticData
53		* @param Factory $factory
		0 ignored issues – show Documentation introduced 2015-12-04 20:50 UTC by Report Bug Copy Issue Report There is no parameter named `$factory`. Did you maybe mean `$appFactory`? This check looks for PHPDoc comments describing methods or function parameters that do not exist on the corresponding method or function. It has, however, found a similar but not annotated parameter which might be a good fit. Consider the following example. The parameter `$ireland` is not defined by the method `finale(...)`. /** * @param array $germany * @param array $ireland */ function finale($germany, $island) { return "2:1"; } The most likely cause is that the parameter was changed, but the annotation was not. Loading history...
54		* @param array $configuration
55		*/
56	10	public function __construct( SemanticData $semanticData, AppFactory $appFactory, array $configuration ) {
57	10	$this->semanticData = $semanticData;
58	10	$this->appFactory = $appFactory;
59	10	$this->configuration = $configuration;
60	10	}
61
62		/**
63		* @since 1.0
64		*
65		* @return boolean
66		* @throws RuntimeException
67		*
68		* @return boolean
69		*/
70	9	public function addAnnotation() {
71
72	9	$subject = $this->semanticData->getSubject();
73
74	9	if ( $subject === null \|\| $subject->getTitle() === null \|\| $subject->getTitle()->isSpecialPage() ) {
75	1	return false;
76		}
77
78	8	if ( isset( $this->configuration['sespSpecialProperties'] ) &&
79	8	is_array( $this->configuration['sespSpecialProperties'] ) ) {
80	7	return $this->addPropertyValues();
81		}
82
83	1	throw new RuntimeException( "Expected a 'sespSpecialProperties' configuration array" );
84		}
85
86		/**
87		* @since 1.0
88		*
89		* @return SemanticData
90		*/
91	7	public function getSemanticData() {
92	7	return $this->semanticData;
93		}
94
95		/**
96		* @since 1.0
97		*
98		* @return WikiPage
99		*/
100	7	public function getWikiPage() {
101
102	7	if ( $this->page === null ) {
103	7	$this->page = $this->appFactory->newWikiPage( $this->semanticData->getSubject()->getTitle() ); //$this->loadRegisteredObject( 'WikiPage' );
		0 ignored issues – show Bug introduced 2015-12-04 20:50 UTC by Report Bug Copy Issue Report It seems like `$this->semanticData->getSubject()->getTitle()` can be `null`; however, `newWikiPage()` does not accept `null`, maybe add an additional type check? Unless you are absolutely sure that the expression can never be null because of other conditions, we strongly recommend to add an additional type check to your code: /** @return stdClass\|null */ function mayReturnNull() { } function doesNotAcceptNull(stdClass $x) { } // With potential error. function withoutCheck() { $x = mayReturnNull(); doesNotAcceptNull($x); // Potential error here. } // Safe - Alternative 1 function withCheck1() { $x = mayReturnNull(); if ( ! $x instanceof stdClass) { throw new \LogicException('$x must be defined.'); } doesNotAcceptNull($x); } // Safe - Alternative 2 function withCheck2() { $x = mayReturnNull(); if ($x instanceof stdClass) { doesNotAcceptNull($x); } } Loading history...
104	7	}
105
106	7	return $this->page;
107		}
108
109	7	protected function addPropertyValues() {
110
111	7	$cachedProperties = array();
112
113	7	foreach ( $this->configuration['sespSpecialProperties'] as $externalId ) {
114
115	7	$propertyId = PropertyRegistry::getInstance()->getPropertyId( $externalId );
116
117	7	if ( $this->hasRegisteredPropertyId( $propertyId, $cachedProperties ) ) {
118		continue;
119		}
120
121	7	$propertyDI = new DIProperty( $propertyId );
122
123	7	if ( $this->getSemanticData()->getPropertyValues( $propertyDI ) !== array() ) {
124		$cachedProperties[ $propertyId ] = true;
125		continue;
126		}
127
128	7	$dataItem = $this->createDataItemById( $externalId, $propertyDI );
129
130	7	if ( $dataItem instanceof DataItem ) {
131	5	$cachedProperties[ $propertyId ] = true;
132	5	$this->getSemanticData()->addPropertyObjectValue( $propertyDI, $dataItem );
133	5	}
134
135	7	}
136
137	7	return true;
138		}
139
140	7	protected function hasRegisteredPropertyId( $propertyId, $cachedProperties ) {
141	7	return ( DIProperty::getPredefinedPropertyTypeId( $propertyId ) === '' ) \|\|
		0 ignored issues – show Deprecated Code introduced 2015-12-04 20:50 UTC by Report Bug Copy Issue Report The method `SMW\DIProperty::getPredefinedPropertyTypeId()` has been deprecated with message: since 2.1, use PropertyRegistry::getPredefinedPropertyTypeId This method has been deprecated. The supplier of the class has supplied an explanatory message. The explanatory message should give you some clue as to whether and when the method will be removed from the class and what other method or class to use instead. Loading history...
142	7	array_key_exists( $propertyId, $cachedProperties );
143		}
144
145	7	protected function createDataItemById( $externalId, $property ) {
146
147	7	$dataItem = null;
148
149		// _REVID was incorrect in the original SESP because getId returns the
150		// page id not the revision Id
151
152		switch ( $externalId ) {
153	7	case '_CUSER' :
154	1	$dataItem = $this->makeFirstAuthorDataItem();
155	1	break;
156	6	case '_VIEWS' :
157		$dataItem = $this->makeNumberOfPageViewsDataItem();
158		break;
159	6	case '_USERREG' :
160	2	$dataItem = $this->makeUserRegistrationDataItem();
161	2	break;
162	4	case '_USEREDITCNT' :
163		$dataItem = $this->makeUserEditCountDataItem();
164		break;
165	4	case '_PAGEID' :
166		$dataItem = $this->makePageIdDataItem();
167		break;
168	4	case '_PAGELGTH' :
169	1	$dataItem = $this->makePageLengthDataItem();
170	1	break;
171	3	case '_REVID' :
172	2	$dataItem = $this->makeRevisionIdDataItem();
173	2	break;
174	1	case '_NREV' :
175	1	$dataItem = $this->makeNumberOfRevisionsDataItem();
176	1	break;
177		case '_NTREV' :
178		$dataItem = $this->makeNumberOfTalkPageRevisionsDataItem();
179		break;
180		case '_EUSER' :
181		$this->addPropertyValuesForPageContributors( $property );
182		break;
183		case '_SUBP' :
184		$this->addPropertyValuesForSubPages( $property );
185		break;
186		case '_MEDIATYPE' :
187		case '_MIMETYPE' :
188		$this->addPropertyValuesForMIMEAndMediaType();
189		break;
190		case '_EXIFDATA' :
191		$this->addPropertyValuesForExifData();
192		break;
193		case '_SHORTURL' :
194		$this->addPropertyValuesForShortUrl();
195		break;
196		}
197
198	7	return $dataItem;
199		}
200
201	2	private function isUserPage() {
202	2	return $this->getWikiPage()->getTitle()->inNamespace( NS_USER );
203		}
204
205		private function isFilePage() {
206		return $this->getWikiPage()->getTitle()->inNamespace( NS_FILE );
207		}
208
209	1	private function makeFirstAuthorDataItem() {
210	1	$creator = $this->getWikiPage()->getCreator();
211
212	1	if ( $creator ) {
213	1	return DIWikiPage::newFromTitle( $creator->getUserPage() );
214		}
215		}
216
217		private function makeNumberOfPageViewsDataItem() {
218		if ( $this->configuration['wgDisableCounters'] ) {
219		return null;
220		}
221
222		$count = $this->getPageViewCount();
223
224		if ( !is_numeric( $count ) ) {
225		return null;
226		}
227
228		return new DINumber( $count );
229		}
230
231		private function getPageViewCount() {
232		if ( class_exists( '\HitCounters\HitCounters' ) ) {
233		return \HitCounters\HitCounters::getCount( $this->getWikiPage()->getTitle() );
234		}
235
236		if ( method_exists( $this->getWikiPage(), 'getCount' ) ) {
237		return $this->getWikiPage()->getCount();
238		}
239
240		return null;
241		}
242
243		private function addPropertyValuesForPageContributors( DIProperty $property ) {
244
245		$user = User::newFromId( $this->getWikiPage()->getUser() );
246		$authors = $this->getWikiPage()->getContributors();
247
248		while ( $user ) {
249		if ( !( in_array( 'bot', $user->getRights() ) &&
250		$this->configuration['wgSESPExcludeBots'] ) &&
251		!$user->isAnon() ) { //no anonymous users (hidden users are not returned)
252
253		$this->getSemanticData()->addPropertyObjectValue(
254		$property,
255		DIWikiPage::newFromTitle( $user->getUserPage() )
256		);
257		}
258
259		$user = $authors->current();
260		$authors->next();
261		}
262		}
263
264		private function makePageIdDataItem() {
265		$pageID = $this->getWikiPage()->getId();
266
267		if ( is_integer( $pageID ) && $pageID > 0 ) {
268		return new DINumber( $pageID );
269		}
270		}
271
272	1	private function makePageLengthDataItem() {
273	1	$pageLen = $this->getWikiPage()->getTitle()->getLength();
274
275	1	if ( is_integer( $pageLen ) && $pageLen > 0 ) {
276	1	return new DINumber( $pageLen );
277		}
278		}
279
280	2	private function makeRevisionIdDataItem() {
281	2	$revID = $this->getWikiPage()->getLatest();
282
283	2	if ( is_integer( $revID ) && $revID > 0 ) {
284	1	return new DINumber( $revID );
285		}
286	1	}
287
288	1	private function getPageRevisionsForId( $pageId ) {
289
290	1	if ( $this->dbConnection === null ) {
291	1	$this->dbConnection = $this->appFactory->newDatabaseConnection(); //( 'DBConnection', 'DatabaseBase' );
292	1	}
293
294	1	return $this->dbConnection->estimateRowCount(
295	1	"revision",
296	1	"*",
297	1	array( "rev_page" => $pageId )
298	1	);
299		}
300
301	1	private function makeNumberOfRevisionsDataItem() {
302	1	$numberOfPageRevisions = $this->getPageRevisionsForId(
303	1	$this->getWikiPage()->getTitle()->getArticleID()
304	1	);
305
306	1	if ( $this->getWikiPage()->getTitle()->exists() && $numberOfPageRevisions > 0 ) {
307	1	return new DINumber( $numberOfPageRevisions );
308		}
309		}
310
311		private function makeNumberOfTalkPageRevisionsDataItem() {
312		$numberOfTalkPageRevisions = $this->getPageRevisionsForId(
313		$this->getWikiPage()->getTitle()->getTalkPage()->getArticleID()
314		);
315
316		if ( $this->getWikiPage()->getTitle()->getTalkPage()->exists() && $numberOfTalkPageRevisions > 0 ) {
317		return new DINumber( $numberOfTalkPageRevisions );
318		}
319		}
320
321		private function addPropertyValuesForMIMEAndMediaType(){
322
323		if ( $this->isFilePage() ) {
324
325		$file = $this->getWikiPage()->getFile();
326		$mimetype = $file->getMimeType();
327		$mediaType = \MimeMagic::singleton()->findMediaType( $mimetype );
328		list( $mimetypemajor, $mimetypeminor ) = $file->splitMime( $mimetype );
		0 ignored issues – show Unused Code introduced 2015-12-04 20:50 UTC by Report Bug Copy Issue Report The assignment to `$mimetypemajor` is unused. Consider omitting it like so `list($first,,$third)`. This checks looks for assignemnts to variables using the `list(...)` function, where not all assigned variables are subsequently used. Consider the following code example. <?php function returnThreeValues() { return array('a', 'b', 'c'); } list($a, $b, $c) = returnThreeValues(); print $a . " - " . $c; Only the variables `$a` and `$c` are used. There was no need to assign `$b`. Instead, the list call could have been. list($a,, $c) = returnThreeValues(); Loading history...
329
330		$this->getSemanticData()->addPropertyObjectValue(
331		new DIProperty( PropertyRegistry::getInstance()->getPropertyId( '_MIMETYPE' ) ),
332		new DIBlob( $mimetypeminor )
333		);
334
335		$this->getSemanticData()->addPropertyObjectValue(
336		new DIProperty( PropertyRegistry::getInstance()->getPropertyId( '_MEDIATYPE' ) ),
337		new DIBlob( $mediaType )
338		);
339		}
340
341		}
342
343		private function addPropertyValuesForSubPages( DIProperty $property ) {
344
345		//-1 = no limit. Returns TitleArray object
346		$subpages = $this->getWikiPage()->getTitle()->getSubpages ( -1 );
347
348		foreach ( $subpages as $title ) {
349		$this->getSemanticData()->addPropertyObjectValue(
350		$property,
351		DIWikiPage::newFromTitle( $title )
352		);
353		}
354		}
355
356		private function addPropertyValuesForExifData() {
357		if ( $this->isFilePage() ) {
358		$this->appFactory->newExifDataAnnotator( $this->getSemanticData(), $this->getWikiPage()->getFile() )->addAnnotation();
359		}
360		}
361
362		private function addPropertyValuesForShortUrl() {
363
364		$shortUrlAnnotator = $this->appFactory->newShortUrlAnnotator( $this->getSemanticData() );
365
366		if ( $shortUrlAnnotator->canUseShortUrl() ) {
367		$shortUrlAnnotator->addAnnotation();
368		}
369		}
370
371	2	private function makeUserRegistrationDataItem() {
372
373	2	if ( !$this->isUserPage() ) {
374		return null;
375		}
376
377	2	$user = $this->appFactory->newUserFromTitle( $this->getWikiPage()->getTitle() );
378
379	2	if ( $user instanceof User ) {
		0 ignored issues – show Bug introduced 2015-12-04 20:50 UTC by Report Bug Copy Issue Report 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? Loading history...
380
381	1	$timestamp = wfTimestamp( TS_ISO_8601, $user->getRegistration() );
382	1	$date = new \DateTime( $timestamp );
383
384	1	return new DITime(
385	1	DITime::CM_GREGORIAN,
386	1	$date->format('Y'),
387	1	$date->format('m'),
		0 ignored issues – show Documentation introduced 2015-12-04 20:50 UTC by Report Bug Copy Issue Report `$date->format('m')` is of type `string`, but the function expects a `boolean`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
388	1	$date->format('d'),
		0 ignored issues – show Documentation introduced 2015-12-04 20:50 UTC by Report Bug Copy Issue Report `$date->format('d')` is of type `string`, but the function expects a `boolean`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
389	1	$date->format('H'),
		0 ignored issues – show Documentation introduced 2015-12-04 20:50 UTC by Report Bug Copy Issue Report `$date->format('H')` is of type `string`, but the function expects a `boolean`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
390	1	$date->format('i')
		0 ignored issues – show Documentation introduced 2015-12-04 20:50 UTC by Report Bug Copy Issue Report `$date->format('i')` is of type `string`, but the function expects a `boolean`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
391	1	);
392		}
393	1	}
394
395		private function makeUserEditCountDataItem() {
396
397		if ( !$this->isUserPage() ) {
398		return;
399		}
400
401		$user = $this->appFactory->newUserFromTitle( $this->getWikiPage()->getTitle() );
402
403		$count = $user instanceof User ? $user->getEditCount() : false;
		0 ignored issues – show Bug introduced 2015-12-04 20:50 UTC by Report Bug Copy Issue Report 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? Loading history...
404
405		if ( $count ) {
406		return new DINumber( $count );
407		}
408		}
409
410		}
411

SemanticMediaWiki / SemanticExtraSpecialProperties

Pull Request — 1.x (#76)

ExtraPropertyAnnotator D

Complexity

Size/Duplication

Coupling/Cohesion

Test Coverage

Importance

25 Methods

How to fix Complexity

Complex Class

Duplication Side-by-Side

Filter issues like