SemanticMediaWiki / SemanticMediaWiki

src/DataValues/ValueFormatters/ReferenceValueFormatter.php

Doc Comments +4 added lines

@@ -120,6 +120,10 @@
 		return $results;
 	}
 
+	/**
+	 * @param boolean $isValue
+	 * @param DataValue $dataValue
+	 */
 	private function findValueOutputFor( $isValue, $type, $dataValue, $linker ) {
 
 		// Turn Uri/Page links into a href representation when not used as value

Unused Use Statements -1 removed lines

@@ -5,7 +5,6 @@
 use SMW\SPARQLStore\QueryEngine\QueryEngine;
 use SMW\SPARQLStore\QueryEngine\EngineOptions;
 use SMW\SPARQLStore\QueryEngine\QueryResultFactory;
-use SMW\DIProperty;
 use SMWQuery as Query;
 
 /**

includes/storage/SMW_ResultArray.php

Doc Comments +2 added lines, -2 removed lines

@@ -134,7 +134,7 @@ 
 	 *
 	 * @since 1.6
 	 *
-	 * @return SMWDataItem|false
+	 * @return null|DataItem
 	 */
 	public function getNextDataItem() {
 		$this->loadContent();
@@ -267,7 +267,7 @@ 
 	 *
 	 * @param boolean $useLimit
 	 *
-	 * @return SMWRequestOptions|null
+	 * @return SMW\RequestOptions|null
 	 */
 	protected function getRequestOptions( $useLimit = true ) {
 		return $this->resultFieldMatchFinder->getRequestOptions( $useLimit );

src/Query/PrintRequest.php

Unused Use Statements -1 removed lines

@@ -2,7 +2,6 @@
 
 use SMW\InTextAnnotationParser;
 use SMW\Query\PrintRequest;
-use SMW\Localizer;
 use SMW\DataValueFactory;
 use SMWDIBlob as DIBlob;
 

Doc Comments +4 added lines, -3 removed lines

@@ -69,7 +69,7 @@ 
 	 * @param integer $mode a constant defining what to printout
 	 * @param string $label the string label to describe this printout
 	 * @param mixed $data optional data for specifying some request, might be a property object, title, or something else; interpretation depends on $mode
-	 * @param mixed $outputformat optional string for specifying an output format, e.g. an output unit
+	 * @param string $outputformat optional string for specifying an output format, e.g. an output unit
 	 * @param array|null $params optional array of further, named parameters for the print request
 	 */
 	public function __construct( $mode, $label, $data = null, $outputformat = false, array $params = null ) {
@@ -137,6 +137,7 @@ 
 
 	/**
 	 * Convenience method for accessing the text in either HTML or Wiki format.
+	 * @param integer $outputMode
 	 */
 	public function getText( $outputMode, $linker = null ) {
 		return Formatter::format( $this, $linker, $outputMode );
@@ -219,7 +220,7 @@ 
 	/**
 	 * Returns the value of a named parameter.
 	 *
-	 * @param $key string the name of the parameter key
+	 * @param string $key string the name of the parameter key
 	 *
 	 * @return string Value of the paramer, if set (else FALSE)
 	 */
@@ -239,7 +240,7 @@ 
 	/**
 	 * Sets a print request parameter.
 	 *
-	 * @param $key string Name of the parameter
+	 * @param string $key string Name of the parameter
 	 * @param $value string Value for the parameter
 	 */
 	public function setParameter( $key, $value ) {

src/Query/Result/EntityListAccumulator.php

Doc Comments +1 added lines

@@ -63,6 +63,7 @@
 	/**
 	 * @since  2.4
 	 *
+	 * @param string $queryID
 	 * @return array
 	 */
 	public function getEntityList( $queryID = null ) {

Unused Use Statements -1 removed lines

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

src/DataValues/PropertyChainValue.php

Doc Comments +5 added lines, -1 removed lines

@@ -76,6 +76,7 @@ 
 
 	/**
 	 * @see DataValue::getShortWikiText
+	 * @param string $linker
 	 */
 	public function getShortWikiText( $linker = null ) {
 
@@ -88,6 +89,7 @@ 
 
 	/**
 	 * @see DataValue::getLongWikiText
+	 * @param string $linker
 	 */
 	public function getLongWikiText( $linker = null ) {
 
@@ -154,7 +156,6 @@ 
 	 * @see DataValue::parseUserValue
 	 * @note called by DataValue::setUserValue
 	 *
-	 * @param string $userValue
 	 */
 	protected function parseUserValue( $value ) {
 
@@ -171,6 +172,9 @@ 
 		$this->m_dataitem = new DIBlob( $value );
 	}
 
+	/**
+	 * @param string $value
+	 */
 	private function initPropertyChain( $value ) {
 
 		$chain = explode( '.', $value );

src/SPARQLStore/QueryEngine/DescriptionInterpreters/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/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 ) {