Skip to content

Commit 2466b23

Browse files
committed
don't overwrite existing lat/long/etc. unless we have new values
Signed-off-by: John Seekins <john@robot-house.us>
1 parent d158481 commit 2466b23

1 file changed

Lines changed: 9 additions & 3 deletions

File tree

enrichers/general.py

Lines changed: 9 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -50,9 +50,15 @@ def _enrich_facility(facility_data: tuple) -> tuple:
5050
enriched_facility["wikipedia"]["search_query"] = wiki_res.get("search_query_steps", "")
5151
enriched_facility["wikidata"]["page_url"] = wd_res.get("url", "")
5252
enriched_facility["wikidata"]["search_query"] = wd_res.get("search_query_steps", "")
53-
enriched_facility["osm"]["latitude"] = osm_res.get("details", {}).get("latitude", osm.default_coords["latitude"])
54-
enriched_facility["osm"]["longitude"] = osm_res.get("details", {}).get("longitude", osm.default_coords["longitude"])
55-
enriched_facility["osm"]["url"] = osm_res.get("url", "")
53+
lat = osm_res.get("details", {}).get("latitude", None)
54+
long = osm_res.get("details", {}).get("longitude", None)
55+
if lat:
56+
enriched_facility["osm"]["latitude"] = lat
57+
if long:
58+
enriched_facility["osm"]["longitude"] = lat
59+
url = osm_res.get("url", None)
60+
if url:
61+
enriched_facility["osm"]["url"] = url
5662
enriched_facility["osm"]["search_query"] = osm_res.get("search_query_steps", "")
5763

5864
logger.debug(enriched_facility)

0 commit comments

Comments
 (0)