Connection: calling query() moved here from ResultSet (BC break)

WIP: begin/commit/rollback is not sent to onQuery

Author: dg

src/Bridges/DatabaseTracy/ConnectionPanel.php

@@ -12,6 +12,7 @@
 use Nette;
 use Nette\Database\Connection;
 use Nette\Database\Helpers;
+use Nette\Database\ResultSet;
 use Tracy;
 
 
@@ -155,7 +156,7 @@ public function getPanel(): ?string
 					$cmd = is_string($this->explain)
 						? $this->explain
 						: 'EXPLAIN';
-					$explain = (new Nette\Database\ResultSet($connection, "$cmd $sql", $params))->fetchAll();
+					$explain = (new ResultSet($connection, $connection->getConnectionDriver()->query("$cmd $sql", $params)))->fetchAll();
 				} catch (\PDOException) {
 				}
 			}

src/Database/Connection.php

@@ -156,7 +156,7 @@ public function beginTransaction(): void
 			throw new \LogicException(__METHOD__ . '() call is forbidden inside a transaction() callback');
 		}
 
-		$this->query('::beginTransaction');
+		$this->getConnectionDriver()->beginTransaction();
 	}
 
 
@@ -166,7 +166,7 @@ public function commit(): void
 			throw new \LogicException(__METHOD__ . '() call is forbidden inside a transaction() callback');
 		}
 
-		$this->query('::commit');
+		$this->getConnectionDriver()->commit();
 	}
 
 
@@ -176,7 +176,7 @@ public function rollBack(): void
 			throw new \LogicException(__METHOD__ . '() call is forbidden inside a transaction() callback');
 		}
 
-		$this->query('::rollBack');
+		$this->getConnectionDriver()->rollBack();
 	}
 
 
@@ -215,14 +215,17 @@ public function query(#[Language('SQL')] string $sql, #[Language('GenericSQL')]
 	{
 		[$this->sql, $params] = $this->preprocess($sql, ...$params);
 		try {
-			$result = new ResultSet($this, $this->sql, $params, $this->rowNormalizer);
+			$time = microtime(true);
+			$result = $this->connection->query($this->sql, $params);
+			$time = microtime(true) - $time;
+			$resultSet = new ResultSet($this, $result, new SqlLiteral($this->sql, $params), $this->rowNormalizer, $time);
 		} catch (DriverException $e) {
 			Arrays::invoke($this->onQuery, $this, $e);
 			throw $e;
 		}
 
-		Arrays::invoke($this->onQuery, $this, $result);
-		return $result;
+		Arrays::invoke($this->onQuery, $this, $resultSet);
+		return $resultSet;
 	}
 
 

src/Database/ResultSet.php

@@ -17,33 +17,24 @@
  */
 class ResultSet implements \Iterator
 {
-	private readonly ?Drivers\Result $result;
-
 	/** @var callable(array, ResultSet): array */
 	private readonly mixed $normalizer;
 	private Row|false|null $lastRow = null;
 	private int $lastRowKey = -1;
 
 	/** @var Row[] */
 	private array $rows;
-	private float $time;
 	private array $converters;
 
 
 	public function __construct(
 		private readonly Connection $connection,
-		private readonly string $queryString,
-		private readonly array $params,
+		private readonly Drivers\Result $result,
+		private readonly ?SqlLiteral $query = null,
 		?callable $normalizer = null,
+		private ?float $time = null,
 	) {
-		$time = microtime(true);
 		$this->normalizer = $normalizer;
-		if (str_starts_with($queryString, '::')) {
-			$connection->getConnectionDriver()->{substr($queryString, 2)}();
-		} else {
-			$this->result = $connection->getConnectionDriver()->query($queryString, $params);
-		}
-		$this->time = microtime(true) - $time;
 	}
 
 
@@ -55,27 +46,27 @@ public function getConnection(): Connection
 	}
 
 
-	public function getQueryString(): string
+	public function getQueryString(): ?string
 	{
-		return $this->queryString;
+		return $this->query?->getSql();
 	}
 
 
 	public function getParameters(): array
 	{
-		return $this->params;
+		return $this->query?->getParameters();
 	}
 
 
 	public function getColumnCount(): ?int
 	{
-		return $this->result?->getColumnCount();
+		return $this->result->getColumnCount();
 	}
 
 
 	public function getRowCount(): ?int
 	{
-		return $this->result?->getRowCount();
+		return $this->result->getRowCount();
 	}
 
 
@@ -150,7 +141,7 @@ public function valid(): bool
 	 */
 	public function fetch(): ?Row
 	{
-		$data = $this->result?->fetch();
+		$data = $this->result->fetch();
 		if ($data === null) {
 			return null;
 
@@ -174,7 +165,7 @@ public function fetch(): ?Row
 	/** @internal */
 	public function fetchAssociative(): ?array
 	{
-		return $this->result?->fetch();
+		return $this->result->fetch();
 	}