SemanticMediaWiki / SemanticMediaWiki

src/SQLStore/EntityStore/CachedEntityLookup.php

Doc Comments +4 added lines

@@ -402,6 +402,10 @@
 		) . $suffix );
 	}
 
+	/**
+	 * @param string $id
+	 * @param DIWikiPage $subject
+	 */
 	private function appendToList( $id, $subject ) {
 
 		// Store the id with the main subject

src/SQLStore/QueryDependency/QueryResultDependencyListResolver.php

Doc Comments +3 added lines

@@ -232,6 +232,9 @@
 		}
 	}
 
+	/**
+	 * @param DIWikiPage|null $subject
+	 */
 	private function doMatchSubproperty( &$subjects, $subject, DIProperty $property ) {
 
 		$subproperties = array();

Unused Use Statements -2 removed lines

@@ -13,8 +13,6 @@
 use SMW\Query\Language\SomeProperty;
 use SMW\Query\Language\ThingDescription;
 use SMW\Query\Language\ValueDescription;
-use SMW\Store;
-use SMWQuery as Query;
 use SMWQueryResult as QueryResult;
 
 /**

src/Factbox/CachedFactbox.php

Doc Comments +4 added lines, -2 removed lines

@@ -79,7 +79,8 @@ 
 	/**
 	 * @since 2.5
 	 *
-	 * @return boolean
+	 * @param integer $expiryInSeconds
+	 * @return boolean|null
 	 */
 	public function setExpiryInSeconds( $expiryInSeconds ) {
 		$this->expiryInSeconds = $expiryInSeconds;
@@ -88,7 +89,7 @@ 
 	/**
 	 * @since 2.5
 	 *
-	 * @return boolean
+	 * @return boolean|null
 	 */
 	public function isEnabled( $isEnabled ) {
 		$this->isEnabled = $isEnabled;
@@ -280,6 +281,7 @@ 
 	/**
 	 * Cached content is serialized in an associative array following:
 	 * { 'revId' => $revisionId, 'text' => (...) }
+	 * @param string $key
 	 */
 	private function saveToCache( $key, array $content ) {
 

src/MediaWiki/Specials/SearchByProperty/PageRequestOptions.php

Doc Comments +7 added lines

@@ -119,6 +119,10 @@ 
 		$this->setNearbySearch();
 	}
 
+	/**
+	 * @param string $value
+	 * @param boolean $escaped
+	 */
 	private function getValue( $value, $escaped ) {
 
 		$this->value = DataValueFactory::getInstance()->newDataValueByProperty(
@@ -131,6 +135,9 @@ 
 		return $this->value->isValid() ? $this->value->getWikiValue() : $value;
 	}
 
+	/**
+	 * @return string
+	 */
 	private function unescape( $value, $escaped ) {
 
 		if ( $this->value instanceof NumberValue ) {

Unused Use Statements -2 removed lines

@@ -4,11 +4,9 @@
 
 use SMW\DataValueFactory;
 use SMW\DataValues\TelephoneUriValue;
-use SMWUriValue as UriValue;
 use SMW\UrlEncoder;
 use SMWNumberValue as NumberValue;
 use SMWPropertyValue as PropertyValue;
-use SMWStringValue as TextValue;
 
 /**
  * @license GNU GPL v2+

src/Query/PrintRequest.php

Doc Comments +3 added lines, -2 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 );
@@ -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 ) {

Unused Use Statements -1 removed lines

@@ -3,7 +3,6 @@
 namespace SMW\Query;
 
 use InvalidArgumentException;
-use SMW\Localizer;
 use SMWDataValue;
 use SMWPropertyValue as PropertyValue;
 use SMW\DataValues\PropertyChainValue;

src/Query/Result/ResultFieldMatchFinder.php

Doc Comments +6 added lines

@@ -156,6 +156,9 @@ 
 		return $options;
 	}
 
+	/**
+	 * @param DataItem $dataItem
+	 */
 	private function getResultsForProperty( $dataItem ) {
 
 		$content = $this->getResultContent(
@@ -212,6 +215,9 @@ 
 		return $content;
 	}
 
+	/**
+	 * @param string $parameter
+	 */
 	private function isMultiValueWithParameter( $parameter ) {
 		return strpos( $this->printRequest->getTypeID(), '_rec' ) !== false && $this->printRequest->getParameter( $parameter ) !== false;
 	}

includes/datavalues/SMW_DV_Record.php

Doc Comments +8 added lines, -1 removed lines

@@ -59,7 +59,7 @@ 
 	 *
 	 * @param string $value
 	 *
-	 * @return array
+	 * @return string[]
 	 */
 	public function getValuesFromString( $value ) {
 		// #664 / T17732
@@ -280,6 +280,10 @@ 
 		return $result;
 	}
 
+	/**
+	 * @param integer $type
+	 * @param SMWDataValue $dataValue
+	 */
 	protected function makeValueOutputText( $type, $dataValue, $linker ) {
 		switch ( $type ) {
 			case 0:
@@ -295,6 +299,9 @@ 
 		}
 	}
 
+	/**
+	 * @param string $value
+	 */
 	private function newContainerSemanticData( $value ) {
 
 		if ( $this->m_contextPage === null ) {

Unused Use Statements -1 removed lines

@@ -5,7 +5,6 @@
 use SMW\ApplicationFactory;
 use SMW\DIProperty;
 use SMW\DIWikiPage;
-use SMWPropertyListValue as PropertyListValue;
 use SMWDataItem as DataItem;
 use SMWContainerSemanticData as ContainerSemanticData;
 use SMWDIContainer as DIContainer;

src/PropertySpecificationLookup.php

Doc Comments +16 added lines, -1 removed lines

@@ -77,7 +77,7 @@ 
 	 *
 	 * @param DIProperty $property
 	 *
-	 * @return false|DataItem
+	 * @return \SMWDataItem
 	 */
 	public function getFieldListBy( DIProperty $property ) {
 
@@ -383,6 +383,10 @@ 
 		return $localPropertyDescription;
 	}
 
+	/**
+	 * @param DIProperty $property
+	 * @param string $languageCode
+	 */
 	private function getPredefinedPropertyDescription( $property, $linker, $languageCode ) {
 
 		$description = '';
@@ -411,6 +415,10 @@ 
 		return $message;
 	}
 
+	/**
+	 * @param DIProperty $property
+	 * @param string $languageCode
+	 */
 	private function findLocalPropertyDescription( $property, $linker, $languageCode ) {
 
 		$text = '';
@@ -428,6 +436,10 @@ 
 		return $text;
 	}
 
+	/**
+	 * @param DIProperty $property
+	 * @param string $languageCode
+	 */
 	private function findPreferredPropertyLabel( $property, $languageCode ) {
 
 		$text = '';
@@ -445,6 +457,9 @@ 
 		return $text;
 	}
 
+	/**
+	 * @param DIProperty $property
+	 */
 	private function findTextValueByLanguage( $dataItems, $property, $languageCode ) {
 
 		if ( $dataItems === null || $dataItems === array() ) {

src/SQLStore/QueryEngine/OrderConditionsComplementor.php

Doc Comments +11 added lines, -1 removed lines

@@ -77,7 +77,7 @@ 
 	/**
 	 * @since 2.5
 	 *
-	 * @return array
+	 * @return string[]
 	 */
 	public function getErrors() {
 		return $this->querySegmentListBuilder->getErrors();
@@ -120,6 +120,9 @@ 
 		return $this->querySegmentListBuilder->getQuerySegmentList();
 	}
 
+	/**
+	 * @param QuerySegment $querySegment
+	 */
 	private function collectExtraDescriptionsFromSortKeys( $querySegment ) {
 
 		$extraDescriptions = array();
@@ -138,6 +141,10 @@ 
 		return $extraDescriptions;
 	}
 
+	/**
+	 * @param string $label
+	 * @param string $order
+	 */
 	private function findExtraDescriptionBy( $querySegment, $label, $order ) {
 
 		$description = null;
@@ -171,6 +178,9 @@ 
 		return $description;
 	}
 
+	/**
+	 * @param QuerySegment $querySegment
+	 */
 	private function addConjunctionFromExtraDescriptions( $querySegment, array $extraDescriptions ) {
 
 		$this->querySegmentListBuilder->setSortKeys(