[prev in list] [next in list] [prev in thread] [next in thread] 

List:       php-qa
Subject:    Bug #77940 [ReO]: test using outdated mon_thousands_sep for Swedish
From:       cmb () php ! net
Date:       2019-04-25 12:21:33
Message-ID: E1hJdNd-0005dH-IT () bugs ! php ! net
[Download RAW message or body]

Edit report at https://bugs.php.net/bug.php?id=77940&edit=1

 ID:                 77940
 Updated by:         cmb@php.net
 Reported by:        v-altruo at microsoft dot com
 Summary:            test using outdated mon_thousands_sep for Swedish
 Status:             Re-Opened
 Type:               Bug
 Package:            Testing related
 Operating System:   Windows
 PHP Version:        7.2.18RC1
 Assigned To:        cmb
 Block user comment: N
 Private report:     N

 New Comment:

Whether it's a period or a non-breaking-space indeed depends on
the system, but it also may depend on any user customization.  So
it seems to me that we shouldn't be too picky here and simply
allow both (which would still not cater to customization), i.e. to
apply the attached 77940.patch.


Previous Comments:
------------------------------------------------------------------------
[2019-04-25 12:21:11] cmb@php.net

The following patch has been added/updated:

Patch Name: 77940.patch
Revision:   1556194871
URL:        https://bugs.php.net/patch-display.php?bug=77940&patch=77940.patch&revision=1556194871


------------------------------------------------------------------------
[2019-04-25 10:27:55] requinix@php.net

Windows must have changed something recently: my desktop Win10 is not up to date with \
patches (shame on me) and says mon_thousands_sep is a period, but my laptop Win10 \
which is up to date says it's a \xA0 non-breaking space.

So I guess AppVeyor's Windows is missing a patch; I didn't see anything in the Win10 \
update history referencing Swedish specifically, but there have been a few updates to \
locale information in the last few months.

------------------------------------------------------------------------
[2019-04-25 09:39:19] cmb@php.net

Hmm, the "fix" breaks the AppVeyor builds.  I'll investigate.

------------------------------------------------------------------------
[2019-04-25 07:00:40] cmb@php.net

Automatic comment on behalf of cmbecker69@gmx.de
Revision: http://git.php.net/?p=php-src.git;a=commit;h=37ca6a7a2a333b5a783b2b8726bb5c9674c6e301
                
Log: Fix #77940: test using outdated mon_thousands_sep for Swedish

------------------------------------------------------------------------
[2019-04-24 23:16:06] requinix@php.net

I'm not entirely sure but I think this changed with CLDR 23 around 2013.
https://unicode.org/cldr/charts/22/by_type/patterns.numbers.html#Western_Digits_(latn)
 https://unicode.org/cldr/charts/23/by_type/patterns.numbers.html#Western_Digits_(latn)
 22 had a <currencyGroup> with lone Swedish as '.' while 23 dropped the category.

Regardless, current chart says the separator should indeed be a space.
https://unicode.org/cldr/charts/latest/verify/numbers/sv.html

------------------------------------------------------------------------


The remainder of the comments for this report are too long. To view
the rest of the comments, please view the bug report online at

    https://bugs.php.net/bug.php?id=77940


--
Edit this bug report at https://bugs.php.net/bug.php?id=77940&edit=1


[prev in list] [next in list] [prev in thread] [next in thread]