SemanticMediaWiki / SemanticMediaWiki

src/SPARQLStore/QueryEngine/DescriptionInterpreters/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/DescriptionInterpreters/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/DescriptionInterpreters/SomePropertyInterpreter.php

Doc Comments +10 added lines

@@ -150,6 +150,10 @@ 
 		return array( $innerOrderByProperty, $innerCondition, $innerJoinVariable );
 	}
 
+	/**
+	 * @param \SMW\SPARQLStore\QueryEngine\Condition\Condition $innerCondition
+	 * @param string $innerJoinVariable
+	 */
 	private function findObjectNameFromInnerCondition( $innerCondition, $innerJoinVariable, &$namespaces ) {
 
 		if ( !$innerCondition instanceof SingletonCondition ) {
@@ -201,6 +205,10 @@ 
 		return TurtleSerializer::getTurtleNameForExpElement( $propertyExpElement );
 	}
 
+	/**
+	 * @param string $objectName
+	 * @param string $joinVariable
+	 */
 	private function doExchangeForWhenInversePropertyIsUsed( DIProperty $property, $objectName, $joinVariable ) {
 
 		$subjectName = '?' . $joinVariable;
@@ -239,6 +247,8 @@ 
 	 * 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 \SMW\SPARQLStore\QueryEngine\Condition\Condition $condition
+	 * @param string $propertyName
 	 */
 	private function tryToAddPropertyPathForSaturatedHierarchy( &$condition, DIProperty $property, &$propertyName ) {
 

src/SPARQLStore/QueryEngine/DescriptionInterpreters/ValueDescriptionInterpreter.php

Doc Comments +19 added lines

@@ -104,10 +104,18 @@ 
 		);
 	}
 
+	/**
+	 * @param string $joinVariable
+	 * @param \SMW\DIProperty|null $orderByProperty
+	 */
 	private function createConditionForEmptyComparator( $joinVariable, $orderByProperty ) {
 		return $this->compoundConditionBuilder->newTrueCondition( $joinVariable, $orderByProperty );
 	}
 
+	/**
+	 * @param string $joinVariable
+	 * @param \SMW\DIProperty|null $orderByProperty
+	 */
 	private function createConditionForEqualityComparator( $dataItem, $property, $joinVariable, $orderByProperty ) {
 
 		$expElement = $this->exporter->getDataItemHelperExpElement( $dataItem );
@@ -153,6 +161,11 @@ 
 		return $condition;
 	}
 
+	/**
+	 * @param string $joinVariable
+	 * @param \SMW\DIProperty|null $orderByProperty
+	 * @param string $comparator
+	 */
 	private function createConditionForRegexComparator( $dataItem, $joinVariable, $orderByProperty, $comparator ) {
 
 		if ( !$dataItem instanceof DIBlob && !$dataItem instanceof DIWikiPage && !$dataItem instanceof DIUri ) {
@@ -196,6 +209,9 @@ 
 		return $condition;
 	}
 
+	/**
+	 * @param string $comparator
+	 */
 	private function createFilterConditionForAnyOtherComparator( $dataItem, $joinVariable, $orderByProperty, $comparator ) {
 
 		$result = new FilterCondition( '', array() );
@@ -228,6 +244,9 @@ 
 		return $result;
 	}
 
+	/**
+	 * @param string $pattern
+	 */
 	private function createFilterConditionToMatchRegexPattern( $dataItem, &$joinVariable, $comparator, $pattern ) {
 
 		if ( $dataItem instanceof DIBlob ) {

src/SQLStore/QueryEngine/DescriptionInterpreters/DispatchingDescriptionInterpreter.php

Doc Comments +1 added lines, -2 removed lines

@@ -44,7 +44,7 @@ 
 	/**
 	 * @param Description $description
 	 *
-	 * @return QuerySegment
+	 * @return \SMW\SQLStore\QueryEngine\QuerySegment
 	 * @throws InvalidArgumentException
 	 */
 	public function interpretDescription( Description $description ) {
@@ -63,7 +63,6 @@ 
 	/**
 	 * @since  2.2
 	 *
-	 * @param DescriptionInterpreter $defaultInterpreter
 	 */
 	public function addInterpreter( DescriptionInterpreter $interpreter ) {
 		$this->interpreters[] = $interpreter;

src/SQLStore/QueryEngine/DescriptionInterpreters/SomePropertyInterpreter.php

Doc Comments +1 added lines, -1 removed lines

@@ -197,7 +197,7 @@
 	 * ValueDescription objects, create and return a plain WHERE condition
 	 * string for it.
 	 *
-	 * @param $query
+	 * @param QuerySegment $query
 	 * @param Description $description
 	 * @param SMWSQLStore3Table $proptable
 	 * @param DataItemHandler $diHandler for that table

includes/storage/SQLStore/SMW_Sql3SmwIds.php

Doc Comments +2 added lines, -3 removed lines

@@ -723,7 +723,7 @@ 
 	 *
 	 * @param integer $sid
 	 * @param DIWikiPage $subject
-	 * @param integer|string|null $interWiki
+	 * @param string $interWiki
 	 */
 	public function updateInterwikiField( $sid, DIWikiPage $subject, $interWiki = null ) {
 
@@ -978,9 +978,8 @@ 
 	/**
 	 * @since 2.3
 	 *
-	 * @param integer $id
 	 *
-	 * @return string[]
+	 * @return SMW\Iterators\MappingIterator
 	 */
 	public function getDataItemPoolHashListFor( array $idlist ) {
 		return $this->idToDataItemMatchFinder->getDataItemPoolHashListFor( $idlist );

src/IteratorFactory.php

Doc Comments +1 added lines, -1 removed lines

@@ -27,7 +27,7 @@
 	/**
 	 * @since 2.5
 	 *
-	 * @param Iterator/array $$iterable
+	 * @param Iterator/array $iterable
 	 * @param callable $callback
 	 *
 	 * @return MappingIterator

tests/phpunit/Unit/Iterators/ResultIteratorTest.php

Unused Use Statements -1 removed lines

@@ -3,7 +3,6 @@
 namespace SMW\Iterators\Tests;
 
 use SMW\Iterators\ResultIterator;
-use ArrayIterator;
 
 /**
  * @covers \SMW\Iterators\ResultIterator