SemanticMediaWiki / SemanticCompoundQueries

SCQ_CompoundQueryApi.php

Doc Comments +2 added lines, -2 removed lines

@@ -62,7 +62,7 @@ 
 	 * @codeCoverageIgnore
 	 * @see ApiBase::getDescription
 	 *
-	 * @return array
+	 * @return string[]
 	 */
 	public function getDescription() {
 		return array(
@@ -74,7 +74,7 @@ 
 	 * @codeCoverageIgnore
 	 * @see ApiBase::getExamples
 	 *
-	 * @return array
+	 * @return string[]
 	 */
 	protected function getExamples() {
 		return array(

Spacing +7 added lines, -7 removed lines

@@ -17,18 +17,18 @@ 
 	 * @see ApiBase::execute
 	 */
 	public function execute() {
-		$parameterFormatter = new ApiRequestParameterFormatter( $this->extractRequestParams() );
+		$parameterFormatter = new ApiRequestParameterFormatter($this->extractRequestParams());
 		$parameters = $parameterFormatter->getAskApiParameters();
 
-		list( $queryParams, $otherParams ) = SCQQueryProcessor::separateParams( $parameters );
-		list( $queryResult ) = SCQQueryProcessor::queryAndMergeResults( $queryParams, $otherParams );
+		list($queryParams, $otherParams) = SCQQueryProcessor::separateParams($parameters);
+		list($queryResult) = SCQQueryProcessor::queryAndMergeResults($queryParams, $otherParams);
 
 		$outputFormat = 'json';
-		if ( $this->getMain()->getPrinter() instanceof \ApiFormatXml ) {
+		if ($this->getMain()->getPrinter() instanceof \ApiFormatXml) {
 			$outputFormat = 'xml';
 		}
 
-		$this->addQueryResult( $queryResult, $outputFormat );
+		$this->addQueryResult($queryResult, $outputFormat);
 	}
 
 	/**
@@ -78,8 +78,8 @@ 
 	 */
 	protected function getExamples() {
 		return array(
-			'api.php?action=compoundquery&query=' . urlencode( '[[Has city::Vienna]]; ?Has coordinates|[[Has city::Graz]]; ?Has coordinates' ),
-			'api.php?action=compoundquery&query=' . urlencode( '|[[Has city::Vienna]]; ?Has coordinates|[[Has city::Graz]]; ?Has coordinates' ),
+			'api.php?action=compoundquery&query=' . urlencode('[[Has city::Vienna]]; ?Has coordinates|[[Has city::Graz]]; ?Has coordinates'),
+			'api.php?action=compoundquery&query=' . urlencode('|[[Has city::Vienna]]; ?Has coordinates|[[Has city::Graz]]; ?Has coordinates'),
 		);
 	}
 

SCQ_QueryProcessor.php

Braces +2 added lines, -5 removed lines

@@ -144,9 +144,7 @@ 
 
 				if ( $c == '[' ) {
 					$uncompleted_square_brackets++;
-				}
-
-				elseif ( $c == ']' ) {
+				} elseif ( $c == ']' ) {
 					$uncompleted_square_brackets--;
 				}
 			}
@@ -240,8 +238,7 @@ 
 			foreach ( $params as $param ) {
 				$parameters[$param->getName()] = $param->getValue();
 			}
-		}
-		else {
+		} else {
 			$parameters = $params;
 		}
 

Doc Comments +7 added lines, -4 removed lines

@@ -123,7 +123,7 @@ 
 	 * 
 	 * @param string $param
 	 * 
-	 * @return array
+	 * @return string[]
 	 */
 	protected static function getSubParams( $param ) {
 		$sub_params = array();
@@ -171,7 +171,7 @@ 
 	 * Combine two arrays of SMWWikiPageValue objects into one
 	 * 
 	 * @param array $result1
-	 * @param array $result2
+	 * @param SMWDIWikiPage[] $result2
 	 * 
 	 * @return array
 	 */
@@ -195,6 +195,9 @@ 
 		return $result1;
 	}
 
+	/**
+	 * @param SMW\Query\PrintRequest[] $printRequests2
+	 */
 	protected static function mergeSMWPrintRequests( $printRequests1, $printRequests2 ) {
 		$existingPrintoutLabels = array();
 		foreach ( $printRequests1 as $p1 ) {
@@ -211,7 +214,7 @@ 
 	}
 
 	/**
-	 * @param $querystring
+	 * @param string $querystring
 	 * @param array $params
 	 * @param $extraPrintouts
 	 * @param $outputMode
@@ -256,7 +259,7 @@ 
 	 * 
 	 * @param SCQQueryResult $res
 	 * @param array $params These need to be the result of a list fed to getProcessedParams as of SMW 1.6.2
-	 * @param $outputmode
+	 * @param integer $outputmode
 	 * @param $context
 	 * @param string $format
 	 * 

Spacing +67 added lines, -67 removed lines

@@ -14,11 +14,11 @@ 
 	/**
 	 * Comparison helper function, used in sorting results.
 	 */
-	public static function compareQueryResults( $a, $b ) {
-		if ( $a->getSerialization() == $b->getSerialization() ) {
+	public static function compareQueryResults($a, $b) {
+		if ($a->getSerialization() == $b->getSerialization()) {
 			return 0;
 		}
-		return ( $a->getSerialization() < $b->getSerialization() ) ? -1 : 1;
+		return ($a->getSerialization() < $b->getSerialization()) ? -1 : 1;
 	}
 
 	/**
@@ -28,20 +28,20 @@ 
 	 * 
 	 * @return string
 	 */
-	public static function doCompoundQuery( Parser &$parser ) {
+	public static function doCompoundQuery(Parser&$parser) {
 		global $smwgQEnabled, $smwgIQRunningNumber;
 
-		if ( !$smwgQEnabled ) {
-			return smwfEncodeMessages( array( wfMessage( 'smw_iq_disabled' )->inContentLanguage()->text() ) );
+		if (!$smwgQEnabled) {
+			return smwfEncodeMessages(array(wfMessage('smw_iq_disabled')->inContentLanguage()->text()));
 		}
 
 		$smwgIQRunningNumber++;
 
 		$params = func_get_args();
-		array_shift( $params ); // We already know the $parser.
+		array_shift($params); // We already know the $parser.
 
-		list( $queryParams, $otherParams ) = self::separateParams( $params );
-		list( $queryResult, $otherParams ) = self::queryAndMergeResults( $queryParams, $otherParams );
+		list($queryParams, $otherParams) = self::separateParams($params);
+		list($queryResult, $otherParams) = self::queryAndMergeResults($queryParams, $otherParams);
 
 		return self::getResultFromQueryResult(
 			$queryResult,
@@ -56,25 +56,25 @@ 
 	 * @param $params
 	 * @return array
 	 */
-	public static function separateParams( $params ) {
+	public static function separateParams($params) {
 		$queryParams = array();
 		$otherParams = array();
 
-		foreach ( $params as $param ) {
+		foreach ($params as $param) {
 			// Very primitive heuristic - if the parameter
 			// includes a square bracket, then it's a
 			// sub-query; otherwise it's a regular parameter.
-			if ( strpos( $param, '[' ) !== false ) {
+			if (strpos($param, '[') !== false) {
 				$queryParams[] = $param;
 			} else {
-				$parts = explode( '=', $param, 2 );
+				$parts = explode('=', $param, 2);
 
-				if ( count( $parts ) >= 2 ) {
-					$otherParams[strtolower( trim( $parts[0] ) )] = $parts[1]; // don't trim here, some params care for " "
+				if (count($parts) >= 2) {
+					$otherParams[strtolower(trim($parts[0]))] = $parts[1]; // don't trim here, some params care for " "
 				}
 			}
 		}
-		return array( $queryParams, $otherParams );
+		return array($queryParams, $otherParams);
 	}
 
 	/**
@@ -84,36 +84,36 @@ 
 	 * @param $otherParams
 	 * @return array
 	 */
-	public static function queryAndMergeResults( $queryParams, $otherParams ) {
+	public static function queryAndMergeResults($queryParams, $otherParams) {
 		$results = array();
 		$printRequests = array();
 
-		foreach ( $queryParams as $param ) {
-			$subQueryParams = self::getSubParams( $param );
+		foreach ($queryParams as $param) {
+			$subQueryParams = self::getSubParams($param);
 
-			if ( array_key_exists( 'format', $otherParams ) && !array_key_exists( 'format', $subQueryParams ) ) {
+			if (array_key_exists('format', $otherParams) && !array_key_exists('format', $subQueryParams)) {
 				$subQueryParams['format'] = $otherParams['format'];
 			}
 
 			$nextResult = self::getQueryResultFromFunctionParams($subQueryParams);
 
-			$results = self::mergeSMWQueryResults( $results, $nextResult->getResults() );
-			$printRequests = self::mergeSMWPrintRequests( $printRequests, $nextResult->getPrintRequests() );
+			$results = self::mergeSMWQueryResults($results, $nextResult->getResults());
+			$printRequests = self::mergeSMWPrintRequests($printRequests, $nextResult->getPrintRequests());
 		}
 
 		// Sort results so that they'll show up by page name
-		if( !isset($otherParams['unsorted']) || !strcmp( $otherParams['unsorted'], 'on' ) ) {
-			uasort( $results, array( 'SCQQueryProcessor', 'compareQueryResults' ) );
+		if (!isset($otherParams['unsorted']) || !strcmp($otherParams['unsorted'], 'on')) {
+			uasort($results, array('SCQQueryProcessor', 'compareQueryResults'));
 		}
 
-		$queryResult = new SCQQueryResult( $printRequests, new SMWQuery(), $results, smwfGetStore() );
+		$queryResult = new SCQQueryResult($printRequests, new SMWQuery(), $results, smwfGetStore());
 
-		if ( version_compare( SMW_VERSION, '1.6.1', '>' ) ) {
-			SMWQueryProcessor::addThisPrintout( $printRequests, $otherParams );
-			$otherParams = self::getProcessedParams( $otherParams, $printRequests );
+		if (version_compare(SMW_VERSION, '1.6.1', '>')) {
+			SMWQueryProcessor::addThisPrintout($printRequests, $otherParams);
+			$otherParams = self::getProcessedParams($otherParams, $printRequests);
 		}
 
-		return array( $queryResult, $otherParams );
+		return array($queryResult, $otherParams);
 	}
 
 	/**
@@ -125,31 +125,31 @@ 
 	 * 
 	 * @return array
 	 */
-	protected static function getSubParams( $param ) {
+	protected static function getSubParams($param) {
 		$sub_params = array();
 		$sub_param = '';
 		$uncompleted_square_brackets = 0;
 
-		for ( $i = 0; $i < strlen( $param ); $i++ ) {
+		for ($i = 0; $i < strlen($param); $i++) {
 			$c = $param[$i];
 
-			if ( ( $c == ';' ) && ( $uncompleted_square_brackets <= 0 ) ) {
-				$sub_params[] = trim( $sub_param );
+			if (($c == ';') && ($uncompleted_square_brackets <= 0)) {
+				$sub_params[] = trim($sub_param);
 				$sub_param = '';
 			} else {
 				$sub_param .= $c;
 
-				if ( $c == '[' ) {
+				if ($c == '[') {
 					$uncompleted_square_brackets++;
 				}
 
-				elseif ( $c == ']' ) {
+				elseif ($c == ']') {
 					$uncompleted_square_brackets--;
 				}
 			}
 		}
 
-		$sub_params[] = trim( $sub_param );
+		$sub_params[] = trim($sub_param);
 
 		return $sub_params;
 	}
@@ -161,10 +161,10 @@ 
 	 * 
 	 * @return SMWQueryResult
 	 */
-	protected static function getQueryResultFromFunctionParams( $rawparams, $context = SMWQueryProcessor::INLINE_QUERY, $showmode = false ) {
+	protected static function getQueryResultFromFunctionParams($rawparams, $context = SMWQueryProcessor::INLINE_QUERY, $showmode = false) {
 		$printouts = array();
-		self::processFunctionParams( $rawparams, $querystring, $params, $printouts, $showmode );
-		return self::getQueryResultFromQueryString( $querystring, $params, $printouts, $context );
+		self::processFunctionParams($rawparams, $querystring, $params, $printouts, $showmode);
+		return self::getQueryResultFromQueryString($querystring, $params, $printouts, $context);
 	}
 
 	/**
@@ -175,19 +175,19 @@ 
 	 * 
 	 * @return array
 	 */
-	protected static function mergeSMWQueryResults( $result1, $result2 ) {
-		if ( $result1 == null ) {
+	protected static function mergeSMWQueryResults($result1, $result2) {
+		if ($result1 == null) {
 			return $result2;
 		}
 
 		$existing_page_names = array();
-		foreach ( $result1 as $r1 ) {
+		foreach ($result1 as $r1) {
 			$existing_page_names[] = $r1->getSerialization();
 		}
 
-		foreach ( $result2 as $r2 ) {
+		foreach ($result2 as $r2) {
 			$page_name = $r2->getSerialization();
-			if ( ! in_array( $page_name, $existing_page_names ) ) {
+			if (!in_array($page_name, $existing_page_names)) {
 				$result1[] = $r2;
 			}
 		}
@@ -195,15 +195,15 @@ 
 		return $result1;
 	}
 
-	protected static function mergeSMWPrintRequests( $printRequests1, $printRequests2 ) {
+	protected static function mergeSMWPrintRequests($printRequests1, $printRequests2) {
 		$existingPrintoutLabels = array();
-		foreach ( $printRequests1 as $p1 ) {
+		foreach ($printRequests1 as $p1) {
 			$existingPrintoutLabels[] = $p1->getLabel();
 		}
 
-		foreach ( $printRequests2 as $p2 ) {
+		foreach ($printRequests2 as $p2) {
 			$label = $p2->getLabel();
-			if ( ! in_array( $label, $existingPrintoutLabels ) ) {
+			if (!in_array($label, $existingPrintoutLabels)) {
 				$printRequests1[] = $p2;
 			}
 		}
@@ -219,21 +219,21 @@ 
 	 * 
 	 * @return SMWQueryResult
 	 */
-	protected static function getQueryResultFromQueryString( $querystring, array $params, $extraPrintouts, $context = SMWQueryProcessor::INLINE_QUERY ) {
-		wfProfileIn( 'SCQQueryProcessor::getQueryResultFromQueryString' );
+	protected static function getQueryResultFromQueryString($querystring, array $params, $extraPrintouts, $context = SMWQueryProcessor::INLINE_QUERY) {
+		wfProfileIn('SCQQueryProcessor::getQueryResultFromQueryString');
 
-		if ( version_compare( SMW_VERSION, '1.6.1', '>' ) ) {
-			SMWQueryProcessor::addThisPrintout( $extraPrintouts, $params );
-			$params = self::getProcessedParams( $params, $extraPrintouts, false );
+		if (version_compare(SMW_VERSION, '1.6.1', '>')) {
+			SMWQueryProcessor::addThisPrintout($extraPrintouts, $params);
+			$params = self::getProcessedParams($params, $extraPrintouts, false);
 		}
 
-		$query = self::createQuery( $querystring, $params, $context, null, $extraPrintouts );
-		$queryResult = smwfGetStore()->getQueryResult( $query );
+		$query = self::createQuery($querystring, $params, $context, null, $extraPrintouts);
+		$queryResult = smwfGetStore()->getQueryResult($query);
 
 		$parameters = array();
 
-		if ( version_compare( SMW_VERSION, '1.7.2', '>' ) ) {
-			foreach ( $params as $param ) {
+		if (version_compare(SMW_VERSION, '1.7.2', '>')) {
+			foreach ($params as $param) {
 				$parameters[$param->getName()] = $param->getValue();
 			}
 		}
@@ -241,11 +241,11 @@ 
 			$parameters = $params;
 		}
 
-		foreach ( $queryResult->getResults() as $wikiPage ) {
+		foreach ($queryResult->getResults() as $wikiPage) {
 			$wikiPage->display_options = $parameters;
 		}
 
-		wfProfileOut( 'SCQQueryProcessor::getQueryResultFromQueryString' );
+		wfProfileOut('SCQQueryProcessor::getQueryResultFromQueryString');
 
 		return $queryResult;
 	}
@@ -262,23 +262,23 @@ 
 	 * 
 	 * @return string
 	 */
-	protected static function getResultFromQueryResult( SCQQueryResult $res, array $params, $outputmode, $context = SMWQueryProcessor::INLINE_QUERY, $format = '' ) {
-		wfProfileIn( 'SCQQueryProcessor::getResultFromQueryResult' );
+	protected static function getResultFromQueryResult(SCQQueryResult $res, array $params, $outputmode, $context = SMWQueryProcessor::INLINE_QUERY, $format = '') {
+		wfProfileIn('SCQQueryProcessor::getResultFromQueryResult');
 
-		if ( version_compare( SMW_VERSION, '1.6.1', '>' ) ) {
+		if (version_compare(SMW_VERSION, '1.6.1', '>')) {
 			$format = $params['format'];
 
-			if ( version_compare( SMW_VERSION, '1.7.2', '>' ) ) {
+			if (version_compare(SMW_VERSION, '1.7.2', '>')) {
 				$format = $format->getValue();
 			}
 		} else {
-			$format = self::getResultFormat( $params );
+			$format = self::getResultFormat($params);
 		}
 
-		$printer = self::getResultPrinter( $format, $context );
-		$result = $printer->getResult( $res, $params, $outputmode );
+		$printer = self::getResultPrinter($format, $context);
+		$result = $printer->getResult($res, $params, $outputmode);
 
-		wfProfileOut( 'SCQQueryProcessor::getResultFromQueryResult' );
+		wfProfileOut('SCQQueryProcessor::getResultFromQueryResult');
 
 		return $result;
 	}

SCQ_QueryResult.php

Spacing +9 added lines, -9 removed lines

@@ -17,29 +17,29 @@
 	 * 
 	 * @param SMWQueryResult $newResult
 	 */
-	public function addResult( SMWQueryResult $newResult ) {
+	public function addResult(SMWQueryResult $newResult) {
 		$existingPageNames = array();
 		
-		while ( $row = $this->getNext() ) {
-			if ( $row[0] instanceof SMWResultArray ) {
+		while ($row = $this->getNext()) {
+			if ($row[0] instanceof SMWResultArray) {
 				$content = $row[0]->getContent();
-				$existingPageNames[] = $content[0]->getLongText( SMW_OUTPUT_WIKI );
+				$existingPageNames[] = $content[0]->getLongText(SMW_OUTPUT_WIKI);
 			}
 		}
 		
-		while ( ( $row = $newResult->getNext() ) !== false ) {
-			if ( property_exists( $newResult, 'display_options' ) ) {
+		while (($row = $newResult->getNext()) !== false) {
+			if (property_exists($newResult, 'display_options')) {
 				$row[0]->display_options = $newResult->display_options;
 			}
 			$content = $row[0]->getContent();
-			$pageName = $content[0]->getLongText( SMW_OUTPUT_WIKI );
+			$pageName = $content[0]->getLongText(SMW_OUTPUT_WIKI);
 			
-			if ( !in_array( $pageName, $existingPageNames ) ) {
+			if (!in_array($pageName, $existingPageNames)) {
 				$this->m_content[] = $row;
 			}
 		}
 		
-		reset( $this->m_content );
+		reset($this->m_content);
 	}
 	
 }

SemanticCompoundQueries.php

Spacing +5 added lines, -5 removed lines

@@ -15,15 +15,15 @@ 
  * @defgroup SemanticCompoundQueries SemanticCompoundQueries
  */
 
-if ( !defined( 'MEDIAWIKI' ) ) die();
+if (!defined('MEDIAWIKI')) die();
 
-define( 'SCQ_VERSION', '0.4.1' );
+define('SCQ_VERSION', '0.4.1');
 
 $wgExtensionCredits['semantic'][] = array(
 	'path' => __FILE__,
 	'name' => 'Semantic Compound Queries',
 	'version' => SCQ_VERSION,
-	'author' => array( 'Yaron Koren' ),
+	'author' => array('Yaron Koren'),
 	'url' => 'https://www.mediawiki.org/wiki/Extension:Semantic_Compound_Queries',
 	'descriptionmsg' => 'semanticcompoundqueries-desc',
 	'license-name' => 'GPL-2.0+'
@@ -40,7 +40,7 @@ 
 
 $wgAPIModules['compoundquery'] = '\SCQCompoundQueryApi';
 
-function scqgRegisterParser( Parser &$parser ) {
-	$parser->setFunctionHook( 'compound_query', array( 'SCQQueryProcessor', 'doCompoundQuery' ) );
+function scqgRegisterParser(Parser&$parser) {
+	$parser->setFunctionHook('compound_query', array('SCQQueryProcessor', 'doCompoundQuery'));
 	return true; // always return true, in order not to stop MW's hook processing!
 }

Braces +3 added lines, -1 removed lines

@@ -15,7 +15,9 @@
  * @defgroup SemanticCompoundQueries SemanticCompoundQueries
  */
 
-if ( !defined( 'MEDIAWIKI' ) ) die();
+if ( !defined( 'MEDIAWIKI' ) ) {
+	die();
+}
 
 define( 'SCQ_VERSION', '0.4.1' );
 

SemanticCompoundQueries.i18n.magic.php

Spacing +14 added lines, -14 removed lines

@@ -7,70 +7,70 @@
 
 /** English (English) */
 $magicWords['en'] = array(
-	'compound_query' => array( 0, 'compound_query' ),
+	'compound_query' => array(0, 'compound_query'),
 );
 
 /** Arabic (العربية) */
 $magicWords['ar'] = array(
-	'compound_query' => array( 0, 'استعلام_مركب' ),
+	'compound_query' => array(0, 'استعلام_مركب'),
 );
 
 /** Egyptian Arabic (مصرى) */
 $magicWords['arz'] = array(
-	'compound_query' => array( 0, 'استعلام_مركب' ),
+	'compound_query' => array(0, 'استعلام_مركب'),
 );
 
 /** Chuvash (Чӑвашла) */
 $magicWords['cv'] = array(
-	'compound_query' => array( 0, 'compound_query' ),
+	'compound_query' => array(0, 'compound_query'),
 );
 
 /** German (Deutsch) */
 $magicWords['de'] = array(
-	'compound_query' => array( 0, 'abfrage_zusammen', 'frage_zusammen' ),
+	'compound_query' => array(0, 'abfrage_zusammen', 'frage_zusammen'),
 );
 
 /** Japanese (日本語) */
 $magicWords['ja'] = array(
-	'compound_query' => array( 0, '複合クエリ', '複合クエリー' ),
+	'compound_query' => array(0, '複合クエリ', '複合クエリー'),
 );
 
 /** Korean (한국어) */
 $magicWords['ko'] = array(
-	'compound_query' => array( 0, '복합_쿼리' ),
+	'compound_query' => array(0, '복합_쿼리'),
 );
 
 /** Macedonian (македонски) */
 $magicWords['mk'] = array(
-	'compound_query' => array( 0, 'мешовито_барање' ),
+	'compound_query' => array(0, 'мешовито_барање'),
 );
 
 /** Low Saxon (Netherlands) (Nedersaksies) */
 $magicWords['nds-nl'] = array(
-	'compound_query' => array( 0, 'samenestelden_zeukopdrachte', 'samengestelde_zoekopdracht' ),
+	'compound_query' => array(0, 'samenestelden_zeukopdrachte', 'samengestelde_zoekopdracht'),
 );
 
 /** Dutch (Nederlands) */
 $magicWords['nl'] = array(
-	'compound_query' => array( 0, 'samengestelde_zoekopdracht' ),
+	'compound_query' => array(0, 'samengestelde_zoekopdracht'),
 );
 
 /** Brazilian Portuguese (português do Brasil) */
 $magicWords['pt-br'] = array(
-	'compound_query' => array( 0, 'consulta_composta' ),
+	'compound_query' => array(0, 'consulta_composta'),
 );
 
 /** Serbian (Latin script) (srpski (latinica)‎) */
 $magicWords['sr-el'] = array(
-	'compound_query' => array( 0, 'sastavi_upit' ),
+	'compound_query' => array(0, 'sastavi_upit'),
 );
 
 /** Simplified Chinese (中文（简体）‎) */
 $magicWords['zh-hans'] = array(
-	'compound_query' => array( 0, '组合查询' ),
+	'compound_query' => array(0, '组合查询'),
 );
 
 /** Traditional Chinese (中文（繁體）‎) */
 $magicWords['zh-hant'] = array(
-	'compound_query' => array( 0, '組合搜尋' ),
+	'compound_query' => array(0, '組合搜尋'),
 );
\ No newline at end of file