Problem/Motivation
The automatic download of translation files has a hidden dependency on the Update Manager (update).
When the user opts out of notifications during the Standard Profile installation, the Update Manager is not enabled and the locale module does not communicate the missing dependency. this can lead to confusing behaviour or false help text in a few places:
On "/admin/config/regional/translate/import"
"Translation files are automatically downloaded and imported when languages are added, or when modules or themes are enabled."
This is true only if the Update Manager is enabled.
On the same page, the different ways to import translations are explained. In this help text a reference to the automatic download via the report page "Available Translations Update" is missing.
On "/admin/reports/translations"
The manual check is performed without notifying the user, that Update Manager is required for it to function. Instead it will show a status message suggesting everything is OK.
Proposed resolution
Creating a hard dependency on Update Manager is not desirable, as the modules can function without it.
Suggestion: show a message of type "warning" or "error" on the report page when a manual check is performed without update enabled, highlighting this problem.
Remaining tasks
Change help text to explain dependency on update module
Add status message to report page if update is disabled.
Beta phase evaluation
Issue category | Bug because translations are never imported, and the user isn't informed on how to get them working without this fix |
---|---|
Unfrozen changes | Unfrozen because it only changes the user interface to be more usable (UX) |
Prioritized changes | The main goal of this issue is usability. |
Comment | File | Size | Author |
---|---|---|---|
#11 | patch_error.JPG | 123.43 KB | Truptti |
#8 | interdiff-5-8.txt | 961 bytes | piyuesh23 |
#8 | 2349743-auto-translate-dependency-5.patch-8.patch | 2.87 KB | piyuesh23 |
#5 | 2349743-auto-translate-dependency-5.patch | 2.87 KB | banviktor |
#3 | 2349743-3-auto-translate-dependency.patch | 2.86 KB | rvilar |
Comments
Comment #1
rvilarI'm starting to work on that
Comment #2
aimeeraeComment #3
rvilarI attach a patch that updates the report page to add the message when you haven't Update module enabled and updated the messatge in language page too
Comment #4
jhedstromPatch no longer applies.
Comment #5
banviktor CreditAttribution: banviktor commentedrerolled
Comment #6
banviktor CreditAttribution: banviktor commentedComment #7
jhedstromThis looks good. One small nit though:
Shouldn't this be
!module
? (Although I can't really imagine anything in the resultant url being escaped since it will be relative).I've added a beta-phase evaluation to the issue summary.
Comment #8
piyuesh23 CreditAttribution: piyuesh23 commentedUpdated the file. Attaching the updated patch & interdiff below.
Comment #9
piyuesh23 CreditAttribution: piyuesh23 commentedComment #10
piyuesh23 CreditAttribution: piyuesh23 commentedComment #11
Truptti CreditAttribution: Truptti at Axelerant commentedTried verifying the issue, but the patch in #8 gave error.Updating the status to Needs Work
Comment #12
Truptti CreditAttribution: Truptti at Axelerant commentedComment #23
quietone CreditAttribution: quietone at PreviousNext commentedI tested this on Drupal 9.5.x, standard install.
Seems to be fixed.
Therefore, closing as cannot reproduce. If you are experiencing this problem on a supported version of Drupal reopen the issue, by setting the status to 'Active', and provide complete steps to reproduce the issue (starting from "Install Drupal core").
Thanks!
Comment #24
quietone CreditAttribution: quietone at PreviousNext commented