Commit 9b330ef
committed
feature #4507 Comply with best practices, Round 2 (WouterJ)
This PR was merged into the 2.3 branch.
Discussion
----------
Comply with best practices, Round 2
| Q | A
| --- | ---
| Doc fix? | yes
| New docs? | no
| Applies to | all
| Fixed tickets | partly #4431
Less other fixes, more find/replace this time to speed up the process. I have one question this time:
* Should we just mention the use of the param converter (the current case), or update all simple controller examples to the use param converter?
Commits
-------
5ee9791 Some fixes
fdc460d Minor standard fix for best practices guide
2a97453 Minor tweak
a29f9fb Don't use form() helper
4ef1ef3 Apply best practices to forms
de4fcf4 Use AppBundle instead of AcmeStoreBundle
c8ce507 Other minor fixes
3c71b6d Use AppBundle instead of AcmeDemoBundleFile tree
4 files changed
+199
-241
lines changed- best_practices
- book
4 files changed
+199
-241
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
15 | 15 | | |
16 | 16 | | |
17 | 17 | | |
18 | | - | |
19 | | - | |
20 | | - | |
| 18 | + | |
21 | 19 | | |
22 | 20 | | |
23 | 21 | | |
| |||
51 | 49 | | |
52 | 50 | | |
53 | 51 | | |
54 | | - | |
55 | | - | |
56 | | - | |
| 52 | + | |
57 | 53 | | |
58 | 54 | | |
59 | 55 | | |
| |||
110 | 106 | | |
111 | 107 | | |
112 | 108 | | |
113 | | - | |
114 | | - | |
115 | | - | |
| 109 | + | |
116 | 110 | | |
117 | 111 | | |
118 | 112 | | |
| |||
0 commit comments