locale_get_display_name no longer translates in the locale specified by the 2nd parameter #8656

Activity

1. gdejong

   added

   Bug

   Status: Needs Triage

   on May 30, 2022
2. cmb69 commented on May 30, 2022

   

   cmb69

   on May 30, 2022

   Member

   More actions

   I cannot reproduce that; neither with ICU 68.2 nor with ICO 70.1. Do you use different ICU versions with your PHP 8.1.5 and 8.1.6? Which one(s)?
3. cmb69

   added

   Extension: intl

   Status: Needs Feedback

   and removed

   Status: Needs Triage

   on May 30, 2022
4. gdejong commented on May 30, 2022

   

   gdejong

   on May 30, 2022

   Author

   More actions

   I should have added: I am running PHP in a Docker container (the cli-alpine version) and using mlocati/php-extension-installer to install the intl extension.

   docker-compose.yml:

   version: '3.5'
   services:
     php:
       build:
         context: .
         dockerfile: Dockerfile
       working_dir: /opt/project
       volumes:
         - .:/opt/project
   

   Dockerfile:

   FROM php:8.1.6-cli-alpine
   
   # Copy PHP Extension installer
   COPY --from=mlocati/php-extension-installer /usr/bin/install-php-extensions /usr/local/bin/
   
   RUN chmod uga+x /usr/local/bin/install-php-extensions && install-php-extensions intl
   

   test.php

   <?php
   
   echo "PHP Version: " . PHP_VERSION . PHP_EOL;
   
   if (!extension_loaded("intl")) {
       die("missing intl extension");
   }
   
   echo locale_get_display_name("ja", "ja") . PHP_EOL;
   

   Build the image:

   docker-compose build
   

   And run it:

   docker-compose run --rm php php test.php
   

   Regarding ICU versions:

   php:8.1.6-cli-alpine is using 71.1
php:8.1.5-cli-alpine is using 69.1

   (Checked that using docker-compose run --rm php php --re intl 2>&1 | grep INTL_ICU_VERSION)

   👍React with 👍1Reacted by Christoph M. Becker
5. github-actions

   added

   Status: Needs Triage

   and removed

   Status: Needs Feedback

   on May 30, 2022
6. saundefined commented on May 30, 2022

   

   saundefined

   on May 30, 2022

   Member

   More actions

   ACK, followed the steps above:

   PHP: 8.1.0, ICU: 69.1
   日本語
   
   PHP: 8.1.5, ICU: 69.1
   日本語
   
   PHP: 8.1.6, ICU: 71.1
   Japanese
   

   👍React with 👍1Reacted by Christoph M. Becker
7. cmb69 commented on May 30, 2022

   

   cmb69

   on May 30, 2022

   Member

   More actions

   Works for me even with ICU 71.1. I guess the the full data are not installed; try something like apk del icu-data-en && apk add icu-data-full. Would that solve the issue for you?

   👍React with 👍4Reacted by David DIVERRES, Julien Wadin, mightyplow and Cyril VERLOOP
8. cmb69

   added

   Status: Needs Feedback

   and removed

   Status: Needs Triage

   on May 30, 2022
9. gdejong commented on May 30, 2022

   

   gdejong

   on May 30, 2022

   Author

   More actions

   OK, the php:8.1.6-cli-alpine image uses Alpine Linux 3.16.0 (cat /etc/alpine-release) and they have split the ICU package in two.

   See https://wiki.alpinelinux.org/wiki/Release_Notes_for_Alpine_3.16.0#ICU_data_split

   

   We can install icu-data-full in our own build process as a workaround.

   Is this something that the PHP alpine Docker images would want to include by default?

   👍React with 👍2Reacted by Adrián Martín and Cyril VERLOOP
10. github-actions

    added

    Status: Needs Triage

    and removed

    Status: Needs Feedback

    on May 30, 2022
11. cmb69 commented on May 30, 2022

    

    cmb69

    on May 30, 2022

    Member

    More actions

    Thanks for checking and reporting back!

    Is this something that the PHP alpine Docker images would want to include by default?

    You need to ask the provider of these packages (whoever that is) about that.

    Anyhow, not a bug in php-src.

    👍React with 👍2Reacted by gdejong and Blurazzle
12. cmb69

    closed this as completedon May 30, 2022
13. cmb69

    added

    Status: Invalid

    and removed

    Status: Needs Triage

    on May 30, 2022