Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.Updated: Comment #N
Problem/Motivation
While cloning a Drupal 8 site with $ drush archive-dump then extract we got error regarding the non-existence of the translation directory. That's ok.
What bothers me is this should not be an absolute path as it breaks CMI IMHO
Locale injects configuration specific value into config/active
locale.settings.yml: path: sites/default/files/translations
Proposed resolution
Why not use public:// (thanks to @helmo)
locale.settings.yml: path: public://translations
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | change-default-translation-path-to-use-streamwrapper-2168733-1.patch | 681 bytes | mavimo |
Comments
Comment #1
mavimo CreditAttribution: mavimo commentedUsing public:// stream wrapper.
@mavimo & @kmox83
Comment #2
clemens.tolboom@mavimo let's consider this a bug now as the patch looks ok to me :)
Comment #4
clemens.tolboomTests seems to use streamwrapper
translations://po_3DVuIk.po... is that correct?Comment #5
mavimo CreditAttribution: mavimo commented@clemens.tolboom no, you can't use
translation://StreamWrapper becouse it will be resolved using 'translation.path' config. Usingtranslation://in translation.path will generate a recursion in StreamWrapper resolving.Comment #6
clemens.tolboom@mavimo I was not clear enough in #4 ... the test failures are using
translation://so you need to fix for that somehow.Comment #7
mavimo CreditAttribution: mavimo commentedComment #8
steva1982 CreditAttribution: steva1982 commentedThis issue can be considered completed, depends from #2263995: Support for recursive path resolution in LocalStream.
Comment #17
quietone CreditAttribution: quietone as a volunteer commentedThe path was changed to being based on the site path in October 2015 in #2384675: Deprecate conf_path().
Closing this as outdated.