[Unicode]   Common Locale Data Repository : Bug Tracking Home | Site Map | Search
 
Modify

CLDR Ticket #11154(accepted data)

Opened 3 months ago

Last modified 2 months ago

Error in VE currency coverage

Reported by: mark Owned by: emmons
Component: currency Data Locale:
Phase: dsub Review: emmons
Weeks: Data Xpath:
Xref:

Description (last modified by mark) (diff)

I just got two failures in running the tests.

Error: (TestLocalCurrency.java:72) No current legal tender currency for locale: es_VE
Error: (TestSupplementalInfo.java:1494) Modern territory missing currency: [VE]

Here is the problem. We say that the 'from' field in the currency defaults to the start of the day, and the 'to' field defaults to the end of the day. http://unicode.org/reports/tr35/#Date_Ranges

But in SupplementalDataInfo.DateRange.java, which is what is used to parse the <currency> attributes in supplementalData.xml, that's not the way it works. The default 'to' time is also start of day. So current data like the following works — there is no point where VN doesn't have a currency (after 1978).

<region iso3166="VN">

<currency iso4217="VND" from="1985-09-14"/>
<currency iso4217="VNN" from="1978-05-03" to="1985-09-14"/>

</region>

For the data for VE that was just changed, the change was in accordance with the spec, but it actually leaves one day uncovered.

<region iso3166="VE">

<currency iso4217="VES" from="2018-06-04"/>
<currency iso4217="VEF" from="2008-01-01" to="2018-06-03"/>
<currency iso4217="VEB" from="1871-05-11" to="2008-06-30"/>

</region>

  1. So I recommend for now, changing VE to have to="2018-06-04", like the other cases. That fixes the problem, and the code won't detect a missing currency on 2018-06-03.
  1. We then have to decide whether to change the spec or change the data and code. So that we can see what the overall repercussions are, we also have to look at the other date range attributes (besides those in <currency>) to see if those have different defaults.
  1. We need to modify the tests so that all modern times are covered. That is, if the test hadn't been run today, the failure wouldn't have been caught.

Attachments

Change History

comment:1 Changed 3 months ago by mark

  • Description modified (diff)

comment:2 Changed 3 months ago by mark

  • Review set to emmons

comment:3 Changed 3 months ago by mark

Not marked for review yet, because we need to resolve some other issues.

comment:4 Changed 2 months ago by emmons

  • Status changed from new to accepted
  • Component changed from unknown to currency
  • Priority changed from assess to major
  • Milestone changed from UNSCH to 34
  • Owner changed from anybody to emmons
  • Type changed from unknown to data
View

Add a comment

Modify Ticket

Action
as accepted
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.