New best-practice for Symfony Forms

[javiereguiluz]

In #5076 we've reworded an important note as follows:

The ``getName()`` method returns the identifier of this form "type". These
identifiers must be unique in the application. Unless you want to override
a built-in type, they should be different from the default Symfony types
and from any type defined by a third-party bundle installed in you application.
Consider prefixing your types with ``app_`` to avoid identifier collisions.

Should we make the app_ prefix recommendation a formal best practice?

• A) Add a note to the best practices (Added a new best practices for custom form field types #5093)
• B) Update usages within the documentation to have the app_ prefix

[xabbuh]

Same question as in #5076: Should we use app. instead to be consistent with service name recommendations?

Besides that I like this idea. 👍

[stof]

@xabbuh dots are forbidden in type names, so the answer is no

[weaverryan]

👍

[weaverryan]

I've just merged this in and updated the description to have a todo to add the app_ prefix to all the existing form types on the docs.

[snoek09]

Part B is done so this can be closed, right?

[xabbuh]

You are right @snoek09.