tobybatch / kimai2

Docker containers for the kimai2 web application including docker-compose and kubernetes/helm deployment.
MIT License
181 stars 96 forks source link

[BUG] Number and Currency Formatting not working due to missing icu-data-full package #511

Closed Birkenstab closed 1 year ago

Birkenstab commented 1 year ago

Number and Currency Formatting seem not to be working correctly in the fpm docker image. image Deutsch should use comma (,) as decimal delimiter

The apache version seems to be working fine.

Reproduce

Executing php code with php -a:

$formatter = new NumberFormatter('de', NumberFormatter::CURRENCY);
echo $formatter->format(1234567.899, NumberFormatter::TYPE_DEFAULT);

Results in: ¤1,234,567.90 German (de) should use a comma (,) as decimal delimiter.

After installing the icu-data-full package via apk the same code results in: 1.234.567,90 XXX Which is correct

Cause

The package icu-data-full seems to be missing, as described in https://stackoverflow.com/questions/73895691/php-numberformatter-always-gives-me-the-default-en-us

tobybatch commented 1 year ago

I'll keep this open while we check it's all working now.

stale[bot] commented 1 year ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.