diff options
| author | Bryan Newbold <bnewbold@robocracy.org> | 2019-05-21 11:48:28 -0700 | 
|---|---|---|
| committer | Bryan Newbold <bnewbold@robocracy.org> | 2019-05-21 11:48:28 -0700 | 
| commit | 0988ac6fed7f1d1ba41233bde56b455ba6cba204 (patch) | |
| tree | 9354db4d9000d610e9c823d77b6a174fcadf3237 /python | |
| parent | 192b44ed9e3b1c388c11cf0467a749470dfe1300 (diff) | |
| download | fatcat-0988ac6fed7f1d1ba41233bde56b455ba6cba204.tar.gz fatcat-0988ac6fed7f1d1ba41233bde56b455ba6cba204.zip | |
fix lint errors in JSTOR importer
Diffstat (limited to 'python')
| -rw-r--r-- | python/fatcat_tools/importers/jstor.py | 33 | 
1 files changed, 16 insertions, 17 deletions
| diff --git a/python/fatcat_tools/importers/jstor.py b/python/fatcat_tools/importers/jstor.py index 4531d926..7d630b6b 100644 --- a/python/fatcat_tools/importers/jstor.py +++ b/python/fatcat_tools/importers/jstor.py @@ -49,6 +49,21 @@ class JstorImporter(EntityImporter):          extra = dict()          extra_jstor = dict() +        release_type = "article-journal" +        if "[Abstract]" in title: +            # TODO: strip the "[Abstract]" bit? +            release_type = "abstract" +        elif "[Editorial" in title: +            release_type = "editorial" +        elif "[Letter" in title: +            release_type = "letter" +        elif "[Poem" in title or "[Photograph" in title: +            release_type = None + +        if title.startswith("[") and title.endswith("]"): +            # strip brackets if that is all that is there (eg, translation or non-english) +            title = title[1:-1] +          # JSTOR journal-id          journal_ids = [j.string for j in journal_meta.find_all('journal-id')]          if journal_ids: @@ -76,8 +91,7 @@ class JstorImporter(EntityImporter):                  issnl=issnl,                  publisher=publisher,                  container_type=self.map_container_type(release_type), -                name=clean(journal_title, force_xml=True), -                extra=journal_extra) +                name=clean(journal_title, force_xml=True))              ce_edit = self.create_container(ce)              container_id = ce_edit.ident              self._issnl_id_map[issnl] = container_id @@ -155,21 +169,6 @@ class JstorImporter(EntityImporter):              if not language:                  warnings.warn("MISSING MARC LANG: {}".format(cm.find("meta-value").string)) -        release_type = "article-journal" -        if "[Abstract]" in title: -            # TODO: strip the "[Abstract]" bit? -            release_type = "abstract" -        elif "[Editorial" in title: -            release_type = "editorial" -        elif "[Letter" in title: -            release_type = "letter" -        elif "[Poem" in title or "[Photograph" in title: -            release_type = None - -        if title.startswith("[") and title.endswith("]"): -            # strip brackets if that is all that is there (eg, translation or non-english) -            title = title[1:-1] -          # JSTOR issue-id          if article_meta.find('issue-id'):              issue_id = clean(article_meta.find('issue-id').string) | 
