SemanticMediaWiki / SemanticMediaWiki

includes/export/SMW_Exporter.php

Doc Comments +3 added lines, -3 removed lines

@@ -403,7 +403,7 @@ 
 	 * represent. Returns null if this attempt failed.
 	 *
 	 * @param ExpElement $expElement
-	 * @return SMWDataItem or null
+	 * @return null|DIWikiPage or null
 	 */
 	public function findDataItemForExpElement( ExpElement $expElement ) {
 		return self::$dataItemByExpElementMatchFinder->tryToFindDataItemForExpElement( $expElement );
@@ -440,7 +440,7 @@ 
 	 * must take the type of the annotated object into account for some
 	 * reason.
 	 *
-	 * @param $propertyKey string the Id of the special property
+	 * @param string $propertyKey string the Id of the special property
 	 * @param $forNamespace integer the namespace of the page which has a value for this property
 	 * @return ExpNsResource|null
 	 */
@@ -620,7 +620,7 @@ 
 	 * elements are always preferred in query answering.
 	 *
 	 * @param $dataItem SMWDataItem
-	 * @return ExpElement|null
+	 * @return ExpLiteral|null
 	 */
 	static public function getDataItemHelperExpElement( SMWDataItem $dataItem ) {
 

includes/queryprinters/FeedResultPrinter.php

Unused Use Statements -1 removed lines

@@ -5,7 +5,6 @@
 use FeedItem;
 use ParserOptions;
 use Sanitizer;
-use SMWDIWikiPage;
 use SMWQueryResult;
 use TextContent;
 use Title;

src/GlobalFunctions.php

Doc Comments +1 added lines, -2 removed lines

@@ -95,7 +95,6 @@ 
  * $icon should be one of: 'warning' (default), 'info'.
  *
  * @param array $messages
- * @param string $icon Acts like an enum. Callers must ensure safety, since this value is used directly in the output.
  * @param string $seperator
  * @param boolean $escape Should the messages be escaped or not (ie when they already are)
  *
@@ -201,7 +200,7 @@ 
  * @param mixed $namespace
  * @param boolean $complete
  *
- * @return true
+ * @return boolean
  *
  * @codeCoverageIgnore
  */

tests/phpunit/Integration/Query/InversePropertyRelationshipDBIntegrationTest.php

Doc Comments +4 added lines

@@ -116,6 +116,10 @@
 		);
 	}
 
+	/**
+	 * @param string $property
+	 * @param string $value
+	 */
 	private function newDataValueForPagePropertyValue( $property, $value ) {
 
 		$property = DIProperty::newFromUserLabel( $property );

includes/queryprinters/DsvResultPrinter.php

Doc Comments -1 removed lines

@@ -81,7 +81,6 @@
 	 *
 	 * @since 1.6
 	 *
-	 * @param SMWQueryResult $res
 	 *
 	 * @return string
 	 */

tests/phpunit/Integration/MediaWiki/Hooks/ParserFirstCallInitIntegrationTest.php

Doc Comments +3 added lines

@@ -168,6 +168,9 @@
 		return $provider;
 	}
 
+	/**
+	 * @param \SMW\ParserOutput|null $parserOutput
+	 */
 	private function findSemanticataFromOutput( $parserOutput ) {
 
 		if ( method_exists( $parserOutput, 'getExtensionData' ) ) {

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
 	 *

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;