Follow up for #1864292-7: Installation in non-English language fails (See step 5.)
Problem/Motivation
Warning during install when installing in another language
at
core/install.php?langcode=nl&profile=standard
Warning: Invalid argument supplied for foreach() in locale_translation_status_delete_languages() (line 949 of core/modules/locale/locale.module).
Proposed resolution
A check if $status exists before it is being processed.
Remaining tasks
Discussion.
Test.
User interface changes
No UI changes anticipated.
API changes
No API changes anticipated.
Comment | File | Size | Author |
---|---|---|---|
#3 | locale-delete-language-1864472-3.patch | 1.03 KB | Sutharsan |
#2 | noenglish-2012-12-12_0052.png | 78.31 KB | YesCT |
Comments
Comment #1
YesCT CreditAttribution: YesCT commentedComment #2
YesCT CreditAttribution: YesCT commentedBut English is deleted.
admin/config/regional/language
Comment #3
Sutharsan CreditAttribution: Sutharsan commentedThe code is missing a check before the foreach is started. Note that to manually test test patch, the latest patch in #1864292: Installation in non-English language fails is required too.
Comment #4
Sutharsan CreditAttribution: Sutharsan commentedgo botty
Comment #5
Gábor HojtsyThis only happens on install, so not really possible to do automated tests. Change looks trivial otherwise. Thanks!
Comment #6
Dries CreditAttribution: Dries commentedCommitted to 8.x.
Comment #7.0
(not verified) CreditAttribution: commentedupdated resolution