SemanticMediaWiki / SemanticMediaWiki

src/SQLStore/SQLStoreFactory.php

Doc Comments +3 added lines, -3 removed lines

@@ -59,7 +59,7 @@ 
 	/**
 	 * @since 2.2
 	 *
-	 * @return QueryEngine
+	 * @return QueryEngine\QueryEngine
 	 */
 	public function newMasterQueryEngine() {
 		return $this->queryEngineFactory->newQueryEngine();
@@ -68,7 +68,7 @@ 
 	/**
 	 * @since 2.2
 	 *
-	 * @return QueryEngine
+	 * @return QueryEngine\QueryEngine
 	 */
 	public function newSlaveQueryEngine() {
 		return $this->newMasterQueryEngine();
@@ -421,7 +421,7 @@ 
 	/**
 	 * @since 2.5
 	 *
-	 * @return LoggerInterface
+	 * @return \Psr\Log\LoggerInterface
 	 */
 	public function getLogger() {
 		return ApplicationFactory::getInstance()->getMediaWikiLogger();

src/SQLStore/ConceptCache.php

Doc Comments -1 removed lines

@@ -38,7 +38,6 @@
 	 * @since 2.2
 	 *
 	 * @param SMWSQLStore3 $store
-	 * @param ConceptQuerySegmentBuilder $conceptQueryResolver
 	 */
 	public function __construct( SMWSQLStore3 $store, ConceptQuerySegmentBuilder $conceptQuerySegmentBuilder ) {
 		$this->store = $store;

src/SQLStore/QueryEngine/QuerySegmentListBuildManager.php

Doc Comments +2 added lines, -2 removed lines

@@ -66,7 +66,7 @@ 
 	/**
 	 * @since 2.5
 	 *
-	 * @return array
+	 * @return string[]
 	 */
 	public function getErrors() {
 		return $this->errors;
@@ -75,7 +75,7 @@ 
 	/**
 	 * @since 2.5
 	 *
-	 * @return array
+	 * @return QuerySegment[]
 	 */
 	public function getQuerySegmentList() {
 		return $this->querySegmentList;

Unused Use Statements -1 removed lines

@@ -2,7 +2,6 @@
 
 namespace SMW\SQLStore\QueryEngine;
 
-use SMW\DIWikiPage;
 use SMWQuery as Query;
 use SMW\MediaWiki\Database;
 use SMW\SQLStore\SQLStore;

tests/phpunit/TestEnvironment.php

Doc Comments +2 added lines, -2 removed lines

@@ -183,7 +183,7 @@ 
 	/**
 	 * @since 2.5
 	 *
-	 * @param $originalClassName
+	 * @param string $originalClassName
 	 * @param array $configuration
 	 *
 	 * @return PHPUnit_Framework_MockObject_MockObject
@@ -228,7 +228,7 @@ 
 	 * @since 2.5
 	 *
 	 * @param integer $index
-	 * @param string $url
+	 * @param string $text
 	 *
 	 * @return string
 	 */

Unused Use Statements -1 removed lines

@@ -5,7 +5,6 @@
 use SMW\ApplicationFactory;
 use SMW\DataValueFactory;
 use SMW\DeferredCallableUpdate;
-use SMW\Store;
 use SMW\Localizer;
 use SMW\Tests\Utils\UtilityFactory;
 use SMW\Tests\Utils\Mock\ConfigurableStub;

includes/dataitems/SMW_DI_Property.php

Doc Comments +5 added lines

@@ -307,6 +307,7 @@ 
 	/**
 	 * @since  2.0
 	 *
+	 * @param string $propertyTypeId
 	 * @return self
 	 * @throws PropertyDataTypeLookupExeption
 	 * @throws InvalidArgumentException
@@ -460,6 +461,7 @@ 
 
 	/**
 	 * @deprecated since 2.1, use PropertyRegistry::findPropertyIdByLabel
+	 * @param string $label
 	 */
 	public static function findPropertyID( $label, $useAlias = true ) {
 		return PropertyRegistry::getInstance()->findPropertyIdByLabel( $label, $useAlias );
@@ -493,6 +495,9 @@ 
 		PropertyRegistry::getInstance()->registerPropertyAlias( $id, $label );
 	}
 
+	/**
+	 * @param string $subobjectName
+	 */
 	private function newDIWikiPage( $dbkey, $subobjectName ) {
 
 		// If an inverse marker is present just omit the marker so a normal

includes/dataitems/SMW_DI_Time.php

Doc Comments +8 added lines, -3 removed lines

@@ -335,7 +335,7 @@ 
 	 *
 	 * @param string $timestamp must be in format
 	 *
-	 * @return SMWDITime|false
+	 * @return SMWDataItem
 	 */
 	public static function newFromTimestamp( $timestamp ) {
 		$timestamp = wfTimestamp( TS_MW, (string)$timestamp );
@@ -508,10 +508,10 @@ 
 	 * calendar model, presicion.
 	 *
 	 * @param double $jdValue
-	 * @param integer|null $calendarmodel
+	 * @param integer|null $calendarModel
 	 * @param integer|null $precision
 	 *
-	 * @return DITime object
+	 * @return SMWDITime object
 	 */
 	public static function newFromJD( $jdValue, $calendarModel = null, $precision = null, $timezone = false ) {
 
@@ -593,6 +593,11 @@ 
 		return $this->m_day > self::getDayNumberForMonth( $this->m_month, $this->m_year, $this->m_model );
 	}
 
+	/**
+	 * @param boolean $month
+	 * @param boolean $day
+	 * @param boolean $hour
+	 */
 	private function setPrecisionLevelBy( $month, $day, $hour ) {
 		if ( $month === false ) {
 			$this->m_precision = self::PREC_Y;

includes/datavalues/SMW_DV_URI.php

Doc Comments +7 added lines

@@ -45,6 +45,9 @@ 
 	 */
 	private $showUrlContextInRawFormat = true;
 
+	/**
+	 * @param string $typeid
+	 */
 	public function __construct( $typeid ) {
 		parent::__construct( $typeid );
 		switch ( $typeid ) {
@@ -182,6 +185,7 @@ 
 	 * Returns true if the argument is a valid RFC 3966 phone number.
 	 * Only global phone numbers are supported, and no full validation
 	 * of parameters (appended via ;param=value) is performed.
+	 * @param string $s
 	 */
 	protected static function isValidTelURI( $s ) {
 		$tel_uri_regex = '<^tel:\+[0-9./-]*[0-9][0-9./-]*(;[0-9a-zA-Z-]+=(%[0-9a-zA-Z][0-9a-zA-Z]|[0-9a-zA-Z._~:/?#[\]@!$&\'()*+,;=-])*)*$>';
@@ -340,6 +344,9 @@ 
 		return str_replace( ':', '&#58;', $url );
 	}
 
+	/**
+	 * @param string $context
+	 */
 	private function decodeUriContext( $context, $linker ) {
 
 		// Prior to decoding turn any `-` into an internal representation to avoid

includes/query/SMW_QueryParser.php

Doc Comments +9 added lines

@@ -142,6 +142,7 @@ 
 	 * but just controls query generation. For general effect, the default namespaces
 	 * should be set to NULL.
 	 *
+	 * @param boolean $setNS
 	 * @return Description|null
 	 */
 	private function getSubqueryDescription( &$setNS ) {
@@ -250,6 +251,7 @@ 
 	 * be the content of "[[ ... ]]". Returns NULL upon error.
 	 *
 	 * Parameters $setNS has the same use as in getSubqueryDescription().
+	 * @param boolean $setNS
 	 */
 	private function getLinkDescription( &$setNS ) {
 		// This method is called when we encountered an opening '[['. The following
@@ -433,6 +435,7 @@ 
 	 * a category or property) and create a suitable description.
 	 * The first chunk behind the "[[" has already been read and is
 	 * passed as a parameter.
+	 * @param string $firstChunk
 	 */
 	private function getArticleDescription( $firstChunk, &$setNS ) {
 		$chunk = $firstChunk;
@@ -478,6 +481,10 @@ 
 		return $this->finishLinkDescription( $chunk, true, $result, $setNS );
 	}
 
+	/**
+	 * @param boolean $hasNamespaces
+	 * @param Description|null $result
+	 */
 	private function finishLinkDescription( $chunk, $hasNamespaces, $result, &$setNS ) {
 		if ( is_null( $result ) ) { // no useful information or concrete error found
 			$this->descriptionProcessor->addErrorWithMsgKey( 'smw_unexpectedpart', $chunk ); // was smw_badqueryatom
@@ -574,6 +581,7 @@ 
 	/**
 	 * Enter a new subblock in the query, which must at some time be terminated by the
 	 * given $endstring delimiter calling popDelimiter();
+	 * @param string $endstring
 	 */
 	private function pushDelimiter( $endstring ) {
 		array_push( $this->separatorStack, $endstring );
@@ -583,6 +591,7 @@ 
 	 * Exit a subblock in the query ending with the given delimiter.
 	 * If the delimiter does not match the top-most open block, false
 	 * will be returned. Otherwise return true.
+	 * @param string $endstring
 	 */
 	private function popDelimiter( $endstring ) {
 		$topdelim = array_pop( $this->separatorStack );

includes/queryprinters/CategoryResultPrinter.php

Doc Comments +4 added lines

@@ -224,6 +224,10 @@
 		return ByLanguageCollationMapper::getInstance()->findFirstLetterForCategory( $sortKey );
 	}
 
+	/**
+	 * @param SMWQueryResult $res
+	 * @param boolean $first_col
+	 */
 	private function addRowFieldsToTemplate( $res, $row, &$first_col, $templateRenderer ) {
 
 		// explicitly number parameters for more robust parsing (values may contain "=")