#!/usr/bin/env python3
import requests
import json
from datetime import datetime
from bs4 import BeautifulSoup


def parse_connections(html):
    soup = BeautifulSoup(html, "lxml")
    connections = list()

    for row in soup.find_all("td", class_="overview timelink"):
        columns = row.parent.find_all("td")

        try:
            price = columns[3].contents[3].string.strip().replace(',', '.')
        except IndexError:
            price = ''

        data = {
            'details': columns[0].a.get('href'),
            'departure': columns[0].a.contents[0].string,
            'arrival': columns[0].a.contents[2].string,
            'transfers': int(columns[2].contents[0]),
            'time': columns[2].contents[2],
            'products': columns[3].contents[0].split(', '),
            'price': price
        }

        if data['price'] == "":
            data['price'] = None
        else:
            data['price'] = float(data['price'])

        if columns[1].find('img'):
            data['canceled'] = True
        elif columns[1].find('span', class_="okmsg"):
            data['ontime'] = True
        elif columns[1].find('span', class_="red"):
            if hasattr(columns[1].contents[0], 'text'):
                delay_departure = columns[1].contents[0].text.replace("ca. +", "")
            else:
                delay_departure = 0
            if hasattr(columns[1].contents[2], 'text'):
                delay_arrival = columns[1].contents[2].text.replace("ca. +", "")
            else:
                delay_arrival = 0
            data['delay'] = {
                'delay_departure': int(delay_departure),
                'delay_arrival': int(delay_arrival)
            }
        connections.append(data)
    return connections

def parse_stations(html):
    """
        Strips JS code, loads JSON
    """
    html = html.replace('SLs.sls=', '').replace(';SLs.showSuggestion();', '')
    html = json.loads(html)
    return html['suggestions']


class Schiene():

    def stations(self, station, limit=10):
        """
        Find stations for given queries

        Args:
            station (str): search query
            limit (int): limit number of results
        """
        query = {
            'start': 1,
            'S': station + '?',
            'REQ0JourneyStopsB': limit
        }
        rsp = requests.get('http://reiseauskunft.bahn.de/bin/ajax-getstop.exe/dn', params=query)
        return parse_stations(rsp.text)


    def connections(self, origin, destination, dt=datetime.now()):
        """
        Find connections between two stations

        Args:
            origin (str): origin station
            destination (str): destination station
            dt (datetime): date and time for query
        """
        query = {
            'S': origin,
            'Z': destination,
            'date': dt.strftime("%d.%m.%y"),
            'time': dt.strftime("%H:%M"),
            'start': 1
        }
        rsp = requests.get('http://mobile.bahn.de/bin/mobil/query.exe/dox?', params=query)
        return parse_connections(rsp.text)

@@ -171,12 +171,19 @@
 l, %22
-lxml
+html.parser
 %22)%0A 

switch to html.parser for better compatibility