SemanticMediaWiki / SemanticMediaWiki

tests/phpunit/Unit/MediaWiki/Api/BrowseBySubjectTest.php

Doc Comments +5 added lines

@@ -221,6 +221,11 @@
 			} );
 	}
 
+	/**
+	 * @param string $field
+	 * @param string $internalType
+	 * @param \Closure $definition
+	 */
 	protected function assertInternalArrayStructure( $setup, $result, $field, $internalType, $definition ) {
 
 		if ( isset( $setup[$field] ) && $setup[$field] ) {

src/InTextAnnotationParser.php

Doc Comments +10 added lines, -4 removed lines

@@ -181,7 +181,7 @@ 
 	 *
 	 * @param string $text
 	 *
-	 * @return text
+	 * @return string
 	 */
 	public static function decodeSquareBracket( $text ) {
 		return Obfuscator::decodeSquareBracket( $text );
@@ -192,7 +192,7 @@ 
 	 *
 	 * @param string $text
 	 *
-	 * @return text
+	 * @return string
 	 */
 	public static function obfuscateAnnotation( $text ) {
 		return Obfuscator::obfuscateAnnotation( $text );
@@ -203,7 +203,7 @@ 
 	 *
 	 * @param string $text
 	 *
-	 * @return text
+	 * @return string
 	 */
 	public static function removeAnnotation( $text ) {
 		return Obfuscator::removeAnnotation( $text );
@@ -245,7 +245,7 @@ 
 	 *
 	 * @since 1.9
 	 *
-	 * @return array
+	 * @return string[]
 	 */
 	protected function getModules() {
 		return array(
@@ -326,6 +326,8 @@ 
 	 * @since 1.9
 	 *
 	 * @param array $properties
+	 * @param DIWikiPage $subject
+	 * @param string $value
 	 *
 	 * @return string
 	 */
@@ -389,6 +391,10 @@ 
 		return $this->applicationFactory->getNamespaceExaminer()->isSemanticEnabled( $title->getNamespace() );
 	}
 
+	/**
+	 * @param DIWikiPage $subject
+	 * @param string $value
+	 */
 	private function getPropertyLink( $subject, $properties, $value, $valueCaption ) {
 
 		// #1855

src/ParserFunctions/InfoParserFunction.php

Doc Comments +1 added lines, -1 removed lines

@@ -31,7 +31,7 @@
 	 * @param Parser $parser
 	 * @param ProcessingResult $result
 	 *
-	 * @return mixed
+	 * @return string
 	 */
 	public function handle( Parser $parser, ProcessingResult $result ) {
 		if ( $result->hasFatal() ) {

tests/phpunit/Benchmark/PageEditCopyBenchmarkRunner.php

Doc Comments -1 removed lines

@@ -74,7 +74,6 @@
 	/**
 	 * @since 2.5
 	 *
-	 * @param integer|null $copyCount
 	 */
 	public function setEditRepetitionCount( $editRepetitionCount = null ) {
 		$this->editRepetitionCount = $editRepetitionCount;

tests/phpunit/Benchmark/PageImportBenchmarkRunner.php

Doc Comments +3 added lines

@@ -96,6 +96,9 @@
 		}
 	}
 
+	/**
+	 * @param string $file
+	 */
 	private function doXmlImport( $file, array $case ) {
 
 		$this->xmlImportRunner->setFile( $file );

tests/phpunit/Benchmark/QueryBenchmarkRunner.php

Doc Comments +3 added lines

@@ -101,6 +101,9 @@
 		);
 	}
 
+	/**
+	 * @param Query $query
+	 */
 	private function doQuery( array $case, $query ) {
 
 		$this->benchmarker->clear();

tests/phpunit/Utils/Runners/XmlImportRunner.php

Doc Comments +7 added lines, -1 removed lines

@@ -31,6 +31,9 @@ 
 	 */
 	private $testEnvironment;
 
+	/**
+	 * @param string $file
+	 */
 	public function __construct( $file = null ) {
 		$this->file = $file;
 		$this->testEnvironment = new TestEnvironment();
@@ -130,7 +133,7 @@ 
 	}
 
 	/**
-	 * @return integer
+	 * @return double
 	 */
 	public function getElapsedImportTimeInSeconds() {
 		return round( $this->importTime, 7 );
@@ -145,6 +148,9 @@ 
 		return $this->requestContext;
 	}
 
+	/**
+	 * @param string $file
+	 */
 	private function assertThatFileIsReadableOrThrowException( $file ) {
 
 		$file = str_replace( array( '\\', '/' ), DIRECTORY_SEPARATOR, $file );

includes/datavalues/SMW_DataValue.php

Doc Comments +6 added lines, -7 removed lines

@@ -168,7 +168,7 @@ 
 	 * label for printout might also be specified.
 	 *
 	 * @param string $value
-	 * @param mixed $caption
+	 * @param boolean $caption
 	 */
 	public function setUserValue( $value, $caption = false ) {
 
@@ -263,7 +263,7 @@ 
 	/**
 	 * @since 2.4
 	 *
-	 * @return DIWikiPage|null
+	 * @return SMWDIWikiPage
 	 */
 	public function getContextPage() {
 		return $this->m_contextPage;
@@ -305,7 +305,7 @@ 
 	 *
 	 * @param string $key
 	 *
-	 * @return mixed|false
+	 * @return string|false
 	 */
 	public function getOption( $key ) {
 
@@ -340,7 +340,6 @@ 
 	/**
 	 * @since 2.4
 	 *
-	 * @param string $caption
 	 */
 	public function getCaption() {
 		return $this->m_caption;
@@ -427,7 +426,6 @@ 
 	 *
 	 * @param $parameters
 	 * @param integer|null $type
-	 * @param integer|null $language
 	 */
 	public function addErrorMsg( $parameters, $type = null ) {
 		$this->mErrors[Message::getHash( $parameters, $type )] = Message::encode( $parameters, $type );
@@ -486,7 +484,7 @@ 
 	 *
 	 * @param string $value
 	 *
-	 * @return Description
+	 * @return SMW\Query\Language\Description
 	 * @throws InvalidArgumentException
 	 */
 	public function getQueryDescription( $value ) {
@@ -507,7 +505,7 @@ 
 	 *
 	 * @since 2.4
 	 *
-	 * @return DataValueFormatter
+	 * @return SMW\DataValues\ValueFormatters\DataValueFormatter
 	 */
 	public function getDataValueFormatter() {
 		return ValueFormatterRegistry::getInstance()->getDataValueFormatterFor( $this );
@@ -591,6 +589,7 @@ 
 	 *
 	 * The parameter $linked controls linking of values such as titles and should
 	 * be non-NULL and non-false if this is desired.
+	 * @return string
 	 */
 	abstract public function getLongWikiText( $linked = null );
 

src/DataValues/ValueFormatters/PropertyValueFormatter.php

Doc Comments +6 added lines

@@ -210,6 +210,9 @@ 
 		return $highlighter->getHtml();
 	}
 
+	/**
+	 * @param string $propertyDescription
+	 */
 	private function canHighlight( &$propertyDescription, $linker ) {
 
 		if ( $this->dataValue->getOption( PropertyValue::OPT_NO_HIGHLIGHT ) === true ) {
@@ -262,6 +265,9 @@ 
 		return $preferredLabelMarker;
 	}
 
+	/**
+	 * @param \SMWDataItem $property
+	 */
 	private function findTranslatedPropertyLabel( $property ) {
 
 		// User-defined properties don't have any translatable label (this is