SemanticMediaWiki / SemanticMediaWiki

src/Query/PrintRequest.php

Doc Comments +11 added lines, -3 removed lines

@@ -46,7 +46,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 ) {
@@ -130,6 +130,8 @@ 
 
 	/**
 	 * Convenience method for accessing the text in either HTML or Wiki format.
+	 * @param integer $outputmode
+	 * @return string
 	 */
 	public function getText( $outputmode, $linker = null ) {
 		switch ( $outputmode ) {
@@ -267,7 +269,7 @@ 
 	/**
 	 * Returns the value of a named parameter.
 	 *
-	 * @param $key string the name of the parameter key
+	 * @param string $key string the name of the parameter key
 	 *
 	 * @return string Value of the paramer, if set (else FALSE)
 	 */
@@ -287,7 +289,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 ) {
@@ -373,11 +375,17 @@ 
 		}
 	}
 
+	/**
+	 * @param string $text
+	 */
 	private static function isCategory( $text ) {
 		return Localizer::getInstance()->getNamespaceTextById( NS_CATEGORY ) == mb_convert_case( $text, MB_CASE_TITLE ) ||
 		$text == 'Category';
 	}
 
+	/**
+	 * @param string $text
+	 */
 	private static function doSplitText( $text ) {
 		// 1464
 		// Temporary encode "=" within a <> entity (<span>...</span>)

src/PropertyAnnotator/DisplayTitlePropertyAnnotator.php

Unused Use Statements -2 removed lines

@@ -2,9 +2,7 @@
 
 namespace SMW\PropertyAnnotator;
 
-use SMW\DIProperty;
 use SMW\PropertyAnnotator;
-use SMWDIBlob as DIBlob;
 
 /**
  * @license GNU GPL v2+

src/CacheFactory.php

Doc Comments +2 added lines, -2 removed lines

@@ -120,7 +120,7 @@ 
 	 *
 	 * @param integer|string $mediaWikiCacheType
 	 *
-	 * @return Cache
+	 * @return null|\Onoi\Cache\Cache
 	 */
 	public function newMediaWikiCompositeCache( $mediaWikiCacheType = null ) {
 
@@ -143,7 +143,7 @@ 
 	 * @param string|integer|null $cacheType
 	 * @param integer $cacheLifetime
 	 *
-	 * @return BlobStore
+	 * @return \Onoi\BlobStore\BlobStore
 	 */
 	public function newBlobStore( $namespace, $cacheType = null, $cacheLifetime = 0 ) {
 

src/SharedCallbackContainer.php

Doc Comments +6 added lines

@@ -29,6 +29,9 @@ 
 		$this->registerCallbackHandlersByFactory( $callbackLoader );
 	}
 
+	/**
+	 * @param CallbackLoader $callbackLoader
+	 */
 	private function registerCallbackHandlers( $callbackLoader ) {
 
 		$callbackLoader->registerExpectedReturnType( 'Settings', '\SMW\Settings' );
@@ -111,6 +114,9 @@ 
 		} );
 	}
 
+	/**
+	 * @param CallbackLoader $callbackLoader
+	 */
 	private function registerCallbackHandlersByFactory( $callbackLoader ) {
 
 		$callbackLoader->registerCallback( 'BlobStore', function( $namespace, $cacheType = null, $ttl = 0 ) use ( $callbackLoader ) {

src/SQLStore/RedirectInfoStore.php

Doc Comments +13 added lines

@@ -103,6 +103,10 @@ 
 		$this->inMemoryPoolCache->getPoolCacheFor( 'sql.store.redirect.infostore' )->delete( $hash );
 	}
 
+	/**
+	 * @param string $title
+	 * @param integer $namespace
+	 */
 	private function select( $title, $namespace ) {
 
 		$row = $this->connection->selectRow(
@@ -118,6 +122,11 @@ 
 		return $row !== false && isset( $row->o_id ) ? (int)$row->o_id : 0;
 	}
 
+	/**
+	 * @param integer $id
+	 * @param string $title
+	 * @param integer $namespace
+	 */
 	private function insert( $id, $title, $namespace ) {
 
 		$this->connection->insert(
@@ -130,6 +139,10 @@ 
 		);
 	}
 
+	/**
+	 * @param string $title
+	 * @param integer $namespace
+	 */
 	private function delete( $title, $namespace ) {
 
 		$this->connection->delete(

tests/phpunit/includes/storage/sqlstore/Sql3StubSemanticDataTest.php

Unused Use Statements -1 removed lines

@@ -5,7 +5,6 @@
 use SMW\DIProperty;
 use SMW\DIWikiPage;
 use SMW\SemanticData;
-use SMW\StoreFactory;
 use SMWDITime as DITime;
 use SMWSql3StubSemanticData as StubSemanticData;
 use Title;

src/DataValues/MonolingualTextValue.php

Doc Comments +4 added lines, -1 removed lines

@@ -72,7 +72,6 @@ 
 	 * @see DataValue::parseUserValue
 	 * @note called by DataValue::setUserValue
 	 *
-	 * @param string $value
 	 */
 	protected function parseUserValue( $userValue ) {
 
@@ -248,6 +247,7 @@ 
 	/**
 	 * @since 2.4
 	 *
+	 * @param string $languageCode
 	 * @return DataValue|null
 	 */
 	public function getTextValueByLanguage( $languageCode ) {
@@ -280,6 +280,9 @@ 
 		return $dataValue;
 	}
 
+	/**
+	 * @param string $value
+	 */
 	private function newContainerSemanticData( $value ) {
 
 		if ( $this->m_contextPage === null ) {

includes/articlepages/SMW_OrderedListPage.php

Doc Comments +2 added lines, -1 removed lines

@@ -119,6 +119,7 @@ 
 
 	/**
 	 * @since 2.4
+	 * @param string $msgKey
 	 */
 	protected function getNavigationLinks( $msgKey, array $diWikiPages, $default = 50 ) {
 		global $wgRequest;
@@ -178,7 +179,7 @@ 
 	 * If the method returns false, nothing will be printed besides
 	 * the original article.
 	 *
-	 * @return true
+	 * @return boolean
 	 */
 	protected function initParameters() {
 		$this->limit = 20;

includes/specials/SMW_SpecialBrowse.php

Doc Comments +8 added lines, -3 removed lines

@@ -170,7 +170,7 @@ 
 	 * @param[in] $left bool  Should properties be displayed on the left side?
 	 * @param[in] $incoming bool  Is this an incoming? Or an outgoing?
 	 *
-	 * @return A string containing the HTML with the factbox
+	 * @return string string containing the HTML with the factbox
 	 */
 	private function displayData( SMWSemanticData $data, $left = true, $incoming = false ) {
 		// Some of the CSS classes are different for the left or the right side.
@@ -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
 	 */
@@ -291,7 +292,7 @@ 
 	/**
 	 * Displays the subject that is currently being browsed to.
 	 *
-	 * @return A string containing the HTML with the subject line
+	 * @return string string containing the HTML with the subject line
 	 */
 	private function displayHead() {
 		global $wgOut;
@@ -324,6 +325,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 ) {
@@ -353,6 +355,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
 	 */
@@ -442,7 +447,7 @@ 
 	/**
 	 * Creates the query form in order to quickly switch to a specific article.
 	 *
-	 * @return A string containing the HTML for the form
+	 * @return string string containing the HTML for the form
 	 */
 	private function queryForm() {