Yaron Koren has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/295695

Change subject: Made more variable names camel-cased
......................................................................

Made more variable names camel-cased

Change-Id: Ifa5ca3d10664aec99bb7e6cd780d0b0f53caf1bc
---
M SCQ_QueryProcessor.php
1 file changed, 16 insertions(+), 16 deletions(-)


  git pull 
ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/SemanticCompoundQueries 
refs/changes/95/295695/1

diff --git a/SCQ_QueryProcessor.php b/SCQ_QueryProcessor.php
index db21e41..5b756a8 100644
--- a/SCQ_QueryProcessor.php
+++ b/SCQ_QueryProcessor.php
@@ -39,7 +39,7 @@
                $params = func_get_args();
                array_shift( $params ); // We already know the $parser.
 
-               $other_params = array();
+               $otherParams = array();
                $results = array();
                $printRequests = array();
                $queryParams = array();
@@ -54,7 +54,7 @@
                                $parts = explode( '=', $param, 2 );
 
                                if ( count( $parts ) >= 2 ) {
-                                       $other_params[strtolower( trim( 
$parts[0] ) )] = $parts[1]; // don't trim here, some params care for " "
+                                       $otherParams[strtolower( trim( 
$parts[0] ) )] = $parts[1]; // don't trim here, some params care for " "
                                }
                        }
                }
@@ -62,32 +62,32 @@
                foreach ( $queryParams as $param ) {
                        $subQueryParams = self::getSubParams( $param );
 
-                       if ( array_key_exists( 'format', $other_params ) && 
!array_key_exists( 'format', $subQueryParams ) ) {
-                               $subQueryParams['format'] = 
$other_params['format'];
+                       if ( array_key_exists( 'format', $otherParams ) && 
!array_key_exists( 'format', $subQueryParams ) ) {
+                               $subQueryParams['format'] = 
$otherParams['format'];
                        }
 
-                       $next_result = self::getQueryResultFromFunctionParams(
+                       $nextResult = self::getQueryResultFromFunctionParams(
                                $subQueryParams,
                                SMW_OUTPUT_WIKI
                        );
 
-                       $results = self::mergeSMWQueryResults( $results, 
$next_result->getResults() );
-                       $printRequests = self::mergeSMWPrintRequests( 
$printRequests, $next_result->getPrintRequests() );
+                       $results = self::mergeSMWQueryResults( $results, 
$nextResult->getResults() );
+                       $printRequests = self::mergeSMWPrintRequests( 
$printRequests, $nextResult->getPrintRequests() );
                }
 
                // Sort results so that they'll show up by page name
                uasort( $results, array( 'SCQQueryProcessor', 
'compareQueryResults' ) );
 
-               $query_result = 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, 
$other_params );
-                       $other_params = parent::getProcessedParams( 
$other_params, $printRequests );
+                       SMWQueryProcessor::addThisPrintout( $printRequests, 
$otherParams );
+                       $otherParams = parent::getProcessedParams( 
$otherParams, $printRequests );
                }
 
                return self::getResultFromQueryResult(
-                       $query_result,
-                       $other_params,
+                       $queryResult,
+                       $otherParams,
                        SMW_OUTPUT_WIKI
                );
        }
@@ -205,7 +205,7 @@
                }
 
                $query = self::createQuery( $querystring, $params, $context, 
null, $extraPrintouts );
-               $query_result = smwfGetStore()->getQueryResult( $query );
+               $queryResult = smwfGetStore()->getQueryResult( $query );
 
                $parameters = array();
 
@@ -218,13 +218,13 @@
                        $parameters = $params;
                }
 
-               foreach ( $query_result->getResults() as $wiki_page ) {
-                       $wiki_page->display_options = $parameters;
+               foreach ( $queryResult->getResults() as $wikiPage ) {
+                       $wikiPage->display_options = $parameters;
                }
 
                wfProfileOut( 
'SCQQueryProcessor::getQueryResultFromQueryString' );
 
-               return $query_result;
+               return $queryResult;
        }
 
        /**

-- 
To view, visit https://gerrit.wikimedia.org/r/295695
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ifa5ca3d10664aec99bb7e6cd780d0b0f53caf1bc
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/SemanticCompoundQueries
Gerrit-Branch: master
Gerrit-Owner: Yaron Koren <[email protected]>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to