Yes, according to the Symfony Book you should put them in the\Form\Type directory.

👍 Though we have twouse statements on the same page. Can you please update them too?

@xabbuh: I am not sure what two use statements you mean. Can you give me the lines?

@HKandulla Sorry, I missed the fact that you already tackled them.

@HKandulla Can you also add the filename comments like started in#5727?

@xabbuh: Good idea. I added the filename comments to all files (Formtype, Controller and Views). Additionally I changed the namespace for the example Controller from "AppBundle\Controller\Admin" to "AppBundle\Controller" to use standard directory order.

I'm not sure I'm 100% behind this PR. A major goal of the best practices guide is to make everything simpler and quicker to write. That's why templates have been moved to/app/Resources (AppBundle:static:home.html.twig vsstatic/home.html.twig).

For the same reason, I would argue that theType prefix of the form type classes already makes it obvious that these are types. Adding a deeper level of namespaces,Type, means we only add more characters to write.

On the other side, things can get out of hand pretty quickly when having a form type, extension and listener. It's also consistent with the internal Form component extension organization.

So I'm leaning a bit towards -0.4 for usingForm\Type in the best practices (but instead, updating to the book to useForm directly). I would also like to see the opinions of@weaverryan@javiereguiluz before merging/not-merging this.

Anyway, as always a huge thanks to you for putting time into making the Symfony documentation more consistent! It's great you noticed this inconsistency, it's just a matter of deciding the practice.

@wouterj here it is the discussion I had with@weaverryan about this issue:symfony/demo#6

Personally, I like to keep FormTypes in an own directory, especially when you start to put more/all busines logik into one Bundle. But I leave it up to you. Let me know, than I can do the changes xabbuh pointed out before you merge.

The Symfony best practices are to put them in theForm directory alone. As@HKandulla hinted, this is totally subjective. I dislike the idea of making a beginner create thisAppBundle/Form/Type/* structure - whereForm will have nothing in it (other than aType directory) for many projects. For the Symfony Demo - which is maturing into a larger project - havingForm\Type makes more sense, but you could still put all the "types" inForm and other things in sub-directories (Form/DataTransformer).

My vote is to keep it inForm.

As there seems to be a general vote to not place form types inForm\Typeby default, I'm going to close this PR. You can of course still comment if you disagree with this decision.

In order to make things more clear and explicitly tell people that this is not a hard rule (nothing in the docs is btw), I've created a new PR adding a best practice box for the form type namespace:#6059

🎄