SemanticMediaWiki / SemanticMediaWiki

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/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 ) ) ||

src/InTextAnnotationParser.php

Doc Comments +4 added lines, -4 removed lines

@@ -145,7 +145,7 @@ 
 	 *
 	 * @param string $text
 	 *
-	 * @return text
+	 * @return string
 	 */
 	public static function decodeSquareBracket( $text ) {
 		return str_replace( array( '%5B', '%5D' ), array( '[', ']' ), $text );
@@ -156,7 +156,7 @@ 
 	 *
 	 * @param string $text
 	 *
-	 * @return text
+	 * @return string
 	 */
 	public static function obscureAnnotation( $text ) {
 		return preg_replace_callback(
@@ -173,7 +173,7 @@ 
 	 *
 	 * @param string $text
 	 *
-	 * @return text
+	 * @return string
 	 */
 	public static function removeAnnotation( $text ) {
 		return preg_replace_callback(
@@ -230,7 +230,7 @@ 
 	 *
 	 * @since 1.9
 	 *
-	 * @return array
+	 * @return string[]
 	 */
 	protected function getModules() {
 		return array(

src/SQLStore/Lookup/UsageStatisticsListLookup.php

Doc Comments +9 added lines, -9 removed lines

@@ -100,7 +100,7 @@ 
 	/**
 	 * @since 2.2
 	 *
-	 * @return number
+	 * @return integer
 	 */
 	public function getImproperValueForCount() {
 		return $this->propertyStatisticsStore->getUsageCount(
@@ -111,7 +111,7 @@ 
 	/**
 	 * @since 1.9
 	 *
-	 * @return number
+	 * @return integer
 	 */
 	public function getQueryCount() {
 		return $this->count( '_ASK' );
@@ -120,7 +120,7 @@ 
 	/**
 	 * @since 1.9
 	 *
-	 * @return number
+	 * @return integer
 	 */
 	public function getQuerySize() {
 		return $this->count( '_ASKSI' );
@@ -129,7 +129,7 @@ 
 	/**
 	 * @since 1.9
 	 *
-	 * @return number
+	 * @return integer
 	 */
 	public function getConceptCount() {
 		return $this->count( '_CONC' );
@@ -138,7 +138,7 @@ 
 	/**
 	 * @since 1.9
 	 *
-	 * @return number
+	 * @return integer
 	 */
 	public function getSubobjectCount() {
 		return $this->count( DIProperty::TYPE_SUBOBJECT );
@@ -147,7 +147,7 @@ 
 	/**
 	 * @since 1.9
 	 *
-	 * @return number
+	 * @return integer
 	 */
 	public function getDeclaredPropertiesCount() {
 		return $this->count( DIProperty::TYPE_HAS_TYPE );
@@ -182,7 +182,7 @@ 
 	/**
 	 * @since 1.9
 	 *
-	 * @return number
+	 * @return integer
 	 */
 	public function getPropertyPageCount() {
 
@@ -204,7 +204,7 @@ 
 	 *
 	 * @since 1.9
 	 *
-	 * @return number
+	 * @return integer
 	 */
 	public function getPropertyUsageCount() {
 		$count = 0;
@@ -224,7 +224,7 @@ 
 	/**
 	 * @since 1.9
 	 *
-	 * @return number
+	 * @return integer
 	 */
 	public function getUsedPropertiesCount() {
 		$count = 0;

src/Exporter/DataItemToExpResourceEncoder.php

Doc Comments +7 added lines

@@ -177,6 +177,10 @@ 
 		return $resource;
 	}
 
+	/**
+	 * @param DIWikiPage $diWikiPage
+	 * @param string $modifier
+	 */
 	private function newExpNsResource( $diWikiPage, $modifier ) {
 
 		 $importDataItem = $this->tryToFindImportDataItem( $diWikiPage, $modifier );
@@ -254,6 +258,9 @@ 
 		);
 	}
 
+	/**
+	 * @return DataItem
+	 */
 	private function tryToFindImportDataItem( DIWikiPage $diWikiPage, $modifier ) {
 
 		$importDataItems = null;

Unused Use Statements -1 removed lines

@@ -3,7 +3,6 @@
 namespace SMW\Tests\SQLStore;
 
 use SMW\SQLStore\PropertyTableOutdatedReferenceDisposer;
-use Title;
 
 /**
  * @covers \SMW\SQLStore\PropertyTableOutdatedReferenceDisposer