Skip to content
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

[2.0] http_build_query extra parameters #4076

Merged
merged 3 commits into from Apr 23, 2012
Merged

[2.0] http_build_query extra parameters #4076

merged 3 commits into from Apr 23, 2012

Conversation

Ziumin
Copy link

@Ziumin Ziumin commented Apr 23, 2012

Bug fix: yes

arg_separator.output is not always "&", so it is better ini_set it or put an extra parameters to http_build_query

@fabpot
Copy link
Member

fabpot commented Apr 23, 2012

Can you squash your commits? It will be much easier to get back to this change later on. Thanks.

@Ziumin
Copy link
Author

Ziumin commented Apr 23, 2012

I have no idea how to do it using web interface. I'm not familiar with git (prefer hg). Sorry.

@fabpot
Copy link
Member

fabpot commented Apr 23, 2012

nevermind.

fabpot added a commit that referenced this pull request Apr 23, 2012
Commits
-------

2e7d3b1 http_build_query fix
de73de0 http_build_query fix
3b7ee9a http_build_query fix

Discussion
----------

[2.0] http_build_query extra parameters

Bug fix: yes

arg_separator.output is not always "&", so it is better ini_set it or put an extra parameters to http_build_query

---------------------------------------------------------------------------

by fabpot at 2012-04-23T10:20:05Z

Can you squash your commits? It will be much easier to get back to this change later on. Thanks.

---------------------------------------------------------------------------

by Ziumin at 2012-04-23T10:46:35Z

I have no idea how to do it using web interface. I'm not familiar with git (prefer hg). Sorry.
@fabpot fabpot merged commit 2e7d3b1 into symfony:2.0 Apr 23, 2012
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.

None yet

2 participants