Skip to content

Commit a09141c

Browse files
committed
minor #6616 Better explain the mandatory/convention location of some elements (rcousens, javiereguiluz)
This PR was submitted for the master branch but it was merged into the 2.7 branch instead (closes #6616). Discussion ---------- Better explain the mandatory/convention location of some elements This finishes #5559 by implementing the comments made by @wouterj Commits ------- eee4750 Fixed some wrong explanations 4a2c92f Tweaked the table contents cd0b889 Fix spacing on third column b853e52 Fix as per comments 99da972 Add third column to specific emplacements table describing relationship
2 parents 8387134 + eee4750 commit a09141c

File tree

1 file changed

+17
-16
lines changed

1 file changed

+17
-16
lines changed

cookbook/bundles/best_practices.rst

Lines changed: 17 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -113,22 +113,23 @@ them under the ``cache/`` or ``log/`` directory of the host application. Tools
113113
can generate files in the bundle directory structure, but only if the generated
114114
files are going to be part of the repository.
115115

116-
The following classes and files have specific emplacements:
117-
118-
=============================== =============================
119-
Type Directory
120-
=============================== =============================
121-
Commands ``Command/``
122-
Controllers ``Controller/``
123-
Service Container Extensions ``DependencyInjection/``
124-
Event Listeners ``EventListener/``
125-
Model classes [1] ``Model/``
126-
Configuration ``Resources/config/``
127-
Web Resources (CSS, JS, images) ``Resources/public/``
128-
Translation files ``Resources/translations/``
129-
Templates ``Resources/views/``
130-
Unit and Functional Tests ``Tests/``
131-
=============================== =============================
116+
The following classes and files have specific emplacements (some are mandatory
117+
and others are just conventions followed by most developers):
118+
119+
=============================== ============================= ================
120+
Type Directory Mandatory?
121+
=============================== ============================= ================
122+
Commands ``Command/`` Yes
123+
Controllers ``Controller/`` No
124+
Service Container Extensions ``DependencyInjection/`` Yes
125+
Event Listeners ``EventListener/`` No
126+
Model classes [1] ``Model/`` No
127+
Configuration ``Resources/config/`` No
128+
Web Resources (CSS, JS, images) ``Resources/public/`` Yes
129+
Translation files ``Resources/translations/`` Yes
130+
Templates ``Resources/views/`` Yes
131+
Unit and Functional Tests ``Tests/`` No
132+
=============================== ============================= ================
132133

133134
[1] See :doc:`/cookbook/doctrine/mapping_model_classes` for how to handle the
134135
mapping with a compiler pass.

0 commit comments

Comments
 (0)