Skip to content

Commit fc8de45

Browse files
committed
bug #1904 [Translator] Fix changing dump directory using AssetMapper (maelanleborgne)
This PR was merged into the 2.x branch. Discussion ---------- [Translator] Fix changing dump directory using AssetMapper | Q | A | ------------- | --- | Bug fix? | yes | New feature? | no | Issues | Fix #1642 | License | MIT The configuration node `ux_translator.dump_directory` wasn't correctly taken into account when using UxTranslator with AssetMapper. This PR fixes the configuration prepend of AssetMapper to use the custom path + adds a "caution" section in the docs to remind developers to also update their `importmap.php`. Commits ------- a27cfc8 Fix changing dump directory using AssetMapper
2 parents 4ca6f2f + a27cfc8 commit fc8de45

File tree

2 files changed

+8
-2
lines changed

2 files changed

+8
-2
lines changed

src/Translator/doc/index.rst

+6
Original file line numberDiff line numberDiff line change
@@ -138,6 +138,12 @@ file. 2 of the new items are::
138138
'path' => 'var/translations/configuration.js',
139139
],
140140

141+
.. caution::
142+
143+
If you change the ``dump_directory`` in your configuration file, you will need to
144+
replace the default ``var/translations/***`` with your new path in the
145+
``importmap.php`` file.
146+
141147
These are then imported in your ``assets/translator.js`` file. This setup is
142148
very similar to working with WebpackEncore. However, the ``var/translations/index.js``
143149
file contains *every* translation in your app, which is not ideal for production

src/Translator/src/DependencyInjection/UxTranslatorExtension.php

+2-2
Original file line numberDiff line numberDiff line change
@@ -43,12 +43,12 @@ public function prepend(ContainerBuilder $container)
4343
if (!$this->isAssetMapperAvailable($container)) {
4444
return;
4545
}
46-
46+
$config = $container->getExtensionConfig('ux_translator')[0];
4747
$container->prependExtensionConfig('framework', [
4848
'asset_mapper' => [
4949
'paths' => [
5050
__DIR__.'/../../assets/dist' => '@symfony/ux-translator',
51-
'%kernel.project_dir%/var/translations' => 'var/translations',
51+
$config['dump_directory'] => '@app/translations',
5252
],
5353
],
5454
]);

0 commit comments

Comments
 (0)