osm_poi_matchmaker.libs.xml.save_downloaded_xml() - Code Metrics - KAMI911/osm_poi_matchmaker - Measure and Improve Code Quality continuously with Scrutinizer

osm_poi_matchmaker.libs.xml.save_downloaded_xml() B
last analyzed 2023-01-03 16:24 UTC

↳ Parent: osm_poi_matchmaker.libs.xml

Complexity

Conditions

Size

Total Lines	14
Code Lines	12

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	7
eloc	12
nop	3
dl	0
loc	14
rs	8
c	0
b	0
f	0

# -*- coding: utf-8 -*-


try:
    import logging
    import sys
    import requests
    import os
    from lxml import etree

    from osm_poi_matchmaker.utils import config
except ImportError as err:
    logging.error('Error %s import module: %s', __name__, err)
    logging.exception('Exception occurred')

    sys.exit(128)


def download_xml(link, verify_link=config.get_download_verify_link()):

    try:
        page = requests.get(link, verify=verify_link)
    except requests.exceptions.ConnectionError as e:

        logging.warning('Unable to open connection.')
        return None
    return page.content if page.status_code == 200 else None


def save_downloaded_xml(link, file, verify=config.get_download_verify_link()):

    if config.get_download_use_cached_data() == True and os.path.isfile(file):

        with open(file, 'r', encoding='utf-8') as content_file:
            page = content_file.read()
    else:
        page = download_xml(link, verify)
        if page is not None:
            if not os.path.exists(config.get_directory_cache_url()):
                os.makedirs(config.get_directory_cache_url())
            with open(file, mode='w', encoding='utf-8') as code:
                code.write(page.decode('utf-8'))
        else:
            logging.warning('Skipping dataset.')
    return page


1			# -- coding: utf-8 --
			0 ignored issues – show introduced 2020-10-11 06:10 UTC by Report Bug Copy Issue Report Missing module docstring Loading history...
2
3			try:
4			import logging
5			import sys
6			import requests
7			import os
8			from lxml import etree
			0 ignored issues – show Unused Code introduced 2020-10-11 06:10 UTC by Report Bug Copy Issue Report Unused etree imported from lxml Loading history...
9			from osm_poi_matchmaker.utils import config
10			except ImportError as err:
11			logging.error('Error %s import module: %s', __name__, err)
12			logging.exception('Exception occurred')
13
14			sys.exit(128)
15
16
17			def download_xml(link, verify_link=config.get_download_verify_link()):
			0 ignored issues – show introduced 2020-10-11 06:10 UTC by Report Bug Copy Issue Report Missing function or method docstring Loading history...
18			try:
19			page = requests.get(link, verify=verify_link)
20			except requests.exceptions.ConnectionError as e:
			0 ignored issues – show Coding Style Naming introduced 2020-10-11 06:10 UTC by Report Bug Copy Issue Report Variable name "e" doesn't conform to snake_case naming style ('([^\\W\\dA-Z][^\\WA-Z]2,\|_[^\\WA-Z]*\|__[^\\WA-Z\\d_][^\\WA-Z]+__)$' pattern) This check looks for invalid names for a range of different identifiers. You can set regular expressions to which the identifiers must conform if the defaults do not match your requirements. If your project includes a Pylint configuration file, the settings contained in that file take precedence. To find out more about Pylint, please refer to their site. Loading history... Unused Code introduced 2020-10-11 06:10 UTC by Report Bug Copy Issue Report The variable `e` seems to be unused. Loading history...
21			logging.warning('Unable to open connection.')
22			return None
23			return page.content if page.status_code == 200 else None
24
25
26			def save_downloaded_xml(link, file, verify=config.get_download_verify_link()):
			0 ignored issues – show introduced 2020-10-11 06:10 UTC by Report Bug Copy Issue Report Missing function or method docstring Loading history...
27			if config.get_download_use_cached_data() == True and os.path.isfile(file):
			0 ignored issues – show introduced 2020-10-11 06:10 UTC by Report Bug Copy Issue Report Comparison to True should be just 'expr' Loading history...
28			with open(file, 'r', encoding='utf-8') as content_file:
29			page = content_file.read()
30			else:
31			page = download_xml(link, verify)
32			if page is not None:
33			if not os.path.exists(config.get_directory_cache_url()):
34			os.makedirs(config.get_directory_cache_url())
35			with open(file, mode='w', encoding='utf-8') as code:
36			code.write(page.decode('utf-8'))
37			else:
38			logging.warning('Skipping dataset.')
39			return page
40

KAMI911 / osm_poi_matchmaker

osm_poi_matchmaker.libs.xml.save_downloaded_xml() B last analyzed 2023-01-03 16:24 UTC

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like

osm_poi_matchmaker.libs.xml.save_downloaded_xml() B
last analyzed 2023-01-03 16:24 UTC