code cleanup

jsonkeeper/__init__.py

@@ -75,7 +75,8 @@ def make_json_error(error):
 
         @app.after_request
         def set_response_headers(response):
-            response.headers['Cache-Control'] = 'private, no-store, no-cache, must-revalidate'
+            response.headers['Cache-Control'] = ('private, no-store, no-cache,'
+                                                 ' must-revalidate')
             # response.headers['Pragma'] = 'no-cache'
             # response.headers['Expires'] = '0'
             return response

jsonkeeper/config.py

@@ -42,8 +42,8 @@ def log_cfg(self, cp, msg):
         # https://www.bugs.python.org/issue27805
         # side note: stat.S_ISCHR(os.stat(fn).st_mode) doesn't seem to work for
         #            in an alpine linux docker container running canvas indexer
-        #            with gunicorn although manually executing it on a python shell
-        #            in the container works
+        #            with gunicorn although manually executing it on a python
+        #            shell in the container works
         if log_file == '/dev/stdout':
             mode = 'w'
         else:

jsonkeeper/subroutines.py

@@ -715,7 +715,7 @@ def remove_document_from_actstr(to_rem_id):
         only the first AS page with a matching activity is deleted.
     """
 
-    return # currently not used because Delete Activities were introduced
+    return  # currently not used because Delete Activities were introduced
 
     coll_json = get_actstr_collection()
     if coll_json:

jsonkeeper/views.py

@@ -4,9 +4,7 @@
     acceptable_content_type,
     CORS_preflight_response,
     add_CORS_headers,
-    get_access_token,
     get_JSON_string_by_ID,
-    get_JSON_metadata_by_ID,
     get_actstr_collection_pages,
     get_actstr_collection,
     handle_post_request,
@@ -136,8 +134,10 @@ def api_json_id_range(json_id, r_num):
         return abort(404, 'JSON document with ID {} not found'.format(json_id))
 
 
-@jk.route('/{}/<regex("{}"):json_id>/status'.format(current_app.cfg.api_path(),
-                                              current_app.cfg.doc_id_patt()),
+@jk.route('/{}/<regex("{}"):json_id>/status'.format(
+                current_app.cfg.api_path(),
+                current_app.cfg.doc_id_patt()
+                ),
           methods=['GET', 'PATCH', 'OPTIONS'])
 def api_json_id_status(json_id):
     """ API endpoint for retrieving and changing JSON documents' metadata.

test.py

@@ -429,7 +429,7 @@ def test_status_endpoint(self):
                                 headers={'Accept': 'application/json',
                                          'Content-Type': 'application/json',
                                          'X-Access-Token': 'foo',
-                                         'X-Unlisted':'true'},
+                                         'X-Unlisted': 'true'},
                                 data='{"baz":"bam"}')
             location = resp.headers.get('Location')
             resp = self.tc.get('{}/status'.format(location),
@@ -458,7 +458,7 @@ def test_anon_AS(self):
             resp = self.tc.post('/{}'.format(self.app.cfg.api_path()),
                                 headers={'Accept': 'application/json',
                                          'Content-Type': 'application/ld+json',
-                                         'X-Unlisted':'false'},
+                                         'X-Unlisted': 'false'},
                                 data=curation_json)
             resp = self.tc.get('/{}'.format(self.app.cfg.as_coll_url()))
             self.assertEqual(resp.status, '404 NOT FOUND')
@@ -476,7 +476,7 @@ def test_unlisted_AS(self):
             resp = self.tc.post('/{}'.format(self.app.cfg.api_path()),
                                 headers={'Accept': 'application/json',
                                          'Content-Type': 'application/ld+json',
-                                         'X-Unlisted':'true'},
+                                         'X-Unlisted': 'true'},
                                 data=curation_json)
 
             resp = self.tc.get('/{}'.format(self.app.cfg.as_coll_url()))
@@ -487,7 +487,7 @@ def test_unlisted_AS(self):
                                 headers={'Accept': 'application/json',
                                          'Content-Type': 'application/ld+json',
                                          'X-Access-Token': 'foo',
-                                         'X-Unlisted':'false'},
+                                         'X-Unlisted': 'false'},
                                 data=curation_json)
             location = resp.headers.get('Location')
 
@@ -508,5 +508,6 @@ def test_unlisted_AS(self):
             # a Delete Action should have been added
             self.assertEqual(json_obj.get('totalItems'), 2)
 
+
 if __name__ == '__main__':
     unittest.main()

util/iiif.py

@@ -2,7 +2,6 @@
 """
 
 import json
-import uuid
 from collections import OrderedDict
 
 
@@ -120,7 +119,6 @@ def get_range_summary(self):
                        '>>> {}'.format(json.dumps(w))))
             ret_lst.append(dic)
             ret_dic[dic['ran']] = dic['man']
-        import pprint
         return ret_lst, ret_dic
 
     def _extract_manifest_id(self, dic):