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.

Sign up for GitHub

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

Jump to bottom

Fix remove concat with more than 2 arguments #746

Closed

soyuka wants to merge 1 commit into api-platform:master from soyuka:fix-concat

Member

soyuka commented Sep 10, 2016

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


          Fix remove concat with more than 2 arguments

7a54dde

Member

dunglas commented Sep 10, 2016

👍

1 similar comment

Contributor

Simperfit commented Sep 11, 2016

👍

Member

dunglas commented Sep 11, 2016 •

edited

Loading

WDYT @teohhanhui? I would like to merge this one ASAP.

Contributor

teohhanhui commented Sep 13, 2016 •

edited

Loading

~~@soyuka Since DQL does not support CONCAT for more than 2 arguments, I'm okay with reverting the relevant changes.~~

See #745 (comment) and #745 (comment)

soyuka closed this

soyuka deleted the fix-concat branch

November 3, 2016 13:43

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment