Lokal Profil has uploaded a new change for review.
https://gerrit.wikimedia.org/r/281930
Change subject: [NOT TESTED] Re-implement checks to not use globals
......................................................................
[NOT TESTED] Re-implement checks to not use globals
* Both check and conv now raise error on unknown input
* Adding lat/lon check to all configs where these have
standard name
Bug: T39422
Change-Id: I28895b6cc94e87af68f88b455ef2a431d4b5f00a
---
M erfgoedbot/monuments_config.py
M erfgoedbot/update_database.py
2 files changed, 217 insertions(+), 4 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/labs/tools/heritage
refs/changes/30/281930/1
diff --git a/erfgoedbot/monuments_config.py b/erfgoedbot/monuments_config.py
index f8c058c..d907d2b 100755
--- a/erfgoedbot/monuments_config.py
+++ b/erfgoedbot/monuments_config.py
@@ -56,10 +56,12 @@
{
'source': u'lat',
'dest': u'lat',
+ 'check': u'checkLat',
},
{
'source': u'lon',
'dest': u'lon',
+ 'check': u'checkLon',
},
{
'source': u'nomcoor',
@@ -138,10 +140,13 @@
{
'source': u'լայն',
'dest': u'lat',
+ 'check': u'checkLat',
+ }
},
{
'source': u'երկ',
'dest': u'lon',
+ 'check': u'checkLon',
},
{
'source': u'հղում',
@@ -214,10 +219,12 @@
{
'source': u'lat',
'dest': u'lat',
+ 'check': u'checkLat',
},
{
'source': u'lon',
'dest': u'lon',
+ 'check': u'checkLon',
},
{
'source': u'area',
@@ -294,10 +301,12 @@
{
'source': u'lat',
'dest': u'lat',
+ 'check': u'checkLat',
},
{
'source': u'long',
'dest': u'lon',
+ 'check': u'checkLon',
},
{
'source': u'dirección',
@@ -436,10 +445,12 @@
{
'source': u'Breitengrad',
'dest': u'lat',
+ 'check': u'checkLat',
},
{
'source': u'Längengrad',
'dest': u'lon',
+ 'check': u'checkLon',
},
],
},
@@ -497,10 +508,12 @@
{
'source': u'lat',
'dest': u'lat',
+ 'check': u'checkLat',
},
{
'source': u'lon',
'dest': u'lon',
+ 'check': u'checkLon',
},
{
'source': u'image',
@@ -578,10 +591,12 @@
{
'source': u'lat',
'dest': u'lat',
+ 'check': u'checkLat',
},
{
'source': u'lon',
'dest': u'lon',
+ 'check': u'checkLon',
},
{
'source': u'image',
@@ -667,10 +682,12 @@
{
'source': u'lat',
'dest': u'lat',
+ 'check': u'checkLat',
},
{
'source': u'lon',
'dest': u'lon',
+ 'check': u'checkLon',
},
{
'source': u'image',
@@ -754,10 +771,12 @@
{
'source': u'lat',
'dest': u'lat',
+ 'check': u'checkLat',
},
{
'source': u'lon',
'dest': u'lon',
+ 'check': u'checkLon',
},
{
'source': u'image',
@@ -936,10 +955,12 @@
{
'source': u'lat',
'dest': u'lat',
+ 'check': u'checkLat',
},
{
'source': u'lon',
'dest': u'lon',
+ 'check': u'checkLon',
},
{
'source': u'architect',
@@ -1038,10 +1059,12 @@
{
'source': u'lat',
'dest': u'lat',
+ 'check': u'checkLat',
},
{
'source': u'lon',
'dest': u'lon',
+ 'check': u'checkLon',
},
{
'source': u'architecte',
@@ -1142,10 +1165,12 @@
{
'source': u'lat',
'dest': u'lat',
+ 'check': u'checkLat',
},
{
'source': u'lon',
'dest': u'lon',
+ 'check': u'checkLon',
},
{
'source': u'architect',
@@ -1227,10 +1252,12 @@
{
'source': u'lat',
'dest': u'lat',
+ 'check': u'checkLat',
},
{
'source': u'long',
'dest': u'lon',
+ 'check': u'checkLon',
},
{
'source': u'dirección',
@@ -1303,10 +1330,12 @@
{
'source': u'шырата',
'dest': u'lat',
+ 'check': u'checkLat',
},
{
'source': u'даўгата',
'dest': u'lon',
+ 'check': u'checkLon',
},
{
'source': u'каардынаты',
@@ -1378,10 +1407,12 @@
{
'source': u'lat',
'dest': u'lat',
+ 'check': u'checkLat',
},
{
'source': u'lon',
'dest': u'lon',
+ 'check': u'checkLon',
},
{
'source': u'image',
@@ -1491,10 +1522,12 @@
{
'source': u'lat',
'dest': u'lat',
+ 'check': u'checkLat',
},
{
'source': u'lon',
'dest': u'lon',
+ 'check': u'checkLon',
},
{
'source': u'construction',
@@ -1587,10 +1620,12 @@
{
'source': u'latitude',
'dest': u'lat',
+ 'check': u'checkLat',
},
{
'source': u'longitude',
'dest': u'lon',
+ 'check': u'checkLon',
},
{
'source': u'région-ISO',
@@ -1670,10 +1705,12 @@
{
'source': u'latitudine',
'dest': u'lat',
+ 'check': u'checkLat',
},
{
'source': u'longitudine',
'dest': u'lon',
+ 'check': u'checkLon',
},
{
'source': u'regione-ISO',
@@ -1761,10 +1798,12 @@
{
'source': u'Breitengrad',
'dest': u'lat',
+ 'check': u'checkLat',
},
{
'source': u'Längengrad',
'dest': u'lon',
+ 'check': u'checkLon',
},
{
'source': u'Region-ISO',
@@ -1844,10 +1883,12 @@
{
'source': u'Breitengrad',
'dest': u'lat',
+ 'check': u'checkLat',
},
{
'source': u'Längengrad',
'dest': u'lon',
+ 'check': u'checkLon',
},
{
'source': u'Region-ISO',
@@ -1927,10 +1968,12 @@
{
'source': u'Breitengrad',
'dest': u'lat',
+ 'check': u'checkLat',
},
{
'source': u'Längengrad',
'dest': u'lon',
+ 'check': u'checkLon',
},
{
'source': u'Region-ISO',
@@ -1983,10 +2026,12 @@
{
'source': u'Breitengrad',
'dest': u'lat',
+ 'check': u'checkLat',
},
{
'source': u'Längengrad',
'dest': u'lon',
+ 'check': u'checkLon',
},
{
'source': u'Foto',
@@ -2180,10 +2225,12 @@
{
'source': u'latitude',
'dest': u'lat',
+ 'check': u'checkLat',
},
{
'source': u'longitude',
'dest': u'lon',
+ 'check': u'checkLon',
},
{
'source': u'Photo',
@@ -2275,10 +2322,12 @@
{
'source': u'latitude',
'dest': u'lat',
+ 'check': u'checkLat',
},
{
'source': u'longitude',
'dest': u'lon',
+ 'check': u'checkLon',
},
{
'source': u'image',
@@ -2341,10 +2390,12 @@
{
'source': u'lat',
'dest': u'lat',
+ 'check': u'checkLat',
},
{
'source': u'lon',
'dest': u'lon',
+ 'check': u'checkLon',
},
{
'source': u'image',
@@ -2427,10 +2478,12 @@
{
'source': u'Zeměpisná_šířka',
'dest': u'lat',
+ 'check': u'checkLat',
},
{
'source': u'Zeměpisná_délka',
'dest': u'lon',
+ 'check': u'checkLon',
},
{
'source': u'Popis',
@@ -2498,10 +2551,12 @@
{
'source': u'lat',
'dest': u'lat',
+ 'check': u'checkLat',
},
{
'source': u'long',
'dest': u'lon',
+ 'check': u'checkLon',
},
{
'source': u'dirección',
@@ -2570,10 +2625,12 @@
{
'source': u'lat',
'dest': u'lat',
+ 'check': u'checkLat',
},
{
'source': u'long',
'dest': u'lon',
+ 'check': u'checkLon',
},
{
'source': u'dirección',
@@ -2671,10 +2728,12 @@
{
'source': u'NS',
'dest': u'lat',
+ 'check': u'checkLat',
},
{
'source': u'EW',
'dest': u'lon',
+ 'check': u'checkLon',
},
],
},
@@ -2734,10 +2793,12 @@
{
'source': u'NS',
'dest': u'lat',
+ 'check': u'checkLat',
},
{
'source': u'EW',
'dest': u'lon',
+ 'check': u'checkLon',
},
{
'source': u'Nummer',
@@ -2817,10 +2878,12 @@
{
'source': u'NS',
'dest': u'lat',
+ 'check': u'checkLat',
},
{
'source': u'EW',
'dest': u'lon',
+ 'check': u'checkLon',
},
],
},
@@ -2875,10 +2938,12 @@
{
'source': u'NS',
'dest': u'lat',
+ 'check': u'checkLat',
},
{
'source': u'EW',
'dest': u'lon',
+ 'check': u'checkLon',
},
{
'source': u'bezeichnung',
@@ -2934,10 +2999,12 @@
{
'source': u'NS',
'dest': u'lat',
+ 'check': u'checkLat',
},
{
'source': u'EW',
'dest': u'lon',
+ 'check': u'checkLon',
},
{
'source': u'Region',
@@ -3034,10 +3101,12 @@
{
'source': u'lat',
'dest': u'lat',
+ 'check': u'checkLat',
},
{
'source': u'lon',
'dest': u'lon',
+ 'check': u'checkLon',
},
{
'source': u'kommunenr',
@@ -3141,10 +3210,12 @@
{
'source': u'lat',
'dest': u'lat',
+ 'check': u'checkLat',
},
{
'source': u'lon',
'dest': u'lon',
+ 'check': u'checkLon',
},
{
'source': u'billede',
@@ -3219,10 +3290,12 @@
{
'source': u'دائرة عرض',
'dest': u'lat',
+ 'check': u'checkLat',
},
{
'source': u'خط طول',
'dest': u'lon',
+ 'check': u'checkLon',
},
{
'source': u'صورة',
@@ -3371,10 +3444,12 @@
{
'source': u'lat',
'dest': u'lat',
+ 'check': u'checkLat',
},
{
'source': u'lon',
'dest': u'lon',
+ 'check': u'checkLon',
},
{
'source': u'nomcoor',
@@ -3461,10 +3536,12 @@
{
'source': u'lat',
'dest': u'lat',
+ 'check': u'checkLat',
},
{
'source': u'lon',
'dest': u'lon',
+ 'check': u'checkLon',
},
{
'source': u'id_aut',
@@ -3562,10 +3639,12 @@
{
'source': u'lat',
'dest': u'lat',
+ 'check': u'checkLat',
},
{
'source': u'lon',
'dest': u'lon',
+ 'check': u'checkLon',
},
{
'source': u'nomcoor',
@@ -3651,10 +3730,12 @@
{
'source': u'lat',
'dest': u'lat',
+ 'check': u'checkLat',
},
{
'source': u'lon',
'dest': u'lon',
+ 'check': u'checkLon',
},
{
'source': u'notas',
@@ -3740,10 +3821,12 @@
{
'source': u'lat',
'dest': u'lat',
+ 'check': u'checkLat',
},
{
'source': u'lon',
'dest': u'lon',
+ 'check': u'checkLon',
},
{
'source': u'nomcoor',
@@ -3826,10 +3909,12 @@
{
'source': u'lat',
'dest': u'lat',
+ 'check': u'checkLat',
},
{
'source': u'lon',
'dest': u'lon',
+ 'check': u'checkLon',
},
{
'source': u'nomcoor',
@@ -3908,10 +3993,12 @@
{
'source': u'latitude',
'dest': u'lat',
+ 'check': u'checkLat',
},
{
'source': u'longitude',
'dest': u'lon',
+ 'check': u'checkLon',
},
{
'source': u'titre coordonnées',
@@ -4022,10 +4109,12 @@
{
'source': u'lat',
'dest': u'lat',
+ 'check': u'checkLat',
},
{
'source': u'long',
'dest': u'lon',
+ 'check': u'checkLon',
},
{
'source': u'image',
@@ -4112,10 +4201,12 @@
{
'source': u'lat',
'dest': u'lat',
+ 'check': u'checkLat',
},
{
'source': u'lon',
'dest': u'lon',
+ 'check': u'checkLon',
},
{
'source': u'show_notes',
@@ -4191,10 +4282,12 @@
{
'source': u'lat',
'dest': u'lat',
+ 'check': u'checkLat',
},
{
'source': u'lon',
'dest': u'lon',
+ 'check': u'checkLon',
},
{
'source': u'type',
@@ -4293,10 +4386,12 @@
{
'source': u'lat',
'dest': u'lat',
+ 'check': u'checkLat',
},
{
'source': u'lon',
'dest': u'lon',
+ 'check': u'checkLon',
},
{
'source': u'lb_type',
@@ -4368,10 +4463,12 @@
{
'source': u'lat',
'dest': u'lat',
+ 'check': u'checkLat',
},
{
'source': u'lon',
'dest': u'lon',
+ 'check': u'checkLon',
},
{
'source': u'function',
@@ -4466,10 +4563,12 @@
{
'source': u'lat',
'dest': u'lat',
+ 'check': u'checkLat',
},
{
'source': u'lon',
'dest': u'lon',
+ 'check': u'checkLon',
},
{
'source': u'image',
@@ -4541,10 +4640,12 @@
{
'source': u'lat',
'dest': u'lat',
+ 'check': u'checkLat',
},
{
'source': u'lon',
'dest': u'lon',
+ 'check': u'checkLon',
},
{
'source': u'name',
@@ -4603,10 +4704,12 @@
{
'source': u'lat',
'dest': u'lat',
+ 'check': u'checkLat',
},
{
'source': u'lon',
'dest': u'lon',
+ 'check': u'checkLon',
},
{
'source': u'name',
@@ -4676,10 +4779,12 @@
{
'source': u'szélesség',
'dest': u'lat',
+ 'check': u'checkLat',
},
{
'source': u'hosszúság',
'dest': u'lon',
+ 'check': u'checkLon',
},
{
'source': u'státusz',
@@ -4746,10 +4851,12 @@
{
'source': u'lat',
'dest': u'lat',
+ 'check': u'checkLat',
},
{
'source': u'lon',
'dest': u'lon',
+ 'check': u'checkLon',
},
{
'source': u'image',
@@ -4809,10 +4916,12 @@
{
'source': u'lat',
'dest': u'lat',
+ 'check': u'checkLat',
},
{
'source': u'lon',
'dest': u'lon',
+ 'check': u'checkLon',
},
{
'source': u'image',
@@ -4913,10 +5022,12 @@
{
'source': u'LAT',
'dest': u'lat',
+ 'check': u'checkLat',
},
{
'source': u'LONG',
'dest': u'lon',
+ 'check': u'checkLon',
},
{
'source': u'תמונה',
@@ -5008,10 +5119,12 @@
{
'source': u'LAT',
'dest': u'lat',
+ 'check': u'checkLat',
},
{
'source': u'LONG',
'dest': u'lon',
+ 'check': u'checkLon',
},
{
'source': u'תמונה',
@@ -5062,10 +5175,12 @@
{
'source': u'lat',
'dest': u'lat',
+ 'check': u'checkLat',
},
{
'source': u'long',
'dest': u'lon',
+ 'check': u'checkLon',
},
{
'source': u'regione',
@@ -5125,10 +5240,12 @@
{
'source': u'lat',
'dest': u'lat',
+ 'check': u'checkLat',
},
{
'source': u'lon',
'dest': u'lon',
+ 'check': u'checkLon',
},
{
'source': u'nomcoor',
@@ -5242,10 +5359,12 @@
{
'source': u'Breitengrad',
'dest': u'lat',
+ 'check': u'checkLat',
},
{
'source': u'Längengrad',
'dest': u'lon',
+ 'check': u'checkLon',
},
{
'source': u'ObjektID',
@@ -5296,10 +5415,12 @@
{
'source': u'عرض',
'dest': u'lat',
+ 'check': u'checkLat',
},
{
'source': u'طول',
'dest': u'lon',
+ 'check': u'checkLon',
},
{
'source': u'صورة',
@@ -5386,10 +5507,12 @@
{
'source': u'lat',
'dest': u'lat',
+ 'check': u'checkLat',
},
{
'source': u'lon',
'dest': u'lon',
+ 'check': u'checkLon',
},
{
'source': u'type',
@@ -5467,10 +5590,12 @@
{
'source': u'lat',
'dest': u'lat',
+ 'check': u'checkLat',
},
{
'source': u'lon',
'dest': u'lon',
+ 'check': u'checkLon',
},
{
'source': u'image',
@@ -5537,10 +5662,12 @@
{
'source': u'lat',
'dest': u'lat',
+ 'check': u'checkLat',
},
{
'source': u'lon',
'dest': u'lon',
+ 'check': u'checkLon',
},
{
'source': u'bild',
@@ -5642,10 +5769,12 @@
{
'source': u'lat',
'dest': u'lat',
+ 'check': u'checkLat',
},
{
'source': u'lon',
'dest': u'lon',
+ 'check': u'checkLon',
},
{
'source': u'image',
@@ -5716,10 +5845,12 @@
{
'source': u'lat',
'dest': u'lat',
+ 'check': u'checkLat',
},
{
'source': u'lon',
'dest': u'lon',
+ 'check': u'checkLon',
},
{
'source': u'image',
@@ -5781,10 +5912,12 @@
{
'source': u'lat',
'dest': u'lat',
+ 'check': u'checkLat',
},
{
'source': u'lon',
'dest': u'lon',
+ 'check': u'checkLon',
},
{
'source': u'image',
@@ -5869,10 +6002,12 @@
{
'source': u'lat',
'dest': u'lat',
+ 'check': u'checkLat',
},
{
'source': u'lon',
'dest': u'lon',
+ 'check': u'checkLon',
},
{
'source': u'image',
@@ -5949,10 +6084,12 @@
{
'source': u'lat',
'dest': u'lat',
+ 'check': u'checkLat',
},
{
'source': u'lon',
'dest': u'lon',
+ 'check': u'checkLon',
},
{
'source': u'image',
@@ -6051,10 +6188,12 @@
{
'source': u'Breitengrad',
'dest': u'lat',
+ 'check': u'checkLat',
},
{
'source': u'Längengrad',
'dest': u'lon',
+ 'check': u'checkLon',
},
{
'source': u'Region-ISO',
@@ -6110,10 +6249,12 @@
{
'source': u'lat',
'dest': u'lat',
+ 'check': u'checkLat',
},
{
'source': u'long',
'dest': u'lon',
+ 'check': u'checkLon',
},
{
'source': u'dirección',
@@ -6197,10 +6338,12 @@
{
'source': u'lat',
'dest': u'lat',
+ 'check': u'checkLat',
},
{
'source': u'lon',
'dest': u'lon',
+ 'check': u'checkLon',
},
{
'source': u'county_iso',
@@ -6308,10 +6451,12 @@
{
'source': u'lat',
'dest': u'lat',
+ 'check': u'checkLat',
},
{
'source': u'lon',
'dest': u'lon',
+ 'check': u'checkLon',
},
{
'source': u'description',
@@ -6377,10 +6522,12 @@
{
'source': u'lat',
'dest': u'lat',
+ 'check': u'checkLat',
},
{
'source': u'lon',
'dest': u'lon',
+ 'check': u'checkLon',
},
{
'source': u'provincia',
@@ -6451,10 +6598,12 @@
{
'source': u'lat',
'dest': u'lat',
+ 'check': u'checkLat',
},
{
'source': u'lon',
'dest': u'lon',
+ 'check': u'checkLon',
},
{
'source': u'image',
@@ -6515,10 +6664,12 @@
{
'source': u'lat',
'dest': u'lat',
+ 'check': u'checkLat',
},
{
'source': u'lon',
'dest': u'lon',
+ 'check': u'checkLon',
},
{
'source': u'color',
@@ -6580,10 +6731,12 @@
{
'source': u'szerokość',
'dest': u'lat',
+ 'check': u'checkLat',
},
{
'source': u'długość',
'dest': u'lon',
+ 'check': u'checkLon',
},
{
'source': u'koordynaty', # To get rid of errors
@@ -6662,10 +6815,12 @@
{
'source': u'szerokość',
'dest': u'lat',
+ 'check': u'checkLat',
},
{
'source': u'długość',
'dest': u'lon',
+ 'check': u'checkLon',
},
{
'source': u'koordynaty', # To get rid of errors
@@ -6748,10 +6903,12 @@
{
'source': u'lat',
'dest': u'lat',
+ 'check': u'checkLat',
},
{
'source': u'lon',
'dest': u'lon',
+ 'check': u'checkLon',
},
{
'source': u'imagem',
@@ -6823,6 +6980,7 @@
{
'source': u'Lat',
'dest': u'lat',
+ 'check': u'checkLat',
},
{
'source': u'Coordonate',
@@ -6831,6 +6989,7 @@
{
'source': u'Lon',
'dest': u'lon',
+ 'check': u'checkLon',
},
{
'source': u'Imagine',
@@ -6901,10 +7060,12 @@
{
'source': u'гшир',
'dest': u'lat',
+ 'check': u'checkLat',
},
{
'source': u'гдуж',
'dest': u'lon',
+ 'check': u'checkLon',
},
{
'source': u'Слика',
@@ -6970,10 +7131,12 @@
{
'source': u'lat',
'dest': u'lat',
+ 'check': u'checkLat',
},
{
'source': u'long',
'dest': u'lon',
+ 'check': u'checkLon',
},
{
'source': u'image',
@@ -7048,10 +7211,12 @@
{
'source': u'lat',
'dest': u'lat',
+ 'check': u'checkLat',
},
{
'source': u'long',
'dest': u'lon',
+ 'check': u'checkLon',
},
{
'source': u'bild',
@@ -7139,10 +7304,12 @@
{
'source': u'lat',
'dest': u'lat',
+ 'check': u'checkLat',
},
{
'source': u'long',
'dest': u'lon',
+ 'check': u'checkLon',
},
{
'source': u'bild',
@@ -7217,10 +7384,12 @@
{
'source': u'lat',
'dest': u'lat',
+ 'check': u'checkLat',
},
{
'source': u'long',
'dest': u'lon',
+ 'check': u'checkLon',
},
{
'source': u'bild',
@@ -7292,10 +7461,12 @@
{
'source': u'lat',
'dest': u'lat',
+ 'check': u'checkLat',
},
{
'source': u'lon',
'dest': u'lon',
+ 'check': u'checkLon',
},
{
'source': u'bild',
@@ -7407,10 +7578,12 @@
{
'source': u'Breitengrad',
'dest': u'lat',
+ 'check': u'checkLat',
},
{
'source': u'Längengrad',
'dest': u'lon',
+ 'check': u'checkLon',
},
],
},
@@ -7522,10 +7695,12 @@
{
'source': u'šírka',
'dest': u'lat',
+ 'check': u'checkLat',
},
{
'source': u'dĺžka',
'dest': u'lon',
+ 'check': u'checkLon',
},
{
'source': u'Beschreibung-de',
@@ -7590,10 +7765,12 @@
{
'source': u'lat',
'dest': u'lat',
+ 'check': u'checkLat',
},
{
'source': u'long',
'dest': u'lon',
+ 'check': u'checkLon',
},
{
'source': u'dirección',
@@ -7672,10 +7849,12 @@
{
'source': u'lat',
'dest': u'lat',
+ 'check': u'checkLat',
},
{
'source': u'lon',
'dest': u'lon',
+ 'check': u'checkLon',
},
{
'source': u'ภาพ',
@@ -7766,10 +7945,12 @@
{
'source': u'latitude',
'dest': u'lat',
+ 'check': u'checkLat',
},
{
'source': u'longitude',
'dest': u'lon',
+ 'check': u'checkLon',
},
{
'source': u'image',
@@ -7958,10 +8139,12 @@
{
'source': u'lat',
'dest': u'lat',
+ 'check': u'checkLat',
},
{
'source': u'lon',
'dest': u'lon',
+ 'check': u'checkLon',
},
{
'source': u'nolatlon',
@@ -8044,10 +8227,12 @@
{
'source': u'lat',
'dest': u'lat',
+ 'check': u'checkLat',
},
{
'source': u'lon',
'dest': u'lon',
+ 'check': u'checkLon',
},
{
'source': u'nolatlon',
@@ -8111,10 +8296,12 @@
{
'source': u'lat',
'dest': u'lat',
+ 'check': u'checkLat',
},
{
'source': u'long',
'dest': u'lon',
+ 'check': u'checkLon',
},
{
'source': u'dirección',
@@ -8198,10 +8385,12 @@
{
'source': u'lat',
'dest': u'lat',
+ 'check': u'checkLat',
},
{
'source': u'long',
'dest': u'lon',
+ 'check': u'checkLon',
},
{
'source': u'dirección',
@@ -8298,10 +8487,12 @@
{
'source': u'Latitude',
'dest': u'lat',
+ 'check': u'checkLat',
},
{
'source': u'Longitude',
'dest': u'lon',
+ 'check': u'checkLon',
},
],
},
@@ -8369,10 +8560,12 @@
{
'source': u'Längengrad',
'dest': u'lon',
+ 'check': u'checkLon',
},
{
'source': u'Breitengrad',
'dest': u'lat',
+ 'check': u'checkLat',
},
{
'source': u'Beschreibung',
@@ -8455,10 +8648,12 @@
{
'source': u'lat',
'dest': u'lat',
+ 'check': u'checkLat',
},
{
'source': u'lon',
'dest': u'lon',
+ 'check': u'checkLon',
},
{
'source': u'idurl',
@@ -8559,10 +8754,12 @@
{
'source': u'lat',
'dest': u'lat',
+ 'check': u'checkLat',
},
{
'source': u'lon',
'dest': u'lon',
+ 'check': u'checkLon',
},
{
'source': u'kuva',
@@ -8645,10 +8842,12 @@
{
'source': u'lat',
'dest': u'lat',
+ 'check': u'checkLat',
},
{
'source': u'long',
'dest': u'lon',
+ 'check': u'checkLon',
},
{
'source': u'image',
diff --git a/erfgoedbot/update_database.py b/erfgoedbot/update_database.py
index 65d6846..62fe526 100755
--- a/erfgoedbot/update_database.py
+++ b/erfgoedbot/update_database.py
@@ -123,6 +123,17 @@
return True
+def run_check(check, fieldValue, monumentKey, countryconfig, sourcePage):
+ """Run a named check."""
+ if check == 'checkLat':
+ return checkLat(fieldValue, monumentKey, countryconfig, sourcePage)
+ elif check == 'checkLon':
+ return checkLon(fieldValue, monumentKey, countryconfig, sourcePage)
+ else:
+ raise pywikibot.Error('Un-defined check in config for %s: %s'
+ % (countryconfig.get('table')), check)
+
+
def convertField(field, contents, countryconfig):
'''
Convert a field
@@ -185,7 +196,9 @@
return u'noFoP'
except ValueError:
return u'noFoP'
- return u''
+ else:
+ raise pywikibot.Error('Un-defined converter in config for %s: %s'
+ % (countryconfig.get('table')),
field.get('conv'))
def unknownFieldsStatistics(countryconfig, unknownFields):
@@ -228,7 +241,7 @@
fieldnames.append(field.get('dest'))
# Do some conversions here
- fieldValue = u''
+ fieldValue = u'' # Should this be None?
if field.get('conv'):
fieldValue = convertField(field, contents, countryconfig)
else:
@@ -237,8 +250,9 @@
if field.get('check'):
# check data
# run function with name field.get('check')
- globals()[field.get('check')](
- fieldValue, monumentKey, countryconfig, sourcePage)
+ if not run_check(field.get('check'), fieldValue, monumentKey,
+ countryconfig, sourcePage):
+ fieldValue = u'' # throw away input if check fails
fieldvalues.append(fieldValue)
if countryconfig.get('countryBbox'):
--
To view, visit https://gerrit.wikimedia.org/r/281930
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I28895b6cc94e87af68f88b455ef2a431d4b5f00a
Gerrit-PatchSet: 1
Gerrit-Project: labs/tools/heritage
Gerrit-Branch: master
Gerrit-Owner: Lokal Profil <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
