Page MenuHomeDevCentral

Localisation for create_surname and create_given_name (part 2)
ClosedPublic

Authored by dereckson on Jul 9 2016, 19:20.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Nov 2, 11:41
Unknown Object (File)
Wed, Oct 16, 23:05
Unknown Object (File)
Wed, Oct 16, 23:05
Unknown Object (File)
Wed, Oct 16, 17:22
Unknown Object (File)
Wed, Oct 16, 17:00
Unknown Object (File)
Oct 4 2024, 08:40
Unknown Object (File)
Oct 3 2024, 13:47
Unknown Object (File)
Sep 30 2024, 10:28
Subscribers
None

Details

Summary

These scripts now actually use the localisation provided.

This is a follow-up for 66c0b27797dd. Fixes T892.

Side edit: import statements by alphabetical order.

Test Plan

Diff Detail

Repository
rDWD Dæghrefn Wikidata access layer
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

dereckson retitled this revision from to .+prenom/.+nom: use localized labels.
dereckson updated this object.
dereckson edited the test plan for this revision. (Show Details)
dereckson added reviewers: Ash-Crow, dachary.
dereckson retitled this revision from .+prenom/.+nom: use localized labels to Localisation for create_surname and create_given_name (part 2).Jul 9 2016, 19:22
dereckson updated this object.
dachary edited edge metadata.

It would be nice to have a comment explaining what is expected in the json files, although it's fairly easy to infer by reading the source. As it's getting more sophisticated, unit tests would also be welcome.

Other than this, the code looks good and the feature useful :-)

This revision is now accepted and ready to land.Jul 10 2016, 08:13
This revision was automatically updated to reflect the committed changes.