SemanticMediaWiki / SemanticMediaWiki

src/SPARQLStore/QueryEngine/Interpreter/ConjunctionInterpreter.php

Doc Comments +10 added lines

@@ -100,6 +100,10 @@ 
 		return $result;
 	}
 
+	/**
+	 * @param string $joinVariable
+	 * @param \SMW\DIProperty|null $orderByProperty
+	 */
 	private function doPreliminarySubDescriptionCheck( $subDescriptions, $joinVariable, $orderByProperty ) {
 
 		$count = count( $subDescriptions );
@@ -126,6 +130,9 @@ 
 		return null;
 	}
 
+	/**
+	 * @param string $joinVariable
+	 */
 	private function doResolveSubDescriptionsRecursively( $subDescriptions, $joinVariable ) {
 
 		// Using a stdClass as data container for simpler handling in follow-up tasks
@@ -195,6 +202,9 @@ 
 		return $subConditionElements;
 	}
 
+	/**
+	 * @param \stdClass $subConditionElements
+	 */
 	private function createConditionFromSubConditionElements( $subConditionElements ) {
 
 		if ( $subConditionElements->singletonMatchElement instanceof ExpElement ) {

src/SPARQLStore/QueryEngine/Interpreter/DisjunctionInterpreter.php

Doc Comments +11 added lines

@@ -107,6 +107,10 @@ 
 		return $result;
 	}
 
+	/**
+	 * @param string $joinVariable
+	 * @param \SMW\DIProperty|null $orderByProperty
+	 */
 	private function doPreliminarySubDescriptionCheck( $subDescriptions, $joinVariable, $orderByProperty ) {
 
 		$count = count( $subDescriptions );
@@ -131,6 +135,10 @@ 
 		return null;
 	}
 
+	/**
+	 * @param string $joinVariable
+	 * @param \SMW\DIProperty|null $orderByProperty
+	 */
 	private function doResolveSubDescriptionsRecursively( $subDescriptions, $joinVariable, $orderByProperty ) {
 
 		// Using a stdClass as data container for simpler handling in follow-up tasks
@@ -213,6 +221,9 @@ 
 		return $subConditionElements;
 	}
 
+	/**
+	 * @param string $joinVariable
+	 */
 	private function createConditionFromSubConditionElements( $subConditionElements, $joinVariable ) {
 
 		if ( $subConditionElements->unionCondition === '' ) {

src/SPARQLStore/QueryEngine/Interpreter/SomePropertyInterpreter.php

Doc Comments +8 added lines

@@ -150,6 +150,9 @@ 
 		return array( $innerOrderByProperty, $innerCondition, $innerJoinVariable );
 	}
 
+	/**
+	 * @param string $innerJoinVariable
+	 */
 	private function findObjectNameFromInnerCondition( $innerCondition, $innerJoinVariable, &$namespaces ) {
 
 		if ( !$innerCondition instanceof SingletonCondition ) {
@@ -201,6 +204,10 @@ 
 		return TurtleSerializer::getTurtleNameForExpElement( $propertyExpElement );
 	}
 
+	/**
+	 * @param string $objectName
+	 * @param string $joinVariable
+	 */
 	private function doExchangeForWhenInversePropertyIsUsed( DIProperty $property, $objectName, $joinVariable ) {
 
 		$subjectName = '?' . $joinVariable;
@@ -239,6 +246,7 @@ 
 	 * can be found using the "zero or more" will resolve the complete path
 	 *
 	 * @see http://www.w3.org/TR/sparql11-query/#propertypath-arbitrary-length
+	 * @param string $propertyName
 	 */
 	private function tryToAddPropertyPathForSaturatedHierarchy( &$condition, DIProperty $property, &$propertyName ) {
 

src/SPARQLStore/QueryEngine/QueryEngine.php

Doc Comments +3 added lines

@@ -213,6 +213,9 @@
 		return DebugOutputFormatter::formatOutputFor( 'SPARQLStore', $entries, $query );
 	}
 
+	/**
+	 * @param Condition $condition
+	 */
 	private function isSingletonConditionWithElementMatch( $condition ) {
 		return $condition instanceof SingletonCondition && $condition->matchElement instanceof Element;
 	}

src/SPARQLStore/QueryEngine/RepositoryResult.php

Doc Comments +1 added lines, -1 removed lines

@@ -73,7 +73,7 @@
 	/**
 	 * Get the number of rows in the result object.
 	 *
-	 * @return interger number of result rows
+	 * @return integer number of result rows
 	 */
 	public function numRows() {
 		return count( $this->data );

src/SPARQLStore/QueryEngine/XmlResponseParser.php

Doc Comments +3 added lines

@@ -126,6 +126,9 @@
 		);
 	}
 
+	/**
+	 * @param string $xmlResultData
+	 */
 	private function parseXml( $xmlResultData ) {
 		return xml_parse( $this->parser, $xmlResultData, true );
 	}

src/SPARQLStore/RedirectLookup.php

Doc Comments +1 added lines, -1 removed lines

@@ -52,7 +52,7 @@
 	 * @since 1.6
 	 *
 	 * @param ExpNsResource $expNsResource string URI to check
-	 * @param boolean $existsthat is set to true if $expNsResource is in the
+	 * @param boolean|null $exists is set to true if $expNsResource is in the
 	 * store; always false for blank nodes; always true for subobjects
 	 *
 	 * @return ExpNsResource

src/SPARQLStore/RepositoryConnector/GenericHttpRepositoryConnector.php

Doc Comments +4 added lines, -4 removed lines

@@ -132,7 +132,7 @@ 
 	 * rdfs, property, xsd, so these do not have to be included in
 	 * $extraNamespaces.
 	 *
-	 * @param $vars mixed array or string, field name(s) to be retrieved, can be '*'
+	 * @param string $vars mixed array or string, field name(s) to be retrieved, can be '*'
 	 * @param $where string WHERE part of the query, without surrounding { }
 	 * @param $options array (associative) of options, e.g. array( 'LIMIT' => '10' )
 	 * @param $extraNamespaces array (associative) of namespaceId => namespaceUri
@@ -549,8 +549,8 @@ 
 	}
 
 	/**
-	 * @param $endpoint string URL of endpoint that was used
-	 * @param $sparql string query that caused the problem
+	 * @param string $endpoint string URL of endpoint that was used
+	 * @param string $sparql string query that caused the problem
 	 */
 	protected function mapHttpRequestError( $endpoint, $sparql ) {
 
@@ -566,7 +566,7 @@ 
 	 *
 	 * @param integer $timeout
 	 *
-	 * @return SparqlDatabase
+	 * @return GenericHttpRepositoryConnector
 	 */
 	public function setConnectionTimeoutInSeconds( $timeout = 10 ) {
 		$this->httpRequest->setOption( CURLOPT_CONNECTTIMEOUT, $timeout );

src/SPARQLStore/SPARQLStore.php

Doc Comments +1 added lines, -1 removed lines

@@ -403,7 +403,7 @@
 	 *
 	 * @param string $connectionTypeId
 	 *
-	 * @return mixed
+	 * @return RepositoryConnection
 	 */
 	public function getConnection( $connectionTypeId = 'sparql' ) {