Skip to content
This repository has been archived by the owner on Jan 28, 2020. It is now read-only.

use Popolo's sort_name for sorting candidates alphabetically #947

Open
mhl opened this issue May 25, 2016 · 1 comment
Open

use Popolo's sort_name for sorting candidates alphabetically #947

mhl opened this issue May 25, 2016 · 1 comment

Comments

@mhl
Copy link
Contributor

mhl commented May 25, 2016

At the moment there are various places (see git grep 'split()\[-1\]') where we pick the last white-space separated part of the name to sort on. This doesn't work even in the UK, where some people have double-barrelled surnames that aren't hyphenated - in other countries, where sorting conventions vary, it's even less likely to be right.

Popolo has a sort_name field which we should use instead. To use this we would need to:

@wfdd
Copy link
Contributor

wfdd commented May 25, 2016

This is still not enough for sorting to work properly in other languages; the various built-in sort functions operate on bytes (i.e. Unicode code units), the order of which may or may not correspond to alphabetical order - see #937.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants