Improve performance of AbstractSqlParser class methods (#1159)
#136
Annotations
10 warnings
|
PHP 8.5:
src/Driver/Pdo/AbstractPdoTransaction.php#L189
Escaped Mutant for Mutator "MethodCallRemoval":
@@ @@
throw new Exception('Failed to set isolation level: transaction was inactive.');
}
$this->logger?->log(LogLevel::DEBUG, 'Setting transaction isolation level to ' . $this->level . ' ' . __METHOD__, ['type' => LogType::TRANSACTION]);
- $this->setTransactionIsolationLevel($level);
+
}
public function createSavepoint(string $name): void
{
|
|
PHP 8.5:
src/Driver/Pdo/AbstractPdoTransaction.php#L186
Escaped Mutant for Mutator "ConcatOperandRemoval":
@@ @@
if (!$this->isActive()) {
throw new Exception('Failed to set isolation level: transaction was inactive.');
}
- $this->logger?->log(LogLevel::DEBUG, 'Setting transaction isolation level to ' . $this->level . ' ' . __METHOD__, ['type' => LogType::TRANSACTION]);
+ $this->logger?->log(LogLevel::DEBUG, 'Setting transaction isolation level to ' . $this->level . ' ', ['type' => LogType::TRANSACTION]);
$this->setTransactionIsolationLevel($level);
}
public function createSavepoint(string $name): void
|
|
PHP 8.5:
src/Driver/Pdo/AbstractPdoTransaction.php#L186
Escaped Mutant for Mutator "ConcatOperandRemoval":
@@ @@
if (!$this->isActive()) {
throw new Exception('Failed to set isolation level: transaction was inactive.');
}
- $this->logger?->log(LogLevel::DEBUG, 'Setting transaction isolation level to ' . $this->level . ' ' . __METHOD__, ['type' => LogType::TRANSACTION]);
+ $this->logger?->log(LogLevel::DEBUG, 'Setting transaction isolation level to ' . $this->level . __METHOD__, ['type' => LogType::TRANSACTION]);
$this->setTransactionIsolationLevel($level);
}
public function createSavepoint(string $name): void
|
|
PHP 8.5:
src/Driver/Pdo/AbstractPdoTransaction.php#L186
Escaped Mutant for Mutator "ConcatOperandRemoval":
@@ @@
if (!$this->isActive()) {
throw new Exception('Failed to set isolation level: transaction was inactive.');
}
- $this->logger?->log(LogLevel::DEBUG, 'Setting transaction isolation level to ' . $this->level . ' ' . __METHOD__, ['type' => LogType::TRANSACTION]);
+ $this->logger?->log(LogLevel::DEBUG, 'Setting transaction isolation level to ' . ' ' . __METHOD__, ['type' => LogType::TRANSACTION]);
$this->setTransactionIsolationLevel($level);
}
public function createSavepoint(string $name): void
|
|
PHP 8.5:
src/Driver/Pdo/AbstractPdoTransaction.php#L83
Escaped Mutant for Mutator "Concat":
@@ @@
$this->logger?->log(LogLevel::DEBUG, 'Set savepoint ' . $this->level . ' ' . __METHOD__, ['type' => LogType::TRANSACTION]);
$this->createSavepoint('LEVEL' . $this->level);
} else {
- $this->logger?->log(LogLevel::DEBUG, 'Transaction not started: nested transaction not supported ' . __METHOD__, ['type' => LogType::TRANSACTION]);
+ $this->logger?->log(LogLevel::DEBUG, __METHOD__ . 'Transaction not started: nested transaction not supported ', ['type' => LogType::TRANSACTION]);
throw new NotSupportedException('Transaction not started: nested transaction not supported.');
}
$this->level++;
|
|
PHP 8.5:
src/Driver/Pdo/AbstractPdoTransaction.php#L81
Escaped Mutant for Mutator "MethodCallRemoval":
@@ @@
$this->logger?->log(LogLevel::DEBUG, 'Set savepoint ' . $this->level . ' ' . __METHOD__, ['type' => LogType::TRANSACTION]);
$this->createSavepoint('LEVEL' . $this->level);
} else {
- $this->logger?->log(LogLevel::DEBUG, 'Transaction not started: nested transaction not supported ' . __METHOD__, ['type' => LogType::TRANSACTION]);
+
throw new NotSupportedException('Transaction not started: nested transaction not supported.');
}
$this->level++;
|
|
PHP 8.5:
src/Driver/Pdo/AbstractPdoTransaction.php#L56
Escaped Mutant for Mutator "MethodCallRemoval":
@@ @@
$this->db->open();
if ($this->level === 0) {
if ($isolationLevel !== null) {
- $this->setTransactionIsolationLevel($isolationLevel);
+
}
$this->logger?->log(LogLevel::DEBUG, 'Begin transaction' . ($isolationLevel ? ' with isolation level ' . $isolationLevel : '') . ' ' . __METHOD__, ['type' => LogType::TRANSACTION]);
$this->db->getPdo()?->beginTransaction();
|
|
PHP 8.5:
src/Connection/AbstractConnection.php#L107
Escaped Mutant for Mutator "MethodCallRemoval":
@@ @@
try {
$result = $closure($this);
if ($transaction->isActive() && $transaction->getLevel() === $level) {
- $transaction->commit();
+
}
} catch (Throwable $e) {
$this->rollbackTransactionOnLevel($transaction, $level);
|
|
PHP 8.5:
src/Connection/AbstractConnection.php#L106
Escaped Mutant for Mutator "LogicalAndNegation":
@@ @@
$level = $transaction->getLevel();
try {
$result = $closure($this);
- if ($transaction->isActive() && $transaction->getLevel() === $level) {
+ if (!($transaction->isActive() && $transaction->getLevel() === $level)) {
$transaction->commit();
}
} catch (Throwable $e) {
|
|
PHP 8.5:
src/Connection/AbstractConnection.php#L106
Escaped Mutant for Mutator "LogicalAnd":
@@ @@
$level = $transaction->getLevel();
try {
$result = $closure($this);
- if ($transaction->isActive() && $transaction->getLevel() === $level) {
+ if ($transaction->isActive() || $transaction->getLevel() === $level) {
$transaction->commit();
}
} catch (Throwable $e) {
|