From 3b531d2f83e9fde67e3c45d751fb80b2d9c815be Mon Sep 17 00:00:00 2001 From: Martin Czygan Date: Tue, 7 Jan 2020 01:58:39 +0100 Subject: datacite: apply pylint suggestions --- python/fatcat_tools/importers/datacite.py | 18 ++++++++++-------- 1 file changed, 10 insertions(+), 8 deletions(-) diff --git a/python/fatcat_tools/importers/datacite.py b/python/fatcat_tools/importers/datacite.py index 53f46bb4..d7fbd269 100644 --- a/python/fatcat_tools/importers/datacite.py +++ b/python/fatcat_tools/importers/datacite.py @@ -8,20 +8,22 @@ problems in content and structure. A few fields habe their own parsing functions (parse_datacite_...), which can be tested more easily. """ -from .common import EntityImporter, clean import collections -import dateparser import datetime -import fatcat_openapi_client import hashlib import json -import pycountry -import langdetect import sqlite3 import sys -from fatcat_tools.transforms import entity_to_dict + +import dateparser +import fatcat_openapi_client +import langdetect +import pycountry + from fatcat_tools.normal import clean_doi +from fatcat_tools.transforms import entity_to_dict +from .common import EntityImporter, clean # Cutoff length for abstracts. MAX_ABSTRACT_LENGTH = 2048 @@ -309,7 +311,7 @@ class DataciteImporter(EntityImporter): for i, c in enumerate(attributes['creators']): nameType = c.get('nameType', '') or '' - if nameType == 'Personal' or nameType == '': + if nameType in ('', 'Personal'): creator_id = None for nid in c.get('nameIdentifiers', []): name_scheme = nid.get('nameIdentifierScheme', '') or '' @@ -493,7 +495,7 @@ class DataciteImporter(EntityImporter): if first_page and last_page: try: - int(first_page) < int(last_page) + _ = int(first_page) < int(last_page) pages = '{}-{}'.format(first_page, last_page) except ValueError as err: # TODO(martin): This is more debug than info. -- cgit v1.2.3