Skip to content

Commit

Permalink
Deprecate unnecessary argument $rawSql of `AbstractCommand::interna…
Browse files Browse the repository at this point in the history
…lExecute()` (#778)
  • Loading branch information
Tigrov authored Nov 21, 2023
1 parent 6845a40 commit 102ee3a
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 5 deletions.
4 changes: 2 additions & 2 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -2,8 +2,8 @@

## 1.2.1 under development

- Bug #777: Fix `Query::count()` when it returns an incorrect value if the result is greater
than `PHP_INT_MAX` (@Tigrov)
- Bug #777: Fix `Query::count()` when it returns an incorrect value if the result is greater than `PHP_INT_MAX` (@Tigrov)
- Enh #778: Deprecate unnecessary argument `$rawSql` of `AbstractCommand::internalExecute()` (@Tigrov)

## 1.2.0 November 12, 2023

Expand Down
4 changes: 2 additions & 2 deletions src/Command/AbstractCommand.php
Original file line number Diff line number Diff line change
Expand Up @@ -548,7 +548,7 @@ abstract protected function internalGetQueryResult(int $queryMode): mixed;
/**
* Executes a prepared statement.
*
* @param string|null $rawSql The rawSql if it has been created.
* @param string|null $rawSql Deprecated. Use `null` value. Will be removed in version 2.0.0.
*
* @throws Exception
* @throws Throwable
Expand Down Expand Up @@ -581,7 +581,7 @@ protected function queryInternal(int $queryMode): mixed
$isReadMode = $this->isReadMode($queryMode);
$this->prepare($isReadMode);

$this->internalExecute($this->getRawSql());
$this->internalExecute(null);

/** @psalm-var mixed $result */
$result = $this->internalGetQueryResult($queryMode);
Expand Down
2 changes: 1 addition & 1 deletion src/Driver/Pdo/AbstractPdoCommand.php
Original file line number Diff line number Diff line change
Expand Up @@ -186,7 +186,7 @@ protected function getQueryMode(int $queryMode): string
*
* It's a wrapper around {@see PDOStatement::execute()} to support transactions and retry handlers.
*
* @param string|null $rawSql The rawSql if it has been created.
* @param string|null $rawSql Deprecated. Use `null` value. Will be removed in version 2.0.0.
*
* @throws Exception
* @throws Throwable
Expand Down

0 comments on commit 102ee3a

Please sign in to comment.