[WebProfiler] Add copy button on runnable and formatted queries (#1056)

[WebProfiler] Add copy button on runnable and formatted queries

Resources/views/Collector/db.html.twig

@@ -144,6 +144,8 @@
         .time-container { position: relative; }
         .time-container .nowrap { position: relative; z-index: 1; text-shadow: 0 0 2px #fff; }
         .time-bar { display: block; position: absolute; top: 0; left: 0; bottom: 0; background: #e0e0e0; }
+        .sql-runnable.sf-toggle-content.sf-toggle-visible { display: flex; flex-direction: column; }
+        .sql-runnable button { align-self: end; }
     </style>
 
     {% if profiler_markup_version > 1 %}
@@ -237,7 +239,7 @@
                                 <td class="nowrap">{{ '%0.2f'|format(query.executionMS * 1000) }}&nbsp;ms</td>
                             {% endif %}
                             <td>
-                                {{ query.sql|doctrine_pretty_query(highlight_only = true) }}
+                                {{ query.sql|doctrine_prettify_sql }}
 
                                 <div>
                                     <strong class="font-normal text-small">Parameters</strong>: {{ profiler_dump(query.params, 2) }}
@@ -263,12 +265,15 @@
                                 </div>
 
                                 <div id="formatted-query-{{ i }}-{{ loop.parent.loop.index }}" class="sql-runnable hidden">
-                                    {{ query.sql|doctrine_pretty_query }}
+                                    {{ query.sql|doctrine_format_sql(highlight = true) }}
+                                    <button class="btn btn-sm" data-clipboard-text="{{ query.sql|doctrine_format_sql(highlight = false)|e('html_attr') }}">Copy</button>
                                 </div>
 
                                 {% if query.runnable %}
                                     <div id="original-query-{{ i }}-{{ loop.parent.loop.index }}" class="sql-runnable hidden">
-                                        {{ (query.sql ~ ';')|doctrine_replace_query_parameters(query.params)|doctrine_pretty_query(highlight_only = true) }}
+                                        {% set runnable_sql = (query.sql ~ ';')|doctrine_replace_query_parameters(query.params) %}
+                                        {{ runnable_sql|doctrine_prettify_sql }}
+                                        <button class="btn btn-sm" data-clipboard-text="{{ runnable_sql|e('html_attr') }}">Copy</button>
                                     </div>
                                 {% endif %}
 
@@ -507,6 +512,12 @@
             }
         {% endif %}
 
+        document.querySelectorAll('[data-clipboard-text]').forEach(function(button) {
+            button.addEventListener('click', function() {
+                navigator.clipboard.writeText(button.getAttribute('data-clipboard-text'));
+            })
+        });
+
         //]]></script>
 {% endblock %}
 

Tests/ProfilerTest.php

@@ -67,7 +67,7 @@ public function testRender() : void
     {
         $this->logger->queries = [
             [
-                'sql' => 'SELECT * FROM foo WHERE bar IN (?, ?)',
+                'sql' => 'SELECT * FROM foo WHERE bar IN (?, ?) AND "" >= ""',
                 'params' => ['foo', 'bar'],
                 'types' => null,
                 'executionMS' => 1,
@@ -88,6 +88,13 @@ public function testRender() : void
         ]);
 
         $output = str_replace(["\e[37m", "\e[0m", "\e[32;1m", "\e[34;1m"], '', $output);
-        $this->assertContains("SELECT * FROM foo WHERE bar IN ('foo', 'bar');", $output);
+        $this->assertContains("SELECT * FROM foo WHERE bar IN ('foo', 'bar') AND \"\" >= \"\";", $output);
+
+        $expectedEscapedSql = 'SELECT 
  * 
FROM 
  foo 
WHERE 
  bar IN (?, ?) 
  AND "" >= ""';
+        $this->assertContains($expectedEscapedSql, $output);
+        $this->assertSame(
+            "SELECT \n  * \nFROM \n  foo \nWHERE \n  bar IN (?, ?) \n  AND \"\" >= \"\"",
+            html_entity_decode($expectedEscapedSql)
+        );
     }
 }

Twig/DoctrineExtension.php

@@ -20,7 +20,9 @@ class DoctrineExtension extends AbstractExtension
     public function getFilters()
     {
         return [
-            new TwigFilter('doctrine_pretty_query', [$this, 'formatQuery'], ['is_safe' => ['html']]),
+            new TwigFilter('doctrine_pretty_query', [$this, 'formatQuery'], ['is_safe' => ['html'], 'deprecated' => true]),
+            new TwigFilter('doctrine_prettify_sql', [$this, 'prettifySql'], ['is_safe' => ['html']]),
+            new TwigFilter('doctrine_format_sql', [$this, 'formatSql'], ['is_safe' => ['html']]),
             new TwigFilter('doctrine_replace_query_parameters', [$this, 'replaceQueryParameters']),
         ];
     }
@@ -164,16 +166,9 @@ static function ($matches) use ($parameters, &$i) {
      */
     public function formatQuery($sql, $highlightOnly = false)
     {
-        SqlFormatter::$pre_attributes            = 'class="highlight highlight-sql"';
-        SqlFormatter::$quote_attributes          = 'class="string"';
-        SqlFormatter::$backtick_quote_attributes = 'class="string"';
-        SqlFormatter::$reserved_attributes       = 'class="keyword"';
-        SqlFormatter::$boundary_attributes       = 'class="symbol"';
-        SqlFormatter::$number_attributes         = 'class="number"';
-        SqlFormatter::$word_attributes           = 'class="word"';
-        SqlFormatter::$error_attributes          = 'class="error"';
-        SqlFormatter::$comment_attributes        = 'class="comment"';
-        SqlFormatter::$variable_attributes       = 'class="variable"';
+        @trigger_error(sprintf('The "%s()" method is deprecated and will be removed in DoctrineBundle 3.0.', __METHOD__), E_USER_DEPRECATED);
+
+        $this->setUpSqlFormatter();
 
         if ($highlightOnly) {
             $html = SqlFormatter::highlight($sql);
@@ -186,6 +181,34 @@ public function formatQuery($sql, $highlightOnly = false)
         return $html;
     }
 
+    public function prettifySql(string $sql) : string
+    {
+        $this->setUpSqlFormatter();
+
+        return SqlFormatter::highlight($sql);
+    }
+
+    public function formatSql(string $sql, bool $highlight) : string
+    {
+        $this->setUpSqlFormatter();
+
+        return SqlFormatter::format($sql, $highlight);
+    }
+
+    private function setUpSqlFormatter() : void
+    {
+        SqlFormatter::$pre_attributes            = 'class="highlight highlight-sql"';
+        SqlFormatter::$quote_attributes          = 'class="string"';
+        SqlFormatter::$backtick_quote_attributes = 'class="string"';
+        SqlFormatter::$reserved_attributes       = 'class="keyword"';
+        SqlFormatter::$boundary_attributes       = 'class="symbol"';
+        SqlFormatter::$number_attributes         = 'class="number"';
+        SqlFormatter::$word_attributes           = 'class="word"';
+        SqlFormatter::$error_attributes          = 'class="error"';
+        SqlFormatter::$comment_attributes        = 'class="comment"';
+        SqlFormatter::$variable_attributes       = 'class="variable"';
+    }
+
     /**
      * Get the name of the extension
      *