rkrx / php-mysql-query-builder

src/Builder/Traits/OffsetBuilder.php

Spacing +1 added lines, -1 removed lines

@@ -21,7 +21,7 @@
 	 * @return string
 	 */
 	protected function buildOffset($query) {
-		if($this->offset !== null) {
+		if ($this->offset !== null) {
 			$query .= "OFFSET\n\t{$this->offset}\n";
 		}
 		return $query;

src/Builder/Traits/UnionBuilder.php

Spacing +4 added lines, -4 removed lines

@@ -30,21 +30,21 @@
 	 * @return string
 	 */
 	protected function buildUnions($query) {
-		$wrap = function ($query) {
+		$wrap = function($query) {
 			$query = trim($query);
 			$query = join("\n\t", explode("\n", $query));
 			return sprintf("(\n\t%s\n)", $query);
 		};
 		$queries = [$wrap($query)];
-		foreach($this->unions as $unionQuery) {
-			if($unionQuery[0] === 'ALL') {
+		foreach ($this->unions as $unionQuery) {
+			if ($unionQuery[0] === 'ALL') {
 				$queries[] = 'UNION ALL';
 			} else {
 				$queries[] = 'UNION';
 			}
 			$queries[] = $wrap($unionQuery[1]);
 		}
-		if(count($queries) > 1) {
+		if (count($queries) > 1) {
 			return join(" ", $queries);
 		}
 		return $query;

src/Builder/Delete.php

Spacing +2 added lines, -2 removed lines

@@ -29,7 +29,7 @@ 
 	 * @return $this
 	 */
 	public function from($alias, $table = null) {
-		if($table !== null) {
+		if ($table !== null) {
 			$this->aliases[] = $alias;
 		}
 		$this->addTable($alias, $table);
@@ -43,7 +43,7 @@ 
 	public function __toString() {
 		$query = "DELETE ";
 		$query .= join(', ', $this->aliases);
-		$query = trim($query) . " FROM\n";
+		$query = trim($query)." FROM\n";
 		$query = $this->buildTables($query);
 		$query = $this->buildJoins($query);
 		$query = $this->buildWhereConditions($query);

src/Builder/Helpers/FieldValueConverter.php

Spacing +2 added lines, -2 removed lines

@@ -8,8 +8,8 @@
 	 * @return array
 	 */
 	public static function convertValues(array $row, array $columnDefinitions) {
-		foreach($row as $key => &$value) {
-			if($value !== null) {
+		foreach ($row as $key => &$value) {
+			if ($value !== null) {
 				$value = self::convertValue($value, $columnDefinitions[$key]);
 			}
 		}

src/Builder/Internal/ConditionBuilder.php

Spacing +4 added lines, -4 removed lines

@@ -12,15 +12,15 @@
 	 * @return string
 	 */
 	public static function build(Database $db, $query, array $conditions, $token) {
-		if(!count($conditions)) {
+		if (!count($conditions)) {
 			return $query;
 		}
 		$query .= "{$token}\n";
 		$arr = [];
-		foreach($conditions as $condition) {
+		foreach ($conditions as $condition) {
 			list($expression, $arguments) = $condition;
-			if(is_array($expression)) {
-				foreach($expression as $key => $value) {
+			if (is_array($expression)) {
+				foreach ($expression as $key => $value) {
 					$arr = self::buildCondition($arr, "`{$key}`=?", [$value], $db);
 				}
 			} else {

src/Builder/Expr/RequiredDBFilterMap.php

Spacing +2 added lines, -2 removed lines

@@ -27,8 +27,8 @@
 	 * @return DBExprFilter
 	 */
 	public function __invoke($expression, $keyPath, $validator = null) {
-		return new DBExprFilter($expression, $this->map, $keyPath, $validator, function ($result, array $data) {
-			if(!$result) {
+		return new DBExprFilter($expression, $this->map, $keyPath, $validator, function($result, array $data) {
+			if (!$result) {
 				throw new RequiredValueNotFoundException(sprintf("Required value %s not found", $data['key']));
 			}
 		});

src/Builder/Helpers/YieldPolyfillIterator.php

Doc Comments +1 added lines, -1 removed lines

@@ -74,7 +74,7 @@
 	}
 
 	/**
-	 * @return mixed
+	 * @return integer
 	 */
 	public function key() {
 		return $this->idx;

Spacing +8 added lines, -8 removed lines

@@ -42,16 +42,16 @@ 
 	 */
 	public function current() {
 		$row = $this->currentItem;
-		if($this->preserveTypes) {
+		if ($this->preserveTypes) {
 			$columnDefinitions = FieldTypeProvider::getFieldTypes($this->getStmt());
 			$row = FieldValueConverter::convertValues($row, $columnDefinitions);
 		}
 		$callback = $this->callback;
-		if($callback !== null) {
+		if ($callback !== null) {
 			$result = $callback($row);
-			if($result instanceof DBIgnoreRow) {
+			if ($result instanceof DBIgnoreRow) {
 				// Do nothing in this case
-			} elseif($result !== null) {
+			} elseif ($result !== null) {
 				return $result;
 			} else {
 				return $row;
@@ -79,7 +79,7 @@ 
 	 */
 	public function valid() {
 		$result = !!$this->currentItem;
-		if(!$result) {
+		if (!$result) {
 			$this->closeCursor();
 		}
 		return $result;
@@ -88,7 +88,7 @@ 
 	/**
 	 */
 	public function rewind() {
-		if($this->stmt !== null) {
+		if ($this->stmt !== null) {
 			throw new RuntimeException("It's not possible to rewind this iterator");
 		}
 		$this->stmt = call_user_func($this->statementFactory);
@@ -100,7 +100,7 @@ 
 	 * @return QueryStatement
 	 */
 	private function getStmt() {
-		if($this->stmt === null) {
+		if ($this->stmt === null) {
 			$this->rewind();
 		}
 		return $this->stmt;
@@ -109,7 +109,7 @@ 
 	/**
 	 */
 	private function closeCursor() {
-		if($this->stmt instanceof QueryStatement) {
+		if ($this->stmt instanceof QueryStatement) {
 			$this->stmt->closeCursor();
 		}
 		$this->stmt = null;

src/Builder/Select.php

Doc Comments +1 added lines, -1 removed lines

@@ -78,7 +78,7 @@
 	}
 
 	/**
-	 * @return array
+	 * @return string[]
 	 */
 	public function getFields() {
 		return $this->fields;

src/Builder/Traits/LimitBuilder.php

Spacing +2 added lines, -2 removed lines

@@ -28,10 +28,10 @@
 	 */
 	protected function buildLimit($query, $offset = null) {
 		$limit = $this->limit;
-		if($limit === null && $offset !== null) {
+		if ($limit === null && $offset !== null) {
 			$limit = '18446744073709551615';
 		}
-		if($limit !== null) {
+		if ($limit !== null) {
 			$query .= "LIMIT\n\t{$limit}\n";
 		}
 		return $query;