feature #15939 Removed the "Delete profiles" action from the web profiler sidebar (javiereguiluz)
This PR was merged into the 2.8 branch.
Discussion
----------
Removed the "Delete profiles" action from the web profiler sidebar
| Q | A
| ------------- | ---
| Bug fix? | no
| New feature? | no
| BC breaks? | no
| Deprecations? | yes
| Tests pass? | yes
| Fixed tickets | #15819
| License | MIT
| Doc PR | -
Commits
-------
8c47eb9
Removed the "Delete profiles" action from the web profiler sidebar
This commit is contained in:
commit
32002d771e
@ -125,6 +125,8 @@ class ProfilerController
|
|||||||
*/
|
*/
|
||||||
public function purgeAction()
|
public function purgeAction()
|
||||||
{
|
{
|
||||||
|
@trigger_error('The '.__METHOD__.' method is deprecated since version 2.8 and will be removed in 3.0.', E_USER_DEPRECATED);
|
||||||
|
|
||||||
if (null === $this->profiler) {
|
if (null === $this->profiler) {
|
||||||
throw new NotFoundHttpException('The profiler must be enabled.');
|
throw new NotFoundHttpException('The profiler must be enabled.');
|
||||||
}
|
}
|
||||||
|
@ -1,7 +0,0 @@
|
|||||||
{% if token is not empty %}
|
|
||||||
<div id="sidebar-admin">
|
|
||||||
<h3>Administration tools</h3>
|
|
||||||
|
|
||||||
<a href="{{ path('_profiler_purge', { token: token }) }}">Delete all profiles</a>
|
|
||||||
</div>
|
|
||||||
{% endif %}
|
|
@ -79,10 +79,6 @@
|
|||||||
{% endfor %}
|
{% endfor %}
|
||||||
</ul>
|
</ul>
|
||||||
{% endif %}
|
{% endif %}
|
||||||
|
|
||||||
<div class="module">
|
|
||||||
{{ include('@WebProfiler/Profiler/admin.html.twig', { token: token|default('') }, with_context = false) }}
|
|
||||||
</div>
|
|
||||||
</div>
|
</div>
|
||||||
</div>
|
</div>
|
||||||
</div>
|
</div>
|
||||||
|
@ -486,32 +486,15 @@ tr.status-warning td {
|
|||||||
padding-bottom: 99999px; {# needed for 'same-height columns' trick #}
|
padding-bottom: 99999px; {# needed for 'same-height columns' trick #}
|
||||||
width: 220px;
|
width: 220px;
|
||||||
}
|
}
|
||||||
#sidebar a {
|
|
||||||
color: #ADD8E6;
|
|
||||||
}
|
|
||||||
#sidebar .module {
|
#sidebar .module {
|
||||||
padding: 10px;
|
padding: 10px;
|
||||||
width: 220px;
|
width: 220px;
|
||||||
}
|
}
|
||||||
#sidebar .module h3 {
|
|
||||||
color: #999;
|
|
||||||
font-size: 12px;
|
|
||||||
font-weight: bold;
|
|
||||||
text-transform: uppercase;
|
|
||||||
margin-bottom: 10px;
|
|
||||||
}
|
|
||||||
|
|
||||||
{# Sidebar Shortcuts
|
{# Sidebar Shortcuts
|
||||||
------------------------------------------------------------------------- #}
|
------------------------------------------------------------------------- #}
|
||||||
#sidebar #sidebar-shortcuts {
|
#sidebar #sidebar-shortcuts {
|
||||||
background: #333;
|
background: #333;
|
||||||
/* needed to prevent visual issues due to the pure CSS slideToggle() */
|
|
||||||
-webkit-touch-callout: none;
|
|
||||||
-webkit-user-select: none;
|
|
||||||
-khtml-user-select: none;
|
|
||||||
-moz-user-select: none;
|
|
||||||
-ms-user-select: none;
|
|
||||||
user-select: none;
|
|
||||||
width: 220px;
|
width: 220px;
|
||||||
}
|
}
|
||||||
#sidebar #sidebar-shortcuts .shortcuts {
|
#sidebar #sidebar-shortcuts .shortcuts {
|
||||||
@ -567,10 +550,6 @@ tr.status-warning td {
|
|||||||
float: right;
|
float: right;
|
||||||
margin-right: 10px;
|
margin-right: 10px;
|
||||||
}
|
}
|
||||||
#sidebar-admin a {
|
|
||||||
font-size: 14px;
|
|
||||||
margin-left: 10px;
|
|
||||||
}
|
|
||||||
|
|
||||||
{# Sidebar Menu
|
{# Sidebar Menu
|
||||||
------------------------------------------------------------------------- #}
|
------------------------------------------------------------------------- #}
|
||||||
|
Reference in New Issue
Block a user