Wikimedia commons support multiple values, Compile regex

api/v1/node.py

@@ -15,7 +15,7 @@
 
 router = APIRouter()
 
-photo_id_re = re.compile(r'view/(?P<id>\S+)\.')
+_photo_id_re = re.compile(r'view/(?P<id>\S+)\.')
 
 
 def _get_timezone(x: float, y: float) -> tuple[str | None, str | None]:
@@ -38,7 +38,7 @@ async def _get_image_data(tags: dict[str, str]) -> dict:
 
     if (
         image_url
-        and (photo_id_match := photo_id_re.search(image_url))
+        and (photo_id_match := _photo_id_re.search(image_url))
         and (photo_id := photo_id_match.group('id'))
         and (await PhotoService.get_by_id(photo_id)) is not None
     ):
@@ -55,7 +55,7 @@ async def _get_image_data(tags: dict[str, str]) -> dict:
             '@photo_source': image_url,
         }
 
-    wikimedia_commons: str = tags.get('wikimedia_commons', '')
+    wikimedia_commons: str = tags.get('wikimedia_commons', '').partition(';')[0]
 
     if wikimedia_commons:
         return {

config.py

@@ -7,7 +7,7 @@
 from pyproj import Transformer
 
 NAME = 'openaedmap-backend'
-VERSION = '2.11.0'
+VERSION = '2.11.1'
 CREATED_BY = f'{NAME} {VERSION}'
 WEBSITE = 'https://openaedmap.org'
 

planet_diffs.py

@@ -12,6 +12,9 @@
 from utils import http_get, retry_exponential
 from xmltodict_postprocessor import xmltodict_postprocessor
 
+_action_open_re = re.compile(r'<(create|modify|delete)>')
+_action_close_re = re.compile(r'</(create|modify|delete)>')
+
 
 @trace
 async def get_planet_diffs(last_update: float) -> tuple[Sequence[dict], float]:
@@ -98,6 +101,6 @@ def _format_actions(xml: str) -> str:
     # <create> -> <action type="create">
     # </create> -> </action>
     # etc.
-    xml = re.sub(r'<(create|modify|delete)>', r'<action type="\1">', xml)
-    xml = re.sub(r'</(create|modify|delete)>', r'</action>', xml)
+    xml = _action_open_re.sub(r'<action type="\1">', xml)
+    xml = _action_close_re.sub(r'</action>', xml)
     return xml