Revert "PhpStan fixes (#363)"

This reverts commit 68f9d0981f194931e475e2c93730d8e8785c20e4.

.github/funding.yml

@@ -1 +0,0 @@
-github: dg

composer.json

@@ -16,7 +16,6 @@
 	"require-dev": {
 		"tracy/tracy": "~2.2",
 		"nette/tester": "~2.0",
-		"nette/di": "^3.0",
 		"phpstan/phpstan": "^0.12"
 	},
 	"replace": {

readme.md

@@ -14,13 +14,7 @@ Introduction
 Database access functions in PHP are not standardised. This library
 hides the differences between them, and above all, it gives you a very handy interface.
 
-
-Support Project
----------------
-
-Do you like Dibi? Are you looking forward to the new features?
-
-[![Donate](https://files.nette.org/icons/donation-1.svg?)](https://nette.org/make-donation?to=dibi)
+If you like Dibi, **[please make a donation now](https://nette.org/make-donation?to=dibi)**. Thank you!
 
 
 Installation
@@ -48,11 +42,11 @@ The database connection is represented by the object `Dibi\Connection`:
 
 ```php
 $database = new Dibi\Connection([
-	'driver'   => 'mysqli',
-	'host'     => 'localhost',
-	'username' => 'root',
-	'password' => '***',
-	'database' => 'table',
+    'driver'   => 'mysqli',
+    'host'     => 'localhost',
+    'username' => 'root',
+    'password' => '***',
+    'database' => 'table',
 ]);
 
 $result = $database->query('SELECT * FROM users');
@@ -62,12 +56,12 @@ Alternatively, you can use the `dibi` static register, which maintains a connect
 
 ```php
 dibi::connect([
-	'driver'   => 'mysqli',
-	'host'     => 'localhost',
-	'username' => 'root',
-	'password' => '***',
-	'database' => 'test',
-	'charset'  => 'utf8',
+    'driver'   => 'mysqli',
+    'host'     => 'localhost',
+    'username' => 'root',
+    'password' => '***',
+    'database' => 'test',
+    'charset'  => 'utf8',
 ]);
 
 $result = dibi::query('SELECT * FROM users');
@@ -235,8 +229,8 @@ Example:
 
 ```php
 $arr = [
-	'a' => 'hello',
-	'b'  => true,
+    'a' => 'hello',
+    'b'  => true,
 ];
 
 $database->query('INSERT INTO table %v', $arr);
@@ -508,7 +502,7 @@ $all = $result->fetchAssoc('customer_id|order_id');
 // we will iterate like this:
 foreach ($all as $customerId => $orders) {
    foreach ($orders as $orderId => $order) {
-	   ...
+       ...
    }
 }
 ```
@@ -540,7 +534,7 @@ $all = $result->fetchAssoc('name[]order_id');
 // we get all the Arnolds in the results
 foreach ($all['Arnold Rimmer'] as $arnoldOrders) {
    foreach ($arnoldOrders as $orderId => $order) {
-	   ...
+       ...
    }
 }
 ```
@@ -554,8 +548,8 @@ foreach ($all as $customerId => $orders) {
    echo "Customer $customerId":
 
    foreach ($orders as $orderId => $order) {
-	   echo "ID number: $order->number";
-	   // customer name is in $order->name
+       echo "ID number: $order->number";
+       // customer name is in $order->name
    }
 }
 ```
@@ -577,7 +571,7 @@ foreach ($all as $customerId => $row) {
    echo "Customer $row->name":
 
    foreach ($row->order_id as $orderId => $order) {
-	   echo "ID number: $order->number";
+       echo "ID number: $order->number";
    }
 }
 ```

src/Dibi/Bridges/Tracy/Panel.php

@@ -105,15 +105,6 @@ class Panel implements Tracy\IBarPanel
 		}
 
 		$totalTime = $s = null;
-
-		$singleConnection = reset($this->events)->connection;
-		foreach ($this->events as $event) {
-			if ($event->connection !== $singleConnection) {
-				$singleConnection = null;
-				break;
-			}
-		}
-
 		foreach ($this->events as $event) {
 			$totalTime += $event->time;
 			$connection = $event->connection;
@@ -144,10 +135,7 @@ class Panel implements Tracy\IBarPanel
 				$s .= Tracy\Helpers::editorLink($event->source[0], $event->source[1]);//->class('tracy-DibiProfiler-source');
 			}
 
-			$s .= "</td><td>{$event->count}</td>";
-			if (!$singleConnection) {
-				$s .= '<td>' . htmlspecialchars($this->getConnectionName($connection)) . '</td></tr>';
-			}
+			$s .= "</td><td>{$event->count}</td></tr>";
 		}
 
 		return '<style> #tracy-debug td.tracy-DibiProfiler-sql { background: white !important }
@@ -155,21 +143,12 @@ class Panel implements Tracy\IBarPanel
 			#tracy-debug tracy-DibiProfiler tr table { margin: 8px 0; max-height: 150px; overflow:auto } </style>
 			<h1>Queries: ' . count($this->events)
 				. ($totalTime === null ? '' : ', time: ' . number_format($totalTime * 1000, 1, '.', ' ') . ' ms') . ', '
-				. htmlspecialchars($this->getConnectionName($singleConnection)) . '</h1>
+				. htmlspecialchars($connection->getConfig('driver') . ($connection->getConfig('name') ? '/' . $connection->getConfig('name') : '')
+				. ($connection->getConfig('host') ? ' @ ' . $connection->getConfig('host') : '')) . '</h1>
 			<div class="tracy-inner tracy-DibiProfiler">
 			<table>
-				<tr><th>Time&nbsp;ms</th><th>SQL Statement</th><th>Rows</th>' . (!$singleConnection ? '<th>Connection</th>' : '') . '</tr>
-				' . $s . '
+				<tr><th>Time&nbsp;ms</th><th>SQL Statement</th><th>Rows</th></tr>' . $s . '
 			</table>
 			</div>';
 	}
-
-
-	private function getConnectionName(Dibi\Connection $connection): string
-	{
-		$driver = $connection->getConfig('driver');
-		return (is_object($driver) ? get_class($driver) : $driver)
-			. ($connection->getConfig('name') ? '/' . $connection->getConfig('name') : '')
-			. ($connection->getConfig('host') ? ' @ ' . $connection->getConfig('host') : '');
-	}
 }

src/Dibi/Drivers/MySqliDriver.php

@@ -130,15 +130,6 @@ class MySqliDriver implements Dibi\Driver
 	}
 
 
-	/**
-	 * Pings a server connection, or tries to reconnect if the connection has gone down.
-	 */
-	public function ping(): bool
-	{
-		return $this->connection->ping();
-	}
-
-
 	/**
 	 * Executes the SQL query.
 	 * @throws Dibi\DriverException
@@ -157,9 +148,6 @@ class MySqliDriver implements Dibi\Driver
 	}
 
 
-	/**
-	 * @param int|string $code
-	 */
 	public static function createException(string $message, $code, string $sql): Dibi\DriverException
 	{
 		if (in_array($code, [1216, 1217, 1451, 1452, 1701], true)) {
@@ -317,7 +305,7 @@ class MySqliDriver implements Dibi\Driver
 		if ($value->y || $value->m || $value->d) {
 			throw new Dibi\NotSupportedException('Only time interval is supported.');
 		}
-		return $value->format("'%r%H:%I:%S.%f'");
+		return $value->format('%r%H:%I:%S.%f');
 	}
 
 

src/Dibi/Drivers/PostgreDriver.php

@@ -23,7 +23,6 @@ use Dibi\Helpers;
  *   - charset => character encoding to set (default is utf8)
  *   - persistent (bool) => try to find a persistent link?
  *   - resource (resource) => existing connection resource
- *   - connect_type (int) => see pg_connect()
  */
 class PostgreDriver implements Dibi\Driver
 {
@@ -63,15 +62,14 @@ class PostgreDriver implements Dibi\Driver
 					}
 				}
 			}
-			$connectType = $config['connect_type'] ?? PGSQL_CONNECT_FORCE_NEW;
 
 			set_error_handler(function (int $severity, string $message) use (&$error) {
 				$error = $message;
 			});
 			if (empty($config['persistent'])) {
-				$this->connection = pg_connect($string, $connectType);
+				$this->connection = pg_connect($string, PGSQL_CONNECT_FORCE_NEW);
 			} else {
-				$this->connection = pg_pconnect($string, $connectType);
+				$this->connection = pg_pconnect($string);
 			}
 			restore_error_handler();
 		}

src/Dibi/Drivers/SqlsrvDriver.php

@@ -200,7 +200,7 @@ class SqlsrvDriver implements Dibi\Driver
 
 	public function escapeBinary(string $value): string
 	{
-		return '0x' . bin2hex($value);
+		return "'" . str_replace("'", "''", $value) . "'";
 	}
 
 

src/Dibi/Fluent.php

@@ -32,12 +32,6 @@ namespace Dibi;
  * @method Fluent and(...$cond)
  * @method Fluent or(...$cond)
  * @method Fluent using(...$cond)
- * @method Fluent update(...$cond)
- * @method Fluent insert(...$cond)
- * @method Fluent delete(...$cond)
- * @method Fluent into(...$cond)
- * @method Fluent values(...$cond)
- * @method Fluent set(...$args)
  * @method Fluent asc()
  * @method Fluent desc()
  */

src/Dibi/Loggers/FileLogger.php

@@ -73,9 +73,8 @@ class FileLogger
 
 	private function writeToFile(Dibi\Event $event, string $message): void
 	{
-		$driver = $event->connection->getConfig('driver');
 		$message .=
-			"\n-- driver: " . (is_object($driver) ? get_class($driver) : $driver) . '/' . $event->connection->getConfig('name')
+			"\n-- driver: " . $event->connection->getConfig('driver') . '/' . $event->connection->getConfig('name')
 			. "\n-- " . date('Y-m-d H:i:s')
 			. "\n\n";
 		file_put_contents($this->file, $message, FILE_APPEND | LOCK_EX);

src/Dibi/Result.php

@@ -19,7 +19,7 @@ class Result implements IDataSource
 {
 	use Strict;
 
-	/** @var ResultDriver|null */
+	/** @var ResultDriver */
 	private $driver;
 
 	/** @var array  Translate table */
@@ -295,7 +295,6 @@ class Result implements IDataSource
 		} while ($row = $this->fetch());
 
 		unset($x);
-		/** @var mixed[] $data */
 		return $data;
 	}
 

src/Dibi/Translator.php

@@ -92,25 +92,24 @@ final class Translator
 					$sql[] = $arg;
 				} else {
 					$sql[] = substr($arg, 0, $toSkip)
-						// note: this can change $this->args & $this->cursor & ...
-						. preg_replace_callback(<<<'XX'
-							/
-							(?=[`['":%?])                       ## speed-up
-							(?:
-								`(.+?)`|                        ## 1) `identifier`
-								\[(.+?)\]|                      ## 2) [identifier]
-								(')((?:''|[^'])*)'|             ## 3,4) string
-								(")((?:""|[^"])*)"|             ## 5,6) "string"
-								('|")|                          ## 7) lone quote
-								:(\S*?:)([a-zA-Z0-9._]?)|       ## 8,9) :substitution:
-								%([a-zA-Z~][a-zA-Z0-9~]{0,5})|  ## 10) modifier
-								(\?)                            ## 11) placeholder
-							)/xs
-XX
-,
+/*
+					. preg_replace_callback('/
+					(?=[`[\'":%?])                    ## speed-up
+					(?:
+						`(.+?)`|                     ## 1) `identifier`
+						\[(.+?)\]|                   ## 2) [identifier]
+						(\')((?:\'\'|[^\'])*)\'|     ## 3,4) 'string'
+						(")((?:""|[^"])*)"|          ## 5,6) "string"
+						(\'|")|                      ## 7) lone quote
+						:(\S*?:)([a-zA-Z0-9._]?)|    ## 8,9) :substitution:
+						%([a-zA-Z~][a-zA-Z0-9~]{0,5})|## 10) modifier
+						(\?)                         ## 11) placeholder
+					)/xs',
+*/                  // note: this can change $this->args & $this->cursor & ...
+					. preg_replace_callback('/(?=[`[\'":%?])(?:`(.+?)`|\[(.+?)\]|(\')((?:\'\'|[^\'])*)\'|(")((?:""|[^"])*)"|(\'|")|:(\S*?:)([a-zA-Z0-9._]?)|%([a-zA-Z~][a-zA-Z0-9~]{0,5})|(\?))/s',
 							[$this, 'cb'],
 							substr($arg, $toSkip)
-						);
+					);
 					if (preg_last_error()) {
 						throw new PcreException;
 					}
@@ -401,22 +400,11 @@ XX
 					$toSkip = strcspn($value, '`[\'":');
 					if (strlen($value) !== $toSkip) {
 						$value = substr($value, 0, $toSkip)
-							. preg_replace_callback(<<<'XX'
-								/
-								(?=[`['":])
-								(?:
-									`(.+?)`|
-									\[(.+?)\]|
-									(')((?:''|[^'])*)'|
-									(")((?:""|[^"])*)"|
-									('|")|
-									:(\S*?:)([a-zA-Z0-9._]?)
-								)/sx
-XX
-,
-								[$this, 'cb'],
-								substr($value, $toSkip)
-							);
+						. preg_replace_callback(
+							'/(?=[`[\'":])(?:`(.+?)`|\[(.+?)\]|(\')((?:\'\'|[^\'])*)\'|(")((?:""|[^"])*)"|(\'|")|:(\S*?:)([a-zA-Z0-9._]?))/s',
+							[$this, 'cb'],
+							substr($value, $toSkip)
+						);
 						if (preg_last_error()) {
 							throw new PcreException;
 						}

tests/dibi/Connection.connect.phpt

@@ -12,7 +12,7 @@ use Tester\Assert;
 require __DIR__ . '/bootstrap.php';
 
 
-test('', function () use ($config) {
+test(function () use ($config) {
 	$conn = new Connection($config);
 	Assert::true($conn->isConnected());
 
@@ -21,7 +21,7 @@ test('', function () use ($config) {
 });
 
 
-test('lazy', function () use ($config) {
+test(function () use ($config) { // lazy
 	$conn = new Connection($config + ['lazy' => true]);
 	Assert::false($conn->isConnected());
 
@@ -30,7 +30,7 @@ test('lazy', function () use ($config) {
 });
 
 
-test('', function () use ($config) {
+test(function () use ($config) {
 	$conn = new Connection($config);
 	Assert::true($conn->isConnected());
 
@@ -40,7 +40,7 @@ test('', function () use ($config) {
 });
 
 
-test('', function () use ($config) {
+test(function () use ($config) {
 	$conn = new Connection($config);
 	Assert::true($conn->isConnected());
 
@@ -52,7 +52,7 @@ test('', function () use ($config) {
 });
 
 
-test('', function () use ($config) {
+test(function () use ($config) {
 	$conn = new Connection($config);
 	Assert::equal('hello', $conn->query('SELECT %s', 'hello')->fetchSingle());
 
@@ -63,7 +63,7 @@ test('', function () use ($config) {
 });
 
 
-test('', function () use ($config) {
+test(function () use ($config) {
 	Assert::exception(function () use ($config) {
 		new Connection($config + ['onConnect' => '']);
 	}, InvalidArgumentException::class, "Configuration option 'onConnect' must be array.");

tests/dibi/PdoDriver.providedConnection.phpt

@@ -29,11 +29,13 @@ Assert::exception(function () {
 }, Dibi\DriverException::class, 'PDO connection in exception or warning error mode is not supported.');
 
 
-test('PDO error mode: explicitly set silent', function () {
+// PDO error mode: explicitly set silent
+test(function () {
 	buildPdoDriver(PDO::ERRMODE_SILENT);
 });
 
 
-test('PDO error mode: implicitly set silent', function () {
+// PDO error mode: implicitly set silent
+test(function () {
 	buildPdoDriver(null);
 });

tests/dibi/Result.normalize.phpt

@@ -24,7 +24,7 @@ class MockResult extends Dibi\Result
 }
 
 
-test('', function () {
+test(function () {
 	$result = new MockResult;
 	$result->setType('col', Type::BOOL);
 
@@ -46,7 +46,7 @@ test('', function () {
 });
 
 
-test('', function () {
+test(function () {
 	$result = new MockResult;
 	$result->setType('col', Type::TEXT);
 
@@ -62,7 +62,7 @@ test('', function () {
 });
 
 
-test('', function () {
+test(function () {
 	$result = new MockResult;
 	$result->setType('col', Type::FLOAT);
 
@@ -139,7 +139,7 @@ test('', function () {
 });
 
 
-test('', function () {
+test(function () {
 	$result = new MockResult;
 	$result->setType('col', Type::INTEGER);
 
@@ -165,7 +165,7 @@ test('', function () {
 });
 
 
-test('', function () {
+test(function () {
 	$result = new MockResult;
 	$result->setType('col', Type::DATETIME);
 
@@ -183,7 +183,7 @@ test('', function () {
 });
 
 
-test('', function () {
+test(function () {
 	$result = new MockResult;
 	$result->setType('col', Type::DATETIME);
 	$result->setFormat(Type::DATETIME, 'Y-m-d H:i:s');
@@ -202,7 +202,7 @@ test('', function () {
 });
 
 
-test('', function () {
+test(function () {
 	$result = new MockResult;
 	$result->setType('col', Type::DATE);
 
@@ -218,7 +218,7 @@ test('', function () {
 });
 
 
-test('', function () {
+test(function () {
 	$result = new MockResult;
 	$result->setType('col', Type::TIME);
 

tests/dibi/bootstrap.php

@@ -37,7 +37,7 @@ if ($config['system'] === 'odbc') {
 }
 
 
-function test(string $title, Closure $function): void
+function test(Closure $function)
 {
 	$function();
 }