Issues in MandatoryTypePropertyAnnotator.php - Fixed Issues - Inspection of "Merge pull request #1870 from SemanticMediaWiki/ti..." - SemanticMediaWiki/SemanticMediaWiki - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( d2d28e...1c2760 )

by mw

created 2016-09-25 06:48 UTC

MandatoryTypePropertyAnnotator.php (1 issue)

Showing only issues like (Show All)

Check for unnecessary assignments in calls to list(...)

Unused Code Minor

mwjames 1

Upgrade to new PHP Analysis Engine

These results are based on our legacy PHP analysis, consider migrating to our new PHP analysis engine instead. Learn more

<?php

namespace SMW\PropertyAnnotator;

use SMW\DataTypeRegistry;
use SMW\DataValueFactory;
use SMW\DIProperty;
use SMW\PropertyAnnotator;
use SMW\Store;

/**
 * @license GNU GPL v2+
 * @since 2.2
 *
 * @author mwjames
 */
class MandatoryTypePropertyAnnotator extends PropertyAnnotatorDecorator {

	protected function addPropertyValues() {

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

		if ( $subject->getNamespace() !== SMW_NS_PROPERTY ) {
			return;
		}

		$property = DIProperty::newFromUserLabel(
			str_replace( '_', ' ', $subject->getDBKey() )
		);

		if ( !$property->isUserDefined() ) {
			return;
		}

		$this->findMandatoryTypeForImportVocabulary();
	}

	private function findMandatoryTypeForImportVocabulary() {

		$property = new DIProperty( '_IMPO' );

		$dataItems = $this->getSemanticData()->getPropertyValues(
			$property
		);

		if ( $dataItems === null || $dataItems === array() ) {
			return;
		}

		$this->addTypeFromImportVocabulary( $property, current( $dataItems ) );
	}

	private function addTypeFromImportVocabulary( $property, $dataItem ) {

		$importValue = DataValueFactory::getInstance()->newDataValueByItem(
			$dataItem,
			$property
		);

		if ( strpos( $importValue->getTermType(), ':' ) === false ) {
			return;
		}

		$property = new DIProperty( '_TYPE' );

		list( $ns, $type ) = explode( ':', $importValue->getTermType(), 2 );
<?php

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

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

print $a . " - " . $c;

		$typeId = DataTypeRegistry::getInstance()->findTypeId( $type );

		if ( $typeId === '' ) {
			return;
		}

		$dataValue = DataValueFactory::getInstance()->newDataValueByProperty(
			$property,
			$typeId
		);

		$this->replaceAnyTypeByImportType( $property, $dataValue );
	}

	private function replaceAnyTypeByImportType( DIProperty $property, $dataValue ) {

		foreach ( $this->getSemanticData()->getPropertyValues( $property ) as $dataItem ) {
			$this->getSemanticData()->removePropertyObjectValue(
				$property,
				$dataItem
			);
		}

		$this->getSemanticData()->addDataValue( $dataValue );
	}

}


1		<?php
2
3		namespace SMW\PropertyAnnotator;
4
5		use SMW\DataTypeRegistry;
6		use SMW\DataValueFactory;
7		use SMW\DIProperty;
8		use SMW\PropertyAnnotator;
9		use SMW\Store;
10
11		/**
12		* @license GNU GPL v2+
13		* @since 2.2
14		*
15		* @author mwjames
16		*/
17		class MandatoryTypePropertyAnnotator extends PropertyAnnotatorDecorator {
18
19	187	protected function addPropertyValues() {
20
21	187	$subject = $this->getSemanticData()->getSubject();
22
23	187	if ( $subject->getNamespace() !== SMW_NS_PROPERTY ) {
24	177	return;
25		}
26
27	146	$property = DIProperty::newFromUserLabel(
28	146	str_replace( '_', ' ', $subject->getDBKey() )
29		);
30
31	146	if ( !$property->isUserDefined() ) {
32	1	return;
33		}
34
35	145	$this->findMandatoryTypeForImportVocabulary();
36	145	}
37
38	145	private function findMandatoryTypeForImportVocabulary() {
39
40	145	$property = new DIProperty( '_IMPO' );
41
42	145	$dataItems = $this->getSemanticData()->getPropertyValues(
43		$property
44		);
45
46	145	if ( $dataItems === null \|\| $dataItems === array() ) {
47	138	return;
48		}
49
50	10	$this->addTypeFromImportVocabulary( $property, current( $dataItems ) );
51	10	}
52
53	10	private function addTypeFromImportVocabulary( $property, $dataItem ) {
54
55	10	$importValue = DataValueFactory::getInstance()->newDataValueByItem(
56		$dataItem,
57		$property
58		);
59
60	10	if ( strpos( $importValue->getTermType(), ':' ) === false ) {
61	1	return;
62		}
63
64	9	$property = new DIProperty( '_TYPE' );
65
66	9	list( $ns, $type ) = explode( ':', $importValue->getTermType(), 2 );
		0 ignored issues – show Unused Code introduced 2015-11-23 10:29 UTC by Report Bug Copy Issue Report Show Similar Issues like this The assignment to `$ns` 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...
67
68	9	$typeId = DataTypeRegistry::getInstance()->findTypeId( $type );
69
70	9	if ( $typeId === '' ) {
71	1	return;
72		}
73
74	8	$dataValue = DataValueFactory::getInstance()->newDataValueByProperty(
75		$property,
76		$typeId
77		);
78
79	8	$this->replaceAnyTypeByImportType( $property, $dataValue );
80	8	}
81
82	8	private function replaceAnyTypeByImportType( DIProperty $property, $dataValue ) {
83
84	8	foreach ( $this->getSemanticData()->getPropertyValues( $property ) as $dataItem ) {
85	2	$this->getSemanticData()->removePropertyObjectValue(
86		$property,
87		$dataItem
88		);
89		}
90
91	8	$this->getSemanticData()->addDataValue( $dataValue );
92	8	}
93
94		}
95

SemanticMediaWiki / SemanticMediaWiki

Push — master ( d2d28e...1c2760 )

MandatoryTypePropertyAnnotator.php (1 issue)

Showing only issues like (Show All)

Introduced By

Upgrade to new PHP Analysis Engine

Duplication Side-by-Side

Filter issues like