Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
bug #28183 [WebProfilerBundle] fix wrong url when base path is the in…
…dex (ismail1432) This PR was submitted for the master branch but it was merged into the 3.4 branch instead (closes #28183). Discussion ---------- [WebProfilerBundle] fix wrong url when base path is the index | Q | A | ------------- | --- | Branch? | >= 3.4 | Bug fix? | yes | New feature? | no <!-- don't forget to update src/**/CHANGELOG.md files --> | BC breaks? | no | Deprecations? | no <!-- don't forget to update UPGRADE-*.md and src/**/CHANGELOG.md files --> | Tests pass? | yes <!-- please add some, will be required by reviewers --> | Fixed tickets | #27846 <!-- #-prefixed issue number(s), if any --> | License | MIT The `FileLinkFormatter` create a wrong url because it's using [getBaseUrl](https://github.com/symfony/symfony/blob/master/src/Symfony/Component/HttpFoundation/Request.php#L870) that render the script filename, the problem is that in this class it's used twice, first time with the [getBaseUrl method](https://github.com/symfony/symfony/blob/master/src/Symfony/Component/HttpKernel/Debug/FileLinkFormatter.php#L99) and another time with the [generate method](https://github.com/symfony/symfony/blob/master/src/Symfony/Component/HttpKernel/Debug/FileLinkFormatter.php#L77). Just switch to `getBasePath` solve the issue. WDYT ? Commits ------- 897615e change baseUrl to basePath to fix wrong profiler url
- Loading branch information