feat(admin): filtr "PBN: status" (skasowany/aktywny/brak) dla wydawnictw#450
Merged
Conversation
Nowy SimpleListFilter (PBNStatusFilter) traversujacy pbn_uid__status, dodany do list_filter adminow Wydawnictwo_Ciagle i Wydawnictwo_Zwarte. Trzy opcje: skasowany w PBN (DELETED), aktywny (ACTIVE), brak powiazania (pbn_uid puste). Ulatwia znalezienie rekordow BPP powiazanych z praca usunieta po stronie PBN. Test regresyjny potwierdza takze, ze ta sama sciezka dziala w DjangoQL admina (pbn_uid.status = "DELETED") — pbn_api nie jest wykluczone z BppQLSchema. Testy TDD: test_admin/test_pbn_status_filter.py. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
mpasternak
added a commit
that referenced
this pull request
Jul 7, 2026
…PBN" dla wyd. ciągłych (#464) Uzupełnia PR #450/#443 o dwa brakujące kryteria filtrowania po statusie PBN: 1. Wydawnictwo ciągłe — nowy filtr "Źródło: status w PBN" (ZrodloUsunieteWPBNFilter), traversuje zrodlo.pbn_uid.status. Wyłapuje prace, których ŹRÓDŁO jest skasowane w PBN, niezależnie od statusu samego rekordu. 2. Źródła — podpięcie istniejącego PBNStatusFilter do ZrodloAdmin (Zrodlo.pbn_uid → Journal, który też ma .status). PBNStatusFilter uogólniony na parametr `pbn_relation` (domyślnie "pbn_uid"), w pełni wstecznie kompatybilny — istniejące testy przechodzą bez zmian. Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Po co
Po zmergowaniu #449 (import nie tworzy już rekordów BPP dla prac DELETED w PBN) pojawiła się potrzeba znalezienia rekordów, które powstały wcześniej — powiązanych z pracą skasowaną po stronie PBN. W adminie
Wydawnictwo_Ciagle/Zwartebył tylko filtr „PBN UID obecny/brak", bez statusu.Co
Nowy
PBNStatusFilter(bpp/admin/filters.py) traversującypbn_uid__status, dodany dolist_filterobu adminów. Trzy opcje:DELETED)ACTIVE)pbn_uidpuste)DjangoQL — działa już bez zmian
Przy okazji zweryfikowałem (test regresyjny):
pbn_uid.status = "DELETED"w polu DjangoQL admina waliduje się i filtruje poprawnie.pbn_apinie jest wykluczone zBppQLSchema, więc traversal popbn_uidjest dostępny. Test pilnuje, że przyszłe uszczelnianie schematu tej ścieżki nie odetnie.Testy (TDD)
src/bpp/tests/test_admin/test_pbn_status_filter.py— RED (ImportError) → GREEN. 8 testów: deleted/active/brak dlaWydawnictwo_Ciagle, kontrola pustej wartości,Wydawnictwo_Zwarte, obecność wlist_filterobu adminów, oraz regresja DjangoQL. Lokalnie 8/8, ruff czysty. Bez migracji.🤖 Generated with Claude Code