Benestar / asparagus

src/ExpressionValidator.php

Spacing +38 added lines, -38 removed lines

@@ -74,19 +74,19 @@ 
 	 * @param int $options
 	 * @throws InvalidArgumentException
 	 */
-	public function validate( $expression, $options ) {
-		if ( !is_string( $expression ) ) {
-			throw new InvalidArgumentException( '$expression has to be a string.' );
+	public function validate($expression, $options) {
+		if (!is_string($expression)) {
+			throw new InvalidArgumentException('$expression has to be a string.');
 		}
 
-		if ( !$this->matches( $expression, $options ) ) {
-			throw new InvalidArgumentException( '$expression has to be a ' .
-				implode( ' or a ', $this->getOptionNames( $options ) ) . ', got ' . $expression
+		if (!$this->matches($expression, $options)) {
+			throw new InvalidArgumentException('$expression has to be a ' .
+				implode(' or a ', $this->getOptionNames($options)) . ', got ' . $expression
 			);
 		}
 	}
 
-	private function getOptionNames( $options ) {
+	private function getOptionNames($options) {
 		$names = array(
 			'variable' => self::VALIDATE_VARIABLE,
 			'IRI' => self::VALIDATE_IRI,
@@ -98,69 +98,69 @@ 
 			'function with variable assignment' => self::VALIDATE_FUNCTION_AS
 		);
 
-		$names = array_filter( $names, function( $key ) use ( $options ) {
+		$names = array_filter($names, function($key) use ($options) {
 			return $options & $key;
 		} );
 
-		return array_keys( $names );
+		return array_keys($names);
 	}
 
-	private function matches( $expression, $options ) {
-		return $this->isVariable( $expression, $options ) ||
-			$this->isIRI( $expression, $options ) ||
-			$this->isPrefix( $expression, $options ) ||
-			$this->isPrefixedIri( $expression, $options ) ||
-			$this->isValue( $expression, $options ) ||
-			$this->isPath( $expression, $options ) ||
-			$this->isFunction( $expression, $options ) ||
-			$this->isFunctionAs( $expression, $options );
+	private function matches($expression, $options) {
+		return $this->isVariable($expression, $options) ||
+			$this->isIRI($expression, $options) ||
+			$this->isPrefix($expression, $options) ||
+			$this->isPrefixedIri($expression, $options) ||
+			$this->isValue($expression, $options) ||
+			$this->isPath($expression, $options) ||
+			$this->isFunction($expression, $options) ||
+			$this->isFunctionAs($expression, $options);
 	}
 
-	private function isVariable( $expression, $options ) {
+	private function isVariable($expression, $options) {
 		return $options & self::VALIDATE_VARIABLE &&
-			$this->regexHelper->matchesRegex( '\{variable}', $expression );
+			$this->regexHelper->matchesRegex('\{variable}', $expression);
 	}
 
-	private function isIRI( $expression, $options ) {
+	private function isIRI($expression, $options) {
 		return $options & self::VALIDATE_IRI &&
-			$this->regexHelper->matchesRegex( '\{iri}', $expression );
+			$this->regexHelper->matchesRegex('\{iri}', $expression);
 	}
 
-	private function isPrefix( $expression, $options ) {
+	private function isPrefix($expression, $options) {
 		return $options & self::VALIDATE_PREFIX &&
-			$this->regexHelper->matchesRegex( '\{prefix}', $expression );
+			$this->regexHelper->matchesRegex('\{prefix}', $expression);
 	}
 
-	private function isPrefixedIri( $expression, $options ) {
+	private function isPrefixedIri($expression, $options) {
 		return $options & self::VALIDATE_PREFIXED_IRI &&
-			$this->regexHelper->matchesRegex( '\{prefixed_iri}', $expression );
+			$this->regexHelper->matchesRegex('\{prefixed_iri}', $expression);
 	}
 
-	private function isValue( $expression, $options ) {
+	private function isValue($expression, $options) {
 		return $options & self::VALIDATE_NATIVE &&
-			$this->regexHelper->matchesRegex( '\{native}', $expression );
+			$this->regexHelper->matchesRegex('\{native}', $expression);
 	}
 
-	private function isPath( $expression, $options ) {
+	private function isPath($expression, $options) {
 		return $options & self::VALIDATE_PATH &&
-			$this->regexHelper->matchesRegex( '\{path}', $expression );
+			$this->regexHelper->matchesRegex('\{path}', $expression);
 	}
 
-	private function isFunction( $expression, $options ) {
+	private function isFunction($expression, $options) {
 		// @todo this might not be complete
 		return $options & self::VALIDATE_FUNCTION &&
-			$this->regexHelper->matchesRegex( '\{function}', $expression ) &&
-			$this->checkBrackets( $expression );
+			$this->regexHelper->matchesRegex('\{function}', $expression) &&
+			$this->checkBrackets($expression);
 	}
 
-	private function checkBrackets( $expression ) {
-		$expression = $this->regexHelper->escapeSequences( $expression );
-		return substr_count( $expression, '(' ) === substr_count( $expression, ')' );
+	private function checkBrackets($expression) {
+		$expression = $this->regexHelper->escapeSequences($expression);
+		return substr_count($expression, '(') === substr_count($expression, ')');
 	}
 
-	private function isFunctionAs( $expression, $options ) {
+	private function isFunctionAs($expression, $options) {
 		return $options & self::VALIDATE_FUNCTION_AS &&
-			$this->regexHelper->matchesRegex( '\(\{function} AS \{variable}\)', $expression );
+			$this->regexHelper->matchesRegex('\(\{function} AS \{variable}\)', $expression);
 	}
 
 }

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));
 	}
 
 }