SemanticMediaWiki / SemanticMediaWiki

src/ParserFunctions/DocumentationParserFunction.php

Doc Comments +1 added lines, -1 removed lines

@@ -31,7 +31,7 @@
 	 * @param Parser $parser
 	 * @param ProcessingResult $result
 	 *
-	 * @return mixed
+	 * @return string
 	 */
 	public function handle( Parser $parser, ProcessingResult $result ) {
 		if ( $result->hasFatal() ) {

src/ParserFunctions/ShowParserFunction.php

Doc Comments -1 removed lines

@@ -48,7 +48,6 @@
 	 *
 	 * @since 1.9
 	 *
-	 * @param array $params
 	 *
 	 * @return string|null
 	 */

src/PropertyLabelFinder.php

Doc Comments +1 added lines, -1 removed lines

@@ -45,7 +45,7 @@
 	/**
 	 * @since 2.2
 	 *
-	 * @return array
+	 * @return string[]
 	 */
 	public function getKownPredefinedPropertyLabels() {
 		return $this->languageDependentPropertyLabels;

src/Query/Parser/DescriptionProcessor.php

Doc Comments +5 added lines

@@ -185,6 +185,7 @@ 
 	 *
 	 * The return value is the expected combined description. The object $currentDescription will
 	 * also be changed (if it was non-NULL).
+	 * @param integer $compoundType
 	 */
 	private function newCompoundDescription( Description $currentDescription = null, Description $newDescription = null, $compoundType = SMW_CONJUNCTION_QUERY ) {
 
@@ -220,6 +221,10 @@ 
 		}
 	}
 
+	/**
+	 * @param Description $currentDescription
+	 * @param Description $newDescription
+	 */
 	private function newCompoundDescriptionFor( $compoundType, $currentDescription, $newDescription ) {
 
 		if ( ( ( $compoundType & SMW_CONJUNCTION_QUERY ) != 0 && ( $currentDescription instanceof Conjunction ) ) ||

src/Query/QueryCreator.php

Doc Comments +9 added lines, -1 removed lines

@@ -104,7 +104,7 @@ 
 	 *
 	 * @param string $queryString
 	 *
-	 * @return Query
+	 * @return \SMWQuery
 	 */
 	public function create( $queryString ) {
 
@@ -243,10 +243,18 @@ 
 		return $orders;
 	}
 
+	/**
+	 * @param string $sort
+	 *
+	 * @return string
+	 */
 	private function getNormalizedSortLabel( $sort ) {
 		return Localizer::getInstance()->getNamespaceTextById( NS_CATEGORY ) == mb_convert_case( $sort, MB_CASE_TITLE ) ? '_INST' : $sort;
 	}
 
+	/**
+	 * @param string $key
+	 */
 	private function getConfiguration( $key, $default ) {
 		return isset( $this->configuration[$key] ) ? $this->configuration[$key] : $default;
 	}

Unused Use Statements -1 removed lines

@@ -3,7 +3,6 @@
 namespace SMW\Query;
 
 use SMW\QueryFactory;
-use SMW\DIWikiPage;
 use SMW\Localizer;
 use SMW\DataValueFactory;
 

src/Query/Result/CachedQueryResultPrefetcher.php

Doc Comments +20 added lines, -2 removed lines

@@ -211,7 +211,6 @@ 
 	/**
 	 * @since 2.5
 	 *
-	 * @param QueryEngine $queryEngine
 	 */
 	public function disableCache() {
 		$this->enabledCache = false;
@@ -283,7 +282,6 @@ 
 	/**
 	 * @since 2.5
 	 *
-	 * @param DIWikiPage|array $list
 	 * @param string $context
 	 */
 	public function resetCacheBy( $item, $context = '' ) {
@@ -315,6 +313,11 @@ 
 		return $this->enabledCache && $this->blobStore->canUse() && ( $query->getContextPage() !== null || ( $query->getContextPage() === null && $this->nonEmbeddedCacheLifetime > 0 ) );
 	}
 
+	/**
+	 * @param string $queryId
+	 * @param Query $query
+	 * @param \Onoi\BlobStore\Container $container
+	 */
 	private function newQueryResultFromCache( $queryId, $query, $container ) {
 
 		$results = array();
@@ -379,6 +382,12 @@ 
 		return $queryResult;
 	}
 
+	/**
+	 * @param QueryResult $queryResult
+	 * @param string $queryId
+	 * @param \Onoi\BlobStore\Container $container
+	 * @param Query $query
+	 */
 	private function addQueryResultToCache( $queryResult, $queryId, $container, $query ) {
 
 		$this->bufferedStatsdCollector->incr( 'misses' );
@@ -465,6 +474,9 @@ 
 		return md5( $subject . self::VERSION . $this->hashModifier );
 	}
 
+	/**
+	 * @param string $message
+	 */
 	private function log( $message, $context = array() ) {
 
 		if ( $this->logger === null ) {
@@ -474,6 +486,9 @@ 
 		$this->logger->info( $message, $context );
 	}
 
+	/**
+	 * @param Query $query
+	 */
 	private function getNoCacheId( $query ) {
 
 		$id = 'noCache.misc';
@@ -493,6 +508,9 @@ 
 		return $id;
 	}
 
+	/**
+	 * @param string $date
+	 */
 	private function initStats( $date ) {
 
 		$this->bufferedStatsdCollector->shouldRecord( $this->isEnabled() );

src/SPARQLStore/QueryEngine/CompoundConditionBuilder.php

Doc Comments +3 added lines, -1 removed lines

@@ -448,7 +448,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
 	 */
@@ -493,6 +492,9 @@ 
 		}
 	}
 
+	/**
+	 * @param string $propertyKey
+	 */
 	private function addOrderForUnknownPropertyKey( Condition &$condition, $propertyKey, $order ) {
 
 		if ( $propertyKey === '' || $propertyKey === '#' ) { // order by result page sortkey

Unused Use Statements -3 removed lines

@@ -10,9 +10,6 @@
 use SMW\Message;
 use SMW\PropertyHierarchyLookup;
 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/QueryEngine.php

Doc Comments +3 added lines

@@ -223,6 +223,9 @@
 		return QueryDebugOutputFormatter::getStringFrom( 'SPARQLStore', $entries, $query );
 	}
 
+	/**
+	 * @param Condition $condition
+	 */
 	private function isSingletonConditionWithElementMatch( $condition ) {
 		return $condition instanceof SingletonCondition && $condition->matchElement instanceof Element;
 	}

src/SQLStore/EntityRebuildDispatcher.php

Doc Comments +4 added lines, -1 removed lines

@@ -100,7 +100,7 @@ 
 	/**
 	 * @since 2.3
 	 *
-	 * @param array|false $namespaces
+	 * @param boolean $namespaces
 	 */
 	public function setRestrictionToNamespaces( $namespaces ) {
 		$this->namespaces = $namespaces;
@@ -369,6 +369,9 @@ 
 		}
 	}
 
+	/**
+	 * @param boolean $emptyRange
+	 */
 	private function findNextIdPosition( &$id, $emptyRange ) {
 
 		$nextPosition = $id + $this->iterationLimit;

Unused Use Statements -1 removed lines

@@ -7,7 +7,6 @@
 use SMW\DIProperty;
 use SMW\DIWikiPage;
 use SMW\SemanticData;
-use SMW\Store;
 use Title;
 
 /**