chillerlan / php-database

src/Query/CreateAbstract.php

Spacing +3 added lines, -3 removed lines

@@ -28,9 +28,9 @@
 		return (new class($this->DBDriver, $this->dialect) extends StatementAbstract implements CreateDatabaseInterface{
 			use CharsetTrait, IfNotExistsTrait, NameTrait;
 
-			public function ifNotExists():CreateDatabaseInterface{return $this->_ifNotExists();}
-			public function name(string $dbname = null):CreateDatabaseInterface{return $this->_name($dbname);}
-			public function collate(string $collation):CreateDatabaseInterface{return $this->_collate($collation);}
+			public function ifNotExists():CreateDatabaseInterface{return $this->_ifNotExists(); }
+			public function name(string $dbname = null):CreateDatabaseInterface{return $this->_name($dbname); }
+			public function collate(string $collation):CreateDatabaseInterface{return $this->_collate($collation); }
 
 			public function sql():string{
 

src/Query/Dialects/MySQL/CreateTable.php

Indentation +1 added lines, -1 removed lines

@@ -53,7 +53,7 @@
 
 		$type = strtoupper(trim($type));
 		$nolengthtypes = ['DATE', 'TINYBLOB', 'TINYTEXT', 'BLOB', 'TEXT', 'MEDIUMBLOB',
-		                  'MEDIUMTEXT', 'LONGBLOB', 'LONGTEXT', 'SERIAL', 'BOOLEAN', 'UUID'];
+						  'MEDIUMTEXT', 'LONGBLOB', 'LONGTEXT', 'SERIAL', 'BOOLEAN', 'UUID'];
 
 		$field[] = (is_int($length) || is_string($length) && count(explode(',', $length)) === 2) && !in_array($type, $nolengthtypes)
 			? $type.'('. $length . ')'

Spacing +2 added lines, -2 removed lines

@@ -19,7 +19,7 @@ 
 class CreateTable extends CreateTableAbstract{
 	use CharsetTrait;
 
-	public function collate(string $collation):CreateTableInterface{return $this->_collate($collation);}
+	public function collate(string $collation):CreateTableInterface{return $this->_collate($collation); }
 
 	public function sql():string{
 #		var_dump($this->cols);
@@ -56,7 +56,7 @@ 
 		                  'MEDIUMTEXT', 'LONGBLOB', 'LONGTEXT', 'SERIAL', 'BOOLEAN', 'UUID'];
 
 		$field[] = (is_int($length) || is_string($length) && count(explode(',', $length)) === 2) && !in_array($type, $nolengthtypes)
-			? $type.'('. $length . ')'
+			? $type.'('.$length.')'
 			: $type;
 
 		if($attribute){

src/Query/CreateTableInterface.php

Doc Comments +8 added lines

@@ -20,6 +20,10 @@ 
 	public function ifNotExists():CreateTableInterface;
 	public function name(string $tablename = null):CreateTableInterface;
 	public function primaryKey(string $field):CreateTableInterface;
+
+	/**
+	 * @param string $defaultValue
+	 */
 	public function field(
 		string $name,
 		string $type,
@@ -31,5 +35,9 @@ 
 		$defaultValue = null,
 		string $extra = null
 	):CreateTableInterface;
+
+	/**
+	 * @return void
+	 */
 	public function index($name);
 }

src/Query/DeleteInterface.php

Doc Comments +3 added lines

@@ -16,6 +16,9 @@
  */
 interface DeleteInterface extends StatementInterface{
 
+	/**
+	 * @return void
+	 */
 	public function from();
 	public function where($val1, $val2, $operator = '=', $bind = true, $join = 'AND'):DeleteInterface;
 

src/Query/StatementInterface.php

Doc Comments +4 added lines

@@ -18,6 +18,10 @@
 
 	public function sql():string;
 	public function bindValues():array;
+
+	/**
+	 * @return void
+	 */
 	public function execute();
 	public function quote(string $str):string;
 

src/Query/Traits/WhereTrait.php

Doc Comments +1 added lines, -1 removed lines

@@ -53,7 +53,7 @@
 	/**
 	 * @param        $val1
 	 * @param null   $val2
-	 * @param null   $operator
+	 * @param string   $operator
 	 * @param bool   $bind
 	 * @param string $join
 	 *

Spacing +1 added lines, -1 removed lines

@@ -141,7 +141,7 @@
 		$where = [];
 
 		foreach($this->where as $k => $v){
-			$last = $this->where[$k-1] ?? false;
+			$last = $this->where[$k - 1] ?? false;
 #			print_r([$last, $v]);
 
 			if(in_array($v, ['AND', 'OR', 'XOR', '(', ')'], true)){

src/Query/UpdateInterface.php

Doc Comments +11 added lines

@@ -16,8 +16,19 @@
  */
 interface UpdateInterface extends StatementInterface{
 
+	/**
+	 * @return void
+	 */
 	public function table();
+
+	/**
+	 * @return void
+	 */
 	public function set();
+
+	/**
+	 * @return UpdateAbstract
+	 */
 	public function where($val1, $val2, $operator = '=', $bind = true, $join = 'AND');
 
 }

tests/Query/QueryTest.php

Unused Use Statements +2 added lines, -2 removed lines

@@ -11,15 +11,15 @@
 
 namespace chillerlan\DatabaseTest\Query;
 
-use chillerlan\Database\Drivers\PDO\PDOMySQLDriver;
+use chillerlan\DatabaseTest\TestAbstract;
 use chillerlan\Database\DBQuery;
+use chillerlan\Database\Drivers\PDO\PDOMySQLDriver;
 use chillerlan\Database\Query\CreateInterface;
 use chillerlan\Database\Query\CreateTableInterface;
 use chillerlan\Database\Query\DeleteInterface;
 use chillerlan\Database\Query\SelectInterface;
 use chillerlan\Database\Query\StatementInterface;
 use chillerlan\Database\Query\UpdateInterface;
-use chillerlan\DatabaseTest\TestAbstract;
 
 /**
  * Class QueryTest

Spacing +1 added lines, -1 removed lines

@@ -54,7 +54,7 @@
 			->where('t2.what', 3)
 			->openBracket('and')
 			->where('t1.what', 't2.what', '>', false)
-			->where('t2.what', [1,2,3], 'in')
+			->where('t2.what', [1, 2, 3], 'in')
 			->closeBracket()
 			->where('t3.what', $this->statement->select->cols(['foo'])->from(['nope'])->where('bar', 42), 'in', true, 'or')
 			->groupby(['foo', 'bar'])

src/Query/SelectAbstract.php

Spacing +1 added lines, -1 removed lines

@@ -90,7 +90,7 @@
 		$from = $table;
 
 		if($ref){
-			$from = sprintf('%s AS %s', $table, $ref);// @todo: index hint
+			$from = sprintf('%s AS %s', $table, $ref); // @todo: index hint
 		}
 
 		$this->from[$ref ?? $table] = $from;