SemanticMediaWiki / SemanticMediaWiki

src/DeferredRequestDispatchManager.php

Doc Comments +12 added lines, -2 removed lines

@@ -85,7 +85,7 @@ 
 	/**
 	 * @since 2.5
 	 *
-	 * @param Title $title
+	 * @param Title|null $title
 	 * @param array $parameters
 	 */
 	public function dispatchSearchTableUpdateJobFor( Title $title, $parameters = array() ) {
@@ -96,7 +96,7 @@ 
 	 * @since 2.3
 	 *
 	 * @param string $type
-	 * @param Title $title
+	 * @param Title|null $title
 	 * @param array $parameters
 	 */
 	public function dispatchJobRequestFor( $type, Title $title, $parameters = array() ) {
@@ -128,6 +128,9 @@ 
 		return true;
 	}
 
+	/**
+	 * @param string $type
+	 */
 	private function getDispatchableCallbackJobFor( $type ) {
 
 		$jobFactory = ApplicationFactory::getInstance()->newJobFactory();
@@ -170,6 +173,9 @@ 
 		return $callback;
 	}
 
+	/**
+	 * @param string $type
+	 */
 	private function doPreliminaryCheckForType( $type, array $parameters ) {
 
 		if ( $type !== 'SMW\ParserCachePurgeJob' && $type !== 'SMW\UpdateJob' && $type !== 'SMW\SearchTableUpdateJob' ) {
@@ -194,6 +200,10 @@ 
 		return self::$canConnectToUrl = $this->httpRequest->ping();
 	}
 
+	/**
+	 * @param string $type
+	 * @param \Closure $dispatchableCallbackJob
+	 */
 	private function doPostJobWith( $type, $title, $parameters, $dispatchableCallbackJob ) {
 
 		$parameters['async-job'] = array(

src/DataValues/ErrorMsgTextValue.php

Doc Comments +1 added lines, -1 removed lines

@@ -41,7 +41,7 @@
 	/**
 	 * @see DataValue::loadDataItem
 	 *
-	 * @param SMWDataItem $dataitem
+	 * @param DataItem $dataItem
 	 *
 	 * @return boolean
 	 */

src/Message.php

Doc Comments +2 added lines, -4 removed lines

@@ -60,7 +60,7 @@ 
 	/**
 	 * @since 2.4
 	 *
-	 * @param $type
+	 * @param string $type
 	 */
 	public static function deregisterHandlerFor( $type ) {
 		unset( self::$messageHandler[$type] );
@@ -96,7 +96,6 @@ 
 	 *
 	 * @since 2.5
 	 *
-	 * @param string|array $parameters
 	 * @param integer|null $type
 	 *
 	 * @return string
@@ -119,7 +118,7 @@ 
 	 *
 	 * @since 2.5
 	 *
-	 * @param string $messageId
+	 * @param string $message
 	 *
 	 * @return boolean
 	 */
@@ -130,7 +129,6 @@ 
 	/**
 	 * @since 2.5
 	 *
-	 * @param string $json
 	 * @param integer|null $type
 	 * @param integer|null $language
 	 *

src/ProcessingErrorMsgHandler.php

Doc Comments +9 added lines, -1 removed lines

@@ -113,7 +113,7 @@ 
 	/**
 	 * @since 2.5
 	 *
-	 * @param array|string $errorMsg
+	 * @param array|string $error
 	 * @param DIProperty|null $property
 	 *
 	 * @return DIContainer|null
@@ -171,6 +171,11 @@ 
 		return new DIContainer( $containerSemanticData );
 	}
 
+	/**
+	 * @param ContainerSemanticData $containerSemanticData
+	 * @param DIProperty|null $property
+	 * @param string $error
+	 */
 	private function addToContainerSemanticData( $containerSemanticData, $property, $error ) {
 
 		if ( $property !== null ) {
@@ -186,6 +191,9 @@ 
 		);
 	}
 
+	/**
+	 * @param string $hash
+	 */
 	private function newContainerSemanticData( $hash ) {
 
 		if ( $this->subject === null ) {

src/SQLStore/QueryEngine/Fulltext/SearchTableRebuilder.php

Doc Comments +7 added lines, -1 removed lines

@@ -86,7 +86,7 @@ 
 	 *
 	 * @since 2.5
 	 *
-	 * @return boolean
+	 * @return null|boolean
 	 */
 	public function run() {
 
@@ -160,6 +160,9 @@ 
 		$this->doRebuildFromRows( $searchTable, $table, $pid, $rows );
 	}
 
+	/**
+	 * @param SearchTable $searchTable
+	 */
 	private function doRebuildFromRows( $searchTable, $table, $pid, $rows ) {
 
 		$i = 0;
@@ -203,6 +206,9 @@ 
 		$this->reportMessage( "\n" );
 	}
 
+	/**
+	 * @param string $message
+	 */
 	private function reportMessage( $message, $verbose = true ) {
 		if ( $verbose ) {
 			$this->messageReporter->reportMessage( $message );

includes/datavalues/SMW_DV_Time.php

Doc Comments +16 added lines, -7 removed lines

@@ -186,14 +186,17 @@ 
 	 * The function sets the provided call-by-ref values to the respective
 	 * values. If errors are encountered, they are added to the objects
 	 * error list and false is returned. Otherwise, true is returned.
-	 * @param $string string input time representation, e.g. "12 May 2007 13:45:23-3:30"
+	 * @param string $string string input time representation, e.g. "12 May 2007 13:45:23-3:30"
 	 * @param $datecomponents array of strings that might belong to the specification of a date
 	 * @param $calendarmodesl string if model was set in input, otherwise false
-	 * @param $era string '+' or '-' if provided, otherwise false
-	 * @param $hours integer set to a value between 0 and 24
-	 * @param $minutes integer set to a value between 0 and 59
-	 * @param $seconds integer set to a value between 0 and 59, or false if not given
-	 * @param $timeoffset double set to a value for time offset (e.g. 3.5), or false if not given
+	 * @param boolean $era string '+' or '-' if provided, otherwise false
+	 * @param boolean $hours integer set to a value between 0 and 24
+	 * @param boolean $minutes integer set to a value between 0 and 59
+	 * @param boolean $seconds integer set to a value between 0 and 59, or false if not given
+	 * @param boolean $timeoffset double set to a value for time offset (e.g. 3.5), or false if not given
+	 * @param boolean $calendarmodel
+	 * @param boolean $microseconds
+	 * @param boolean $timezone
 	 * @return boolean stating if the parsing succeeded
 	 * @todo This method in principle allows date parsing to be internationalized further. Should be done.
 	 */
@@ -409,7 +412,7 @@ 
 	 * messages are added to the objects list of errors, and false is
 	 * returned. Otherwise, true is returned.
 	 * @param $datecomponents array of strings that might belong to the specification of a date
-	 * @param $date array set to result
+	 * @param boolean $date array set to result
 	 * @return boolean stating if successful
 	 */
 	protected function interpretDateComponents( $datecomponents, &$date ) {
@@ -807,10 +810,16 @@ 
 		}
 	}
 
+	/**
+	 * @param string $value
+	 */
 	private function isInterpretableAsYearOnly( $value ) {
 		return strpos( $value, ' ' ) === false && is_numeric( strval( $value ) ) && ( strval( $value ) < 0 || strlen( $value ) < 6 );
 	}
 
+	/**
+	 * @param string $value
+	 */
 	private function isInterpretableAsTimestamp( $value ) {
 		// 1200-11-02T12:03:25 or 20120320055913
 		return ( ( strlen( $value ) > 4 && substr( $value, 10, 1 ) === 'T' ) || strlen( $value ) == 14 ) && wfTimestamp( TS_MW, $value ) !== false;

includes/specials/SMW_SpecialAsk.php

Doc Comments +4 added lines

@@ -770,6 +770,9 @@ 
 		return '(' . $downloadLinks . ')';
 	}
 
+	/**
+	 * @param string $html
+	 */
 	private function doFinalModificationsOnBorrowedOutput( &$html, &$searchInfoText ) {
 
 		if ( !$this->getRequest()->getCheck( 'bTitle' ) ) {
@@ -790,6 +793,7 @@ 
 
 	/**
 	 * FIXME MW 1.25
+	 * @param string $key
 	 */
 	private function addExternalHelpLinkFor( $key ) {
 

src/Exporter/ResourceBuilders/PropertyValueResourceBuilder.php

Doc Comments +6 added lines

@@ -94,6 +94,9 @@ 
 		}
 	}
 
+	/**
+	 * @param DIProperty $property
+	 */
 	protected function getResourceElementForProperty( $property ) {
 
 		$key = 'resource:builder:' . $property->getKey();
@@ -112,6 +115,9 @@ 
 		return $resourceElement;
 	}
 
+	/**
+	 * @param DIProperty $property
+	 */
 	protected function getResourceElementHelperForProperty( $property ) {
 
 		$key = 'resource:builder:aux:' . $property->getKey();

src/DataValues/ExternalIdentifierValue.php

Doc Comments +3 added lines, -1 removed lines

@@ -40,7 +40,6 @@ 
 	/**
 	 * @see DataValue::getShortWikiText
 	 *
-	 * @param string $value
 	 */
 	public function getShortWikiText( $linker = null ) {
 
@@ -137,6 +136,9 @@ 
 		return $dataValue->getDataItem();
 	}
 
+	/**
+	 * @return boolean
+	 */
 	private function getExternalFormattedUri( $value ) {
 
 		if ( $this->externalFormattedUri !== null ) {