SemanticMediaWiki / SemanticFormsSelect

tests/bootstrap.php

Spacing +10 added lines, -10 removed lines

@@ -1,23 +1,23 @@ 
 <?php
 
-if ( PHP_SAPI !== 'cli' ) {
-	die( 'Not an entry point' );
+if (PHP_SAPI !== 'cli') {
+	die('Not an entry point');
 }
 
-error_reporting( E_ALL | E_STRICT );
-date_default_timezone_set( 'UTC' );
-ini_set( 'display_errors', 1 );
+error_reporting(E_ALL | E_STRICT);
+date_default_timezone_set('UTC');
+ini_set('display_errors', 1);
 
 global $IP;
 
 //if ( !is_readable( $autoloaderClassPath = __DIR__ . '/../../SemanticMediaWiki/tests/autoloader.php' ) ) {
-if ( !is_readable( $autoloaderClassPath = $IP . '/extensions/SemanticMediaWiki/tests/autoloader.php' ) ) {
-	die( "\nThe Semantic MediaWiki test autoloader is not available\n" );
+if (!is_readable($autoloaderClassPath = $IP.'/extensions/SemanticMediaWiki/tests/autoloader.php')) {
+	die("\nThe Semantic MediaWiki test autoloader is not available\n");
 }
 
 // if ( !class_exists( 'SemanticFormsSelect' ) || ( $version = SemanticFormsSelect::getVersion() ) === null ) {
-if ( ExtensionRegistry::getInstance()->isLoaded( 'SemanticFormsSelect' ) ) {
-	die( "\nSemantic Forms Select is not available, please check your Composer or LocalSettings.\n" );
+if (ExtensionRegistry::getInstance()->isLoaded('SemanticFormsSelect')) {
+	die("\nSemantic Forms Select is not available, please check your Composer or LocalSettings.\n");
 }
 
 //print sprintf( "\n%-20s%s\n", "Semantic Forms Select: ", ExtensionRegistry::getInstance()->isLoaded( 'SemanticFormsSelect' ) );
@@ -26,4 +26,4 @@ 
 $autoloader = require $autoloaderClassPath;
 //$autoloader->addPsr4( 'SFS\\Tests\\', __DIR__ . '/phpunit/Unit' );
 //$autoloader->addPsr4( 'SFS\\Tests\\Integration\\', __DIR__ . '/phpunit/Integration' );
-unset( $autoloader );
+unset($autoloader);

tests/phpunit/Unit/ApiSemanticFormsSelectRequestProcessorTest.php

Spacing +20 added lines, -20 removed lines

@@ -24,13 +24,13 @@ 
 
 	protected function setUp() {
 		parent::setUp();
-		$parser = $this->getMockBuilder( '\Parser' )
+		$parser = $this->getMockBuilder('\Parser')
 			->disableOriginalConstructor()->getMock();
-		$this->ApiSFSRP = new ApiSemanticFormsSelectRequestProcessor( $parser );
+		$this->ApiSFSRP = new ApiSemanticFormsSelectRequestProcessor($parser);
 	}
 
 	protected function tearDown() {
-		unset( $this->ApiSFSRP );
+		unset($this->ApiSFSRP);
 		parent::tearDown();
 	}
 
@@ -44,7 +44,7 @@ 
 
 		$parameters = [];
 
-		$this->setExpectedException( 'InvalidArgumentException' );
+		$this->setExpectedException('InvalidArgumentException');
 		$this->ApiSFSRP->getJsonDecodedResultValuesForRequestParameters(
 			$parameters
 		);
@@ -52,7 +52,7 @@ 
 
 	public function testJsonResultValuesFromRequestParameters() {
 
-		$parameters = [ 'query' => 'foo', 'sep' => ',' ];
+		$parameters = ['query' => 'foo', 'sep' => ','];
 
 		$this->assertInternalType(
 			'object',
@@ -65,8 +65,8 @@ 
 	public function testJsonResultValuesFromRequestParameters_doProcessQueryFor(
 	) {
 
-		$parameters = [ 'approach' => 'smw', 'query' => 'foo, baa, gaah',
-		                     'sep'      => ',' ];
+		$parameters = ['approach' => 'smw', 'query' => 'foo, baa, gaah',
+		                     'sep'      => ','];
 
 		$this->assertInternalType(
 			'object',
@@ -77,8 +77,8 @@ 
 	}
 
 	public function testSetDebugFlag() {
-		$this->ApiSFSRP->setDebugFlag( true );
-		$parameters = [ 'query' => 'foo , function', 'sep' => ',' ];
+		$this->ApiSFSRP->setDebugFlag(true);
+		$parameters = ['query' => 'foo , function', 'sep' => ','];
 
 		$this->assertInternalType(
 			'object',
@@ -89,9 +89,9 @@ 
 	}
 
 	public function testSetDebugFlag_doProcessQueryFor() {
-		$this->ApiSFSRP->setDebugFlag( true );
-		$parameters = [ 'approach' => 'smw', 'query' => 'my Query,query2',
-		                     'sep'      => ',' ];
+		$this->ApiSFSRP->setDebugFlag(true);
+		$parameters = ['approach' => 'smw', 'query' => 'my Query,query2',
+		                     'sep'      => ','];
 
 		$this->assertInternalType(
 			'object',
@@ -104,11 +104,11 @@ 
 	public function testGetFormattedValuesFrom() {
 		$sep = ",";
 		$values = "my Query,query2";
-		$result = [ "", "my Query", "query2" ];
+		$result = ["", "my Query", "query2"];
 		$formattedValues = $this->invokeMethod(
-			$this->ApiSFSRP, 'getFormattedValuesFrom', [ $sep, $values ]
+			$this->ApiSFSRP, 'getFormattedValuesFrom', [$sep, $values]
 		);
-		$this->assertEquals( $result, $formattedValues );
+		$this->assertEquals($result, $formattedValues);
 	}
 
 	/**
@@ -120,14 +120,14 @@ 
 	 *
 	 * @return mixed Method return.
 	 */
-	public function invokeMethod( &$object, $methodName,
+	public function invokeMethod(&$object, $methodName,
 		array $parameters = []
 	) {
-		$reflection = new \ReflectionClass( get_class( $object ) );
-		$method = $reflection->getMethod( $methodName );
-		$method->setAccessible( true );
+		$reflection = new \ReflectionClass(get_class($object));
+		$method = $reflection->getMethod($methodName);
+		$method->setAccessible(true);
 
-		return $method->invokeArgs( $object, $parameters );
+		return $method->invokeArgs($object, $parameters);
 	}
 
 

tests/phpunit/Unit/SemanticFormsSelectInputTest.php

Spacing +10 added lines, -10 removed lines

@@ -25,17 +25,17 @@ 
 		$isMandatory = false;
 		$isDisabled = false;
 
-		$otherArgs = [ 'template' => 'Foo', 'field' => '',
-		                    'function' => 'Bar', 'is_list' => true ];
+		$otherArgs = ['template' => 'Foo', 'field' => '',
+		                    'function' => 'Bar', 'is_list' => true];
 
-		$parserOutput = $this->getMockBuilder( '\ParserOutput' )
+		$parserOutput = $this->getMockBuilder('\ParserOutput')
 			->disableOriginalConstructor()->getMock();
 
-		$parser = $this->getMockBuilder( '\Parser' )
+		$parser = $this->getMockBuilder('\Parser')
 			->disableOriginalConstructor()->getMock();
 
-		$parser->expects( $this->any() )->method( 'getOutput' )->will(
-				$this->returnValue( $parserOutput )
+		$parser->expects($this->any())->method('getOutput')->will(
+				$this->returnValue($parserOutput)
 			);
 		$this->SFSInput = new SemanticFormsSelectInput(
 			$value, $inputName, $isMandatory, $isDisabled, $otherArgs
@@ -43,7 +43,7 @@ 
 	}
 
 	protected function tearDown() {
-		unset( $this->SelectField );
+		unset($this->SelectField);
 		parent::tearDown();
 	}
 
@@ -70,14 +70,14 @@ 
 
 	public function testGetParameters() {
 
-		$this->assertInternalType( 'array', $this->SFSInput->getParameters() );
+		$this->assertInternalType('array', $this->SFSInput->getParameters());
 	}
 
 
 	public function testGetResourceModuleNames() {
-		$rsmn = [ 'ext.sf_select.scriptselect' ];
+		$rsmn = ['ext.sf_select.scriptselect'];
 
-		$this->assertEquals( $rsmn, $this->SFSInput->getResourceModuleNames() );
+		$this->assertEquals($rsmn, $this->SFSInput->getResourceModuleNames());
 	}
 
 }

tests/phpunit/Unit/ApiSemanticFormsSelectTest.php

Spacing +16 added lines, -16 removed lines

@@ -24,11 +24,11 @@ 
 
 	protected function setUp() {
 		parent::setUp();
-		$parameters = [ 'action' => 'sformsselect', 'approach' => 'smw',
-		                     'query'  => 'abc', 'sep' => ',' ];
+		$parameters = ['action' => 'sformsselect', 'approach' => 'smw',
+		                     'query'  => 'abc', 'sep' => ','];
 
 		$this->ApiMain = new ApiMain(
-			$this->newRequestContext( $parameters ), true
+			$this->newRequestContext($parameters), true
 		);
 		$this->ApiSFS = new ApiSemanticFormsSelect(
 			$this->ApiMain, 'sformsselect'
@@ -36,15 +36,15 @@ 
 	}
 
 	protected function tearDown() {
-		unset( $this->ApiSFS );
-		unset( $this->ApiMain );
+		unset($this->ApiSFS);
+		unset($this->ApiMain);
 		parent::tearDown();
 	}
 
 
 	public function testCanConstruct() {
 
-		$apiMain = new ApiMain( $this->newRequestContext( [] ), true );
+		$apiMain = new ApiMain($this->newRequestContext([]), true);
 
 		$instance = new ApiSemanticFormsSelect(
 			$apiMain, 'sformsselect'
@@ -63,30 +63,30 @@ 
 	}
 
 	public function testGetDescription() {
-		$tdata = [ 'API for providing SemanticFormsSelect values' ];
-		$this->assertEquals( $this->ApiSFS->getDescription(), $tdata );
+		$tdata = ['API for providing SemanticFormsSelect values'];
+		$this->assertEquals($this->ApiSFS->getDescription(), $tdata);
 	}
 
 	public function testGetParamDescription() {
-		$tdata = [ 'approach' => 'The actual approach: function or smw',
-		                'query'    => 'The query of the former' ];
-		$this->assertEquals( $this->ApiSFS->getParamDescription(), $tdata );
+		$tdata = ['approach' => 'The actual approach: function or smw',
+		                'query'    => 'The query of the former'];
+		$this->assertEquals($this->ApiSFS->getParamDescription(), $tdata);
 	}
 
 	public function testGetVersion() {
 		$tdata = 'SFS\ApiSemanticFormsSelect: 1.1';
-		$this->assertEquals( $this->ApiSFS->getVersion(), $tdata );
+		$this->assertEquals($this->ApiSFS->getVersion(), $tdata);
 	}
 
 
-	private function newRequestContext( $request = [] ) {
+	private function newRequestContext($request = []) {
 
 		$context = new RequestContext();
 
-		if ( $request instanceof WebRequest ) {
-			$context->setRequest( $request );
+		if ($request instanceof WebRequest) {
+			$context->setRequest($request);
 		} else {
-			$context->setRequest( new FauxRequest( $request, true ) );
+			$context->setRequest(new FauxRequest($request, true));
 		}
 
 		return $context;

tests/phpunit/Unit/OutputTest.php

Spacing +6 added lines, -6 removed lines

@@ -24,24 +24,24 @@
 	}
 
 	protected function tearDown() {
-		unset( $this->data );
+		unset($this->data);
 		parent::tearDown();
 	}
 
 	public function testCanConstruct() {
-		$this->assertInstanceOf( '\SFS\Output', new Output() );
+		$this->assertInstanceOf('\SFS\Output', new Output());
 	}
 
 	public function testAddToHeadItem() {
-		$ret = Output::addToHeadItem( $this->data );
+		$ret = Output::addToHeadItem($this->data);
 
-		$this->assertArrayHasKey( 'Foo', $ret );
-		$this->assertArrayHasKey( 'Spam', $ret );
+		$this->assertArrayHasKey('Foo', $ret);
+		$this->assertArrayHasKey('Spam', $ret);
 	}
 
 	public function testCommitToParserOutput() {
 		global $wgOut;
-		$expected_result = '[' . json_encode( $this->data ) . ']';
+		$expected_result = '['.json_encode($this->data).']';
 		Output::commitToParserOutput();
 		$this->assertEquals(
 			$expected_result, $wgOut->getJsConfigVars()['sf_select']

src/Output.php

Spacing +3 added lines, -3 removed lines

@@ -26,7 +26,7 @@ 
 	 *
 	 * @param array $data
 	 */
-	public static function addToHeadItem( Array $data = [] ) {
+	public static function addToHeadItem(Array $data = []) {
 		return self::$headItems[] = $data;
 	}
 
@@ -35,11 +35,11 @@ 
 	 *
 	 */
 	public static function commitToParserOutput() {
-		global $wgOut;	# is there a better way to get $output/$parser without using a global? (testability!)
+		global $wgOut; # is there a better way to get $output/$parser without using a global? (testability!)
 
 		// to be used in JS like:
 		// var SFSelect_fobjs = $.parseJSON( mw.config.get( 'sf_select' ) );
-		$wgOut->addJsConfigVars('sf_select', json_encode( self::$headItems ));
+		$wgOut->addJsConfigVars('sf_select', json_encode(self::$headItems));
 
 		//self::$resourceModules = array();
 		//self::$headItems = array();

SemanticFormsSelect.hooks.php

Spacing +6 added lines, -6 removed lines

@@ -16,14 +16,14 @@ 
 	 * @param $parser Parser

 	 * @return bool

 	 */

-	public static function onSemanticFormsSelectSetup ( & $parser ) {

+	public static function onSemanticFormsSelectSetup(& $parser) {

 

-		if ( !defined( 'PF_VERSION' ) ) {

-			die( '<b>Error:</b><a href="https://github.com/SemanticMediaWiki/SemanticFormsSelect/">Semantic Forms Select</a> requires the <a href="https://www.mediawiki.org/wiki/Extension:PageForms">Page Forms</a> extension. Please install and activate this extension first.' );

+		if (!defined('PF_VERSION')) {

+			die('<b>Error:</b><a href="https://github.com/SemanticMediaWiki/SemanticFormsSelect/">Semantic Forms Select</a> requires the <a href="https://www.mediawiki.org/wiki/Extension:PageForms">Page Forms</a> extension. Please install and activate this extension first.');

 		}

 

-		if ( isset( $GLOBALS['wgPageFormsFormPrinter'] ) ) {

-			$GLOBALS['wgPageFormsFormPrinter']->registerInputType( \SFS\SemanticFormsSelectInput::class );

+		if (isset($GLOBALS['wgPageFormsFormPrinter'])) {

+			$GLOBALS['wgPageFormsFormPrinter']->registerInputType(\SFS\SemanticFormsSelectInput::class);

 		}

 

 		return true;

@@ -31,7 +31,7 @@ 
 	

 	public static function onRegistration() {

 

-		if ( isset( $GLOBALS['wgAPIModules'] ) ) {

+		if (isset($GLOBALS['wgAPIModules'])) {

 			$GLOBALS['wgAPIModules']['sformsselect'] = \SFS\ApiSemanticFormsSelect::class;

 		}

 	}

src/SelectField.php

Spacing +51 added lines, -51 removed lines

@@ -21,7 +21,7 @@ 
 	private $mValues = null;

 	private $mHasStaticValues = false;

 

-	private $mData = [];    # array with all parameters

+	private $mData = []; # array with all parameters

 	private $mQuery = "";

 	private $mFunction = "";

 	private $mSelectIsMultiple = false;

@@ -33,18 +33,18 @@ 
 	private $mLabel = false;

 	private $mDelimiter = ",";

 

-	public function __construct( & $parser ) {

+	public function __construct(& $parser) {

 		$this->mParser = $parser;

 	}

 

 	/**

 	 * Convenience function to process all parameters at once

 	 */

-	public function processParameters( $input_name = "", $other_args ) {

-		if ( array_key_exists( "query", $other_args ) ) {

-			$this->setQuery( $other_args );

-		} elseif ( array_key_exists( "function", $other_args ) ) {

-			$this->setFunction( $other_args );

+	public function processParameters($input_name = "", $other_args) {

+		if (array_key_exists("query", $other_args)) {

+			$this->setQuery($other_args);

+		} elseif (array_key_exists("function", $other_args)) {

+			$this->setFunction($other_args);

 		}

 	}

 

@@ -57,88 +57,88 @@ 
 		return $this->mData;

 	}

 

-	public function setQuery( $other_args ) {

+	public function setQuery($other_args) {

 		$querystr = $other_args["query"];

-		$querystr = str_replace( [ "~", "(", ")" ], [ "=", "[", "]" ], $querystr );

+		$querystr = str_replace(["~", "(", ")"], ["=", "[", "]"], $querystr);

 

 		//$this->mSelectField["query"] = $query;

 		$this->mQuery = $querystr;

 		$this->mData['selectquery'] = $querystr;

 

 		// unparametrized query

-		if ( strpos( $querystr, '@@@@' ) === false ) {

-			$rawparams = explode( ";", $querystr );

+		if (strpos($querystr, '@@@@') === false) {

+			$rawparams = explode(";", $querystr);

 

 			// there is no need to run the parser, $query has been parsed already

 			//$params[0] = $wgParser->replaceVariables( $params[0] );

 

-			list( $query, $params ) = QueryProcessor::getQueryAndParamsFromFunctionParams( $rawparams, SMW_OUTPUT_WIKI, QueryProcessor::INLINE_QUERY, false );

+			list($query, $params) = QueryProcessor::getQueryAndParamsFromFunctionParams($rawparams, SMW_OUTPUT_WIKI, QueryProcessor::INLINE_QUERY, false);

 			

-			$result = QueryProcessor::getResultFromQuery( $query, $params, SMW_OUTPUT_WIKI, QueryProcessor::INLINE_QUERY );

+			$result = QueryProcessor::getResultFromQuery($query, $params, SMW_OUTPUT_WIKI, QueryProcessor::INLINE_QUERY);

 		

-			$this->mValues = $this->getFormattedValuesFrom( $this->mDelimiter, $result );

+			$this->mValues = $this->getFormattedValuesFrom($this->mDelimiter, $result);

 		

-			$this->setHasStaticValues( true );

+			$this->setHasStaticValues(true);

 		}

 	}

 

-	public function setFunction( $other_args ) {

+	public function setFunction($other_args) {

 		#global $wgParser;

 

 		$function = $other_args["function"];

-		$function = '{{#' . $function . '}}';

-		$function = str_replace( [ "~", "(", ")" ], [ "=", "[", "]" ], $function );

+		$function = '{{#'.$function.'}}';

+		$function = str_replace(["~", "(", ")"], ["=", "[", "]"], $function);

 

 		//$this->mSelectField["function"] = $function;

 		$this->mFunction = $function;

 		$this->mData['selectfunction'] = $function;

 

 		// unparametrized function

-		if ( strpos( $function, '@@@@' ) === false ) {

-			$f = str_replace( ";", "|", $function );

+		if (strpos($function, '@@@@') === false) {

+			$f = str_replace(";", "|", $function);

 

-			$this->setValues( $this->mParser->replaceVariables( $f ) );

+			$this->setValues($this->mParser->replaceVariables($f));

 

-			$this->setHasStaticValues( true );

+			$this->setHasStaticValues(true);

 		}

 	}

 

-	public function setSelectIsMultiple( Array $other_args ) {

-		$this->mSelectIsMultiple = array_key_exists( "part_of_multiple", $other_args );

+	public function setSelectIsMultiple(Array $other_args) {

+		$this->mSelectIsMultiple = array_key_exists("part_of_multiple", $other_args);

 		$this->mData["selectismultiple"] = $this->mSelectIsMultiple;

 	}

 

-	public function setSelectTemplate( $input_name = "" ) {

-		$index = strpos( $input_name, "[" );

-		$this->mSelectTemplate = substr( $input_name, 0, $index );

+	public function setSelectTemplate($input_name = "") {

+		$index = strpos($input_name, "[");

+		$this->mSelectTemplate = substr($input_name, 0, $index);

 		$this->mData['selecttemplate'] = $this->mSelectTemplate;

 	}

 

-	public function setSelectField( $input_name = "" ) {

-		$index = strrpos( $input_name, "[" );

-		$this->mSelectField = substr( $input_name, $index + 1, strlen( $input_name ) - $index - 2 );

+	public function setSelectField($input_name = "") {

+		$index = strrpos($input_name, "[");

+		$this->mSelectField = substr($input_name, $index + 1, strlen($input_name) - $index - 2);

 		$this->mData['selectfield'] = $this->mSelectField;

 	}

 

-	public function setValueTemplate( Array $other_args ) {

+	public function setValueTemplate(Array $other_args) {

 		$this->mValueTemplate =

-			array_key_exists( "sametemplate", $other_args ) ? $this->mSelectTemplate : $other_args["template"];

+			array_key_exists("sametemplate", $other_args) ? $this->mSelectTemplate : $other_args["template"];

 		$this->mData["valuetemplate"] = $this->mValueTemplate;

 	}

 

-	public function setValueField( Array $other_args ) {

+	public function setValueField(Array $other_args) {

 		$this->mValueField = $other_args["field"];

 		$this->mData["valuefield"] = $this->mValueField;

 

 	}

 

-	public function setSelectRemove( Array $other_args ) {

-		$this->mSelectRemove = array_key_exists( 'rmdiv', $other_args );

+	public function setSelectRemove(Array $other_args) {

+		$this->mSelectRemove = array_key_exists('rmdiv', $other_args);

 		$this->mData['selectrm'] = $this->mSelectRemove;

 	}

 

-	public function setLabel( Array $other_args ) {

-		$this->mLabel = array_key_exists( 'label', $other_args );

+	public function setLabel(Array $other_args) {

+		$this->mLabel = array_key_exists('label', $other_args);

 		$this->mData['label'] = $this->mLabel;

 	}

 

@@ -146,15 +146,15 @@ 
 	 * setDelimiter

 	 * @param array $other_args

 	 */

-	public function setDelimiter( Array $other_args ) {

+	public function setDelimiter(Array $other_args) {

 		

 		$this->mDelimiter = $GLOBALS['wgPageFormsListSeparator'];

 		

-		if ( array_key_exists( 'sep', $other_args ) ) {

+		if (array_key_exists('sep', $other_args)) {

 			$this->mDelimiter = $other_args['sep'];

 		} else {

 			// Adding Backcompatibility

-			if ( array_key_exists( 'delimiter', $other_args ) ) {

+			if (array_key_exists('delimiter', $other_args)) {

 				$this->mDelimiter = $other_args['delimiter'];

 			}

 		}

@@ -174,10 +174,10 @@ 
 	 * setValues

 	 * @param string $values (comma separated, fully parsed list of values)

 	 */

-	private function setValues( $values ) {

-		$values = explode( $this->mDelimiter, $values );

-		$values = array_map( "trim", $values );

-		$values = array_unique( $values );

+	private function setValues($values) {

+		$values = explode($this->mDelimiter, $values);

+		$values = array_map("trim", $values);

+		$values = array_unique($values);

 		$this->mValues = $values;

 	}

 

@@ -185,21 +185,21 @@ 
 		return $this->mHasStaticValues;

 	}

 

-	private function setHasStaticValues( $StaticValues ) {

+	private function setHasStaticValues($StaticValues) {

 		$this->mHasStaticValues = $StaticValues;

 	}

 	

 	/** Copied from ApiSemanticFormsSelectRequestProcessor */

 	

-	private function getFormattedValuesFrom( $sep, $values ) {

+	private function getFormattedValuesFrom($sep, $values) {

 

-		if ( strpos( $values, $sep ) === false ) {

-			return [ $values ];

+		if (strpos($values, $sep) === false) {

+			return [$values];

 		}

 

-		$values = explode( $sep, $values );

-		$values = array_map( "trim", $values );

-		$values = array_unique( $values );

+		$values = explode($sep, $values);

+		$values = array_map("trim", $values);

+		$values = array_unique($values);

 

 		// TODO: sorting here will destroy any sort defined in the query, e.g. in case sorting for labels (instead of mainlable)

 		//sort( $values );

src/ApiSemanticFormsSelectRequestProcessor.php

Spacing +40 added lines, -40 removed lines

@@ -33,7 +33,7 @@ 
 	 *
 	 * @param Parser $parser
 	 */
-	public function __construct( Parser $parser ) {
+	public function __construct(Parser $parser) {
 		$this->parser = $parser;
 	}
 
@@ -42,7 +42,7 @@ 
 	 *
 	 * @param boolean $debugFlag
 	 */
-	public function setDebugFlag( $debugFlag ) {
+	public function setDebugFlag($debugFlag) {
 		$this->debugFlag = $debugFlag;
 	}
 
@@ -53,95 +53,95 @@ 
 	 *
 	 * @return string
 	 */
-	public function getJsonDecodedResultValuesForRequestParameters( array $parameters ) {
+	public function getJsonDecodedResultValuesForRequestParameters(array $parameters) {
 
-		if ( !isset( $parameters['query'] ) || !isset( $parameters['sep'] ) ) {
-			throw new InvalidArgumentException( 'Missing an query parameter' );
+		if (!isset($parameters['query']) || !isset($parameters['sep'])) {
+			throw new InvalidArgumentException('Missing an query parameter');
 		}
 
 		$this->parser->firstCallInit();
 		$json = [];
 
-		if ( isset( $parameters['approach'] ) && $parameters['approach'] == 'smw' ) {
-			$json = $this->doProcessQueryFor( $parameters['query'], $parameters['sep'] );
+		if (isset($parameters['approach']) && $parameters['approach'] == 'smw') {
+			$json = $this->doProcessQueryFor($parameters['query'], $parameters['sep']);
 		} else {
-			$json = $this->doProcessFunctionFor( $parameters['query'], $parameters['sep'] );
+			$json = $this->doProcessFunctionFor($parameters['query'], $parameters['sep']);
 		}
 
 		// I have no idea why we first encode and and then decode here??
 
-		return json_decode( $json );
+		return json_decode($json);
 	}
 
-	private function doProcessQueryFor( $querystr, $sep = "," ) {
+	private function doProcessQueryFor($querystr, $sep = ",") {
 
 		$querystr = str_replace(
-			[ "<", ">", "sep=;" ],
-			[ "<", ">", "sep={$sep};" ],
+			["&lt;", "&gt;", "sep=;"],
+			["<", ">", "sep={$sep};"],
 			$querystr
 		);
 
-		$rawparams = explode( ";", $querystr );
-		$f = str_replace( ";", "|", $rawparams[0] );
+		$rawparams = explode(";", $querystr);
+		$f = str_replace(";", "|", $rawparams[0]);
 
-		$rawparams[0] = $this->parser->replaceVariables( $f );
+		$rawparams[0] = $this->parser->replaceVariables($f);
 
-		if ( $this->debugFlag ) {
-			error_log( implode( "|", $rawparams ) );
+		if ($this->debugFlag) {
+			error_log(implode("|", $rawparams));
 		}
 
 		
-		list( $query, $params ) = QueryProcessor::getQueryAndParamsFromFunctionParams( $rawparams, SMW_OUTPUT_WIKI, QueryProcessor::INLINE_QUERY, false );
+		list($query, $params) = QueryProcessor::getQueryAndParamsFromFunctionParams($rawparams, SMW_OUTPUT_WIKI, QueryProcessor::INLINE_QUERY, false);
 			
-		$result = QueryProcessor::getResultFromQuery( $query, $params, SMW_OUTPUT_WIKI, QueryProcessor::INLINE_QUERY );
+		$result = QueryProcessor::getResultFromQuery($query, $params, SMW_OUTPUT_WIKI, QueryProcessor::INLINE_QUERY);
 		
 		
-		$values = $this->getFormattedValuesFrom( $sep, $result );
+		$values = $this->getFormattedValuesFrom($sep, $result);
 
-		return json_encode( [
+		return json_encode([
 			"values" => $values,
-			"count"  => count( $values )
-		] );
+			"count"  => count($values)
+		]);
 	}
 
-	private function doProcessFunctionFor( $query, $sep = "," ) {
+	private function doProcessFunctionFor($query, $sep = ",") {
 
 		$query = str_replace(
-			[ "&lt;", "&gt;", "sep=;" ],
-			[ "<", ">", "sep={$sep};" ],
+			["&lt;", "&gt;", "sep=;"],
+			["<", ">", "sep={$sep};"],
 			$query
 		);
 
-		$f = str_replace( ";", "|", $query );
+		$f = str_replace(";", "|", $query);
 
-		if ( $this->debugFlag ) {
-			error_log( $f );
+		if ($this->debugFlag) {
+			error_log($f);
 		}
 
 		$values = $this->getFormattedValuesFrom(
 			$sep,
-			$this->parser->replaceVariables( $f )
+			$this->parser->replaceVariables($f)
 		);
 
-		return json_encode( [
+		return json_encode([
 			"values" => $values,
-			"count"  => count( $values )
-		] );
+			"count"  => count($values)
+		]);
 	}
 
-	private function getFormattedValuesFrom( $sep, $values ) {
+	private function getFormattedValuesFrom($sep, $values) {
 
-		if ( strpos( $values, $sep ) === false ) {
-			return [ $values ];
+		if (strpos($values, $sep) === false) {
+			return [$values];
 		}
 
-		$values = explode( $sep, $values );
-		$values = array_map( "trim", $values );
-		$values = array_unique( $values );
+		$values = explode($sep, $values);
+		$values = array_map("trim", $values);
+		$values = array_unique($values);
 
 		// TODO: sorting here will destroy any sort defined in the query, e.g. in case sorting for labels (instead of mainlable)
 		//sort( $values );
-		array_unshift( $values, "" );
+		array_unshift($values, "");
 
 		return $values;
 	}