Skip to content

Conversation

chalasr
Copy link
Member

@chalasr chalasr commented Aug 16, 2016

The given example gives extra quotes (Invalid database driver ""value"" instead of (Invalid database driver "value"), because the value is automatically quoted due to the use of json_encode.

@wouterj
Copy link
Member

wouterj commented Aug 17, 2016

Very nice catch, Robin. Thanks for testing and improving our code examples!

@wouterj wouterj merged commit dd54426 into symfony:2.7 Aug 17, 2016
wouterj added a commit that referenced this pull request Aug 17, 2016
… (chalasr)

This PR was merged into the 2.7 branch.

Discussion
----------

Remove extra quotes from ExprBuilder::thenInvalid() usage

The given example gives extra quotes (`Invalid database driver ""value""` instead of (`Invalid database driver "value"`), because the value is automatically quoted due to the use of `json_encode`.

Commits
-------

dd54426 Remove extra quotes from ExprBuilder::thenInvalid() usage
@chalasr chalasr deleted the fix_theninvalid_use branch August 19, 2016 09:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants