Skip to content

Commit

Permalink
bug #26368 [WebProfilerBundle] Fix Debug toolbar breaks app (xkobal)
Browse files Browse the repository at this point in the history
This PR was squashed before being merged into the 2.7 branch (closes #26368).

Discussion
----------

[WebProfilerBundle] Fix Debug toolbar breaks app

| Q             | A
| ------------- | ---
| Branch?       | 2.7
| Bug fix?      | yes
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes    <!-- please add some, will be required by reviewers -->
| Fixed tickets | #26364
| License       | MIT
| Doc PR        | symfony/symfony-docs#... <!-- required for new features -->

Wrong method is used to count `tbody.rows`

Commits
-------

52f187e [WebProfilerBundle] Fix Debug toolbar breaks app
  • Loading branch information
nicolas-grekas committed Mar 2, 2018
2 parents 31c69ec + 52f187e commit 0c5f839
Showing 1 changed file with 1 addition and 1 deletion.
Expand Up @@ -108,7 +108,7 @@
var rows = document.createDocumentFragment();
if (requestStack.length) {
var nbOfAjaxRequest = tbodies.rows.count();
var nbOfAjaxRequest = tbodies.rows.length;
if (nbOfAjaxRequest >= 100) {
tbodies.deleteRow(nbOfAjaxRequest - 1);
}
Expand Down

1 comment on commit 0c5f839

@jezikk
Copy link

@jezikk jezikk commented on 0c5f839 Mar 4, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi, please when this bugfix is going to be released?

Please sign in to comment.