Benestar / asparagus

src/RegexHelper.php

Spacing +14 added lines, -14 removed lines

@@ -67,8 +67,8 @@ 
 	 * @param string $expression
 	 * @return bool
 	 */
-	public function matchesRegex( $regex, $expression ) {
-		return preg_match( '/^' . $this->resolveMagic( $regex ) . '$/i', $expression ) === 1;
+	public function matchesRegex($regex, $expression) {
+		return preg_match('/^' . $this->resolveMagic($regex) . '$/i', $expression) === 1;
 	}
 
 	/**
@@ -80,12 +80,12 @@ 
 	 * @param int $group
 	 * @return string[]
 	 */
-	public function getMatches( $regex, $expression, $group = 1 ) {
-		if ( preg_match_all(
-			'/' . $this->resolveMagic( $regex ) . '/',
-			$this->escapeSequences( $expression ),
+	public function getMatches($regex, $expression, $group = 1) {
+		if (preg_match_all(
+			'/' . $this->resolveMagic($regex) . '/',
+			$this->escapeSequences($expression),
 			$matches
-		) ) {
+		)) {
 			return $matches[$group];
 		}
 
@@ -99,14 +99,14 @@ 
 	 * @param string[] $replacements
 	 * @return string
 	 */
-	public function escapeSequences( $expression, &$replacements = null ) {
+	public function escapeSequences($expression, &$replacements = null) {
 		$replacements = array();
 		// @todo this is not completely safe but works in most cases
 		// @todo for strings use http://stackoverflow.com/questions/171480/regex-grabbing-values-between-quotation-marks
 		return preg_replace_callback(
 			'/("([^\"]*)"|\<([^\>]*)\>)/',
-			function( $match ) use ( &$replacements ) {
-				$key = '<' . md5( $match[0] ) . '>';
+			function($match) use (&$replacements) {
+				$key = '<' . md5($match[0]) . '>';
 				$replacements[$key] = $match[0];
 				return $key;
 			},
@@ -114,7 +114,7 @@ 
 		);
 	}
 
-	private function resolveMagic( $regex ) {
+	private function resolveMagic($regex) {
 		$magics = array(
 			'\{variable}' => self::$variable,
 			'\{iri}' => self::$iri,
@@ -126,7 +126,7 @@ 
 			'\{function}' => $this->getFunctionRegex()
 		);
 
-		return strtr( $regex, $magics );
+		return strtr($regex, $magics);
 	}
 
 	private function getPrefixedIriRegex() {
@@ -139,8 +139,8 @@ 
 	}
 
 	private function getFunctionRegex() {
-		$allowed = array_merge( self::$functions, array( '\<' . self::$iri . '\>', self::$prefix . ':', self::$variable, '!' ) );
-		return '(' . implode( '|', $allowed ) . ').*';
+		$allowed = array_merge(self::$functions, array('\<' . self::$iri . '\>', self::$prefix . ':', self::$variable, '!'));
+		return '(' . implode('|', $allowed) . ').*';
 	}
 
 }

src/GraphBuilder.php

Spacing +49 added lines, -49 removed lines

@@ -67,7 +67,7 @@ 
 	 * @param UsageValidator $usageValidator
 	 * @throws InvalidArgumentException
 	 */
-	public function __construct( UsageValidator $usageValidator ) {
+	public function __construct(UsageValidator $usageValidator) {
 		$this->expressionValidator = new ExpressionValidator();
 		$this->usageValidator = $usageValidator;
 	}
@@ -81,19 +81,19 @@ 
 	 * @return self
 	 * @throws InvalidArgumentException
 	 */
-	public function where( $subject, $predicate, $object ) {
-		$this->expressionValidator->validate( $subject,
+	public function where($subject, $predicate, $object) {
+		$this->expressionValidator->validate($subject,
 			ExpressionValidator::VALIDATE_VARIABLE | ExpressionValidator::VALIDATE_PREFIXED_IRI
 		);
-		$this->expressionValidator->validate( $predicate,
+		$this->expressionValidator->validate($predicate,
 			ExpressionValidator::VALIDATE_VARIABLE | ExpressionValidator::VALIDATE_PATH
 		);
-		$this->expressionValidator->validate( $object,
+		$this->expressionValidator->validate($object,
 			ExpressionValidator::VALIDATE_VARIABLE | ExpressionValidator::VALIDATE_PREFIXED_IRI | ExpressionValidator::VALIDATE_NATIVE
 		);
 
-		$this->usageValidator->trackUsedPrefixes(  $subject . ' ' . $predicate . ' ' . $object );
-		$this->usageValidator->trackDefinedVariables( $subject  . ' ' . $predicate . ' ' . $object );
+		$this->usageValidator->trackUsedPrefixes($subject . ' ' . $predicate . ' ' . $object);
+		$this->usageValidator->trackDefinedVariables($subject . ' ' . $predicate . ' ' . $object);
 
 		$this->currentSubject = $subject;
 		$this->currentPredicate = $predicate;
@@ -112,13 +112,13 @@ 
 	 * @return self
 	 * @throws InvalidArgumentException
 	 */
-	public function also( $subject, $predicate = null, $object = null ) {
-		if ( $predicate === null ) {
-			$this->where( $this->currentSubject, $this->currentPredicate, $subject );
-		} else if ( $object === null ) {
-			$this->where( $this->currentSubject, $subject, $predicate );
+	public function also($subject, $predicate = null, $object = null) {
+		if ($predicate === null) {
+			$this->where($this->currentSubject, $this->currentPredicate, $subject);
+		} else if ($object === null) {
+			$this->where($this->currentSubject, $subject, $predicate);
 		} else {
-			$this->where( $subject, $predicate, $object );
+			$this->where($subject, $predicate, $object);
 		}
 
 		return $this;
@@ -131,11 +131,11 @@ 
 	 * @return self
 	 * @throws InvalidArgumentException
 	 */
-	public function filter( $expression ) {
-		$this->expressionValidator->validate( $expression, ExpressionValidator::VALIDATE_FUNCTION );
+	public function filter($expression) {
+		$this->expressionValidator->validate($expression, ExpressionValidator::VALIDATE_FUNCTION);
 
-		$this->usageValidator->trackUsedPrefixes( $expression );
-		$this->usageValidator->trackUsedVariables( $expression );
+		$this->usageValidator->trackUsedPrefixes($expression);
+		$this->usageValidator->trackUsedVariables($expression);
 
 		$this->filters[] = '(' . $expression . ')';
 
@@ -151,8 +151,8 @@ 
 	 * @return self
 	 * @throws InvalidArgumentException
 	 */
-	public function filterExists( $subject, $predicate = null, $object = null ) {
-		$graphBuilder = $this->getGraphBuilder( $subject, $predicate, $object );
+	public function filterExists($subject, $predicate = null, $object = null) {
+		$graphBuilder = $this->getGraphBuilder($subject, $predicate, $object);
 		$this->filters[] = 'EXISTS {' . $graphBuilder->getSPARQL() . ' }';
 
 		return $this;
@@ -167,8 +167,8 @@ 
 	 * @return self
 	 * @throws InvalidArgumentException
 	 */
-	public function filterNotExists( $subject, $predicate = null, $object = null ) {
-		$graphBuilder = $this->getGraphBuilder( $subject, $predicate, $object );
+	public function filterNotExists($subject, $predicate = null, $object = null) {
+		$graphBuilder = $this->getGraphBuilder($subject, $predicate, $object);
 		$this->filters[] = 'NOT EXISTS {' . $graphBuilder->getSPARQL() . ' }';
 
 		return $this;
@@ -183,19 +183,19 @@ 
 	 * @return self
 	 * @throws InvalidArgumentException
 	 */
-	public function optional( $subject, $predicate = null, $object = null ) {
-		$graphBuilder = $this->getGraphBuilder( $subject, $predicate, $object );
+	public function optional($subject, $predicate = null, $object = null) {
+		$graphBuilder = $this->getGraphBuilder($subject, $predicate, $object);
 		$this->optionals[] = $graphBuilder->getSPARQL();
 		return $this;
 	}
 
-	private function getGraphBuilder( $subject, $predicate, $object ) {
-		if ( $subject instanceof GraphBuilder ) {
+	private function getGraphBuilder($subject, $predicate, $object) {
+		if ($subject instanceof GraphBuilder) {
 			return $subject;
 		}
 
-		$graphBuilder = new GraphBuilder( $this->usageValidator );
-		return $graphBuilder->where( $subject, $predicate, $object );
+		$graphBuilder = new GraphBuilder($this->usageValidator);
+		return $graphBuilder->where($subject, $predicate, $object);
 	}
 
 	/**
@@ -205,12 +205,12 @@ 
 	 * @return self
 	 * @throws InvalidArgumentException
 	 */
-	public function union( $graphs /* graphs ... */ ) {
-		$graphs = is_array( $graphs ) ? $graphs : func_get_args();
+	public function union($graphs /* graphs ... */) {
+		$graphs = is_array($graphs) ? $graphs : func_get_args();
 
-		$this->unions[] = implode( ' UNION', array_map( function( GraphBuilder $graph ) {
+		$this->unions[] = implode(' UNION', array_map(function(GraphBuilder $graph) {
 			return ' {' . $graph->getSPARQL() . ' }';
-		}, $graphs ) );
+		}, $graphs));
 
 		return $this;
 	}
@@ -222,14 +222,14 @@ 
 	 * @return self
 	 * @throws InvalidArgumentException
 	 */
-	public function subquery( QueryBuilder $queryBuilder ) {
-		$this->subqueries[] = $queryBuilder->getSPARQL( false );
-		$this->usageValidator->trackDefinedVariables( implode( ' ', $queryBuilder->getSelects() ) );
+	public function subquery(QueryBuilder $queryBuilder) {
+		$this->subqueries[] = $queryBuilder->getSPARQL(false);
+		$this->usageValidator->trackDefinedVariables(implode(' ', $queryBuilder->getSelects()));
 
 		// @todo temp hack to add AS definitions to defined variables
 		$regexHelper = new RegexHelper();
-		$matches = $regexHelper->getMatches( 'AS \{variable}', implode( ' ', $queryBuilder->getSelects() ) );
-		$this->usageValidator->trackDefinedVariables( $matches );
+		$matches = $regexHelper->getMatches('AS \{variable}', implode(' ', $queryBuilder->getSelects()));
+		$this->usageValidator->trackDefinedVariables($matches);
 
 		return $this;
 	}
@@ -244,9 +244,9 @@ 
 		// add subqueries to the beginning because they are logically evaluated first
 		$sparql = $this->formatSubqueries();
 
-		foreach ( $this->conditions as $subject => $predicates ) {
+		foreach ($this->conditions as $subject => $predicates) {
 			$sparql .= ' ' . $subject;
-			$sparql .= $this->formatPredicates( $predicates ) . ' .';
+			$sparql .= $this->formatPredicates($predicates) . ' .';
 		}
 
 		$sparql .= $this->formatOptionals();
@@ -256,32 +256,32 @@ 
 		return $sparql;
 	}
 
-	private function formatPredicates( array $predicates ) {
-		return implode( ' ;', array_map( function( $predicate, $objects ) {
-			return ' ' . $predicate . ' ' . implode( ' , ', $objects );
-		}, array_keys( $predicates ), $predicates ) );
+	private function formatPredicates(array $predicates) {
+		return implode(' ;', array_map(function($predicate, $objects) {
+			return ' ' . $predicate . ' ' . implode(' , ', $objects);
+		}, array_keys($predicates), $predicates));
 	}
 
 	private function formatOptionals() {
-		return implode( array_map( function( $optional ) {
+		return implode(array_map(function($optional) {
 			return ' OPTIONAL {' . $optional . ' }';
-		}, $this->optionals ) );
+		}, $this->optionals));
 	}
 
 	private function formatFilters() {
-		return implode( array_map( function( $filter ) {
+		return implode(array_map(function($filter) {
 			return ' FILTER ' . $filter;
-		}, $this->filters ) );
+		}, $this->filters));
 	}
 
 	private function formatUnions() {
-		return implode( $this->unions );
+		return implode($this->unions);
 	}
 
 	private function formatSubqueries() {
-		return implode( array_map( function( $subquery ) {
+		return implode(array_map(function($subquery) {
 			return ' { ' . $subquery . ' }';
-		}, $this->subqueries ) );
+		}, $this->subqueries));
 	}
 
 }

tests/unit/QueryExecuterTest.php

Spacing +26 added lines, -26 removed lines

@@ -13,67 +13,67 @@
  */
 class QueryExecuterTest extends \PHPUnit_Framework_TestCase {
 
-	private function getHttpMock( $params ) {
-		$http = $this->getMockBuilder( 'Asparagus\Http' )
+	private function getHttpMock($params) {
+		$http = $this->getMockBuilder('Asparagus\Http')
 			->disableOriginalConstructor()
 			->getMock();
 
-		$http->expects( $this->once() )
-			->method( 'request' )
+		$http->expects($this->once())
+			->method('request')
 			->with(
-				$this->equalTo( 'test.example.com' ),
-				$this->equalTo( $params )
+				$this->equalTo('test.example.com'),
+				$this->equalTo($params)
 			)
-			->will( $this->returnValue( '{"results":{"bindings":"~=[,,_,,]:3"}}' ) );
+			->will($this->returnValue('{"results":{"bindings":"~=[,,_,,]:3"}}'));
 
 		return $http;
 	}
 
 	public function testExecute() {
-		$http = $this->getHttpMock( array(
+		$http = $this->getHttpMock(array(
 			'query' => 'FooBar',
 			'format' => 'json'
-		) );
+		));
 
-		$queryExecuter = new QueryExecuter( 'test.example.com', array(), $http );
-		$result = $queryExecuter->execute( 'FooBar' );
+		$queryExecuter = new QueryExecuter('test.example.com', array(), $http);
+		$result = $queryExecuter->execute('FooBar');
 
-		$this->assertEquals( '~=[,,_,,]:3', $result );
+		$this->assertEquals('~=[,,_,,]:3', $result);
 	}
 
 	public function testExecuteCustomParams() {
-		$http = $this->getHttpMock( array(
+		$http = $this->getHttpMock(array(
 			'fancy-query' => 'FooBar',
 			'format-nyan' => 'json'
-		) );
+		));
 
-		$queryExecuter = new QueryExecuter( 'test.example.com', array(
+		$queryExecuter = new QueryExecuter('test.example.com', array(
 			'queryParam' => 'fancy-query',
 			'formatParam' => 'format-nyan'
-		), $http );
+		), $http);
 
-		$result = $queryExecuter->execute( 'FooBar' );
+		$result = $queryExecuter->execute('FooBar');
 
-		$this->assertEquals( '~=[,,_,,]:3', $result );
+		$this->assertEquals('~=[,,_,,]:3', $result);
 	}
 
 	public function testExecuteQueryBuilder() {
-		$http = $this->getHttpMock( array(
+		$http = $this->getHttpMock(array(
 			'query' => 'SELECT * WHERE { }',
 			'format' => 'json'
-		) );
+		));
 
-		$queryExecuter = new QueryExecuter( 'test.example.com', array(), $http );
-		$result = $queryExecuter->execute( new QueryBuilder() );
+		$queryExecuter = new QueryExecuter('test.example.com', array(), $http);
+		$result = $queryExecuter->execute(new QueryBuilder());
 
-		$this->assertEquals( '~=[,,_,,]:3', $result );
+		$this->assertEquals('~=[,,_,,]:3', $result);
 	}
 
 	public function testExecuteInvalidArgument() {
-		$queryExecuter = new QueryExecuter( 'test.example.com' );
-		$this->setExpectedException( 'InvalidArgumentException' );
+		$queryExecuter = new QueryExecuter('test.example.com');
+		$this->setExpectedException('InvalidArgumentException');
 
-		$queryExecuter->execute( null );
+		$queryExecuter->execute(null);
 	}
 
 }