Closed (fixed)
Project:
Drupal core
Version:
8.0.x-dev
Component:
install system
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
3 Oct 2011 at 01:00 UTC
Updated:
29 Jul 2014 at 20:01 UTC
Jump to comment: Most recent file
Comments
Comment #1
alberto56 commented_locale_import_read_po() and _locale_import_one_string() seem to have moved from locale.inc to gettext.inc, but install.inc is still referring to locale.inc.
This patch does the trick for me.
Comment #2
clemens.tolboomSubscribe and XRef #1266318: Make English a first class language
Comment #3
gábor hojtsyI've had #1219236: Locale module includes 1350+ lines of unneeded code on all page loads reopened for this bug a week ago but got no attention there. Also, I've included the locale.bulk.inc file, which does include the gettext.inc file automatically. You are right however, we are fine only using the gettext.inc file after all. I've reproduced your issue and fixes it the right way. Thanks!
Comment #4
catchTrivial fix, committed and pushed to 8.x
Comment #5
webchickHm. Is there no way of testing this? That's a pretty nasty bug.
Comment #6
catchThere's no way to test the installer, simpletest installs Drupal via a different routine.
Comment #7
webchickDarn. :(
Comment #8
catchBut stranger things have happened, so #1297676: Allow testing the installer.
Comment #9
alberto56 commentedSee #630446: Allow SimpleTest to test the non-interactive installer.