diff options
| author | bnewbold <bnewbold@archive.org> | 2020-06-29 19:35:06 +0000 |
|---|---|---|
| committer | bnewbold <bnewbold@archive.org> | 2020-06-29 19:35:06 +0000 |
| commit | cc6361e2b87c5ed7af82be8e0c79ddb1f91af1ce (patch) | |
| tree | d693068634bfa2ecb7e82990ea586a09b1895585 | |
| parent | ed1893bbc1e5e26c095a5e1f6ba3d1b15a959bab (diff) | |
| parent | 9f0e3f2af03161337b86308db0d1142b158282db (diff) | |
| download | fatcat-cc6361e2b87c5ed7af82be8e0c79ddb1f91af1ce.tar.gz fatcat-cc6361e2b87c5ed7af82be8e0c79ddb1f91af1ce.zip | |
Merge branch 'martin-datacite-fix-strptime-36559' into 'master'
datacite: hard cast possible date value to string
See merge request webgroup/fatcat!59
| -rw-r--r-- | python/fatcat_tools/importers/datacite.py | 2 | ||||
| -rw-r--r-- | python/tests/import_datacite.py | 1 |
2 files changed, 2 insertions, 1 deletions
diff --git a/python/fatcat_tools/importers/datacite.py b/python/fatcat_tools/importers/datacite.py index d998f266..98393451 100644 --- a/python/fatcat_tools/importers/datacite.py +++ b/python/fatcat_tools/importers/datacite.py @@ -1027,7 +1027,7 @@ def parse_datacite_dates(dates): ) def parse_item(item): - result, value, year_only = None, item.get('date', '') or '', False + result, value, year_only = None, str(item.get('date', '')) or '', False release_date, release_month, release_year = None, None, None for layout, granularity in common_patterns: diff --git a/python/tests/import_datacite.py b/python/tests/import_datacite.py index c9210ea4..afee06cc 100644 --- a/python/tests/import_datacite.py +++ b/python/tests/import_datacite.py @@ -174,6 +174,7 @@ def test_parse_datacite_dates(): Case('empty list is None', [], (None, None, None)), Case('empty item is None', [{}], (None, None, None)), Case('year only yields year only', [{'date': '2019'}], (None, None, 2019)), + Case('int year', [{'date': 2019}], (None, None, 2019)), Case('first wins', [{'date': '2019'}, {'date': '2020'}], (None, None, 2019)), Case('skip bogus year', [{'date': 'abc'}, {'date': '2020'}], (None, None, 2020)), Case('first with type', [ |