Some linting.

etiquette/__init__.py

@@ -6,3 +6,14 @@ from . import objects
 from . import photodb
 from . import searchhelpers
 from . import tag_export
+
+__all__ = [
+    'constants',
+    'decorators',
+    'exceptions',
+    'helpers',
+    'objects',
+    'photodb',
+    'searchhelpers',
+    'tag_export',
+]

etiquette/helpers.py

@@ -7,7 +7,6 @@ import hashlib
 import mimetypes
 import os
 import PIL.Image
-import re
 import zipstream
 
 from voussoirkit import bytestring

etiquette/objects.py

@@ -357,8 +357,8 @@ class Album(ObjectBase, GroupableMixin):
 
     @decorators.required_feature('album.edit')
     @decorators.transaction
-    def add_child(self, *args, **kwargs):
+    def add_child(self, member):
-        return super().add_child(*args, **kwargs)
+        return super().add_child(member)
 
     @decorators.required_feature('album.edit')
     @decorators.transaction
@@ -1308,7 +1308,11 @@ class Photo(ObjectBase):
 
         new_path.assert_not_exists()
 
-        self.photodb.log.info('Renaming file "%s" -> "%s".', old_path.absolute_path, new_path.absolute_path)
+        self.photodb.log.info(
+            'Renaming file "%s" -> "%s".',
+            old_path.absolute_path,
+            new_path.absolute_path,
+        )
 
         new_path.parent.makedirs(exist_ok=True)
 
@@ -1476,7 +1480,7 @@ class Tag(ObjectBase, GroupableMixin):
     @decorators.required_feature('tag.edit')
     @decorators.transaction
     def add_child(self, member):
-        ret = self.__add_child(member)
+        ret = super().add_child(member)
         if ret is BAIL:
             return BAIL
 
@@ -1486,10 +1490,8 @@ class Tag(ObjectBase, GroupableMixin):
     @decorators.required_feature('tag.edit')
     @decorators.transaction
     def add_children(self, members):
-        bail = True
+        ret = super().add_children(members)
-        for member in members:
+        if ret is BAIL:
-            bail = (self.__add_child(member) is BAIL) and bail
-        if bail:
             return BAIL
 
         self.photodb.caches['tag_exports'].clear()

etiquette/tag_export.py

@@ -57,7 +57,8 @@ def flat_dict(tags, include_synonyms=True):
     for equaling the main tag versus existing in the rest of the subtree.
     '''
     result = {}
-    def recur(tag):
+
+    def recurse(tag):
         try:
             return result[tag]
         except KeyError:
@@ -67,7 +68,7 @@ def flat_dict(tags, include_synonyms=True):
         my_result.add(tag)
 
         for child in tag.get_children():
-            my_result.update(recur(child))
+            my_result.update(recurse(child))
 
         result[tag] = my_result
 
@@ -77,7 +78,7 @@ def flat_dict(tags, include_synonyms=True):
         return my_result
 
     for tag in tags:
-        recur(tag)
+        recurse(tag)
 
     return result
 

frontends/etiquette_cli.py

@@ -246,7 +246,6 @@ def easybake_argparse(args):
         photodb.commit()
 
 def export_symlinks_argparse(args):
-    photodb = find_photodb()
     destination = pathclass.Path(args.destination)
     destination.makedirs(exist_ok=True)
 
@@ -594,12 +593,12 @@ digest:
     flags:
     --exclude_directories A B C:
         Any directories matching any pattern of A, B, C... will be skipped.
-        These patterns may be absolute paths like 'D:\temp', plain names like
+        These patterns may be absolute paths like 'D:\\temp', plain names like
         'thumbnails' or glob patterns like 'build_*'.
 
     --exclude_filenames A B C:
         Any filenames matching any pattern of A, B, C... will be skipped.
-        These patterns may be absolute paths like 'D:\somewhere\config.json',
+        These patterns may be absolute paths like 'D:\\somewhere\\config.json',
         plain names like 'thumbs.db' or glob patterns like '*.temp'.
 
     --glob_directories A B C:

frontends/etiquette_flask/backend/__init__.py

@@ -4,3 +4,11 @@ from . import endpoints
 from . import sessions
 
 site = common.site
+
+__all__ = [
+    'common',
+    'decorators',
+    'endpoints',
+    'sessions',
+    'site',
+]

frontends/etiquette_flask/backend/client_caching.py

@@ -3,8 +3,6 @@ import hashlib
 from voussoirkit import cacheclass
 from voussoirkit import spinal
 
-import etiquette
-
 class FileEtagManager:
     '''
     The FileEtagManager serves ETag and Cache-Control headers for disk files to

frontends/etiquette_flask/backend/endpoints/__init__.py

@@ -4,3 +4,12 @@ from . import bookmark_endpoints
 from . import photo_endpoints
 from . import tag_endpoints
 from . import user_endpoints
+
+__all__ = [
+    'album_endpoints',
+    'basic_endpoints',
+    'bookmark_endpoints',
+    'photo_endpoints',
+    'tag_endpoints',
+    'user_endpoints',
+]

frontends/etiquette_flask/backend/endpoints/album_endpoints.py

@@ -1,6 +1,5 @@
 import flask; from flask import request
 import os
-import time
 import urllib.parse
 
 from voussoirkit import flasktools

frontends/etiquette_flask/backend/endpoints/tag_endpoints.py

@@ -1,5 +1,4 @@
 import flask; from flask import request
-import time
 
 from voussoirkit import flasktools
 

utilities/database_upgrader.py

@@ -188,7 +188,8 @@ def upgrade_6_to_7(photodb):
     Most of the indices were renamed.
     '''
     photodb.sql_execute('BEGIN')
-    indices = photodb.sql_select('SELECT name FROM sqlite_master WHERE type == "index" AND name NOT LIKE "sqlite_%"')
+    query = 'SELECT name FROM sqlite_master WHERE type == "index" AND name NOT LIKE "sqlite_%"'
+    indices = photodb.sql_select(query)
     indices = [name for (name,) in indices]
     for index in indices:
         photodb.sql_execute(f'DROP INDEX {index}')