SemanticMediaWiki / SemanticMediaWiki

src/PropertyAnnotator/MandatoryTypePropertyAnnotator.php

Doc Comments +6 added lines

@@ -51,6 +51,9 @@ 
 		$this->addTypeFromImportVocabulary( $property, current( $dataItems ) );
 	}
 
+	/**
+	 * @param DIProperty $property
+	 */
 	private function addTypeFromImportVocabulary( $property, $dataItem ) {
 
 		$importValue = DataValueFactory::getInstance()->newDataItemValue(
@@ -80,6 +83,9 @@ 
 		$this->replaceAnyTypeByImportType( $property, $dataValue );
 	}
 
+	/**
+	 * @param \SMWDataValue $dataValue
+	 */
 	private function replaceAnyTypeByImportType( DIProperty $property, $dataValue ) {
 
 		foreach ( $this->getSemanticData()->getPropertyValues( $property ) as $dataItem ) {

Unused Use Statements -1 removed lines

@@ -3,7 +3,6 @@
 namespace SMW\Tests\SQLStore;
 
 use SMW\SQLStore\SQLStoreFactory;
-use SMW\Store;
 use SMWSQLStore3;
 
 /**

src/PropertyHierarchyLookup.php

Doc Comments +8 added lines

@@ -122,6 +122,10 @@ 
 		return $this->findMatchesFor( '_SUBC', $category->getDBKey(), $category );
 	}
 
+	/**
+	 * @param string $id
+	 * @param string $key
+	 */
 	private function hasMatchFor( $id, $key, DIWikiPage $subject ) {
 
 		$key = 'm#' . $id . '#' . $key;
@@ -147,6 +151,10 @@ 
 		return $result !== array();
 	}
 
+	/**
+	 * @param string $id
+	 * @param string $key
+	 */
 	private function findMatchesFor( $id, $key, DIWikiPage $subject ) {
 
 		$key = 'f#' . $id . '#' . $key;

src/PropertySpecificationChangeNotifier.php

Doc Comments -1 removed lines

@@ -54,7 +54,6 @@
 	 *
 	 * @since 2.2
 	 *
-	 * @param array $declarationProperties
 	 */
 	public function setPropertiesToCompare( array $propertiesToCompare ) {
 		$this->propertiesToCompare = $propertiesToCompare;

src/Query/Language/ClassDescription.php

Doc Comments +4 added lines

@@ -99,6 +99,10 @@
 		return SMW_CATEGORY_QUERY;
 	}
 
+	/**
+	 * @param integer $maxsize
+	 * @param integer $maxdepth
+	 */
 	public function prune( &$maxsize, &$maxdepth, &$log ) {
 
 		if ( $maxsize >= $this->getSize() ) {

src/SPARQLStore/QueryEngine/CompoundConditionBuilder.php

Doc Comments +3 added lines, -1 removed lines

@@ -442,7 +442,6 @@ 
 	 * Extend the given SPARQL condition by a suitable order by variable,
 	 * possibly adding conditions if required for the type of data.
 	 *
-	 * @param Condition $sparqlCondition condition to modify
 	 * @param string $mainVariable the variable that represents the value to be ordered
 	 * @param integer $diType DataItem type id
 	 */
@@ -487,6 +486,9 @@ 
 		}
 	}
 
+	/**
+	 * @param string $propertyKey
+	 */
 	private function addOrderForUnknownPropertyKey( Condition &$condition, $propertyKey ) {
 
 		if ( $propertyKey === '' ) { // order by result page sortkey

Unused Use Statements -1 removed lines

@@ -11,7 +11,6 @@
 use SMW\Query\Language\Description;
 use SMW\Query\Language\SomeProperty;
 use SMW\Query\Language\ThingDescription;
-use SMW\SPARQLStore\HierarchyFinder;
 use SMW\SPARQLStore\QueryEngine\Condition\Condition;
 use SMW\SPARQLStore\QueryEngine\Condition\SingletonCondition;
 use SMW\SPARQLStore\QueryEngine\Condition\TrueCondition;

src/SPARQLStore/QueryEngine/Interpreter/ClassDescriptionInterpreter.php

Doc Comments +6 added lines

@@ -82,6 +82,9 @@ 
 		return $result;
 	}
 
+	/**
+	 * @param string $joinVariable
+	 */
 	private function mapCategoriesToConditionElements( array $categories, $joinVariable ) {
 
 		$condition = '';
@@ -109,6 +112,9 @@ 
 		return array( $condition, $namespaces );
 	}
 
+	/**
+	 * @param string $categoryExpName
+	 */
 	private function tryToAddClassHierarchyPattern( $category, &$categoryExpName ) {
 
 		if ( !$this->compoundConditionBuilder->canUseQFeature( SMW_SPARQL_QF_SUBC ) ) {

src/SPARQLStore/QueryEngine/Interpreter/ConjunctionInterpreter.php

Doc Comments +10 added lines

@@ -100,6 +100,10 @@ 
 		return $result;
 	}
 
+	/**
+	 * @param string $joinVariable
+	 * @param \SMW\DIProperty|null $orderByProperty
+	 */
 	private function doPreliminarySubDescriptionCheck( $subDescriptions, $joinVariable, $orderByProperty ) {
 
 		$count = count( $subDescriptions );
@@ -126,6 +130,9 @@ 
 		return null;
 	}
 
+	/**
+	 * @param string $joinVariable
+	 */
 	private function doResolveSubDescriptionsRecursively( $subDescriptions, $joinVariable ) {
 
 		// Using a stdClass as data container for simpler handling in follow-up tasks
@@ -195,6 +202,9 @@ 
 		return $subConditionElements;
 	}
 
+	/**
+	 * @param \stdClass $subConditionElements
+	 */
 	private function createConditionFromSubConditionElements( $subConditionElements ) {
 
 		if ( $subConditionElements->singletonMatchElement instanceof ExpElement ) {

src/SPARQLStore/QueryEngine/Interpreter/DisjunctionInterpreter.php

Doc Comments +11 added lines

@@ -107,6 +107,10 @@ 
 		return $result;
 	}
 
+	/**
+	 * @param string $joinVariable
+	 * @param \SMW\DIProperty|null $orderByProperty
+	 */
 	private function doPreliminarySubDescriptionCheck( $subDescriptions, $joinVariable, $orderByProperty ) {
 
 		$count = count( $subDescriptions );
@@ -131,6 +135,10 @@ 
 		return null;
 	}
 
+	/**
+	 * @param string $joinVariable
+	 * @param \SMW\DIProperty|null $orderByProperty
+	 */
 	private function doResolveSubDescriptionsRecursively( $subDescriptions, $joinVariable, $orderByProperty ) {
 
 		// Using a stdClass as data container for simpler handling in follow-up tasks
@@ -213,6 +221,9 @@ 
 		return $subConditionElements;
 	}
 
+	/**
+	 * @param string $joinVariable
+	 */
 	private function createConditionFromSubConditionElements( $subConditionElements, $joinVariable ) {
 
 		if ( $subConditionElements->unionCondition === '' ) {

src/SPARQLStore/QueryEngine/Interpreter/SomePropertyInterpreter.php

Doc Comments +8 added lines

@@ -150,6 +150,9 @@ 
 		return array( $innerOrderByProperty, $innerCondition, $innerJoinVariable );
 	}
 
+	/**
+	 * @param string $innerJoinVariable
+	 */
 	private function findObjectNameFromInnerCondition( $innerCondition, $innerJoinVariable, &$namespaces ) {
 
 		if ( !$innerCondition instanceof SingletonCondition ) {
@@ -201,6 +204,10 @@ 
 		return TurtleSerializer::getTurtleNameForExpElement( $propertyExpElement );
 	}
 
+	/**
+	 * @param string $objectName
+	 * @param string $joinVariable
+	 */
 	private function doExchangeForWhenInversePropertyIsUsed( DIProperty $property, $objectName, $joinVariable ) {
 
 		$subjectName = '?' . $joinVariable;
@@ -239,6 +246,7 @@ 
 	 * can be found using the "zero or more" will resolve the complete path
 	 *
 	 * @see http://www.w3.org/TR/sparql11-query/#propertypath-arbitrary-length
+	 * @param string $propertyName
 	 */
 	private function tryToAddPropertyPathForSaturatedHierarchy( &$condition, DIProperty $property, &$propertyName ) {