Call to undefined function _locale_import_read_po() in ../includes/install.inc on line 1113 when installing Drupal in French.

CommentFileSizeAuthor
#1 1297506-1-gettext-on-install.patch773 bytesalberto56
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

alberto56’s picture

alberto56 CreditAttribution: alberto56 commented
Status: Active » Needs review
FileSize
1297506-1-gettext-on-install.patch773 bytes

_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.

clemens.tolboom’s picture

clemens.tolboom
Primary language Dutch
Location Groningen, 🇳🇱/🇪🇺
CreditAttribution: clemens.tolboom commented

Subscribe and XRef #1266318: Make English a first class language

Gábor Hojtsy’s picture

Gábor Hojtsy
he/him
Primary language Hungarian
Location Hungary
CreditAttribution: Gábor Hojtsy commented
Status: Needs review » Reviewed & tested by the community

I'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!

catch’s picture

catch
he/him
Primary language English
CreditAttribution: catch commented
Status: Reviewed & tested by the community » Fixed

Trivial fix, committed and pushed to 8.x

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
CreditAttribution: webchick commented

Hm. Is there no way of testing this? That's a pretty nasty bug.

catch’s picture

catch
he/him
Primary language English
CreditAttribution: catch commented

There's no way to test the installer, simpletest installs Drupal via a different routine.

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
CreditAttribution: webchick commented

Darn. :(

catch’s picture

catch
he/him
Primary language English
CreditAttribution: catch commented

But stranger things have happened, so #1297676: Allow testing the installer.

alberto56’s picture

alberto56 CreditAttribution: alberto56 commented

See #630446: Allow SimpleTest to test the non-interactive installer.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.