SemanticMediaWiki / SemanticMediaWiki

src/MediaWiki/Specials/Browse/HtmlContentBuilder.php

Doc Comments +5 added lines

@@ -264,6 +264,7 @@ 
 	 * @param[in] $property SMWPropertyValue  The property this value is linked to the subject with
 	 * @param[in] $value SMWDataValue  The actual value
 	 * @param[in] $incoming bool  If this is an incoming or outgoing link
+	 * @param boolean $incoming
 	 *
 	 * @return string  HTML with the link to the article, browse, and search pages
 	 */
@@ -337,6 +338,7 @@ 
 	 * Creates the HTML for the bottom bar including the links with further navigation options.
 	 *
 	 * @param[in] $more bool  Are there more inproperties to be displayed?
+	 * @param boolean $more
 	 * @return string  HTMl with the bottom bar
 	 */
 	private function displayBottom( $more ) {
@@ -366,6 +368,9 @@ 
 	 * @param[in] $out bool  Should the linked to page include outgoing properties?
 	 * @param[in] $in bool  Should the linked to page include incoming properties?
 	 * @param[in] $offset int  What is the offset for the incoming properties?
+	 * @param boolean $out
+	 * @param boolean $in
+	 * @param integer $offset
 	 *
 	 * @return string  HTML with the link to this page
 	 */

Unused Use Statements -1 removed lines

@@ -3,7 +3,6 @@
 namespace SMW\Tests;
 
 use SMW\CacheFactory;
-use SMW\ApplicationFactory;
 
 /**
  * @covers \SMW\CacheFactory

src/MediaWiki/Specials/SpecialBrowse.php

Unused Use Statements -3 removed lines

@@ -4,9 +4,6 @@
 
 use SMW\ApplicationFactory;
 use SMW\DataValueFactory;
-use SMW\DIProperty;
-use SMW\Localizer;
-use SMW\SemanticData;
 use SMW\UrlEncoder;
 use SMW\MediaWiki\Specials\Browse\HtmlContentBuilder;
 use SMW\Message;

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/SQLStore/QueryEngine/Fulltext/SearchTableRebuilder.php

Unused Use Statements -1 removed lines

@@ -7,7 +7,6 @@
 use SMW\SQLStore\QueryEngine\QuerySegmentListBuilder;
 use SMW\Query\Language\ConceptDescription;
 use SMW\DIWikiPage;
-use SMWDIBlob as DIBlob;
 
 /**
  * @covers \SMW\SQLStore\QueryEngine\Interpreter\ConceptDescriptionInterpreter

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 );

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 ) {

Unused Use Statements -1 removed lines

@@ -3,7 +3,6 @@
 use SMW\DataTypeRegistry;
 use SMW\DIWikiPage;
 use SMW\SQLStore\TableDefinition;
-use SMWDataItem as DataItem;
 
 /**
  * Class to provide all basic read methods for SMWSQLStore3.

tests/phpunit/Unit/ProcessingErrorMsgHandlerTest.php

Unused Use Statements -1 removed lines

@@ -4,7 +4,6 @@
 use SMW\DataValueFactory;
 use SMW\DIProperty;
 use SMW\Highlighter;
-use SMW\Message;
 
 /**
  * Objects of this class represent properties in SMW.

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;