Skip to content

[WebProfilerBundle] Fixed profiler seach/homepage with empty token #10820

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
Apr 30, 2014

Conversation

tucksaun
Copy link
Contributor

Q A
Bug fix? yes
New feature? no
BC breaks? no
Deprecations? no
Tests pass? yes
Fixed tickets #10806
License MIT
Doc PR none

@fabpot
Copy link
Member

fabpot commented Apr 30, 2014

Thank you @tucksaun.

@fabpot fabpot merged commit 16dd0e5 into symfony:2.3 Apr 30, 2014
fabpot added a commit that referenced this pull request Apr 30, 2014
…ty token (tucksaun)

This PR was merged into the 2.3 branch.

Discussion
----------

[WebProfilerBundle] Fixed profiler seach/homepage with empty token

| Q             | A
| ------------- | ---
| Bug fix?      | yes
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | #10806
| License       | MIT
| Doc PR        | none

Commits
-------

16dd0e5 [WebProfilerBundle] added test case for #10773
5b91e70 [WebProfilerBundle] fixed profiler homepage, fixed #10806
7b425d2 [WebProfilerBundle] Added test case for #10806
@perajovic
Copy link
Contributor

Shouldn't be merged this PR in 2.4 and master as well?

@tucksaun tucksaun deleted the fix-profiler-seach-with-empty-token branch April 30, 2014 09:43
@stloyd
Copy link
Contributor

stloyd commented Apr 30, 2014

@perajovic This is done from time to time, so no need to worry about, it will be done for sure.

@tucksaun
Copy link
Contributor Author

@perajovic 2.3 is regularly merge in 2.4 and 2.4 in master

@perajovic
Copy link
Contributor

Yes, I know. But a few moments ago I checked a master's log and it wasn't there. Some previous commit was already merged in all current branches 2.3/2.4/master, so I thought that something is wrong...

Cheers!

@stof
Copy link
Member

stof commented Apr 30, 2014

@perajovic 2.4 was merged into master 5 hours ago. this PR has only been merged 2 hours ago

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants