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

Improve cookbook form unit testing. #6210

Closed
wants to merge 1 commit into from
Closed

Improve cookbook form unit testing. #6210

wants to merge 1 commit into from

Conversation

cn0047
Copy link
Contributor

@cn0047 cn0047 commented Feb 1, 2016

Q A
Doc fix? yes
New docs? no
Applies to all
Fixed tickets #6203

I tweak this example from cookbook, and receive error:

Symfony\Component\Form\Exception\UnexpectedTypeException: Expected argument of type "string", "AppBundle\Form\CategoryType" given

and i just fix it like in this pr - and all become ok.

| Q             | A
| ------------- | ---
| Doc fix?      | yes
| New docs?     | no
| Applies to    | all
| Fixed tickets |  #6203|

I tweak this example from cookbook, and receive error:
````
Symfony\Component\Form\Exception\UnexpectedTypeException: Expected argument of type "string", "AppBundle\Form\CategoryType" given
````
and i just fix it like in this pr - and all become ok.
@xabbuh
Copy link
Member

xabbuh commented Feb 1, 2016

Thank you very much for this great pull request @cn007b. Unfortunately, I have to close here though as someone else fixed the issue in the meantime in #6199.

@xabbuh xabbuh closed this Feb 1, 2016
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.

2 participants