add styling for queries which are run inside a transaction

cakephp · Dec 29, 2024 · d97e298 · d97e298
1 parent bf377ad
commit d97e298
Show file tree

Hide file tree

Showing 3 changed files with 36 additions and 3 deletions.
diff --git a/src/Database/Log/DebugLog.php b/src/Database/Log/DebugLog.php
@@ -65,6 +65,13 @@ class DebugLog extends AbstractLogger
      */
     protected bool $_includeSchema = false;
 
+    /**
+     * Whether a transaction is currently open or not.
+     *
+     * @var bool
+     */
+    protected bool $inTransaction = false;
+
     /**
      * Constructor
      *
@@ -162,11 +169,25 @@ public function log($level, string|Stringable $message, array $context = []): vo
 
         $this->_totalTime += $data['took'];
 
+        $sql = (string)$query;
+        $isBegin = $sql === 'BEGIN';
+        $isCommitOrRollback = $sql === 'COMMIT' || $sql === 'ROLLBACK';
+
+        if ($isBegin) {
+            $this->inTransaction = true;
+        }
+
         $this->_queries[] = [
-            'query' => (string)$query,
+            'query' => $sql,
             'took' => $data['took'],
             'rows' => $data['numRows'],
+            'inTransaction' => $this->inTransaction,
+            'isCommitOrRollback' => $isCommitOrRollback,
         ];
+
+        if ($isCommitOrRollback) {
+            $this->inTransaction = false;
+        }
     }
 
     /**

diff --git a/templates/element/sql_log_panel.php b/templates/element/sql_log_panel.php
@@ -73,7 +73,7 @@
                     </thead>
                     <tbody>
                         <?php foreach ($queries as $query) : ?>
-                        <tr>
+                        <tr<?= $query['inTransaction'] ? ' class="in-transaction"' : '' ?>>
                             <td>
                                 <?=
                                     (new SqlFormatter(
@@ -91,6 +91,12 @@
                             <td><?= h($query['rows']) ?></td>
                             <td><?= h($query['took']) ?></td>
                         </tr>
+                        <?php if($query['isCommitOrRollback']): ?>
+                            <tr>
+                                <td colspan="3" class="commit-or-rollback">
+                                </td>
+                            </tr>
+                        <?php endif; ?>
                         <?php endforeach; ?>
                     </tbody>
                 </table>
@@ -99,6 +105,6 @@
     <?php endif; ?>
 
     <?php if ($noOutput) : ?>
-    <div class="c-flash c-flash--warning">No active database connections</div>
+        <div class="c-flash c-flash--warning">No active database connections</div>
     <?php endif ?>
 </div>
diff --git a/webroot/css/style.css b/webroot/css/style.css
@@ -60,6 +60,8 @@
     --packages-warning: #ffcc00;
     --packages-link: #000;
 
+    --sql-log-transaction: #d8d8d8;
+
     --routes-btn-active-bg: #fff;
     --routes-btn-active-text: #555;
     --routes-btn-active-border: #6f6f6f;
@@ -605,6 +607,10 @@ strong {
     box-shadow: 0 2px 0 var(--routes-btn-active-border);
 }
 
+.c-sql-log-panel__entry .in-transaction {
+    background: var(--sql-log-transaction);
+}
+
 .c-toolbar {
     display: flex;
     background: var(--toolbar-bg);