Commit 18a18d1
committed
bug #1913 [Translator] Revert Fix changing dump directory using AssetMapper (maelanleborgne)
This PR was merged into the 2.x branch.
Discussion
----------
[Translator] Revert Fix changing dump directory using AssetMapper
| Q | A
| ------------- | ---
| Bug fix? | yes
| New feature? |no
| Issues | #1904 (comment)
| License | MIT
[This PR ](#1904) introduced a bug : when using 'var/translations' in the `importmap.php` file instead of a relative path, it would break the twig `importmap()` function.
~~This PR aims to rectify this regression by keeping a 'var/translations' namespace pointing to the dump_directory. If this is not accepted, the previous PR should be reverted to prevent users using 'var/translations' in there `importmap.php` to face this issue, but the documentation should still be updated to display an example using the relative path instead of a misleading namespace.~~
This PR reverts the change. A word in the doc to say that the `dump_directory` conf is not compatible with asset_mapper would be nice.
Commits
-------
e551cff Revert "Fix changing dump directory using AssetMapper"File tree
2 files changed
+2
-8
lines changed- src/Translator
- doc
- src/DependencyInjection
2 files changed
+2
-8
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
138 | 138 | | |
139 | 139 | | |
140 | 140 | | |
141 | | - | |
142 | | - | |
143 | | - | |
144 | | - | |
145 | | - | |
146 | | - | |
147 | 141 | | |
148 | 142 | | |
149 | 143 | | |
| |||
Lines changed: 2 additions & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
43 | 43 | | |
44 | 44 | | |
45 | 45 | | |
46 | | - | |
| 46 | + | |
47 | 47 | | |
48 | 48 | | |
49 | 49 | | |
50 | 50 | | |
51 | | - | |
| 51 | + | |
52 | 52 | | |
53 | 53 | | |
54 | 54 | | |
| |||
0 commit comments