Bramka regresyjna na N+1 w adminie (FETCH_RAISE) + dwa realne N+1

src/bpp/admin/jednostka.py

@@ -162,7 +162,39 @@ def get_import_resource_classes(self, request):
         "wydzial",
         "rodzaj",
         "uczelnia",
+        # kolumna ``parent_nazwa`` czyta ``item.parent.nazwa``
+        "parent",
     ]
+
+    def get_queryset(self, request):
+        """Wymuś ``list_select_related`` — ChangeList by go tutaj POMINĄŁ.
+
+        ``ChangeList.get_queryset`` aplikuje ``list_select_related``
+        warunkowo::
+
+            if not qs.query.select_related:
+                qs = self.apply_select_related(qs)
+
+        czyli TYLKO gdy queryset bazowy nie ma jeszcze ŻADNEGO
+        ``select_related``. A ``JednostkaManager.get_queryset()`` dokłada
+        ``.select_related("wydzial")`` (denormalizowany self-FK), więc
+        warunek jest fałszywy i CAŁA deklaracja wyżej przepada — do
+        zapytania trafia sam ``wydzial``, bez ``rodzaj`` i ``uczelnia``.
+
+        Efekt był niewidoczny gołym okiem, bo obie relacje i tak są
+        dotykane w każdym wierszu (``rodzaj`` to kolumna ``list_display``,
+        ``uczelnia`` czyta ``Jednostka.__str__`` sprawdzając
+        ``uzywaj_wydzialow``) — po prostu leniwie, per wiersz. Dokładamy je
+        wprost; ``select_related`` się scala, więc ``wydzial`` z managera
+        nie ginie.
+
+        Regresję pilnuje ``test_fetch_raise_gate.py`` (tryb ``FETCH_RAISE``
+        wywala się z nazwą pola, gdy changelista dotknie relacji spoza
+        deklaracji).
+        """
+        qs = super().get_queryset(request)
+        return qs.select_related(*self.list_select_related)
+
     fields = None
     list_filter = (
         WydzialFilter,

src/bpp/admin/wydawnictwo_zwarte.py

@@ -541,6 +541,11 @@ class Wydawnictwo_ZwarteAdmin(
         "wydawnictwo_nadrzedne": ["wydawnictwo_nadrzedne"],
         "wydawnictwo_nadrzedne_col": ["wydawnictwo_nadrzedne"],
         "wydawca": ["wydawca"],
+        # Kolumna ``wydawnictwo`` (DOMYŚLNIE widoczna) to property modelu
+        # ``get_wydawnictwo``, które sięga po ``self.wydawca.nazwa``. Bez
+        # tego wpisu JOIN na wydawcę wchodził tylko przy jawnie włączonej
+        # kolumnie ``wydawca`` — czyli praktycznie nigdy.
+        "wydawnictwo": ["wydawca"],
     }
 
     autocomplete_fields = [

src/bpp/newsfragments/admin-brakujace-joiny.bugfix.rst

@@ -0,0 +1,5 @@
+Listy jednostek oraz wydawnictw zwartych w panelu administracyjnym dociągały
+część danych osobno dla każdego wiersza, mimo że deklaracja w kodzie mówiła co
+innego. Lista jednostek pobierała pojedynczo rodzaj jednostki, uczelnię
+i jednostkę nadrzędną, a lista wydawnictw zwartych — wydawcę. Powiązania te są
+teraz pobierane jednym zapytaniem dla całej strony.