Updates the order of Python imports

openlibrary/core/models.py

@@ -1,7 +1,6 @@
 """Models of various OL objects.
 """
 
-import json
 import logging
 from collections import defaultdict
 from dataclasses import dataclass, field

openlibrary/plugins/upstream/addtag.py

@@ -8,18 +8,13 @@
 from infogami.infobase.client import ClientException
 from infogami.utils import delegate
 from infogami.utils.view import add_flash_message, public
-
 from openlibrary.accounts import get_current_user
 from openlibrary.plugins.upstream import spamcheck
 from openlibrary.plugins.upstream.addbook import safe_seeother, trim_doc
 from openlibrary.plugins.upstream.models import Tag
 from openlibrary.plugins.upstream.utils import render_template
 
 
-SUBJECT_SUB_TYPES = ["subject", "person", "place", "time"]
-TAG_TYPES = SUBJECT_SUB_TYPES + ["collection"]
-
-
 @public
 def get_tag_types():
     return TAG_TYPES
@@ -30,6 +25,10 @@ def get_subject_tag_types():
     return SUBJECT_SUB_TYPES
 
 
+SUBJECT_SUB_TYPES = ["subject", "person", "place", "time"]
+TAG_TYPES = SUBJECT_SUB_TYPES + ["collection"]
+
+
 def validate_tag(tag):
     return tag.get('name', '') and tag.get('tag_type', '') in get_tag_types()
 
@@ -227,7 +226,7 @@ def POST(self, tag_type):
                 "message.html", "Oops", 'Something went wrong. Please try again later.'
             )
         if not (patron := get_current_user()):
-            raise web.seeother(f'/account/login')
+            raise web.seeother('/account/login')
         if not self.has_permission(patron):
             raise web.unauthorized()
 

openlibrary/plugins/worksearch/subjects.py

@@ -11,7 +11,6 @@
 from infogami.plugins.api.code import jsonapi
 from infogami.utils import delegate
 from infogami.utils.view import render_template, safeint
-
 from openlibrary.core.lending import add_availability
 from openlibrary.core.models import Subject, Tag
 from openlibrary.solr.query_utils import query_dict_to_str