Skip to content

Commit

Permalink
bug #25557 [WebProfilerBundle] add a way to limit ajax request (Simpe…
Browse files Browse the repository at this point in the history
…rfit)

This PR was merged into the 2.7 branch.

Discussion
----------

[WebProfilerBundle] add a way to limit ajax request

| Q             | A
| ------------- | ---
| Branch?       | master
| Bug fix?      | yes
| New feature?  | yes <!-- don't forget to update src/**/CHANGELOG.md files -->
| BC breaks?    | no
| Deprecations? | no <!-- don't forget to update UPGRADE-*.md files -->
| Tests pass?   | yes
| Fixed tickets | #22688
| License       | MIT
| Doc PR        | symfony/symfony-docs#... <!--highly recommended for new features-->

I need to add the doc entry and the reproducer to test that everything is ok.

Commits
-------

9ff86d6 [WebProfilerBundle] limit ajax request to 100 and remove the last one
  • Loading branch information
fabpot committed Feb 19, 2018
2 parents 862e347 + 9ff86d6 commit cb53f4c
Showing 1 changed file with 5 additions and 0 deletions.
Expand Up @@ -108,6 +108,11 @@
var rows = document.createDocumentFragment();
if (requestStack.length) {
var nbOfAjaxRequest = tbodies.rows.count();
if (nbOfAjaxRequest >= 100) {
tbodies.deleteRow(nbOfAjaxRequest - 1);
}
for (var i = 0; i < requestStack.length; i++) {
var request = requestStack[i];
Expand Down

0 comments on commit cb53f4c

Please sign in to comment.