Skip to content

Commit

Permalink
test @id as lowercase
Browse files Browse the repository at this point in the history
  • Loading branch information
mhosken committed Oct 7, 2020
1 parent f7bbefa commit a51cdab
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions lib/sldr/ldml.py
Original file line number Diff line number Diff line change
Expand Up @@ -762,6 +762,8 @@ def keyel(x):
for k, a in ((l, x.get(l)) for l in xl):
if k == 'id' and all(q in _digits for q in a):
res += (k, float(a))
elif k == 'id':
res += (k, a.lower())
elif k == 'type':
res += (k, self._typeSortKey(a))
else:
Expand Down

1 comment on commit a51cdab

@DavidLRowe
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@mhosken This doesn't seem to have fixed issue #1 for nnh.xml, that is element dateFormatItem with id="yMd" sorts after id="yMEd" but (following CLDR) it should sort before.

Please sign in to comment.