SemanticMediaWiki / SemanticMediaWiki

includes/queryprinters/ResultPrinter.php

Doc Comments +3 added lines, -3 removed lines

@@ -188,7 +188,7 @@ 
 	 *
 	 * @param $results SMWQueryResult
 	 * @param $fullParams array
-	 * @param $outputMode integer
+	 * @param integer $outputMode integer
 	 *
 	 * @return string
 	 */
@@ -465,7 +465,7 @@ 
 	/**
 	 * @see SMWIResultPrinter::getQueryMode
 	 *
-	 * @param $context
+	 * @param integer $context
 	 *
 	 * @return integer
 	 */
@@ -607,7 +607,7 @@ 
 	 *
 	 * @param ParamDefinition[] $definitions
 	 *
-	 * @return array
+	 * @return \ParamProcessor\IParamDefinition[]
 	 */
 	public function getParamDefinitions( array $definitions ) {
 		return array_merge( $definitions, $this->getParameters() );

includes/SemanticData.php

Doc Comments +3 added lines, -3 removed lines

@@ -173,7 +173,7 @@ 
 	 * output, then the Semdata container will at least look as if properly
 	 * initialised (though empty).
 	 *
-	 * @return array
+	 * @return string[]
 	 */
 	public function __sleep() {
 		return array( 'mSubject', 'mPropVals', 'mProperties', 'subSemanticData', 'mHasVisibleProps', 'mHasVisibleSpecs', 'lastModified' );
@@ -367,7 +367,7 @@ 
 	 * Store a value for a given property identified by its text label
 	 * (without namespace prefix).
 	 *
-	 * @param $propertyName string
+	 * @param string $propertyName string
 	 * @param $dataItem SMWDataItem
 	 */
 	public function addPropertyValue( $propertyName, SMWDataItem $dataItem ) {
@@ -592,7 +592,7 @@ 
 	 *
 	 * @since 1.9
 	 *
-	 * @param string $subobjectName|null
+	 * @param string $subobjectName
 	 *
 	 * @return boolean
 	 */

includes/SMW_Infolink.php

Doc Comments +7 added lines, -5 removed lines

@@ -76,8 +76,8 @@ 
 	 *
 	 * @param string $caption The label for the link.
 	 * @param string $target The actual link target.
-	 * @param mixed $style CSS class of a span to embedd the link into, or false if no extra style is required.
-	 * @param array $params Array of parameters, format $name => $value, if any.
+	 * @param string|false $style CSS class of a span to embedd the link into, or false if no extra style is required.
+	 * @param string[] $params Array of parameters, format $name => $value, if any.
 	 *
 	 * @return SMWInfolink
 	 */
@@ -125,8 +125,8 @@ 
 	 *
 	 * @param string $caption The label for the link.
 	 * @param string $subject
-	 * @param string $propertyName
-	 * @param mixed $style CSS class of a span to embed the link into, or false if no extra style is required.
+	 * @param string $propertyname
+	 * @param string $style CSS class of a span to embed the link into, or false if no extra style is required.
 	 *
 	 * @return SMWInfolink
 	 */
@@ -145,7 +145,7 @@ 
 	 *
 	 * @param string $caption The label for the link.
 	 * @param string $titleText
-	 * @param mixed $style CSS class of a span to embedd the link into, or false if no extra style is required.
+	 * @param string $style CSS class of a span to embedd the link into, or false if no extra style is required.
 	 *
 	 * @return SMWInfolink
 	 */
@@ -188,6 +188,7 @@ 
 
 	/**
 	 * Change the link text.
+	 * @param string $caption
 	 */
 	public function setCaption( $caption ) {
 		$this->mCaption = $caption;
@@ -195,6 +196,7 @@ 
 
 	/**
 	 * Change the link's CSS class.
+	 * @param string $style
 	 */
 	public function setStyle( $style ) {
 		$this->mStyle = $style;

includes/SMW_Outputs.php

Doc Comments +1 added lines, -2 removed lines

@@ -65,7 +65,6 @@ 
 	 * recorded multiple times in SMWOutputs.
 	 *
 	 * @param string $id
-	 * @param string $item
 	 */
 	public static function requireScript( $id, $script ) {
 		self::$scripts[$id] = $script;
@@ -81,7 +80,7 @@ 
 	 * Support for calling this with the old constants SMW_HEADER_STYLE
 	 * and SMW_HEADER_TOOLTIP will vanish in SMW 1.7 at the latest.
 	 *
-	 * @param mixed $id
+	 * @param string $id
 	 * @param string $item
 	 */
 	public static function requireHeadItem( $id, $item = '' ) {

includes/specials/SMW_SpecialBrowse.php

Doc Comments +9 added lines, -3 removed lines

@@ -151,7 +151,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.
@@ -245,6 +245,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
 	 */
@@ -267,7 +268,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;
@@ -300,6 +301,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 ) {
@@ -329,6 +331,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
 	 */
@@ -418,7 +423,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() {
 		SMWOutputs::requireResource( 'ext.smw.browse' );
@@ -445,6 +450,7 @@ 
 
 	/**
 	 * FIXME MW 1.25
+	 * @param string $key
 	 */
 	private function addExternalHelpLinkFor( $key ) {
 

includes/specials/SpecialConcepts.php

Doc Comments +1 added lines, -1 removed lines

@@ -42,7 +42,7 @@
 	 * @param integer $from
 	 * @param integer $until
 	 *
-	 * @return DIWikiPage[]
+	 * @return \SMWDIWikiPage[]
 	 */
 	public function getResults( $limit, $from, $until ) {
 		$description = new SomeProperty( new DIProperty( '_CONC' ), new ThingDescription() );

Unused Use Statements -1 removed lines

@@ -6,7 +6,6 @@
 use ImportReporter;
 use WikiImporter;
 use RequestContext;
-
 use RuntimeException;
 
 /**

includes/storage/SMW_ResultArray.php

Doc Comments +1 added lines, -1 removed lines

@@ -104,7 +104,7 @@
 	 *
 	 * @since 1.6
 	 *
-	 * @return SMWDataItem|false
+	 * @return null|SMWDataItem
 	 */
 	public function getNextDataItem() {
 		$this->loadContent();

includes/storage/SMW_SQLHelpers.php

Doc Comments +2 added lines, -5 removed lines

@@ -63,8 +63,7 @@ 
 	 * @note The function partly ignores the order in which fields are set up.
 	 * Only if the type of some field changes will its order be adjusted explicitly.
 	 *
-	 * @param string $tableName The table name. Does not need to have been passed to DatabaseBase->tableName yet.
-	 * @param array $columns The fields and their types the table should have.
+	 * @param string $rawTableName The table name. Does not need to have been passed to DatabaseBase->tableName yet.
 	 * @param DatabaseBase or Database $db
 	 * @param object $reportTo Object to report back to.
 	 */
@@ -88,7 +87,6 @@ 
 	 * Creates a new database table with the specified fields.
 	 *
 	 * @param string $tableName The table name.
-	 * @param array $columns The fields and their types the table should have.
 	 * @param DatabaseBase|Database $db
 	 */
 	private static function createTable( $tableName, array $fields, $db ) {
@@ -116,7 +114,6 @@ 
 	 * Update a table given an array of field names and field types.
 	 *
 	 * @param string $tableName The table name.
-	 * @param array $columns The fields and their types the table should have.
 	 * @param DatabaseBase|Database $db
 	 * @param object $reportTo Object to report back to.
 	 */
@@ -344,7 +341,7 @@ 
 	 * Make sure that each of the column descriptions in the given array is
 	 * indexed by *one* index in the given DB table.
 	 *
-	 * @param string $tableName table name. Does not need to have been passed to DatabaseBase->tableName yet.
+	 * @param string $rawTableName table name. Does not need to have been passed to DatabaseBase->tableName yet.
 	 * @param array $indexes array of strings, each a comma separated list with column names to index
 	 * @param DatabaseBase|Database $db DatabaseBase or Database
 	 * @param object $reportTo object to report messages to; since 1.8

includes/storage/SQLStore/SMW_Sql3SmwIds.php

Doc Comments +2 added lines, -3 removed lines

@@ -637,7 +637,7 @@ 
 	 *
 	 * @param integer $sid
 	 * @param DIWikiPage $subject
-	 * @param integer|string|null $interWiki
+	 * @param string $interWiki
 	 */
 	public function updateInterwikiField( $sid, DIWikiPage $subject, $interWiki = null ) {
 
@@ -891,9 +891,8 @@ 
 	/**
 	 * @since 2.3
 	 *
-	 * @param integer $id
 	 *
-	 * @return string[]
+	 * @return DIWikiPage[]
 	 */
 	public function getDataItemPoolHashListFor( array $idlist ) {
 		return $this->byIdDataItemFinder->getDataItemPoolHashListFor( $idlist );

Unused Use Statements -1 removed lines

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