SemanticMediaWiki / SemanticMediaWiki

src/ParserFunctions/DocumentationParserFunction.php

Doc Comments +1 added lines, -1 removed lines

@@ -22,7 +22,7 @@
 	 * @param Parser $parser
 	 * @param ProcessingResult $result
 	 *
-	 * @return mixed
+	 * @return string
 	 */
 	public function handle( Parser $parser, ProcessingResult $result ) {
 		$parameters = $result->getParameters();

src/SQLStore/Lookup/PropertyUsageListLookup.php

Doc Comments +1 added lines, -1 removed lines

@@ -40,7 +40,7 @@
 	 *
 	 * @param Store $store
 	 * @param PropertyStatisticsStore $propertyStatisticsStore
-	 * @param RequestOptions $requestOptions|null
+	 * @param RequestOptions $requestOptions
 	 */
 	public function __construct( Store $store, PropertyStatisticsStore $propertyStatisticsStore, RequestOptions $requestOptions = null ) {
 		$this->store = $store;

src/SQLStore/QueryEngine/ConceptQueryResolver.php

Doc Comments +1 added lines, -1 removed lines

@@ -177,7 +177,7 @@
 	/**
 	 * @since 2.2
 	 *
-	 * @return array
+	 * @return string[]
 	 */
 	public function getErrors() {
 		return $this->errors;

tests/phpunit/includes/GlobalFunctionsTest.php

Doc Comments +1 added lines, -1 removed lines

@@ -43,7 +43,7 @@
 	/**
 	 * Returns the name of the class to be tested
 	 *
-	 * @return string|false
+	 * @return boolean
 	 */
 	public function getClass() {
 		return false;

tests/phpunit/includes/querypages/PropertiesQueryPageTest.php

Doc Comments +2 added lines, -1 removed lines

@@ -23,7 +23,7 @@ 
 class UnusedPropertiesQueryPageTest extends SemanticMediaWikiTestCase {
 
 	/**
-	 * @return string|false
+	 * @return string
 	 */
 	public function getClass() {
 		return '\SMW\UnusedPropertiesQueryPage';
@@ -55,6 +55,7 @@ 
 	/**
 	 * @since 1.9
 	 *
+	 * @param string $result
 	 * @return UnusedPropertiesQueryPage
 	 */
 	private function newInstance( $result = null, $values = array() ) {

tests/phpunit/includes/querypages/QueryPageTest.php

Doc Comments +1 added lines, -1 removed lines

@@ -35,7 +35,7 @@
 	/**
 	 * Returns the name of the class to be tested
 	 *
-	 * @return string|false
+	 * @return string
 	 */
 	public function getClass() {
 		return '\SMW\AggregatablePrinter';

includes/query/SMW_QueryParser.php

Doc Comments +9 added lines

@@ -134,6 +134,7 @@ 
 	 * but just controls query generation. For general effect, the default namespaces
 	 * should be set to NULL.
 	 *
+	 * @param boolean $setNS
 	 * @return Description|null
 	 */
 	private function getSubqueryDescription( &$setNS ) {
@@ -242,6 +243,7 @@ 
 	 * be the content of "[[ ... ]]". Returns NULL upon error.
 	 *
 	 * Parameters $setNS has the same use as in getSubqueryDescription().
+	 * @param boolean $setNS
 	 */
 	private function getLinkDescription( &$setNS ) {
 		// This method is called when we encountered an opening '[['. The following
@@ -424,6 +426,7 @@ 
 	 * a category or property) and create a suitable description.
 	 * The first chunk behind the "[[" has already been read and is
 	 * passed as a parameter.
+	 * @param string $firstChunk
 	 */
 	private function getArticleDescription( $firstChunk, &$setNS ) {
 		$chunk = $firstChunk;
@@ -469,6 +472,10 @@ 
 		return $this->finishLinkDescription( $chunk, true, $result, $setNS );
 	}
 
+	/**
+	 * @param boolean $hasNamespaces
+	 * @param Description|null $result
+	 */
 	private function finishLinkDescription( $chunk, $hasNamespaces, $result, &$setNS ) {
 		if ( is_null( $result ) ) { // no useful information or concrete error found
 			$this->descriptionProcessor->addErrorForMsgCode( 'smw_unexpectedpart', $chunk ); // was smw_badqueryatom
@@ -565,6 +572,7 @@ 
 	/**
 	 * Enter a new subblock in the query, which must at some time be terminated by the
 	 * given $endstring delimiter calling popDelimiter();
+	 * @param string $endstring
 	 */
 	private function pushDelimiter( $endstring ) {
 		array_push( $this->separatorStack, $endstring );
@@ -574,6 +582,7 @@ 
 	 * Exit a subblock in the query ending with the given delimiter.
 	 * If the delimiter does not match the top-most open block, false
 	 * will be returned. Otherwise return true.
+	 * @param string $endstring
 	 */
 	private function popDelimiter( $endstring ) {
 		$topdelim = array_pop( $this->separatorStack );

src/Query/Parser/DescriptionProcessor.php

Doc Comments +5 added lines

@@ -166,6 +166,7 @@ 
 	 *
 	 * The return value is the expected combined description. The object $currentDescription will
 	 * also be changed (if it was non-NULL).
+	 * @param integer $compoundType
 	 */
 	private function getCompoundDescription( Description $currentDescription = null, Description $newDescription = null, $compoundType = SMW_CONJUNCTION_QUERY ) {
 
@@ -201,6 +202,10 @@ 
 		}
 	}
 
+	/**
+	 * @param Description $currentDescription
+	 * @param Description $newDescription
+	 */
 	private function newCompoundDescriptionFor( $compoundType, $currentDescription, $newDescription ) {
 
 		if ( ( ( $compoundType & SMW_CONJUNCTION_QUERY ) != 0 && ( $currentDescription instanceof Conjunction ) ) ||

includes/parserhooks/SubobjectParserFunction.php

Doc Comments +5 added lines, -2 removed lines

@@ -84,9 +84,9 @@ 
 	/**
 	 * @since 1.9
 	 *
-	 * @param ParserParameterProcessor $params
+	 * @param ParserParameterProcessor $parameters
 	 *
-	 * @return string|null
+	 * @return string
 	 */
 	public function parse( ParserParameterProcessor $parameters ) {
 
@@ -172,6 +172,9 @@ 
 		return $parameters->toArray();
 	}
 
+	/**
+	 * @param DIWikiPage $subject
+	 */
 	private function addErrorWithMsg( $subject, $errorMsg ) {
 
 		$error = new Error( $subject );