Rozdział: dziedziczenie wydawcy po wydawnictwie nadrzędnym + podświetlenie wymaganego pola (Freshdesk #385)#401
Merged
Merged
Conversation
…nym + podświetlenie (Freshdesk #385) Przy wprowadzaniu rozdziału (Wydawnictwo_Zwarte z wydawnictwem nadrzędnym) operatorzy łatwo przeoczali pole "wydawca". Dwie zmiany: 1. Dziedziczenie wydawcy (autorytatywnie po stronie serwera, bez JS): Wydawnictwo_ZwarteForm.clean() podstawia wydawcę z wydawnictwa nadrzędnego, gdy pole "wydawca" pozostaje puste. Jawnie wpisany wydawca NIE jest nadpisywany. Operator dostaje komunikat (warning) o odziedziczeniu. Logika wydzielona do testowalnego helpera _dziedzicz_wydawce_po_nadrzednym(). 2. Podświetlenie pola "wydawca" jako wymaganego przy rozdziale — warstwa wyłącznie wizualna (admin JS + CSS), aktywna gdy wskazano wydawnictwo nadrzędne, a pole "wydawca" jest puste. Dodano też help_text na polu. Brak migracji (zmiany tylko w formularzu/adminie). Testy pytest pokrywają: dziedziczenie przy pustym wydawcy, brak nadpisywania jawnie podanego, nadrzędne bez wydawcy oraz brak nadrzędnego. 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
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.
Co
Dla rozdziałów (rekord
Wydawnictwo_Zwarteze wskazanym wydawnictwem nadrzędnym) w adminie:Wydawnictwo_ZwarteForm.clean()podstawia wydawcę z wydawnictwa nadrzędnego. Jawnie wpisany wydawca nie jest nadpisywany. Operator dostaje komunikat (messages.warning) o odziedziczeniu. Logika wydzielona do testowalnego helpera_dziedzicz_wydawce_po_nadrzednym().help_textwyjaśniający mechanizm dziedziczenia.Dlaczego
Freshdesk #385: przy wprowadzaniu rozdziału system nie sygnalizował, że obok wydawnictwa należy podać także wydawcę — łatwo było to przeoczyć. Prośba zgłaszającego: (1) wyraźnie oznaczać wymagane pole „wydawca", (2) wprowadzić dziedziczenie wydawcy po wydawnictwie nadrzędnym z możliwością ręcznej korekty.
Jak testowano
uv run pytest src/bpp/tests/test_admin/test_wydawnictwo_zwarte.py— 5 passed. Nowe testy pytest pokrywają:Dodatkowo:
ruff format/ruff checkczysto,uv run python src/manage.py checkbez uwag,node --checkna pliku JS OK. Brak migracji (zmiany tylko w formularzu/adminie i statykach).Co odłożono
clean(), więc JS-owy prefill byłby tylko kosmetyką wymagającą nowego endpointu.🤖 Generated with Claude Code