5.0: Update django.contrib.admin

Author: ngnpope

django-stubs/contrib/admin/__init__.pyi

@@ -15,6 +15,7 @@ from .filters import SimpleListFilter as SimpleListFilter
 from .options import HORIZONTAL as HORIZONTAL
 from .options import VERTICAL as VERTICAL
 from .options import ModelAdmin as ModelAdmin
+from .options import ShowFacets as ShowFacets
 from .options import StackedInline as StackedInline
 from .options import TabularInline as TabularInline
 from .sites import AdminSite as AdminSite

django-stubs/contrib/admin/exceptions.pyi

@@ -2,3 +2,5 @@ from django.core.exceptions import SuspiciousOperation
 
 class DisallowedModelAdminLookup(SuspiciousOperation): ...
 class DisallowedModelAdminToField(SuspiciousOperation): ...
+class AlreadyRegistered(Exception): ...
+class NotRegistered(Exception): ...

django-stubs/contrib/admin/filters.pyi

@@ -4,10 +4,12 @@ from typing import Any, ClassVar
 
 from django.contrib.admin.options import ModelAdmin
 from django.contrib.admin.views.main import ChangeList
+from django.db.models.aggregates import Count
 from django.db.models.base import Model
 from django.db.models.fields import Field
 from django.db.models.fields.related import RelatedField
 from django.db.models.query import QuerySet
+from django.db.models.query_utils import Q
 from django.http.request import HttpRequest
 from django.utils.datastructures import _ListOrTuple
 from django.utils.functional import _StrOrPromise
@@ -32,13 +34,17 @@ class ListFilter:
     def queryset(self, request: HttpRequest, queryset: QuerySet) -> QuerySet | None: ...
     def expected_parameters(self) -> list[str | None]: ...
 
-class SimpleListFilter(ListFilter):
+class FacetsMixin:
+    def get_facet_counts(self, pk_attname: str, filtered_qs: QuerySet[Model]) -> dict[str, Count]: ...
+    def get_facet_queryset(self, changelist: ChangeList) -> dict[str, int]: ...
+
+class SimpleListFilter(FacetsMixin, ListFilter):
     parameter_name: str | None
     lookup_choices: list[tuple[str, _StrOrPromise]]
     def value(self) -> str | None: ...
     def lookups(self, request: HttpRequest, model_admin: ModelAdmin) -> Iterable[tuple[str, _StrOrPromise]] | None: ...
 
-class FieldListFilter(ListFilter):
+class FieldListFilter(FacetsMixin, ListFilter):
     list_separator: ClassVar[str]
     field: Field
     field_path: str
@@ -117,3 +123,4 @@ class RelatedOnlyFieldListFilter(RelatedFieldListFilter): ...
 class EmptyFieldListFilter(FieldListFilter):
     lookup_kwarg: str
     lookup_val: str | None
+    def get_lookup_condition(self) -> Q: ...

django-stubs/contrib/admin/options.pyi

@@ -38,6 +38,7 @@ from typing_extensions import TypeAlias, TypedDict
 
 IS_POPUP_VAR: str
 TO_FIELD_VAR: str
+IS_FACETS_VAR: str
 HORIZONTAL: Literal[1]
 VERTICAL: Literal[2]
 

django-stubs/contrib/admin/sites.pyi

@@ -28,9 +28,6 @@ else:
 _ViewType = TypeVar("_ViewType", bound=Callable[..., HttpResponse])
 _ActionCallback: TypeAlias = Callable[[ModelAdmin, HttpRequest, QuerySet], TemplateResponse | None]
 
-class AlreadyRegistered(Exception): ...
-class NotRegistered(Exception): ...
-
 class AdminSite:
     site_title: _StrOrPromise
     site_header: _StrOrPromise

django-stubs/contrib/admin/utils.pyi

@@ -13,6 +13,7 @@ from django.db.models.deletion import Collector
 from django.db.models.fields import Field, reverse_related
 from django.db.models.options import Options
 from django.db.models.query import QuerySet
+from django.db.models.query_utils import Q
 from django.forms.forms import BaseForm
 from django.forms.formsets import BaseFormSet
 from django.http.request import HttpRequest
@@ -24,7 +25,11 @@ _T = TypeVar("_T")
 class FieldIsAForeignKeyColumnName(Exception): ...
 
 def lookup_spawns_duplicates(opts: Options, lookup_path: str) -> bool: ...
-def prepare_lookup_value(key: str, value: datetime.datetime | str) -> bool | datetime.datetime | str: ...
+def get_last_value_from_parameters(parameters: dict[str, list[str] | str], key: str) -> str | None: ...
+def prepare_lookup_value(
+    key: str, value: list[bool | datetime.datetime | str] | datetime.datetime | str, separator: str
+) -> list[bool | datetime.datetime | str] | bool | datetime.datetime | str: ...
+def build_q_object_from_lookup_parameters(parameters: dict[str, list[str]]) -> Q: ...
 def quote(s: int | str | UUID) -> str: ...
 def unquote(s: str) -> str: ...
 def flatten(fields: Any) -> list[Callable | str]: ...

django-stubs/contrib/admin/views/main.pyi

@@ -77,7 +77,7 @@ class ChangeList:
     def get_ordering_field(self, field_name: Callable | str) -> Expression | str | None: ...
     def get_ordering(self, request: HttpRequest, queryset: QuerySet) -> list[Expression | str]: ...
     def get_ordering_field_columns(self) -> dict[int, Literal["desc", "asc"]]: ...
-    def get_queryset(self, request: HttpRequest) -> QuerySet: ...
+    def get_queryset(self, request: HttpRequest, exclude_parameters: list[str | None] | None = ...) -> QuerySet: ...
     filter_specs: list[ListFilter]
     has_filters: bool
     has_active_filters: bool

scripts/stubtest/allowlist_todo_django50.txt

@@ -2,30 +2,6 @@
 # Only discrepancies that appeared after Django 4.2 -> 5.0 update.
 # Unsorted: there are real problems and things we can really ignore.
 
-django.contrib.admin.AllValuesFieldListFilter.get_facet_counts
-django.contrib.admin.BooleanFieldListFilter.get_facet_counts
-django.contrib.admin.ChoicesFieldListFilter.get_facet_counts
-django.contrib.admin.DateFieldListFilter.get_facet_counts
-django.contrib.admin.EmptyFieldListFilter.get_facet_counts
-django.contrib.admin.EmptyFieldListFilter.get_lookup_condition
-django.contrib.admin.RelatedFieldListFilter.get_facet_counts
-django.contrib.admin.ShowFacets
-django.contrib.admin.SimpleListFilter.get_facet_counts
-django.contrib.admin.exceptions.AlreadyRegistered
-django.contrib.admin.exceptions.NotRegistered
-django.contrib.admin.filters.AllValuesFieldListFilter.get_facet_counts
-django.contrib.admin.filters.BooleanFieldListFilter.get_facet_counts
-django.contrib.admin.filters.ChoicesFieldListFilter.get_facet_counts
-django.contrib.admin.filters.DateFieldListFilter.get_facet_counts
-django.contrib.admin.filters.EmptyFieldListFilter.get_facet_counts
-django.contrib.admin.filters.EmptyFieldListFilter.get_lookup_condition
-django.contrib.admin.filters.FacetsMixin
-django.contrib.admin.filters.RelatedFieldListFilter.get_facet_counts
-django.contrib.admin.filters.SimpleListFilter.get_facet_counts
-django.contrib.admin.options.IS_FACETS_VAR
-django.contrib.admin.utils.build_q_object_from_lookup_parameters
-django.contrib.admin.utils.get_last_value_from_parameters
-django.contrib.admin.views.main.ChangeList.get_queryset
 django.contrib.contenttypes.fields.GenericForeignKey.get_content_type
 django.contrib.contenttypes.fields.GenericForeignKey.get_prefetch_querysets
 django.contrib.contenttypes.prefetch