fix(bibtex): klucz doktoratu i habilitacji znów zawiera nazwisko autora

[mpasternak]

Problem

Praca_Doktorska.autorzy_dla_opisu() zwraca FakeSet — listę udającą
queryset, bo doktorat ma dokładnie jednego autora w polu autor, a nie relację
wiele-do-wielu. FakeSet implementuje all(), select_related(),
exclude(), odpiete_dyscypliny(), exists() — ale nie first().

bpp/export/bibtex.py:93 woła właśnie first():

first_author = wydawnictwo.autorzy_dla_opisu().first()

→ AttributeError: 'FakeSet' object has no attribute 'first'
(Rollbar #447,
6 wystąpień).

Wyjątek jest tam połykany przez zaloguj_polkniety_wyjatek, więc eksport nie
padał — klucz cichcem gubił nazwisko i zostawało samo <rok>_id<pk>.

Rozwiązanie

FakeSet.first() zwracające pierwszy element albo None.

Testy

Nowy src/bpp/tests/test_export/test_bibtex_klucz_doktoratu.py — TDD, oba
testy najpierw padały ('2020_id…' != 'Kowalski_2020_id…').

• src/bpp/tests/test_export/ — 39 passed

Uwaga na przyszłość (NIE ruszam w tym PR)

FakeSet.exists() zwraca twarde False, mimo że lista ma jeden element.
To wygląda na celowe (jakiś kod pewnie sprawdza exists(), żeby nie renderować
listy autorów dla doktoratu), ale jest mylące. Zmiana wymagałaby przejrzenia
wszystkich wywołań autorzy_set.exists() — osobny temat.

🤖 Generated with Claude Code

https://claude.ai/code/session_01NcAqeqyqBzNEkkVnhpHDaH

[mpasternak]

Poprawki po self-review

Review trafnie zauważył, że pierwszy commit naprawiał objaw, nie przyczynę.
Naniesione w a7fa3c56a:

Przyczyna: autorzy_dla_opisu() zwraca cztery różne kształty, a
generate_bibtex_key zakładał queryset dla wszystkich. bpp/models/util.py:138:

kształt	ma first()?
queryset (autorzy_set.select_related(...))	tak
[] gdy not self.pk	nie
lista z Prefetch(to_attr=PREFETCH_AUTORZY_ATTR)	nie
FakeSet doktoratu/habilitacji	nie (przed tym PR-em)

Review oceniał ścieżkę listową jako „dziś nieosiągalną, bo dwa widoki się nie
spotykają". Zweryfikowałem — jest osiągalna. Nowy test
test_klucz_bibtex_dziala_gdy_autorzy_sa_zwykla_lista woła
prefetch_dane_strony_rekordu(wyd), asertuje że autorzy_dla_opisu() to
faktycznie list, i padał dokładnie tak samo jak doktorat:

AssertionError: assert '2019_id6388619' == 'Nowak_2019_id6388619'

Czyli dodanie samego FakeSet.first() zostawiłoby tę samą pułapkę obok.
generate_bibtex_key bierze teraz pierwszy element tym samym idiomem, co
bpp/models/util.py:180: first() gdy jest, inaczej next(iter(...), None).

FakeSet.exists() — zostaje False, ale z komentarzem. Review potwierdził
niezależnie to, co podejrzewałem: exists() jest osiągalne wyłącznie po
łańcuchu exclude(...) (Rekord.ma_odpiete_dyscypliny,
src/bpp/models/cache/rekord.py:344), a FakeSet.exclude() ignoruje warunki
i zwraca self. Gdyby exists() zwracało True, doktoraty dostałyby
ma_odpiete_dyscypliny=True i strona rekordu renderowałaby pusty box „odpięte
dyscypliny". Bez komentarza sąsiedztwo działającego first() i twardego
False w exists() czyta się jak sprzeczność.

do_rollbar=False w tym except — celowo NIE dodane. Review ma rację:
blok pozostaje osiągalny (nazwisko None → TypeError w re.sub, błędy bazy),
a item #447 był jedynym sygnałem, że coś jest nie tak. Hałas usuwamy naprawą
przyczyny, nie zatkaniem czujnika.

40 passed w src/bpp/tests/test_export/.

Do sprawdzenia po deployu

Review słusznie zwraca uwagę, że skoro istniały dwie ścieżki, część z 6 wystąpień
#447 mogła iść z listowej. Po wdrożeniu warto potwierdzić, że item faktycznie
przestał przyrastać — teraz obie ścieżki są pokryte, więc powinien.

Odnotowane, bez zmian

Drugi test (habilitacja) faktycznie przechodzi tą samą ścieżką co doktorat —
Praca_Habilitacyjna dziedziczy Praca_Doktorska_Baza i nie nadpisuje ani
autorzy_set, ani autorzy_dla_opisu. Zostawiam jako tani strażnik przed
przyszłym rozjazdem, ale nie jest to „drugi przypadek".