diff options
author | Christian Struck <christian@struck.se> | 2017-11-03 10:43:34 +0100 |
---|---|---|
committer | GitHub <noreply@github.com> | 2017-11-03 10:43:34 +0100 |
commit | 7664fb26df0c4ba0f03504b55e15022769bab861 (patch) | |
tree | 1bb38dbd2973cc74b95172f6787e9b398587396b | |
parent | b74f3795d4c65f1f98a58d6644c80d71b4f86e5d (diff) | |
parent | a02bbe591bdf72423f4246815402ab23c61247b5 (diff) | |
download | bvg-grabber-7664fb26df0c4ba0f03504b55e15022769bab861.tar.gz bvg-grabber-7664fb26df0c4ba0f03504b55e15022769bab861.tar.bz2 bvg-grabber-7664fb26df0c4ba0f03504b55e15022769bab861.zip |
Merge pull request #12 from c7h/develop
updated to BS4.4.1
-rwxr-xr-x | bvg-grabber.py | 6 | ||||
-rw-r--r-- | bvggrabber/api/actualdeparture.py | 2 | ||||
-rw-r--r-- | bvggrabber/api/scheduleddeparture.py | 2 | ||||
-rwxr-xr-x | setup.py | 2 |
4 files changed, 5 insertions, 7 deletions
diff --git a/bvg-grabber.py b/bvg-grabber.py index 3964f83..3a2d623 100755 --- a/bvg-grabber.py +++ b/bvg-grabber.py @@ -59,7 +59,5 @@ if __name__ == '__main__': if args.file in ('stdout', '-'): print(res.to_json) else: - file = open(args.file, 'w') - print(res.to_json, file=file) - file.close() - + with open(args.file, 'w') as f: + print(res.to_json, file=f)
\ No newline at end of file diff --git a/bvggrabber/api/actualdeparture.py b/bvggrabber/api/actualdeparture.py index a1b3b5a..e7c084b 100644 --- a/bvggrabber/api/actualdeparture.py +++ b/bvggrabber/api/actualdeparture.py @@ -30,7 +30,7 @@ class ActualDepartureQueryApi(QueryApi): } response = requests.get(ACTUAL_API_ENDPOINT, params=params) if response.ok: - soup = BeautifulSoup(response.text) + soup = BeautifulSoup(response.text, "html.parser") if soup.find_all('form'): # The station we are looking for is ambiguous or does not exist stations = soup.find_all('option') diff --git a/bvggrabber/api/scheduleddeparture.py b/bvggrabber/api/scheduleddeparture.py index 7a30d5f..f316b6e 100644 --- a/bvggrabber/api/scheduleddeparture.py +++ b/bvggrabber/api/scheduleddeparture.py @@ -50,7 +50,7 @@ class ScheduledDepartureQueryApi(QueryApi): 'start': 'yes'} response = requests.get(SCHEDULED_API_ENDPOINT, params=params) if response.ok: - soup = BeautifulSoup(response.text) + soup = BeautifulSoup(response.text, "html.parser") if soup.find('span', 'error'): # The station we are looking for is ambiguous or does not exist stations = soup.find('span', 'select').find_all('a') @@ -22,7 +22,7 @@ setup( packages=find_packages(exclude=['tests']), include_package_data=True, install_requires=[ - 'beautifulsoup4>=4.2.1', + 'beautifulsoup4>=4.4.1', 'python-dateutil>=2.1', 'requests>=1.2.3', 'six>=1.3.0', |