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

CLDR Ticket #8246(closed: fixed)

Opened 4 years ago

Last modified 4 years ago

Replacement attribute in supplementalMetadata aliases should be required

Reported by: emmons Owned by: emmons
Component: xxx-dtd Data Locale:
Phase: final Review: mark
Weeks: Data Xpath:
Xref:

Description

In our ldmlSupplemental.dtd, for the various type of aliases, we have, for example:

<!ELEMENT languageAlias EMPTY >
<!ATTLIST languageAlias type NMTOKEN #IMPLIED >
<!ATTLIST languageAlias replacement NMTOKEN #IMPLIED >
<!ATTLIST languageAlias reason ( deprecated | overlong | macrolanguage | legacy | bibliographic ) #IMPLIED >

In order for an alias to be truly valid, it MUST have both a type ( i.e. the "from" value ) and a replacement value ( the "to" value ).

I would propose that we change these from #IMPLIED to #REQUIRED , and removing those aliases that don't have replacements. We have had a few of these trickle into our data because a code got deprecated, but no replacement was provided. Bogus aliases such as this are currently causing some of our unit tests to crash, when using the modified ldml2ICU conversion structure as outlined in Cldrbug:6673

Attachments

Change History

comment:1 Changed 4 years ago by emmons

  • Owner changed from anybody to emmons
  • Priority changed from assess to major
  • Status changed from new to assigned
  • Component changed from unknown to dtd
  • Milestone changed from UNSCH to 27

comment:2 Changed 4 years ago by emmons

  • Status changed from assigned to reviewing
  • Review set to mark

comment:3 Changed 4 years ago by mark

  • Status changed from reviewing to closed
  • Resolution set to fixed
View

Add a comment

Modify Ticket

Action
as closed
Next status will be 'new'
Next status will be 'closed'
Author


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

 
Note: See TracTickets for help on using tickets.