]> ToastFreeware Gitweb - philipp/winterrodeln/wrpylib.git/blobdiff - wrpylib/wrmwmarkup.py
Remove unused imports.
[philipp/winterrodeln/wrpylib.git] / wrpylib / wrmwmarkup.py
index e47d9e9a8c0f51022f99fe2b7d9bcb69ea8ae09f..2ab28d227efa0ef3ba64ad6141b54a153c48e706 100644 (file)
@@ -1,12 +1,12 @@
-#!/usr/bin/python3.4
-# $Id$
-# $HeadURL$
 """This module contains winterrodeln specific functions that are processing the MediaWiki markup.
 """
 import re
 import xml.etree.ElementTree
 import collections
-import mwparserfromhell
+from typing import Tuple, Optional, List, OrderedDict, Union
+
+from mwparserfromhell.nodes import Template
+
 import wrpylib.wrvalidators
 import wrpylib.mwmarkup
 import wrpylib.wrmwdb
@@ -15,17 +15,29 @@ from wrpylib.wrvalidators import LonLat, opt_lonlat_from_str, opt_lonlat_to_str,
     opt_phone_comment_enum_to_str, lift_german_from_str, GASTHAUSBOX_DICT
 
 
-def sledrun_from_rodelbahnbox(value, sledrun):
+def split_lon_lat(value: Optional[LonLat]) -> Union[LonLat, Tuple[None, None]]:
+    if value is None:
+        return None, None
+    return value
+
+
+def join_lon_lat(lon: Optional[float], lat: Optional[float]) -> Optional[LonLat]:
+    if lon is None or lat is None:
+        return None
+    return LonLat(lon, lat)
+
+
+def sledrun_from_rodelbahnbox(value: OrderedDict, sledrun: object):
     """Takes a Rodelbahnbox as returned by rodelbahnbox_from_str (that is, an OrderedDict) and
     updates the sledrun instance with all values present in the Rodelbahnbox. Other values are not
     updated. Does not validate the arguments."""
     # sledrun.page_id = None # this field is not updated because it is not present in the RodelbahnBox
     # sledrun.page_title = None # this field is not updated because it is not present in the RodelbahnBox
     # sledrun.name_url = None # this field is not updated because it is not present in the RodelbahnBox
-    sledrun.position_longitude, sledrun.position_latitude = value['Position']
-    sledrun.top_longitude, sledrun.top_latitude = value['Position oben']
+    sledrun.position_longitude, sledrun.position_latitude = split_lon_lat(value['Position'])
+    sledrun.top_longitude, sledrun.top_latitude = split_lon_lat(value['Position oben'])
     sledrun.top_elevation = value['Höhe oben']
-    sledrun.bottom_longitude, sledrun.bottom_latitude = value['Position unten']
+    sledrun.bottom_longitude, sledrun.bottom_latitude = split_lon_lat(value['Position unten'])
     sledrun.bottom_elevation = value['Höhe unten']
     sledrun.length = value['Länge']
     sledrun.difficulty = value['Schwierigkeit']
@@ -51,14 +63,14 @@ def sledrun_from_rodelbahnbox(value, sledrun):
     return sledrun
 
 
-def sledrun_to_rodelbahnbox(sledrun):
+def sledrun_to_rodelbahnbox(sledrun) -> collections.OrderedDict:
     """Takes a sledrun instance that might come from the database and converts it to a OrderedDict ready
     to be formatted as RodelbahnBox."""
     value = collections.OrderedDict()
-    value['Position'] = LonLat(sledrun.position_longitude, sledrun.position_latitude)
-    value['Position oben'] = LonLat(sledrun.top_longitude, sledrun.top_latitude)
+    value['Position'] = join_lon_lat(sledrun.position_longitude, sledrun.position_latitude)
+    value['Position oben'] = join_lon_lat(sledrun.top_longitude, sledrun.top_latitude)
     value['Höhe oben'] = sledrun.top_elevation
-    value['Position unten'] = LonLat(sledrun.bottom_longitude, sledrun.bottom_latitude)
+    value['Position unten'] = join_lon_lat(sledrun.bottom_longitude, sledrun.bottom_latitude)
     value['Höhe unten'] = sledrun.bottom_elevation
     value['Länge'] = sledrun.length
     value['Schwierigkeit'] = sledrun.difficulty
@@ -86,18 +98,18 @@ def inn_from_gasthausbox(value, inn):
     value is a dict of properties as returned by gasthausbox_from_str."""
     # page_id = None # this field is not updated because it is not present in the Gasthausbox
     # page_title = None # this field is not updated because it is not present in the Gasthausbox
-    def convtodb(value, key):
-        v = value[key]
-        if v is not None:
-            v = GASTHAUSBOX_DICT[key].to_str(v)
+    def convtodb(val, key):
+        v = GASTHAUSBOX_DICT[key].to_str(val[key])
+        if v == '':
+            return None
         return v
-    inn.position_longitude, inn.position_latitude = value['Position']
+    inn.position_longitude, inn.position_latitude = split_lon_lat(value['Position'])
     inn.position_elevation = value['Höhe']
     inn.operator = value['Betreiber']
     inn.seats = value['Sitzplätze']
     inn.overnight, inn.overnight_comment = value['Übernachtung']
-    inn.smoker_area = value['Rauchfrei'] < 0.9
-    inn.nonsmoker_area = value['Rauchfrei'] > 0.1
+    inn.smoker_area = None if value['Rauchfrei'] is None else value['Rauchfrei'] < 0.9
+    inn.nonsmoker_area = None if value['Rauchfrei'] is None else value['Rauchfrei'] > 0.1
     inn.sled_rental, inn.sled_rental_comment = value['Rodelverleih']
     inn.mobile_provider = convtodb(value, 'Handyempfang')
     inn.homepage = convtodb(value, 'Homepage')
@@ -109,35 +121,39 @@ def inn_from_gasthausbox(value, inn):
     return inn
 
 
-def inn_to_gasthausbox(inn):
-    """Converts an inn class to a dict of Gasthausbox properties. value is an Inn instance."""
+def inn_to_gasthausbox(inn) -> collections.OrderedDict:
+    """Converts an inn class to a dict of Gasthausbox properties. inn is an Inn instance."""
+    def convfromdb(val, key):
+        v = '' if val is None else val
+        return GASTHAUSBOX_DICT[key].from_str(v)
     value = collections.OrderedDict()
-    value['Position'] = LonLat(inn.position_longitude, inn.position_latitude)
+    value['Position'] = join_lon_lat(inn.position_longitude, inn.position_latitude)
     value['Höhe'] = inn.position_elevation
     value['Betreiber'] = inn.operator
     value['Sitzplätze'] = inn.seats
     value['Übernachtung'] = (inn.overnight, inn.overnight_comment)
-    value['Rauchfrei'] = (inn.nonsmoker_area, inn.smoker_area)
+    value['Rauchfrei'] = {(False, True): 0.0, (True, True): 0.5, (True, False): 1.0} \
+        .get((inn.nonsmoker_area, inn.smoker_area), None)
     value['Rodelverleih'] = (inn.sled_rental, inn.sled_rental_comment)
-    value['Handyempfang'] = inn.mobile_provider
-    value['Homepage'] = inn.homepage
-    value['E-Mail'] = inn.email_list
-    value['Telefon'] = inn.phone_list
+    value['Handyempfang'] = convfromdb(inn.mobile_provider, 'Handyempfang')
+    value['Homepage'] = convfromdb(inn.homepage, 'Homepage')
+    value['E-Mail'] = convfromdb(inn.email_list, 'E-Mail')
+    value['Telefon'] = convfromdb(inn.phone_list, 'Telefon')
     value['Bild'] = inn.image
-    value['Rodelbahnen'] = inn.sledding_list
+    value['Rodelbahnen'] = convfromdb(inn.sledding_list, 'Rodelbahnen')
     return value
 
 
-def lonlat_ele_from_template(template):
-    """Template is a mwparserfromhell.nodes.template.Template instance. Returns (lonlat, ele)."""
+def lonlat_ele_from_template(template) -> Tuple[LonLat, Optional[int]]:
+    """Template is a `mwparserfromhell.nodes.template.Template` instance. Returns (lonlat, ele)."""
     lonlat = opt_lonlat_from_str(template.params[0].strip())
     ele = opt_uint_from_str(template.params[1].strip())
     return lonlat, ele
 
 
-def latlon_ele_to_template(lonlat_ele, name):
+def latlon_ele_to_template(lonlat_ele, name) -> Template:
     lonlat, ele = lonlat_ele
-    template = mwparserfromhell.nodes.template.Template(name)
+    template = Template(name)
     template.add(1, opt_lonlat_to_str(lonlat))
     template.add(2, opt_uint_to_str(ele))
     wrpylib.mwmarkup.format_template_oneline(template)
@@ -149,97 +165,7 @@ class ParseError(RuntimeError):
     pass
 
 
-
-def find_template_PositionOben(wikitext):
-    """Same as find_template_latlon_ele with template '{{Position oben|47.076207 N 11.453553 E|1890}}'"""
-    return find_template_latlon_ele(wikitext, 'Position oben')
-
-
-def create_template_PositionOben(lat, lon, ele):
-    return create_template_latlon_ele('Position, oben', lat, lon, ele)
-
-
-def find_template_PositionUnten(wikitext):
-    """Same as find_template_latlon_ele with template '{{Position unten|47.076207 N 11.453553 E|1890}}'"""
-    return find_template_latlon_ele(wikitext, 'Position unten')
-
-
-def find_template_unsigned(wikitext, template_title):
-    """Finds the first occurance of the '{{template_title|1890}}' template
-    and returns the tuple (start, end, unsigned_value) or (None, None, None) if the
-    template was not found. If the template has no valid format, an exception is thrown."""
-    start, end = wrpylib.mwmarkup.find_template(wikitext, template_title)
-    if start is None: return (None,) * 3
-    title, params = wrpylib.mwmarkup.split_template(wikitext[start:end])
-    unsigned_value = wrpylib.wrvalidators.UnsignedNone().to_python(params['1'].strip())
-    return start, end, unsigned_value
-
-
-def create_template_unsigned(template_title, unsigned):
-    unsigned = wrpylib.wrvalidators.UnsignedNone().from_python(unsigned)
-    if len(unsigned) == 0: unsigned = ' '
-    return wrpylib.mwmarkup.create_template(template_title, [unsigned])
-
-
-def find_template_Hoehenunterschied(wikitext):
-    """Same as find_template_unsigned with template '{{Höhenunterschied|350}}'"""
-    return find_template_unsigned(wikitext, 'Höhenunterschied')
-
-
-def create_template_Hoehenunterschied(ele_diff):
-    return create_template_unsigned('Höhenunterschied', ele_diff)
-
-
-def find_template_Bahnlaenge(wikitext):
-    """Same as find_template_unsigned with template '{{Bahnlänge|4500}}'"""
-    return find_template_unsigned(wikitext, 'Bahnlänge')
-
-
-def create_template_Bahnlaenge(length):
-    return create_template_unsigned('Bahnlänge', length)
-
-
-def find_template_Gehzeit(wikitext):
-    """Same as find_template_unsigned with template '{{Gehzeit|60}}'"""
-    return find_template_unsigned(wikitext, 'Gehzeit')
-
-
-def create_template_Gehzeit(walkup_time):
-    return create_template_unsigned('Gehzeit', walkup_time)
-
-
-def find_template_Forumlink(wikitext):
-    """Same as find_template_unsigned with template '{{Forumlink|26}}'"""
-    start, end = wrpylib.mwmarkup.find_template(wikitext, 'Forumlink')
-    if start is None: return (None,) * 3
-    title, params = wrpylib.mwmarkup.split_template(wikitext[start:end])
-    forumid = params['1'].strip()
-    if forumid == '<nummer einfügen>': unsigned_value = None
-    else: unsigned_value = wrpylib.wrvalidators.UnsignedNone().to_python(forumid)
-    return start, end, unsigned_value
-    # return find_template_unsigned(wikitext, u'Forumlink')
-
-
-def find_template_Parkplatz(wikitext):
-    """Same as find_template_latlon_ele with template '{{Parkplatz|47.076207 N 11.453553 E|1890}}'"""
-    return find_template_latlon_ele(wikitext, 'Parkplatz')
-
-
-def find_template_Haltestelle(wikitext):
-    """Finds the first occurance of the '{{Haltestelle|Ortsname|Haltestellenname|47.076207 N 11.453553 E|1890}}' template
-    and returns the tuple (start, end, city, stop, lat, lon, ele) or (None, None, None, None, None, None, None) if the
-    template was not found. If the template has no valid format, an exception is thrown."""
-    start, end = wrpylib.mwmarkup.find_template(wikitext, 'Haltestelle')
-    if start is None: return (None,) * 7
-    title, params = wrpylib.mwmarkup.split_template(wikitext[start:end])
-    city = wrpylib.wrvalidators.UnicodeNone().to_python(params['1'].strip())
-    stop = wrpylib.wrvalidators.UnicodeNone().to_python(params['2'].strip())
-    lat, lon = wrpylib.wrvalidators.GeoNone().to_python(params['3'].strip())
-    ele = wrpylib.wrvalidators.UnsignedNone().to_python(params['4'].strip())
-    return start, end, city, stop, lat, lon, ele
-
-
-def parse_wrmap_coordinates(coords):
+def parse_wrmap_coordinates(coords: str) -> List[List[float]]:
     """gets a string coordinates and returns an array of lon/lat coordinate pairs, e.g.
     47.12 N 11.87 E
     47.13 N 11.70 E
@@ -255,7 +181,7 @@ def parse_wrmap_coordinates(coords):
     else:
         if pos == len(coords):
             return result
-    raise RuntimeError('Wrong coordinate format: {}'.format(coords))
+    raise RuntimeError(f'Wrong coordinate format: {coords}')
 
 
 WRMAP_POINT_TYPES = ['gasthaus', 'haltestelle', 'parkplatz', 'achtung', 'foto', 'verleih', 'punkt']
@@ -263,7 +189,7 @@ WRMAP_LINE_TYPES = ['rodelbahn', 'gehweg', 'alternative', 'lift', 'anfahrt', 'li
 
 
 def parse_wrmap(wikitext):
-    """Parses the (unicode) u'<wrmap ...>content</wrmap>' of the Winterrodeln wrmap extension.
+    """Parses the '<wrmap ...>content</wrmap>' of the Winterrodeln wrmap extension.
     If wikitext does not contain the <wrmap> tag or if the <wrmap> tag contains 
     invalid formatted lines, a ParseError is raised.
     Use wrpylib.mwmarkup.find_tag(wikitext, 'wrmap') to find the wrmap tag within an arbitrary
@@ -271,7 +197,7 @@ def parse_wrmap(wikitext):
 
     :param wikitext: wikitext containing only the template. Example:
 
-    wikitext = u'''
+    wikitext = '''
     <wrmap lat="47.2417134" lon="11.21408895" zoom="14" width="700" height="400">
     <gasthaus name="Rosskogelhütte" wiki="Rosskogelhütte">47.240689 11.190454</gasthaus>
     <parkplatz>47.245789 11.238971</parkplatz>
@@ -287,10 +213,10 @@ def parse_wrmap(wikitext):
     """
     # parse XML
     try:
-        wrmap_xml = xml.etree.ElementTree.fromstring(wikitext.encode('utf-8'))
+        wrmap_xml = xml.etree.ElementTree.fromstring(wikitext)
     except xml.etree.ElementTree.ParseError as e:
         row, column = e.position
-        raise ParseError("XML parse error on row {}, column {}: {}".format(row, column, e))
+        raise ParseError(f"XML parse error on row {row}, column {column}: {e}")
     if wrmap_xml.tag not in ['wrmap', 'wrgmap']:
         raise ParseError('No valid tag name')
 
@@ -300,8 +226,8 @@ def parse_wrmap(wikitext):
         # determine feature type
         is_point = feature.tag in WRMAP_POINT_TYPES
         is_line = feature.tag in WRMAP_LINE_TYPES
-        if (not is_point and not is_line):
-            raise ParseError('Unknown element <{}>.'.format(feature.tag))
+        if not is_point and not is_line:
+            raise ParseError(f'Unknown element <{feature.tag}>.')
 
         # point
         if is_point:
@@ -309,11 +235,11 @@ def parse_wrmap(wikitext):
             allowed_properties = {'name', 'wiki'}
             wrong_properties = set(feature.attrib.keys()) - allowed_properties
             if len(wrong_properties) > 0:
-                raise ParseError("The attribute '{}' is not allowed at <{}>.".format(list(wrong_properties)[0], feature.tag))
+                raise ParseError(f"The attribute '{list(wrong_properties)[0]}' is not allowed at <{feature.tag}>.")
             properties.update(feature.attrib)
             coordinates = parse_wrmap_coordinates(feature.text)
             if len(coordinates) != 1:
-                raise ParseError('The element <{}> has to have exactly one coordinate pair.'.format(feature.tag))
+                raise ParseError(f'The element <{feature.tag}> has to have exactly one coordinate pair.')
             json_features.append({
                 'type': 'Feature',
                 'geometry': {'type': 'Point', 'coordinates': coordinates[0]},
@@ -325,14 +251,14 @@ def parse_wrmap(wikitext):
             allowed_properties = {'farbe', 'dicke'}
             wrong_properties = set(feature.attrib.keys()) - allowed_properties
             if len(wrong_properties) > 0:
-                raise ParseError("The attribute '{}' is not allowed at <{}>.".format(list(wrong_properties)[0], feature.tag))
+                raise ParseError(f"The attribute '{list(wrong_properties)[0]}' is not allowed at <{feature.tag}>.")
             if 'farbe' in feature.attrib: 
                 if not re.match('#[0-9a-fA-F]{6}$', feature.attrib['farbe']):
                     raise ParseError('The attribute "farbe" has to have a format like "#a0bb43".')
-                properties['strokeColor'] = feature.attrib['farbe'] # e.g. #a200b7
+                properties['strokeColor'] = feature.attrib['farbe']  # e.g. #a200b7
             if 'dicke' in feature.attrib:
                 try:
-                    properties['strokeWidth'] = int(feature.attrib['dicke']) # e.g. 6
+                    properties['strokeWidth'] = int(feature.attrib['dicke'])  # e.g. 6
                 except ValueError:
                     raise ParseError('The attribute "dicke" has to be an integer.')
             json_features.append({
@@ -347,14 +273,14 @@ def parse_wrmap(wikitext):
             try:
                 properties[k] = float(v)
             except ValueError:
-                raise ParseError('Attribute "{}" has to be a float value.'.format(k))
+                raise ParseError(f'Attribute "{k}" has to be a float value.')
         elif k in ['zoom', 'width', 'height']:
             try:
                 properties[k] = int(v)
             except ValueError:
-                raise ParseError('Attribute "{}" has to be an integer value.'.format(k))
+                raise ParseError(f'Attribute "{k}" has to be an integer value.')
         else:
-            raise ParseError('Unknown attribute "{}".'.format(k))
+            raise ParseError(f'Unknown attribute "{k}".')
 
     geojson = {
         'type': 'FeatureCollection',
@@ -367,7 +293,7 @@ def parse_wrmap(wikitext):
 def create_wrmap_coordinates(coords):
     result = []
     for coord in coords:
-        result.append('{:.6f} N {:.6f} E'.format(coord[1], coord[0]))
+        result.append(f'{coord[1]:.6f} N {coord[0]:.6f} E')
     return '\n'.join(result)
  
 
@@ -377,7 +303,7 @@ def create_wrmap(geojson):
     wrmap_xml.text = '\n\n'
     for k, v in geojson['properties'].items():
         if k in ['lon', 'lat']:
-            wrmap_xml.attrib[k] = '{:.6f}'.format(v)
+            wrmap_xml.attrib[k] = f'{v:.6f}'
         else:
             wrmap_xml.attrib[k] = str(v)