codex 1.7.3a2__py3-none-any.whl → 1.7.5__py3-none-any.whl

codex/views/browser/browser.py

@@ -173,13 +173,13 @@ class BrowserView(BrowserTitleView):
         else:
             zero_pad = 1
 
-        # self._debug_queries(page_group_count, page_book_count, group_qs, book_qs)
+        # self._debug_queries(page_group_count, page_book_count, group_qs, book_qs) # noqa: ERA001
 
         total_page_count = page_group_count + page_book_count
         mtime = self._get_page_mtime()
         return group_qs, book_qs, num_pages, total_page_count, zero_pad, mtime
 
-    def get_object(self):  # type: ignore
+    def get_object(self):
         """Validate settings and get the querysets."""
         group_qs, book_qs, num_pages, total_count, zero_pad, mtime = (
             self._get_group_and_books()

codex/views/browser/choices.py

@@ -8,7 +8,7 @@ from django.db.models import QuerySet
 from drf_spectacular.utils import extend_schema
 from rest_framework.response import Response
 
-from codex.choices import DUMMY_NULL_NAME, VUETIFY_NULL_CODE
+from codex.choices.browser import DUMMY_NULL_NAME, VUETIFY_NULL_CODE
 from codex.logger.logging import get_logger
 from codex.models import (
     Comic,
@@ -18,8 +18,6 @@ from codex.models import (
 from codex.models.named import IdentifierType
 from codex.serializers.browser.choices import (
     BrowserChoicesFilterSerializer,
-)
-from codex.serializers.browser.filters import (
     BrowserFilterChoicesSerializer,
 )
 from codex.serializers.browser.settings import BrowserFilterChoicesInputSerilalizer
@@ -98,7 +96,7 @@ class BrowserChoicesViewBase(BrowserFilterView):
 
         return rel, model
 
-    def get_object(self) -> QuerySet:  # type: ignore
+    def get_object(self) -> QuerySet:
         """Get the comic subquery use for the choices."""
         return self.get_filtered_queryset(Comic)
 
@@ -136,12 +134,13 @@ class BrowserChoicesAvailableView(BrowserChoicesViewBase):
 
         return count
 
-    def get_object(self) -> dict[str, Any]:  # type: ignore
+    def get_object(self) -> dict[str, Any]:  # type: ignore[reportIncompatibleMethodOverride]
         """Get choice counts."""
         qs = super().get_object()
         filters = self.params.get("filters", {})
         data = {}
-        for field_name in self.serializer_class().get_fields():  # type: ignore
+        serializer: BrowserFilterChoicesSerializer = self.serializer_class()  # type: ignore[reportOptionalCall, reportAssignmentType]
+        for field_name in serializer.get_fields():
             if field_name == "story_arcs" and qs.model is StoryArc:
                 # don't allow filtering on story arc in story arc view.
                 continue
@@ -154,7 +153,7 @@ class BrowserChoicesAvailableView(BrowserChoicesViewBase):
 
             try:
                 is_filter_set = bool(filters.get(field_name))
-                flag = is_filter_set or count > 1  # type: ignore
+                flag = is_filter_set or count > 1
             except TypeError:
                 flag = False
             data[field_name] = flag
@@ -185,7 +184,7 @@ class BrowserChoicesView(BrowserChoicesViewBase):
         field_name = self.kwargs.get("field_name", "")
         return snakecase(field_name)
 
-    def get_object(self) -> dict[str, Any]:  # type: ignore
+    def get_object(self) -> dict[str, Any]:  # type: ignore[reportIncompatibleMethodOverride]
         """Return choices with more than one choice."""
         qs = super().get_object()
         field_name = self._get_field_name()

codex/views/browser/cover.py

@@ -52,7 +52,7 @@ class CoverView(BrowserAnnotateOrderView):
         BrowserAnnotateOrderView.REPARSE_JSON_FIELDS | {"parent"}
     )
 
-    def get_group_filter(self, group=None, pks=None, page_mtime=False):
+    def get_group_filter(self, group=None, pks=None, page_mtime=False):  # noqa: FBT002
         """Get group filter for First Cover View."""
         if self.params.get("dynamic_covers") or self.model in (Volume, Folder):
             return super().get_group_filter(group=group, pks=pks, page_mtime=page_mtime)
@@ -60,7 +60,10 @@ class CoverView(BrowserAnnotateOrderView):
         # First cover group filter relies on sort names to look outside the browser supplied pks
         # For multi_groups not in the browser query.
         pks = self.kwargs["pks"]
-        qs = self.model.objects.filter(pk__in=pks)  # type: ignore
+        if not self.model:
+            qs = Comic.objects.none()
+        else:
+            qs = self.model.objects.filter(pk__in=pks)
         sort_names = qs.values_list("sort_name", flat=True).distinct()
         model_rel = GROUP_RELATION[self.model_group]
         group_filter = {f"{model_rel}__sort_name__in": sort_names}
@@ -142,7 +145,7 @@ class CoverView(BrowserAnnotateOrderView):
         parameters=[BrowserAnnotateOrderView.input_serializer_class],
         responses={(200, content_type): OpenApiTypes.BINARY},
     )
-    def get(self, *args, **kwargs):  # type: ignore
+    def get(self, *args, **kwargs):
         """Get comic cover."""
         try:
             try:

codex/views/browser/download.py

@@ -54,7 +54,8 @@ class GroupDownloadView(BrowserFilterView):
         filename = kwargs.get("filename")
         if not filename:
             pks = self.kwargs.get("pks")
-            filename = f"{self.model.__name__} {pks} Comics.zip"  # type: ignore
+            name = self.model.__name__ if self.model else "No Model"
+            filename = f"{name} {pks} Comics.zip"
 
         headers = {"Content-Length": len(zs), "Last-Modified": zs.last_modified}
         return FileResponse(

codex/views/browser/filters/bookmark.py

@@ -11,7 +11,7 @@ class BrowserFilterBookmarkView(BookmarkFilterMixin, BrowserValidateView):
 
     def get_bookmark_filter(self, model):
         """Build bookmark query."""
-        choice: str = self.params.get("filters", {}).get("bookmark", "")  # type: ignore
+        choice: str = self.params.get("filters", {}).get("bookmark", "")
         if choice:
             bm_rel = self.get_bm_rel(model)
             my_bookmark_filter = self.get_my_bookmark_filter(bm_rel)

codex/views/browser/filters/field.py

@@ -55,5 +55,5 @@ class ComicFieldFilterView(GroupFilterView):
     def get_comic_field_filter(self, model):
         """Filter the comics based on the form filters."""
         rel_prefix = self.get_rel_prefix(model)
-        filters = self.params["filters"]  # type: ignore
+        filters = self.params["filters"]
         return self.get_all_comic_field_filters(rel_prefix, filters)

codex/views/browser/filters/filter.py

@@ -13,8 +13,6 @@ LOG = get_logger(__name__)
 class BrowserFilterView(BrowserFilterBookmarkView):
     """Browser Filters."""
 
-    TARGET = ""
-
     def force_inner_joins(self, qs):
         """Force INNER JOINS to filter empty groups."""
         demote_tables = {"codex_library"}
@@ -28,10 +26,10 @@ class BrowserFilterView(BrowserFilterBookmarkView):
     def _get_query_filters(
         self,
         model,
+        page_mtime,
+        bookmark_filter,
         group=None,
         pks=None,
-        page_mtime=False,
-        bookmark_filter=True,
     ) -> Q:
         """Return all the filters except the group filter."""
         big_include_filter = Q()
@@ -54,16 +52,16 @@ class BrowserFilterView(BrowserFilterBookmarkView):
         model,
         group=None,
         pks=None,
-        page_mtime=False,
-        bookmark_filter=True,
+        page_mtime=False,  # noqa: FBT002
+        bookmark_filter=True,  # noqa: FBT002
     ) -> QuerySet:
         """Get a filtered queryset for the model."""
         query_filters = self._get_query_filters(
             model,
-            group=group,
-            pks=pks,
             page_mtime=page_mtime,
             bookmark_filter=bookmark_filter,
+            group=group,
+            pks=pks,
         )
         # Distinct necissary for folder view with search
         return model.objects.filter(query_filters).distinct()

codex/views/browser/filters/group.py

@@ -9,7 +9,7 @@ from codex.views.const import (
     GROUP_RELATION,
 )
 
-_GROUP_REL_TARGETS = frozenset({"cover", "choices"})
+_GROUP_REL_TARGETS = frozenset({"cover", "choices", "bookmark"})
 _PK_REL_TARGETS = frozenset({"metadata", "mtime"})
 
 
@@ -18,16 +18,16 @@ class GroupFilterView(BrowserParamsView):
 
     SESSION_KEY = BrowserParamsView.BROWSER_SESSION_KEY
 
-    def _get_rel_for_pks(self, group, page_mtime=False):
+    TARGET = ""
+
+    def _get_rel_for_pks(self, group, page_mtime: bool):
         """Get the relation from the model to the pks."""
-        # XXX these TARGET refs might be better as subclass get rel methods.
-        target: str = self.TARGET  # type: ignore
-        if target in _GROUP_REL_TARGETS:
+        if self.TARGET in _GROUP_REL_TARGETS:
             rel = FILTER_ONLY_GROUP_RELATION[group]
-        elif target in _PK_REL_TARGETS or page_mtime:
+        elif self.TARGET in _PK_REL_TARGETS or page_mtime:
             # metadata, mtime, browser.page_mtime
             rel = "pk"
-        elif target == "download":
+        elif self.TARGET == "download":
             rel = "comic__folders" if group == "f" else "pk"
         else:
             # browser.group, opds
@@ -36,7 +36,7 @@ class GroupFilterView(BrowserParamsView):
         rel += "__in"
         return rel
 
-    def get_group_filter(self, group=None, pks=None, page_mtime=False):
+    def get_group_filter(self, group=None, pks=None, page_mtime=False):  # noqa: FBT002
         """Get filter for the displayed group."""
         if group is None:
             group = self.kwargs["group"]

codex/views/browser/filters/search/field/expression.py

@@ -79,8 +79,7 @@ def _cast_value(rel, rel_class, value):
         value = parse(value)
         if rel_class == DateField and value:
             value = value.date()
-    # elif rel_class in (CharField, TextField):
-    #    value = value
+    # (CharField, TextField):
     return value
 
 

codex/views/browser/filters/search/field/optimize.py

@@ -1,5 +1,5 @@
 """Optimize Text Lookups."""
-# XXX Unused XXX multiple like queries test better
+# UNUSED CODE multiple like queries test better
 
 import re
 

codex/views/browser/filters/search/field/parse.py

@@ -22,7 +22,7 @@ if TYPE_CHECKING:
     from pyparsing.helpers import InfixNotationOperatorSpec
 
 _QUOTES_REXP = r"(?:\".*?\")"
-_OPERATORS_REXP = "|".join(("and not", "or not", "and", "or"))
+_OPERATORS_REXP = "and not|or not|and|or"
 _BEGIN_NOT_REXP = r"^\s*\(?\s*(?P<not>not)"
 _IMPLICIT_AND_REXP = (
     rf"{_QUOTES_REXP}|\ (?P<ok>{_OPERATORS_REXP})\ |(?P<bare>(?:\ not)?\ )\S"
@@ -183,5 +183,5 @@ def get_field_query(rel, rel_class, exp, model, many_to_many):
     bool_expr = _create_context_expression(context)
 
     parsed_result = bool_expr.parse_string(exp)
-    root_bool_operand: BoolOperand = parsed_result[0]  # type:ignore
+    root_bool_operand: BoolOperand = parsed_result[0]  # type:ignore[reportAssignmentType]
     return root_bool_operand.to_query()

codex/views/browser/filters/search/parse.py

@@ -149,7 +149,7 @@ class SearchFilterView(BrowserFTSFilter):
 
     def _preparse_search_query(self):
         """Preparse search fields out of query text."""
-        text = self.params.get("q")  # type: ignore
+        text = self.params.get("q")
         field_tokens = {}
         if not text:
             return field_tokens, text
@@ -177,7 +177,7 @@ class SearchFilterView(BrowserFTSFilter):
                 )
             else:
                 # AND and OR
-                # XXX cannot do OR queries with MATCH, it decontextualizes MATCH somehow.
+                # Cannot do OR queries with MATCH, it decontextualizes MATCH somehow.
                 if preop == "or":
                     self.search_error = "OR preceding column tokens with operator expressions will act as AND"
                 include_q_list, exclude_q_list = self.get_search_field_filters(
@@ -227,24 +227,9 @@ class SearchFilterView(BrowserFTSFilter):
 
         return include_q, exclude_q, fts_q
 
-    def _is_search_results_limited(self) -> bool:
-        """Get search result limit from params."""
-        # user = self.request.user  # type: ignore
-        # if user and user.is_authenticated:
-        #    limited = bool(
-        #        self.params.get(  # type: ignore
-        #            "search_results_limit",
-        #            MAX_OBJ_PER_PAGE,
-        #        )
-        #    )
-        # else:
-        #    limited = True
-        # return limited
-        return True
-
     def get_search_limit(self):
         """Get search scores for choices and metadata."""
-        if not self.search_mode or not self._is_search_results_limited():
+        if not self.search_mode:
             return 0
-        page = self.kwargs.get("page", 1)  # type: ignore
+        page = self.kwargs.get("page", 1)
         return page * MAX_OBJ_PER_PAGE + 1

codex/views/browser/group_mtime.py

@@ -54,7 +54,7 @@ class BrowserGroupMtimeView(BrowserFilterView):
             args["distinct"] = True
         return agg_func(bmua_rel, **args)
 
-    def get_group_mtime(self, model, group=None, pks=None, page_mtime=False):
+    def get_group_mtime(self, model, group=None, pks=None, page_mtime=False):  # noqa: FBT002
         """Get a filtered mtime for browser pages and mtime checker."""
         qs = self.get_filtered_queryset(
             model,

codex/views/browser/metadata/copy_intersections.py

@@ -47,7 +47,7 @@ class MetadataCopyIntersectionsView(MetadataQueryIntersectionsView):
     @classmethod
     def _copy_m2m_intersections(cls, obj, m2m_intersections):
         """Copy the m2m intersections into the object."""
-        # XXX It might even be faster to copy everything to a dict and not use the obj.
+        # It might even be faster to copy everything to a dict and not use the obj.
         for key, qs in m2m_intersections.items():
             serializer_key = (
                 f"{PREFETCH_PREFIX}{key}"

codex/views/browser/metadata/metadata.py

@@ -63,7 +63,7 @@ class MetadataView(MetadataCopyIntersectionsView):
             obj = qs[0]
             if not obj:
                 reason = "Empty obj"
-                raise ValueError(reason)  # noqa TRY301
+                raise ValueError(reason)  # noqa: TRY301
         except (IndexError, ValueError) as exc:
             return self._raise_not_found(exc)
 

codex/views/browser/mtime.py

@@ -37,7 +37,7 @@ class MtimeView(BrowserGroupMtimeView):
         """Get max mtime for all groups."""
         max_mtime = None
 
-        for item in self.params["groups"]:  # type: ignore
+        for item in self.params["groups"]:
             mtime = self._get_group_mtime(item)
             max_mtime = max_none(max_mtime, mtime)
         return max_mtime

codex/views/browser/order_by.py

@@ -55,7 +55,7 @@ class BrowserOrderByView(BrowserGroupMtimeView):
                 order_fields_head += ["bookmark_updated_at"]
         return order_fields_head
 
-    def add_order_by(self, qs, order_key="", do_reverse=True, comic_sort_names=None):
+    def add_order_by(self, qs, order_key="", comic_sort_names=None):
         """Create the order_by list."""
         order_fields_head = ()
         if qs.model is Comic:
@@ -65,11 +65,9 @@ class BrowserOrderByView(BrowserGroupMtimeView):
 
         order_fields = (*order_fields_head, "pk")
 
-        prefix = "-" if do_reverse and self.params.get("order_reverse") else ""
-        order_by = []
+        prefix = "-" if self.params.get("order_reverse") else ""
         if prefix:
-            for field in order_fields:
-                order_by.append(prefix + field)
+            order_by = [prefix + field for field in order_fields]
         else:
             order_by = order_fields
 

codex/views/browser/params.py

@@ -58,7 +58,7 @@ class BrowserParamsView(SessionView):
             validated_data = serializer.validated_data
 
             if validated_data:
-                params.update(validated_data)  # type: ignore
+                params.update(validated_data)
             self._params = MappingProxyType(params)
             self.validate_settings()
         return self._params

codex/views/browser/settings.py

@@ -9,7 +9,7 @@ class BrowserSettingsView(SettingsView):
     """Get Browser Settings."""
 
     # Put Browser Settings is normally done through BrowserView.get()
-    serializer_class = BrowserSettingsSerializer  # type: ignore
+    serializer_class = BrowserSettingsSerializer
 
     SESSION_KEY = SettingsView.BROWSER_SESSION_KEY
 

codex/views/browser/validate.py

@@ -3,10 +3,11 @@
 from collections.abc import Mapping
 from copy import deepcopy
 from types import MappingProxyType
+from typing import Any
 
 from rest_framework.exceptions import NotFound
 
-from codex.choices import DEFAULT_BROWSER_ROUTE
+from codex.choices.browser import DEFAULT_BROWSER_ROUTE
 from codex.exceptions import SeeOtherRedirectError
 from codex.logger.logging import get_logger
 from codex.models.groups import BrowserGroupModel
@@ -73,17 +74,18 @@ class BrowserValidateView(SearchFilterView):
         self, reason, route_mask=None, settings_mask: Mapping | None = None
     ):
         """Redirect the client to a valid group url."""
-        route = mapping_to_dict(self.DEFAULT_ROUTE)
+        route: dict[str, Any] = mapping_to_dict(self.DEFAULT_ROUTE)  # type:ignore[reportAssignmentType]
         if route_mask:
-            route["params"].update(route_mask)  # type: ignore
-        settings: dict = deepcopy(mapping_to_dict(self.params))  # type: ignore
+            route["params"].update(route_mask)
+        settings: dict[str, Any] = deepcopy(mapping_to_dict(self.params))  # type: ignore[reportAssignmentType]
         if settings_mask:
             settings.update(settings_mask)
         detail = {"route": route, "settings": settings, "reason": reason}
         raise SeeOtherRedirectError(detail=detail)
 
     def _get_valid_browse_top_groups(self):
-        """Get valid top groups for the current settings.
+        """
+        Get valid top groups for the current settings.
 
         Valid top groups are determined by the Browser Settings.
         """
@@ -118,7 +120,8 @@ class BrowserValidateView(SearchFilterView):
             self.raise_redirect(reason, route, settings_mask)
 
     def _get_valid_browse_nav_groups(self, valid_top_groups):
-        """Get valid nav groups for the current settings.
+        """
+        Get valid nav groups for the current settings.
 
         Valid nav groups are the top group and below that are also
         enabled in browser settings.

codex/views/const.py

@@ -76,9 +76,6 @@ METADATA_GROUP_RELATION = MappingProxyType(
     {
         **GROUP_NAME_MAP,
         COMIC_GROUP: "comic",
-        # Possible to list all groups in a non group view, but could get crazy.
-        # FOLDER_GROUP: "comic__folder",
-        # STORY_ARC_GROUP: "comic__story_arc_number__story_arc"
     }
 )
 CUSTOM_COVER_GROUP_RELATION = MappingProxyType(

codex/views/frontend.py

@@ -17,7 +17,7 @@ class IndexView(SessionView):
     SESSION_KEY = SessionView.BROWSER_SESSION_KEY
 
     permission_classes = (AllowAny,)
-    renderer_classes: ClassVar[list] = [TemplateHTMLRenderer]  # type: ignore
+    renderer_classes: ClassVar[list] = [TemplateHTMLRenderer]  # type: ignore[reportIncompatibleVariableOverride]
     template_name = "index.html"
 
     def _get_last_route(self):

codex/views/mixins.py

@@ -4,6 +4,7 @@ from django.db.models.expressions import F
 
 from codex.logger.logging import get_logger
 from codex.models.comic import Comic, Imprint, Volume
+from codex.views.browser.filters.filter import BrowserFilterView
 from codex.views.const import GROUP_NAME_MAP
 
 LOG = get_logger(__name__)
@@ -12,7 +13,7 @@ _GROUP_NAME_TARGETS = frozenset({"browser", "opds1", "opds2", "reader"})
 _VARIABLE_SHOW = "pi"
 
 
-class SharedAnnotationsMixin:
+class SharedAnnotationsMixin(BrowserFilterView):
     """Cross view annotation methods."""
 
     @staticmethod
@@ -54,7 +55,7 @@ class SharedAnnotationsMixin:
     def annotate_group_names(cls, qs):
         """Annotate name fields by hoisting them up."""
         # Optimized to only lookup what is used on the frontend
-        target = cls.TARGET  # type: ignore
+        target = cls.TARGET
         if target not in _GROUP_NAME_TARGETS:
             return qs
         group_names = {}

codex/views/opds/authentication_v1.py

@@ -40,7 +40,7 @@ _DOC = MappingProxyType(
                 "labels": {"login": "Username", "password": "Password"},
             },
             {
-                # XXX Out of spec type
+                # Out of spec type
                 "type": "cookie",
                 "links": [
                     {

codex/views/opds/const.py

@@ -48,9 +48,9 @@ class MimeType:
 
     ATOM = "application/atom+xml"
     _PROFILE_CATALOG = "profile=opds-catalog"
-    NAV = ";".join((ATOM, _PROFILE_CATALOG, "kind=navigation"))
-    ACQUISITION = ";".join((ATOM, _PROFILE_CATALOG, "kind=acquisition"))
-    ENTRY_CATALOG = ";".join((ATOM, "type=entry", _PROFILE_CATALOG))
+    NAV = f"{ATOM};{_PROFILE_CATALOG};kind=navigation"
+    ACQUISITION = f"{ATOM};{_PROFILE_CATALOG};kind=acquisition"
+    ENTRY_CATALOG = f"{ATOM};type=entry;{_PROFILE_CATALOG}"
     AUTHENTICATION = "application/opds-authentication+json"
     OPENSEARCH = "application/opensearchdescription+xml"
     STREAM = "image/jpeg"
@@ -58,7 +58,7 @@ class MimeType:
     OPDS_PUB = "application/opds-publication+json"
     PROGRESSION = "application/vnd.readium.progression+json"
     BOOK = "http://schema.org/Book"
-    # COMIC = "https://schema.org/ComicStory"
+    # COMIC = "https://schema.org/ComicStory" unused
     JPEG = "image/jpeg"
     WEBP = "image/webp"
     HTML = "text/html"

codex/views/opds/urls.py

@@ -3,7 +3,7 @@
 from django.urls import reverse
 from rest_framework.response import Response
 
-from codex.choices import DEFAULT_BROWSER_ROUTE
+from codex.choices.browser import DEFAULT_BROWSER_ROUTE
 from codex.serializers.opds.urls import OPDSURLsSerializer
 from codex.views.auth import AuthGenericAPIView
 from codex.views.util import pop_name

codex/views/opds/util.py

@@ -5,7 +5,7 @@ from django.http.response import HttpResponseRedirect
 from django.urls import reverse
 from django.utils.http import urlencode
 
-from codex.choices import DEFAULT_BROWSER_ROUTE
+from codex.choices.browser import DEFAULT_BROWSER_ROUTE
 from codex.models import (
     Contributor,
     ContributorPerson,
@@ -31,7 +31,7 @@ def update_href_query_params(href, old_query_params, new_query_params=None):
     return href
 
 
-def get_contributor_people(comic_pks, roles, exclude=False):
+def get_contributor_people(comic_pks, roles, exclude):
     """Get contributors that are not authors."""
     people = ContributorPerson.objects.filter(
         contributor__comic__in=comic_pks,
@@ -43,7 +43,7 @@ def get_contributor_people(comic_pks, roles, exclude=False):
     return people.distinct().only("name")
 
 
-def get_contributors(comic_pks, roles, exclude=False):
+def get_contributors(comic_pks, roles, exclude):
     """Get credits that are not part of other roles."""
     contributors = Contributor.objects.filter(comic__in=comic_pks)
     if exclude:
@@ -78,7 +78,7 @@ def full_redirect_view(url_name):
         path = request.get_full_path()
         if path:
             parts = path.split("?")
-            if len(parts) >= 2:  # noqa PLR2004
+            if len(parts) >= 2:  # noqa: PLR2004
                 parts[0] = url
                 url = "?".join(parts)
 

codex/views/opds/v1/entry/entry.py

@@ -47,8 +47,10 @@ class OPDS1Entry(OPDS1EntryLinksMixin):
                 elif group == "c":
                     title = Comic.get_title(
                         self.obj,
-                        zero_pad=self.zero_pad,
+                        volume=True,
+                        name=True,
                         filename_fallback=self.title_filename_fallback,
+                        zero_pad=self.zero_pad,
                     )
                     parts.append(title)
 
@@ -133,7 +135,7 @@ class OPDS1Entry(OPDS1EntryLinksMixin):
         """Get Author names."""
         if not self.metadata:
             return []
-        people = get_contributor_people(self.obj.ids, AUTHOR_ROLES)
+        people = get_contributor_people(self.obj.ids, AUTHOR_ROLES, exclude=False)
         return self._add_url_to_obj(people, "contributors")
 
     @property

codex/views/opds/v1/entry/links.py

@@ -20,7 +20,7 @@ class OPDS1EntryLinksMixin:
     """OPDS v1 Entry Links Methods."""
 
     def __init__(
-        self, obj, query_params, data: OPDS1EntryData, title_filename_fallback=False
+        self, obj, query_params, data: OPDS1EntryData, title_filename_fallback: bool
     ):
         """Initialize params."""
         self.obj = obj
@@ -35,7 +35,6 @@ class OPDS1EntryLinksMixin:
     def _cover_link(self, rel):
         if self.fake:
             return None
-        # cover_pk = getattr(self.obj, "cover_pk", None)
         try:
             kwargs = {"group": self.obj.group, "pks": self.obj.ids}
             ts = floor(datetime.timestamp(self.obj.updated_at))
@@ -50,7 +49,7 @@ class OPDS1EntryLinksMixin:
         except Exception:
             LOG.exception("create thumb")
 
-    def _nav_href(self, metadata=False):
+    def _nav_href(self, metadata: bool):
         try:
             pks = sorted(self.obj.ids)
             kwargs = {"group": self.obj.group, "pks": pks, "page": 1}
@@ -72,7 +71,7 @@ class OPDS1EntryLinksMixin:
             LOG.exception(msg)
             raise
 
-    def _nav_link(self, metadata=False):
+    def _nav_link(self, metadata: bool):
         href = self._nav_href(metadata)
 
         group = self.obj.group
@@ -154,7 +153,7 @@ class OPDS1EntryLinksMixin:
 
             if self.obj.group == "c" and not self.fake:
                 result += self._links_comic()
-            elif nav := self._nav_link():
+            elif nav := self._nav_link(metadata=False):
                 result += [nav]
 
         except Exception:

codex/views/opds/v1/facets.py

@@ -169,11 +169,14 @@ class OPDS1FacetsView(BrowserView):
         )
         qps = {**self.request.GET}
         qps.update(query_params)
-        zero_pad: int = self.obj["zero_pad"]  # type: ignore
+        zero_pad: int = self.obj["zero_pad"]
         data = OPDS1EntryData(
-            self.opds_acquisition_groups, zero_pad, False, self.mime_type_map
+            self.opds_acquisition_groups,
+            zero_pad,
+            metadata=False,
+            mime_type_map=self.mime_type_map,
         )
-        return OPDS1Entry(entry_obj, qps, data)
+        return OPDS1Entry(entry_obj, qps, data, title_filename_fallback=False)
 
     def _is_facet_active(self, facet_group, facet):
         compare = [facet.value]
@@ -231,7 +234,8 @@ class OPDS1FacetsView(BrowserView):
                 facets += [facet_obj]
         return facets
 
-    def facets(self, entries=False, root=True):
+    def facets(self, entries: bool, root: bool):
+        # entries false, root true
         """Return facets."""
         facets = []
         group = self.kwargs.get("group")