SemanticMediaWiki / SemanticMediaWiki

tests/phpunit/Integration/JSONScript/QueryTestCaseProcessor.php

Doc Comments +9 added lines

@@ -42,6 +42,9 @@ 
 	 * @since 2.2
 	 *
 	 * @param Store $store
+	 * @param \SMW\Tests\Utils\Validators\QueryResultValidator $queryResultValidator
+	 * @param \SMW\Tests\Utils\Validators\StringValidator $stringValidator
+	 * @param \SMW\Tests\Utils\Validators\NumberValidator $numberValidator
 	 */
 	public function __construct( Store $store, $queryResultValidator, $stringValidator, $numberValidator ) {
 		$this->store = $store;
@@ -243,6 +246,9 @@ 
 		);
 	}
 
+	/**
+	 * @param string $about
+	 */
 	private function printDescriptionToOutput( $about, $description ) {
 
 		if ( !$this->debug ) {
@@ -253,6 +259,9 @@ 
 		print_r( $description );
 	}
 
+	/**
+	 * @param \SMWQueryResult $queryResult
+	 */
 	private function printQueryResultToOutput( $queryResult ) {
 
 		if ( is_string( $queryResult ) ) {

src/SQLStore/EntityStore/EntitySubobjectListIterator.php

Doc Comments +3 added lines, -3 removed lines

@@ -56,7 +56,7 @@ 
 	 *
 	 * @param DIWikiPage $subject
 	 *
-	 * @return Iterator
+	 * @return \SMW\Iterators\MappingIterator
 	 */
 	public function newListIteratorFor( DIWikiPage $subject, $skipOn = array() ) {
 		$this->skipOn = $skipOn;
@@ -70,7 +70,7 @@ 
 	 *
 	 * @since 2.5
 	 *
-	 * @return Iterator
+	 * @return \SMW\Iterators\MappingIterator
 	 */
 	public function getIterator() {
 
@@ -89,7 +89,7 @@ 
 	 *
 	 * @param DIWikiPage $subject
 	 *
-	 * @return MappingIterator
+	 * @return \SMW\Iterators\MappingIterator
 	 */
 	private function newMappingIterator( DIWikiPage $subject ) {
 

Unused Use Statements -1 removed lines

@@ -5,7 +5,6 @@
 use SMW\IteratorFactory;
 use SMW\DIProperty;
 use SMW\DIWikiPage;
-use SMWDataItem as DataItem;
 use SMW\SQLStore\SQLStore;
 use IteratorAggregate;
 use RuntimeException;

src/SQLStore/QueryEngine/QuerySegmentListBuilder.php

Doc Comments +4 added lines, -1 removed lines

@@ -179,7 +179,7 @@ 
 	/**
 	 * @since 2.2
 	 *
-	 * @return array
+	 * @return string[]
 	 */
 	public function getErrors() {
 		return $this->errors;
@@ -258,6 +258,9 @@ 
 		return $query->queryNumber;
 	}
 
+	/**
+	 * @param string $fingerprint
+	 */
 	private function findDuplicates( $fingerprint ) {
 
 		if ( $this->errors !== array() || $this->isFilterDuplicates === false ) {

Unused Use Statements -1 removed lines

@@ -6,7 +6,6 @@
 use OutOfBoundsException;
 use SMW\Utils\CircularReferenceGuard;
 use SMW\Query\Language\Description;
-use SMW\Query\Language\Conjuncton;
 use SMW\Store;
 use SMW\Message;
 

includes/dataitems/SMW_DI_Container.php

Doc Comments +3 added lines

@@ -97,6 +97,9 @@
 		// return $this->m_semanticData->getHash();
 	}
 
+	/**
+	 * @param SMWSemanticData $semanticData
+	 */
 	private function getValueHash( $semanticData ) {
 
 		$hash = array();

includes/Subobject.php

Doc Comments +2 added lines, -1 removed lines

@@ -125,6 +125,7 @@ 
 
 	/**
 	 * @deprecated since 2.0
+	 * @param string $identifier
 	 */
 	public function setSemanticData( $identifier ) {
 		$this->setEmptyContainerForId( $identifier );
@@ -171,7 +172,7 @@ 
 	/**
 	 * @since 1.9
 	 *
-	 * @param DataValue $dataValue
+	 * @param SMWDataValue $dataValue
 	 *
 	 * @throws SubSemanticDataException
 	 */

src/Deserializers/SemanticDataDeserializer.php

Doc Comments +1 added lines

@@ -95,6 +95,7 @@
 	}
 
 	/**
+	 * @param DIProperty $property
 	 * @return DataItem
 	 */
 	private function doDeserializeDataItem( $property, $data, $value, $semanticData ) {

src/ParserFunctions/SubobjectParserFunction.php

Doc Comments +4 added lines, -1 removed lines

@@ -129,7 +129,7 @@ 
 	/**
 	 * @since 1.9
 	 *
-	 * @param ParserParameterProcessor $params
+	 * @param ParserParameterProcessor $parameters
 	 *
 	 * @return string|null
 	 */
@@ -229,6 +229,9 @@ 
 		return array( $parameters, $id );
 	}
 
+	/**
+	 * @param boolean $useFirstElementAsPropertyLabel
+	 */
 	private function doPrepareParameters( ParserParameterProcessor $parserParameterProcessor, $useFirstElementAsPropertyLabel ) {
 
 		if ( $parserParameterProcessor->hasParameter( self::PARAM_LINKWITH ) ) {

Unused Use Statements -1 removed lines

@@ -10,7 +10,6 @@
 use SMW\HashBuilder;
 use SMW\DataValueFactory;
 use SMW\DIProperty;
-use Parser;
 
 /**
  * @private This class should not be instantiated directly, please use

src/ProcessingErrorMsgHandler.php

Doc Comments +8 added lines, -1 removed lines

@@ -138,7 +138,7 @@ 
 	/**
 	 * @since 2.5
 	 *
-	 * @param array|string $errorMsg
+	 * @param array|string $error
 	 * @param DIProperty|null $property
 	 *
 	 * @return DIContainer
@@ -196,6 +196,10 @@ 
 		return new DIContainer( $containerSemanticData );
 	}
 
+	/**
+	 * @param DIProperty|null $property
+	 * @param string $error
+	 */
 	private function addToContainerSemanticData( $containerSemanticData, $property, $error ) {
 
 		if ( $property !== null ) {
@@ -211,6 +215,9 @@ 
 		);
 	}
 
+	/**
+	 * @param string $hash
+	 */
 	private function newContainerSemanticData( $hash ) {
 
 		if ( $this->subject === null ) {

Unused Use Statements -1 removed lines

@@ -3,7 +3,6 @@
 namespace SMW;
 
 use SMWContainerSemanticData as ContainerSemanticData;
-use SMWDataItem as DataItem;
 use SMWDataValue as DataValue;
 use SMWDIContainer as DIContainer;
 use SMWDIBlob as DIBlob;

src/DataModel/SubSemanticData.php

Doc Comments +6 added lines, -2 removed lines

@@ -95,7 +95,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( 'subject', 'subSemanticData' );
@@ -144,7 +144,7 @@ 
 	/**
 	 * @since 2.5
 	 *
-	 * @param string $subobjectName|null
+	 * @param string $subobjectName
 	 *
 	 * @return boolean
 	 */
@@ -236,6 +236,10 @@ 
 		}
 	}
 
+	/**
+	 * @param SemanticData $semanticData
+	 * @param string $subobjectName
+	 */
 	private function appendSubSemanticData( $semanticData, $subobjectName ) {
 
 		if ( $this->hasSubSemanticData( $subobjectName ) ) {