From 7ce9e859c240ec3612d032e37d4131e47dd196e1 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Micha=C5=82=20Pasternak?= <michal.dtz@gmail.com>
Date: Sun, 26 Jul 2026 17:14:42 +0200
Subject: [PATCH 01/11] docs(import_pracownikow): spec opcji nadpisywania dat
 zatrudnienia z pliku [ci skip]

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
---
 ...rt-nadpisywanie-dat-zatrudnienia-design.md | 291 ++++++++++++++++++
 1 file changed, 291 insertions(+)
 create mode 100644 docs/superpowers/specs/2026-07-26-import-nadpisywanie-dat-zatrudnienia-design.md

diff --git a/docs/superpowers/specs/2026-07-26-import-nadpisywanie-dat-zatrudnienia-design.md b/docs/superpowers/specs/2026-07-26-import-nadpisywanie-dat-zatrudnienia-design.md
new file mode 100644
index 000000000..ffc5d0b8d
--- /dev/null
+++ b/docs/superpowers/specs/2026-07-26-import-nadpisywanie-dat-zatrudnienia-design.md
@@ -0,0 +1,291 @@
+# Import pracowników: opcja „Nadpisuj daty zatrudnienia (od/do) wartościami z pliku"
+
+Data: 2026-07-26
+Status: projekt (do akceptacji)
+
+## 1. Problem
+
+Scenariusz zgłoszony przez użytkownika: tydzień temu wykonano import
+pracowników z pliku **bez dat** — wszyscy zaimportowani dostali „datę od"
+= 19 lipca (patrz §2.4: fallback „data zmian → dziś" dla nowych okresów).
+Teraz jest do zaimportowania poprawiony plik, w którym „data od" bywa
+np. 5 lat wcześniejsza.
+
+Pytanie brzmiało: czy ponowny import **zmodyfikuje** istniejącą datę, czy
+**doda nowe miejsce pracy** (w tej samej jednostce)? Odpowiedź (zweryfikowana
+w kodzie, §2): **ani jedno, ani drugie** — import wyceluje w istniejący
+otwarty okres, pokaże różnicę dat w podglądzie, ale przy zapisie jej
+**nie zastosuje**. Poprawiony plik niczego więc nie naprawi.
+
+Pierwotny pomysł („wyczyść daty od/do w całej bazie" + import wypełnia
+NULL-e) odrzucono na rzecz bezpieczniejszej opcji **nadpisywania dat
+wartościami z pliku** — dotyka wyłącznie osób obecnych w pliku, nikt inny
+nie traci dat (decyzja użytkownika w rozmowie projektowej).
+
+## 2. Aktualne zachowanie (stan na dziś, zweryfikowane w kodzie)
+
+### 2.1. Skąd biorą się daty w wierszu importu
+
+Kolumny pliku mapują się (ekran mapowania) na pola wewnętrzne
+`data_zatrudnienia` („Data zatrudnienia") i `data_końca_zatrudnienia`
+(„Data końca zatrudnienia") — `mapping.py:29-30`; „Data od"/„Data do" to
+synonimy nagłówków kolumn pliku (`mapping.py:116-117`). Wiersz odczytuje
+je przez `_plik_od()` / `_plik_do()`
+(`src/import_pracownikow/models.py:1020-1033`) — kontrakt: zawsze
+`date | None`, nigdy string.
+
+### 2.2. Wybór okresu docelowego (resolver)
+
+`rozwiaz_okres_zatrudnienia(autor, jednostka, plik_od)`
+(`src/import_pracownikow/okresy.py:73-123`) — „data od" jest TOŻSAMOŚCIĄ
+okresu (spójne z `unique_together = (autor, jednostka, rozpoczal_prace)`).
+Kolejność rozstrzygania przy niepustym `plik_od`:
+
+1. **Dokładne trafienie** — istnieje AJ z `rozpoczal_prace == plik_od`
+   → celuj w niego (`okresy.py:98-100`).
+2. **Okres niedatowany** — istnieje AJ z `rozpoczal_prace IS NULL`
+   → celuj w niego („do wypełnienia", `okresy.py:101-106`).
+3. **Okres OTWARTY** (`zakonczyl_prace IS NULL`) — celuj w najświeższy
+   aktywny okres; różnica „daty od" jest POKAZYWANA w podglądzie, ale NIE
+   powstaje nowy okres i stary NIE jest domykany (świadoma decyzja „nowy
+   okres tylko gdy stary zamknięty", `okresy.py:107-115`).
+4. **Wszystkie okresy zamknięte** (osoba odeszła i wraca) → NOWY okres
+   z datą z pliku (`okresy.py:116-118`).
+
+Pusty `plik_od` → celuj w najświeższy aktywny/istniejący okres, a bez
+żadnego → nowy okres bez daty (`okresy.py:120-123`).
+
+### 2.3. Zapis dat na istniejącym okresie: wypełnij-tylko-NULL
+
+`_integruj_daty_aj` (`src/import_pracownikow/models.py:1333-1357`):
+
+- „Data od": zapis **wyłącznie gdy** `rozpoczal_prace IS NULL` w bazie
+  a plik niesie datę. Istniejąca data **nigdy nie jest nadpisywana**.
+- „Data do": identycznie — wstaw-tylko-gdy-pusta; różnica wobec
+  istniejącej daty jest pokazywana w porównywarce, ale nie zapisywana.
+- Pusta komórka w pliku → nic nie zmieniaj (nawet nie wypełnia NULL-a
+  „datą zmian").
+
+Po ustawieniu dat, PRZED save, walidowany jest niezmiennik
+`rozpoczal_prace < zakonczyl_prace` (`models.py:1370-1388`) — naruszenie
+→ `BPPDatabaseError` → wiersz izolowany (reszta importu idzie dalej).
+Walidacja celowo w Pythonie przed save, bo DB-owy CHECK
+(`poczatek_przed_koncem`, mig. bpp 0469) wewnątrz transakcji dałby
+nieizolowany `CheckViolation`.
+
+### 2.4. Nowe okresy i pochodzenie „19 lipca"
+
+Nowy AJ tworzy `_materializuj_diff`
+(`src/import_pracownikow/pipeline/integrate.py:67-126`). Gdy plik nie
+niesie „daty od", nowy okres dostaje fallback **`data_zmian_personalnych`
+importu, a w jej braku — dzisiejszą datę** (`integrate.py:98-106`).
+Dokładnie stąd wszyscy z zeszłotygodniowego importu (plik bez dat) mają
+`rozpoczal_prace` = 19 lipca. Świeży AJ ma od razu konkretną datę, więc
+gałąź „wypełnij NULL" z §2.3 już go nie stempluje.
+
+### 2.5. Podgląd (porównywarka)
+
+`_porownaj_data` (`models.py:1001-1018`) oznacza pole jako „zmienione"
+(`rozne=True`) także wtedy, gdy obie strony są niepuste i różne — z
+docstringiem wprost: „różnica pokazana **bez nadpisania**". UWAGA:
+„zmienione" obejmuje też wypełnienie NULL-a i nowy okres (`models.py:1017`,
+ekstraktory `_stan_data_od/_do` w `roznice.py:65-84`) — to istotne dla
+liczenia N w §3.5. Stany pól zasilają siatkę porównań i filtr stanu
+(`stany_pol`, `models.py:1204-1219`; liczenie `stany_pol_live`
+`models.py:1177-1190`); zamrożenie w `stany_pol_snapshot` następuje NA
+POCZĄTKU integracji, przed zmianą bazy (`integrate.py:203`), i stan „po
+integracji" jest serwowany z tego snapshotu.
+
+### 2.5a. Bramka „zmiany potrzebne" (kluczowe dla projektu)
+
+Integracja wiersza wykonuje się tylko, gdy analiza uznała, że są zmiany
+do zapisania: `_check_autor_jednostka_needs_update` (`models.py:1259-1266`)
+liczy daty **wyłącznie jako wypełnienie NULL-a** — różnica „baza ma datę,
+plik ma inną" NIE ustawia `zmiany_potrzebne` (`analyze.py:766`,
+`pewnosc.py:134-137`), wiersz nie wchodzi do `zmiany_potrzebne_set`
+(`models.py:399-400`), a nawet włączony do integracji odpadłby na świeżym
+re-checku (`integrate.py:227` → `pominiety_bo_nieaktualny`). Wniosek:
+samo nadpisywanie w `_integruj_daty_aj` NIE wystarczy — bramka musi być
+świadoma flagi (§3.3a).
+
+### 2.6. Constrainty bazodanowe na `Autor_Jednostka`
+
+`src/bpp/models/autor.py:769-809`:
+
+- `unique_together (autor, jednostka, rozpoczal_prace)`;
+- częściowy `UniqueConstraint` „jeden niedatowany okres na parę"
+  (`rozpoczal_prace IS NULL`);
+- `ExclusionConstraint` `bpp_autor_jednostka_okresy_bez_nakladan`:
+  datowane okresy tej samej pary autor+jednostka NIE mogą się nakładać
+  (GiST, `daterange(rozpoczal_prace, zakonczyl_prace)` z `&&`).
+
+`daterange(rozpoczal_prace, zakonczyl_prace)` w constraincie ma granice
+DOMKNIĘTE `'[]'` (`autor.py:713-732`), a `zakonczyl_prace IS NULL` daje
+przedział otwarty w prawo `[od, ∞)`.
+
+Konsekwencja dla projektu: cofnięcie „daty od" wstecz może najechać na
+wcześniejszy zamknięty okres → trzeba to walidować w Pythonie PRZED save
+(jak w §2.3 — naruszenie constraintu psuje transakcję i izolację wiersza).
+
+### 2.6a. Inne miejsca pipeline'u dotykające dat AJ
+
+- **Odpięcia** (`integrate.py:264-313`, `_wykonaj_odpiecia`) ustawiają
+  `zakonczyl_prace` (= wczoraj) parom autor+jednostka SPOZA pliku —
+  rozłączne z projektowaną flagą (guard G1 pomija pary obecne w pliku).
+- **Defragmentacja** (`Autor.save()` → `defragmentuj_jednostke`,
+  `autor.py:431-438`, scalanie `autor.py:620-710`) modyfikuje daty i
+  KASUJE przyległe AJ; biegnie wewnątrz integracji (`_integrate_autor` →
+  `a.save()`) tuż PRZED `_integruj_daty_aj`. Interakcja z flagą: gdy
+  nadpisana „data od" uczyni okres przyległym (dzień-po-dniu) do
+  zamkniętego okresu, przejdzie constraint i pre-check, a scalenie nastąpi
+  przy najbliższym `Autor.save()` — akceptowalne (dla świeżych okresów
+  istnieje już mechanizm `_przepnij_aj_po_defragmentacji`,
+  `models.py:1451-1484`).
+- `pewnosc.odtworz_autor_jednostka` nie pisze dat na istniejących AJ
+  (odkłada tylko datę nowego okresu w diffie, `pewnosc.py:142`) — bez
+  konfliktu.
+
+### 2.7. Wniosek
+
+W scenariuszu z §1 (otwarty okres z 19 lipca, plik z datą wcześniejszą):
+resolver celuje w istniejący okres (§2.2 pkt 3), podgląd pokazuje różnicę
+(§2.5), integracja daty nie zmienia (§2.3). **Potrzebna jest opcja
+nadpisywania.**
+
+## 3. Projektowane rozwiązanie
+
+### 3.1. Model i migracja
+
+Nowe pole `ImportPracownikow.nadpisuj_daty_zatrudnienia`
+(`BooleanField`, `default=False`, verbose_name „Nadpisuj daty zatrudnienia
+(od/do) wartościami z pliku") + migracja `import_pracownikow.0028`.
+Flaga per-import → zakres z natury ograniczony do osób z pliku (bez ryzyka
+multi-hosted). Baseline odświeżamy raz, przy scalaniu (polityka repo).
+
+### 3.2. Formularz nowego importu: szuflada + popup
+
+- Pole dochodzi do `NowyImportForm` (`src/import_pracownikow/forms.py`)
+  i ląduje w istniejącym zwiniętym `<details>` obok
+  `przepnij_wszystkie_prace`; nagłówek szuflady zmienia się na ogólne
+  „Opcje zaawansowane".
+- W `src/import_pracownikow/templates/import_pracownikow/`
+  `importpracownikow_form.html` rozszerzamy istniejący wzorzec JS
+  (`importpracownikow_form.html:26-45`, confirm na
+  `id_przepnij_wszystkie_prace`): przy ZAZNACZANIU checkboxa `confirm(...)`
+  z ostrzeżeniem; anulowanie odznacza pole. Odznaczanie — bez pytania.
+
+### 3.3. Semantyka integracji (`_integruj_daty_aj`)
+
+Flaga czytana z `self.parent.nadpisuj_daty_zatrudnienia`:
+
+- **OFF** → zachowanie dzisiejsze, bit w bit (§2.3).
+- **ON** → gdy plik NIESIE datę różną od bazy, data na okresie docelowym
+  zostaje nadpisana — osobno „od", osobno „do" — z wpisem w `log_zmian`
+  („data rozpoczęcia pracy: X → Y (nadpisano z pliku)").
+- Pusta komórka w pliku NIGDY nie kasuje daty z bazy (zasada „import
+  ustawia, nigdy nie kasuje" — spójna z tytułem/stopniem, §11.2 starego
+  specu).
+- Wybór okresu docelowego (resolver, §2.2) — BEZ ZMIAN. W scenariuszu
+  z §1: data 19 lipca na istniejącym otwartym okresie zostaje zastąpiona
+  datą z pliku; nie powstaje duplikat miejsca pracy.
+- Gałąź „nowy okres" (§2.2 pkt 4 i §2.4) — BEZ ZMIAN (nowy AJ i tak
+  dostaje daty z pliku).
+
+### 3.3a. Bramka `zmiany_potrzebne` świadoma flagi (WARUNEK KONIECZNY)
+
+Bez tej zmiany feature jest martwy dla scenariusza tytułowego (§2.5a):
+wiersz, którego JEDYNĄ różnicą jest data, ma dziś `zmiany_potrzebne=False`
+i integracja go pomija. Zmiany:
+
+- `_check_autor_jednostka_needs_update` (`models.py:1259-1266`): przy
+  fladze ON różnica dat „obie strony niepuste i różne" liczy się jako
+  zmiana potrzebna (przy OFF — jak dziś: tylko wypełnienie NULL-a);
+- przez to samo przechodzi analiza (`analyze.py:766`, `pewnosc.py:134-137`)
+  i świeży re-check przy integracji (`integrate.py:227`) — wiersz z samą
+  korektą dat wchodzi do `zmiany_potrzebne_set` (`models.py:399-400`)
+  i nie kończy jako `pominiety_bo_nieaktualny`.
+
+### 3.4. Walidacja przed zapisem
+
+- Istniejący check `od < do` (§2.3) obejmuje wartości po nadpisaniu —
+  bez zmian, tylko test.
+- NOWY pythonowy pre-check nakładania się okresów: po nadpisaniu dat
+  porównaj przedział z pozostałymi datowanymi okresami tej samej pary
+  autor+jednostka — lustro `ExclusionConstraint` z §2.6 z IDENTYCZNĄ
+  semantyką granic: przedziały DOMKNIĘTE `[od, do]`, a `zakonczyl_prace
+  IS NULL` = otwarty w prawo `[od, ∞)`. Kolizja → `BPPDatabaseError`
+  z komunikatem wskazującym kolidujący okres → wiersz izolowany, reszta
+  importu idzie dalej.
+- Warunek uruchomienia pre-checku (jednoznacznie): TYLKO gdy flaga ON
+  faktycznie NADPISAŁA niepustą wartość w bazie inną wartością z pliku.
+  Wypełnienie NULL-a (także przy fladze ON) idzie dzisiejszą ścieżką BEZ
+  pre-checku — to zachowanie istnieje dziś i celowo go nie zmieniamy.
+
+### 3.5. Ostrzeżenie w końcowym formularzu zapisu
+
+`src/import_pracownikow/templates/import_pracownikow/przeglad.html`,
+sekcja zapisu osób (Krok 2):
+
+- Gdy flaga ON: callout alert (Foundation; ikona `fi-icon`, NIE emoji —
+  to publiczny frontend): „Włączono nadpisywanie dat zatrudnienia:
+  **N** wierszy ma daty różne od bazy — zostaną nadpisane wartościami
+  z pliku."
+- Liczenie N — UWAGA na semantykę: stan „zmienione" ze `stany_pol`
+  obejmuje też wypełnienia NULL-i i nowe okresy (§2.5), które NIE są
+  nadpisaniami. N liczymy jako „obie strony niepuste i różne" (dla
+  `data_od` lub `data_do`), w widoku podglądu (`PodgladImportuView`,
+  `views.py:944-1039`), live z `wstepnie_zaladuj_okresy` (przeciw N+1;
+  `stany_pol_snapshot` bywa NULL do backfillu — `views.py:826-828` —
+  więc SQL po snapshotcie by niedoszacował).
+- `onsubmit`-owy `confirm` przycisku końcowego zapisu dostaje tę samą
+  treść, połączoną z istniejącym ostrzeżeniem o pominiętych wierszach
+  (`przeglad.html:205`).
+
+### 3.6. Testy
+
+Pytest (funkcje, `baker`, bez klas):
+
+1. flaga ON: „data od" i „data do" nadpisane na istniejącym otwartym
+   okresie (scenariusz §1: 19 lipca → data sprzed lat), wpisy w
+   `log_zmian`;
+2. bramka (§3.3a): wiersz z JEDYNĄ różnicą w datach — przy ON
+   `zmiany_potrzebne=True`, integracja go wykonuje (nie kończy jako
+   `pominiety_bo_nieaktualny`); przy OFF `zmiany_potrzebne=False` jak
+   dziś;
+3. flaga OFF: zachowanie dzisiejsze (data zostaje);
+4. pusta komórka przy ON: data w bazie NIE jest kasowana;
+5. kolizja przedziałów przy ON (wcześniejszy zamknięty okres) →
+   `BPPDatabaseError`, wiersz izolowany; wypełnienie NULL-a nie odpala
+   pre-checku;
+6. `od >= do` po nadpisaniu → `BPPDatabaseError` (istniejący mechanizm);
+7. formularz: pole w szufladzie, default OFF;
+8. podgląd: callout z poprawnym N (tylko realne nadpisania: obie strony
+   niepuste i różne; NIE wypełnienia NULL-i ani nowe okresy) renderuje
+   się tylko przy fladze ON;
+9. confirm końcowego zapisu zawiera ostrzeżenie przy fladze ON.
+
+### 3.7. Drobiazgi
+
+- Newsfragment `feature` po polsku w `src/bpp/newsfragments/`
+  (kanoniczny katalog).
+- Dokumentacja użytkownika importu — jeśli istnieje strona o imporcie
+  pracowników w `docs/`, dopisać akapit o opcji.
+
+## 4. Poza zakresem (YAGNI)
+
+- Czyszczenie dat całej bazy/uczelni (pierwotny pomysł — zbędny przy
+  nadpisywaniu; kasowałby daty osobom spoza pliku).
+- Per-wierszowe decyzje „nadpisz/nie nadpisuj".
+- Zmiany w resolverze okresów (`okresy.py`) i w fallbacku „data zmian →
+  dziś" dla nowych okresów.
+- Kasowanie dat pustą komórką pliku.
+
+## 5. Decyzje projektowe (z rozmowy)
+
+1. Nadpisywanie datami z pliku zamiast czyszczenia całej bazy —
+   wybór użytkownika po przedstawieniu zweryfikowanego zachowania (§2).
+2. Opcja umieszczona w szufladzie formularza NOWEGO importu (nie w
+   końcowym formularzu) — spójne z `przepnij_wszystkie_prace`; podgląd
+   od początku wie, że różnice dat będą nadpisane.
+3. Potwierdzenie: popup przy zaznaczaniu + callout i confirm w końcowym
+   formularzu zapisu osób.

From 3276c0a4a189b8d5fb7c4fe0d1ec4fff7c84809b Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Micha=C5=82=20Pasternak?= <michal.dtz@gmail.com>
Date: Sun, 26 Jul 2026 17:41:15 +0200
Subject: [PATCH 02/11] docs(import_pracownikow): plan implementacji
 nadpisywania dat zatrudnienia [ci skip]

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
---
 ...26-import-nadpisywanie-dat-zatrudnienia.md | 1086 +++++++++++++++++
 1 file changed, 1086 insertions(+)
 create mode 100644 docs/superpowers/plans/2026-07-26-import-nadpisywanie-dat-zatrudnienia.md

diff --git a/docs/superpowers/plans/2026-07-26-import-nadpisywanie-dat-zatrudnienia.md b/docs/superpowers/plans/2026-07-26-import-nadpisywanie-dat-zatrudnienia.md
new file mode 100644
index 000000000..ee70daf0b
--- /dev/null
+++ b/docs/superpowers/plans/2026-07-26-import-nadpisywanie-dat-zatrudnienia.md
@@ -0,0 +1,1086 @@
+# Import: nadpisywanie dat zatrudnienia z pliku — plan implementacji
+
+> **For agentic workers:** REQUIRED SUB-SKILL: Use superpowers:subagent-driven-development (recommended) or superpowers:executing-plans to implement this plan task-by-task. Steps use checkbox (`- [ ]`) syntax for tracking.
+
+**Goal:** Opcja per-import „Nadpisuj daty zatrudnienia (od/do) wartościami
+z pliku" — schowana w szufladzie formularza, z confirm-em przy zaznaczaniu
+i ostrzeżeniem (callout + confirm) w końcowym formularzu zapisu osób.
+
+**Architecture:** Flaga `BooleanField` na `ImportPracownikow`; bramka
+`zmiany_potrzebne` i `_integruj_daty_aj` stają się flag-aware; pythonowy
+pre-check nakładania okresów (lustro `ExclusionConstraint`) przed save;
+licznik realnych nadpisań zasila ostrzeżenie finalizacji.
+
+**Tech Stack:** Django 4.x, pytest + model_bakery, crispy-forms
+(Foundation), szablony Django.
+
+**Spec:** `docs/superpowers/specs/2026-07-26-import-nadpisywanie-dat-zatrudnienia-design.md`
+
+## Global Constraints
+
+- ZAWSZE `uv run` przed każdym poleceniem Pythona (`uv run pytest`, `uv run python src/manage.py ...`).
+- Testy: pytest, funkcje bez klas, `@pytest.mark.django_db`, `model_bakery.baker.make`. NIGDY unittest.TestCase.
+- Pytest z `-n auto` przy pełnych przebiegach; pojedyncze testy mogą iść bez.
+- Wyjście testów ZAWSZE do pliku: `... 2>&1 | tee /tmp/wynik.log` — potem grep; nie uruchamiaj tego samego przebiegu dwa razy.
+- Max 88 znaków linii (ruff). Po edycjach: `ruff format <pliki>` i `ruff check <pliki>` (tylko zmienione pliki, nigdy `--all-files`).
+- Komentarze szablonów Django `{# ... #}` są JEDNOLINIOWE — każda linia z własnym `{#` i `#}`.
+- Publiczny frontend = Foundation Icons (`<span class="fi-...">`), NIE emoji.
+- NIE modyfikować istniejących (wydanych) migracji; nowa migracja = `0028`.
+- Newsfragment: `src/bpp/newsfragments/<slug>.feature.rst`, po polsku.
+- Baseline bazy NIE odświeżać na gałęzi — raz, przy scalaniu (`make baseline-update`).
+- Praca na gałęzi `feat/import-nadpisywanie-dat` od `dev`; commity małe, po każdym tasku.
+- Nazwy z specu (spójność międzytaskowa): pole modelu `nadpisuj_daty_zatrudnienia`; metoda wiersza `nadpisze_daty()`; metoda parenta `liczba_nadpisan_dat()`; metoda pre-checku `_sprawdz_nakladanie_okresow(aj)`.
+
+---
+
+### Task 0: Gałąź robocza
+
+**Files:** brak zmian w plikach.
+
+- [ ] **Step 1: Utwórz gałąź od dev**
+
+```bash
+cd /Users/mpasternak/Programowanie/bpp
+git checkout dev && git pull --ff-only && git checkout -b feat/import-nadpisywanie-dat
+```
+
+---
+
+### Task 1: Pole modelu + migracja 0028
+
+**Files:**
+- Modify: `src/import_pracownikow/models.py` (klasa `ImportPracownikow`, po polu `przepnij_wszystkie_prace`, ok. linii 171)
+- Create: `src/import_pracownikow/migrations/0028_nadpisuj_daty_zatrudnienia.py` (generowana)
+- Test: `src/import_pracownikow/tests/test_nadpisywanie_dat.py` (nowy plik)
+
+**Interfaces:**
+- Produces: `ImportPracownikow.nadpisuj_daty_zatrudnienia: bool` (default `False`) — czytane w Taskach 2-6.
+
+- [ ] **Step 1: Napisz failujący test**
+
+Utwórz `src/import_pracownikow/tests/test_nadpisywanie_dat.py`:
+
+```python
+"""Opcja „Nadpisuj daty zatrudnienia (od/do) wartościami z pliku" —
+flaga modelu, bramka zmian, nadpisywanie w integracji, pre-check
+nakładania okresów, licznik ostrzeżenia finalizacji (spec
+2026-07-26-import-nadpisywanie-dat-zatrudnienia-design.md)."""
+
+import pytest
+from model_bakery import baker
+
+from import_pracownikow.models import ImportPracownikow
+
+
+@pytest.mark.django_db
+def test_flaga_nadpisywania_domyslnie_wylaczona():
+    parent = baker.make(ImportPracownikow)
+    assert parent.nadpisuj_daty_zatrudnienia is False
+```
+
+- [ ] **Step 2: Uruchom test — ma paść**
+
+```bash
+uv run pytest src/import_pracownikow/tests/test_nadpisywanie_dat.py -x 2>&1 | tee /tmp/t1.log | tail -5
+```
+
+Oczekiwane: FAIL/ERROR (`AttributeError` / pole nie istnieje).
+
+- [ ] **Step 3: Dodaj pole do modelu**
+
+W `src/import_pracownikow/models.py`, w klasie `ImportPracownikow`,
+bezpośrednio PO polu `przepnij_wszystkie_prace` (przed
+`zakres_integracji`):
+
+```python
+    nadpisuj_daty_zatrudnienia = models.BooleanField(
+        "Nadpisuj daty zatrudnienia (od/do) wartościami z pliku",
+        default=False,
+        # HTML w help_text (crispy renderuje przez |safe) — wzorzec jak w
+        # przepnij_wszystkie_prace. Ten sam string dosłownie w migracji 0028.
+        help_text="Gdy zaznaczone, daty rozpoczęcia i zakończenia pracy "
+        "<strong>ISTNIEJĄCYCH</strong> okresów zatrudnienia zostaną "
+        "<strong>NADPISANE</strong> wartościami z pliku — tam, gdzie plik "
+        "niesie datę różną od bazy.<br>"
+        "Użyj do KOREKTY dat (np. po wcześniejszym imporcie pliku bez "
+        "dat, który ostemplował wszystkich datą importu).<br>"
+        "Puste komórki pliku niczego nie kasują. Dotyczy wyłącznie osób "
+        "obecnych w pliku.",
+    )
+```
+
+- [ ] **Step 4: Wygeneruj migrację**
+
+```bash
+DJANGO_BPP_SKIP_DOTENV=1 uv run python src/manage.py makemigrations import_pracownikow -n nadpisuj_daty_zatrudnienia 2>&1 | tail -3
+```
+
+Oczekiwane: powstaje `src/import_pracownikow/migrations/0028_nadpisuj_daty_zatrudnienia.py`
+(AddField). Obejrzyj plik — ma zawierać wyłącznie to jedno AddField.
+
+- [ ] **Step 5: Uruchom test — ma przejść**
+
+```bash
+uv run pytest src/import_pracownikow/tests/test_nadpisywanie_dat.py -x 2>&1 | tee /tmp/t1b.log | tail -5
+```
+
+Oczekiwane: PASS.
+
+- [ ] **Step 6: Commit**
+
+```bash
+git add src/import_pracownikow/models.py src/import_pracownikow/migrations/0028_nadpisuj_daty_zatrudnienia.py src/import_pracownikow/tests/test_nadpisywanie_dat.py
+git commit -m "feat(import_pracownikow): pole nadpisuj_daty_zatrudnienia (mig 0028)"
+```
+
+---
+
+### Task 2: Formularz (szuflada) + confirm przy zaznaczaniu
+
+**Files:**
+- Modify: `src/import_pracownikow/forms.py` (`NowyImportForm`, linie 16-70)
+- Modify: `src/import_pracownikow/templates/import_pracownikow/importpracownikow_form.html` (blok `<script>`, linie 25-47)
+- Test: `src/import_pracownikow/tests/test_nadpisywanie_dat.py`
+
+**Interfaces:**
+- Consumes: `ImportPracownikow.nadpisuj_daty_zatrudnienia` (Task 1).
+- Produces: pole `nadpisuj_daty_zatrudnienia` w `NowyImportForm` (checkbox w `<details>`), confirm JS na `#id_nadpisuj_daty_zatrudnienia`.
+
+- [ ] **Step 1: Napisz failujące testy**
+
+Dopisz do `test_nadpisywanie_dat.py`:
+
+```python
+from import_pracownikow.forms import NowyImportForm
+
+
+def test_formularz_ma_pole_nadpisywania_dat():
+    form = NowyImportForm()
+    assert "nadpisuj_daty_zatrudnienia" in form.fields
+    assert form.fields["nadpisuj_daty_zatrudnienia"].initial in (None, False)
+
+
+def test_formularz_pole_nadpisywania_w_szufladzie():
+    # Pole ma być w zwiniętym <details> (szuflada opcji zaawansowanych).
+    from crispy_forms.utils import render_crispy_form
+
+    html = render_crispy_form(NowyImportForm())
+    assert "<details" in html
+    pozycja_details = html.index("<details")
+    pozycja_pola = html.index("nadpisuj_daty_zatrudnienia")
+    assert pozycja_pola > pozycja_details
+```
+
+- [ ] **Step 2: Uruchom testy — mają paść**
+
+```bash
+uv run pytest src/import_pracownikow/tests/test_nadpisywanie_dat.py -x 2>&1 | tee /tmp/t2.log | tail -5
+```
+
+Oczekiwane: FAIL (`KeyError: 'nadpisuj_daty_zatrudnienia'`).
+
+- [ ] **Step 3: Dodaj pole do formularza**
+
+W `src/import_pracownikow/forms.py`, w `NowyImportForm.Meta.fields` dopisz
+`"nadpisuj_daty_zatrudnienia"`:
+
+```python
+        fields = [
+            "plik_xls",
+            "data_zmian_personalnych",
+            "przepnij_wszystkie_prace",
+            "nadpisuj_daty_zatrudnienia",
+        ]
+```
+
+W layoutcie zmień nagłówek szuflady i dodaj Row (blok `HTML(...)` +
+`Row(...)` between details, linie 49-57) — całość po zmianie:
+
+```python
+                # Opcje groźne i rzadko potrzebne — chowamy je w domyślnie
+                # ZWINIĘTYM <details> (natywny collapsible, bez JS). Input
+                # zwiniętego <details> normalnie się wysyła, a confirm-y (po
+                # #id_...) dalej działają.
+                HTML(
+                    '<details class="callout secondary">'
+                    '<summary><span class="fi-widget"></span> '
+                    "Opcje zaawansowane</summary>"
+                ),
+                Row(
+                    Column("przepnij_wszystkie_prace", css_class="large-12 small-12"),
+                ),
+                Row(
+                    Column(
+                        "nadpisuj_daty_zatrudnienia", css_class="large-12 small-12"
+                    ),
+                ),
+                HTML("</details>"),
+```
+
+- [ ] **Step 4: Dodaj confirm JS w szablonie**
+
+W `importpracownikow_form.html` w istniejącym `<script>` (po handlerze
+`id_przepnij_wszystkie_prace`, przed zamknięciem `DOMContentLoaded`)
+dopisz:
+
+```javascript
+        var cbDaty = document.getElementById('id_nadpisuj_daty_zatrudnienia');
+        if (cbDaty) {
+            cbDaty.addEventListener('change', function () {
+                if (!cbDaty.checked) { return; }
+                var ok = confirm(
+                    "UWAGA — NADPISYWANIE DAT ZATRUDNIENIA\n\n" +
+                    "Zaznaczasz opcję, która przy zapisie osób NADPISZE " +
+                    "istniejące daty rozpoczęcia/zakończenia pracy " +
+                    "wartościami z pliku (dla osób obecnych w pliku).\n\n" +
+                    "Ręcznie ustawione daty w bazie zostaną UTRACONE tam, " +
+                    "gdzie plik niesie inną datę.\n\n" +
+                    "Użyj tylko, gdy plik jest źródłem prawdy o datach " +
+                    "(np. korekta po imporcie bez dat).\n\n" +
+                    "Czy na pewno wiesz, co robisz?"
+                );
+                if (!ok) { cbDaty.checked = false; }
+            });
+        }
+```
+
+Zaktualizuj też komentarz `{# ... #}` nad `<script>` (każda linia z
+własnym `{#` i `#}`), np.:
+
+```django
+    {# Opcje z szuflady („przepnij wszystkie prace", „nadpisuj daty #}
+    {# zatrudnienia") są groźne — przy ZAZNACZANIU wymuszamy świadome #}
+    {# potwierdzenie (confirm). Anulowanie cofa zaznaczenie. Vanilla JS na #}
+    {# DOMContentLoaded (nie zależy od kolejności $(document).ready). #}
+```
+
+- [ ] **Step 5: Uruchom testy — mają przejść**
+
+```bash
+uv run pytest src/import_pracownikow/tests/test_nadpisywanie_dat.py src/import_pracownikow/tests/test_nowy_import_form.py 2>&1 | tee /tmp/t2b.log | tail -5
+```
+
+Oczekiwane: PASS (także istniejące testy formularza — nagłówek szuflady
+się zmienił; jeśli któryś asertował stary tekst „masowe przepięcie prac",
+zaktualizuj GO (asercję), nie formularz).
+
+- [ ] **Step 6: Commit**
+
+```bash
+git add src/import_pracownikow/forms.py src/import_pracownikow/templates/import_pracownikow/importpracownikow_form.html src/import_pracownikow/tests/
+git commit -m "feat(import_pracownikow): checkbox nadpisywania dat w szufladzie + confirm"
+```
+
+---
+
+### Task 3: Bramka `zmiany_potrzebne` świadoma flagi (§3.3a specu)
+
+**Files:**
+- Modify: `src/import_pracownikow/models.py` (`_check_autor_jednostka_needs_update`, linie 1249-1282)
+- Test: `src/import_pracownikow/tests/test_nadpisywanie_dat.py`
+
+**Interfaces:**
+- Consumes: `self.parent.nadpisuj_daty_zatrudnienia` (Task 1), `self._plik_od()` / `self._plik_do()` (istniejące, `models.py:1020-1033`, zwracają `date | None`).
+- Produces: `check_if_integration_needed()` zwraca `True` dla wiersza z samą różnicą dat przy fladze ON — na tym polegają analiza (`analyze.py:766`), pewnosc (`pewnosc.py:134-137`) i świeży re-check (`integrate.py:227`), które wołają tę metodę bez zmian.
+
+- [ ] **Step 1: Napisz failujące testy**
+
+Dopisz do `test_nadpisywanie_dat.py`:
+
+```python
+from datetime import date
+
+from bpp.models import Autor, Autor_Jednostka, Jednostka
+from import_pracownikow.models import ImportPracownikowRow
+
+
+def _row_z_data(parent, dane, autor, jednostka, aj):
+    return baker.make(
+        ImportPracownikowRow,
+        parent=parent,
+        autor=autor,
+        jednostka=jednostka,
+        autor_jednostka=aj,
+        dane_znormalizowane=dane,
+    )
+
+
+def _scenariusz_tytulowy(nadpisuj):
+    """Otwarty okres od 2026-07-19 (fallback z poprzedniego importu),
+    plik niesie 2021-10-01. Jedyna różnica wiersza = data od."""
+    parent = baker.make(
+        ImportPracownikow, nadpisuj_daty_zatrudnienia=nadpisuj
+    )
+    autor, jednostka = baker.make(Autor), baker.make(Jednostka)
+    aj = baker.make(
+        Autor_Jednostka,
+        autor=autor,
+        jednostka=jednostka,
+        rozpoczal_prace=date(2026, 7, 19),
+        zakonczyl_prace=None,
+        podstawowe_miejsce_pracy=True,
+    )
+    row = _row_z_data(
+        parent, {"data_zatrudnienia": "2021-10-01"}, autor, jednostka, aj
+    )
+    return row, aj
+
+
+@pytest.mark.django_db
+def test_bramka_roznica_dat_flaga_on():
+    row, _ = _scenariusz_tytulowy(nadpisuj=True)
+    assert row.check_if_integration_needed() is True
+
+
+@pytest.mark.django_db
+def test_bramka_roznica_dat_flaga_off_jak_dzis():
+    row, _ = _scenariusz_tytulowy(nadpisuj=False)
+    assert row.check_if_integration_needed() is False
+```
+
+- [ ] **Step 2: Uruchom testy — ON ma paść, OFF ma przejść**
+
+```bash
+uv run pytest src/import_pracownikow/tests/test_nadpisywanie_dat.py -x -k bramka 2>&1 | tee /tmp/t3.log | tail -8
+```
+
+Oczekiwane: `test_bramka_roznica_dat_flaga_on` FAIL (bramka nie zna
+flagi), `..._off_jak_dzis` PASS.
+
+- [ ] **Step 3: Rozszerz `_check_autor_jednostka_needs_update`**
+
+W `src/import_pracownikow/models.py` przed listą `checks` dodaj
+zmienne, a do listy dwa nowe warunki (na końcu listy):
+
+```python
+        # §3.3a specu nadpisywania dat: przy fladze „nadpisuj daty" różnica
+        # wobec ISTNIEJĄCEJ (niepustej) daty też wymaga integracji — bez
+        # flagi liczy się, jak dotąd, wyłącznie wypełnienie NULL-a.
+        nadpisywanie = self.parent.nadpisuj_daty_zatrudnienia
+        plik_od, plik_do = self._plik_od(), self._plik_do()
+        checks = [
+            ...(istniejące warunki bez zmian)...,
+            nadpisywanie
+            and plik_od is not None
+            and aj.rozpoczal_prace is not None
+            and aj.rozpoczal_prace != plik_od,
+            nadpisywanie
+            and plik_do is not None
+            and aj.zakonczyl_prace is not None
+            and aj.zakonczyl_prace != plik_do,
+        ]
+```
+
+(`...(istniejące warunki bez zmian)...` = pozostaw 8 obecnych pozycji
+listy dokładnie jak są — dopisz DWIE nowe na końcu.)
+
+- [ ] **Step 4: Uruchom testy — mają przejść (plus regresja bramki)**
+
+```bash
+uv run pytest src/import_pracownikow/tests/test_nadpisywanie_dat.py src/import_pracownikow/tests/test_pewnosc.py src/import_pracownikow/tests/test_okresy_resolver.py 2>&1 | tee /tmp/t3b.log | tail -5
+```
+
+Oczekiwane: PASS wszystkie.
+
+- [ ] **Step 5: Commit**
+
+```bash
+git add src/import_pracownikow/models.py src/import_pracownikow/tests/test_nadpisywanie_dat.py
+git commit -m "feat(import_pracownikow): bramka zmiany_potrzebne swiadoma flagi nadpisywania dat"
+```
+
+---
+
+### Task 4: Nadpisywanie w `_integruj_daty_aj` + log zmian
+
+**Files:**
+- Modify: `src/import_pracownikow/models.py` (`_integruj_daty_aj`, linie 1333-1357; wywołanie w `_integrate_autor_jednostka`, linia 1368)
+- Test: `src/import_pracownikow/tests/test_nadpisywanie_dat.py`
+
+**Interfaces:**
+- Consumes: `self.parent.nadpisuj_daty_zatrudnienia`, `dane` = `dane_bardziej_znormalizowane` (daty już jako `date`, `models.py:902-912`).
+- Produces: `_integruj_daty_aj(aj, dane) -> bool` — `True` TYLKO gdy nadpisano niepustą wartość (sygnał dla pre-checku w Task 5; wypełnienia NULL-i → `False`).
+
+- [ ] **Step 1: Napisz failujące testy**
+
+Dopisz do `test_nadpisywanie_dat.py`:
+
+```python
+@pytest.mark.django_db
+def test_integracja_nadpisuje_date_od_flaga_on():
+    row, aj = _scenariusz_tytulowy(nadpisuj=True)
+    row.integrate()
+    aj.refresh_from_db()
+    assert aj.rozpoczal_prace == date(2021, 10, 1)
+    assert any(
+        "nadpisano z pliku" in wpis
+        for wpis in row.log_zmian["autor_jednostka"]
+    )
+
+
+@pytest.mark.django_db
+def test_integracja_nie_nadpisuje_flaga_off():
+    row, aj = _scenariusz_tytulowy(nadpisuj=False)
+    # OFF: bramka nie przepuści wiersza; wołamy integrate() wprost, żeby
+    # potwierdzić, że nawet wtedy data NIE jest ruszana.
+    row.integrate()
+    aj.refresh_from_db()
+    assert aj.rozpoczal_prace == date(2026, 7, 19)
+
+
+@pytest.mark.django_db
+def test_integracja_nadpisuje_date_do():
+    parent = baker.make(ImportPracownikow, nadpisuj_daty_zatrudnienia=True)
+    autor, jednostka = baker.make(Autor), baker.make(Jednostka)
+    aj = baker.make(
+        Autor_Jednostka,
+        autor=autor,
+        jednostka=jednostka,
+        rozpoczal_prace=date(2020, 1, 1),
+        zakonczyl_prace=date(2026, 12, 31),
+    )
+    row = _row_z_data(
+        parent,
+        {"data_zatrudnienia": "2020-01-01", "data_końca_zatrudnienia": "2024-06-30"},
+        autor,
+        jednostka,
+        aj,
+    )
+    row.integrate()
+    aj.refresh_from_db()
+    assert aj.zakonczyl_prace == date(2024, 6, 30)
+
+
+@pytest.mark.django_db
+def test_pusta_komorka_nie_kasuje_daty_przy_on():
+    parent = baker.make(ImportPracownikow, nadpisuj_daty_zatrudnienia=True)
+    autor, jednostka = baker.make(Autor), baker.make(Jednostka)
+    aj = baker.make(
+        Autor_Jednostka,
+        autor=autor,
+        jednostka=jednostka,
+        rozpoczal_prace=date(2020, 1, 1),
+        zakonczyl_prace=date(2024, 6, 30),
+    )
+    row = _row_z_data(parent, {}, autor, jednostka, aj)
+    row.integrate()
+    aj.refresh_from_db()
+    assert aj.rozpoczal_prace == date(2020, 1, 1)
+    assert aj.zakonczyl_prace == date(2024, 6, 30)
+```
+
+Uwaga wykonawcza: jeśli `row.integrate()` wymaga w tych testach
+dodatkowych pól wiersza (np. słowników z `dane_znormalizowane`), wzoruj
+się na istniejących testach integracji
+(`src/import_pracownikow/tests/test_integrate_slowniki.py`) — dane
+minimalne, `baker.make`.
+
+- [ ] **Step 2: Uruchom testy — nadpisywanie ma paść**
+
+```bash
+uv run pytest src/import_pracownikow/tests/test_nadpisywanie_dat.py -x -k integracja 2>&1 | tee /tmp/t4.log | tail -8
+```
+
+Oczekiwane: `test_integracja_nadpisuje_date_od_flaga_on` i
+`test_integracja_nadpisuje_date_do` FAIL (data niezmieniona); pozostałe
+PASS.
+
+- [ ] **Step 3: Przebuduj `_integruj_daty_aj`**
+
+Zastąp ciało metody (docstring zaktualizuj — dopisz zwrot i gałąź
+nadpisywania, zostaw opis zachowania bez flagi):
+
+```python
+    def _integruj_daty_aj(self, aj, dane):
+        """Ustawia daty zatrudnienia na powiązaniu z danych wiersza.
+
+        Bez flagi „nadpisuj daty": „data od"/„data do" na ISTNIEJĄCYM AJ
+        wypełniamy TYLKO gdy baza ma ``NULL`` a plik NIESIE datę (§3:
+        „wypełnienie NULL") — istniejącej daty nie ruszamy; pusty plik →
+        nic nie zmieniaj (§5). Fallback ``data zmian → dziś`` dla NOWEGO
+        okresu stempluje materializacja (``integrate._materializuj_diff``).
+
+        Z flagą ``parent.nadpisuj_daty_zatrudnienia`` (spec §3.3): datę
+        różną od niepustej wartości w bazie NADPISUJEMY wartością z pliku
+        (osobno „od" i „do"); puste komórki nadal niczego nie kasują.
+
+        Zwraca ``True`` TYLKO gdy nadpisano niepustą wartość — sygnał dla
+        pre-checku nakładania okresów; wypełnienia NULL-i zwracają
+        ``False`` (idą dzisiejszą ścieżką, bez pre-checku)."""
+        nadpisywanie = self.parent.nadpisuj_daty_zatrudnienia
+        nadpisano = False
+
+        plik_od = dane.get("data_zatrudnienia")
+        if plik_od:
+            if aj.rozpoczal_prace is None:
+                aj.rozpoczal_prace = plik_od
+                self.log_zmian["autor_jednostka"].append(
+                    f"data rozpoczęcia pracy na {aj.rozpoczal_prace}"
+                )
+            elif nadpisywanie and aj.rozpoczal_prace != plik_od:
+                self.log_zmian["autor_jednostka"].append(
+                    f"data rozpoczęcia pracy: {aj.rozpoczal_prace} → "
+                    f"{plik_od} (nadpisano z pliku)"
+                )
+                aj.rozpoczal_prace = plik_od
+                nadpisano = True
+
+        data_konca = dane.get("data_końca_zatrudnienia")
+        if data_konca:
+            if aj.zakonczyl_prace is None:
+                aj.zakonczyl_prace = data_konca
+                self.log_zmian["autor_jednostka"].append(
+                    f"data końca zatrudnienia na {data_konca}"
+                )
+            elif nadpisywanie and aj.zakonczyl_prace != data_konca:
+                self.log_zmian["autor_jednostka"].append(
+                    f"data końca zatrudnienia: {aj.zakonczyl_prace} → "
+                    f"{data_konca} (nadpisano z pliku)"
+                )
+                aj.zakonczyl_prace = data_konca
+                nadpisano = True
+
+        return nadpisano
+```
+
+W `_integrate_autor_jednostka` (linia 1368) zmień wywołanie:
+
+```python
+        nadpisano_daty = self._integruj_daty_aj(aj, dane)
+```
+
+(`nadpisano_daty` zostanie użyte w Task 5 — na razie przypisanie
+wystarczy; NIE dodawaj `noqa`, zmienna będzie użyta w następnym commicie
+— jeśli ruff F841 blokuje commit, dodaj pre-check z Task 5 w tym samym
+commicie.)
+
+- [ ] **Step 4: Uruchom testy — mają przejść (plus regresja dat)**
+
+```bash
+uv run pytest src/import_pracownikow/tests/test_nadpisywanie_dat.py src/import_pracownikow/tests/test_porownywarka_daty.py src/import_pracownikow/tests/test_pipeline 2>&1 | tee /tmp/t4b.log | tail -5
+```
+
+Oczekiwane: PASS wszystkie.
+
+- [ ] **Step 5: Commit**
+
+```bash
+git add src/import_pracownikow/models.py src/import_pracownikow/tests/test_nadpisywanie_dat.py
+git commit -m "feat(import_pracownikow): nadpisywanie dat AJ wartosciami z pliku (flaga ON)"
+```
+
+---
+
+### Task 5: Pre-check nakładania okresów (lustro ExclusionConstraint)
+
+**Files:**
+- Modify: `src/import_pracownikow/models.py` (`_integrate_autor_jednostka`, ok. linii 1368-1388; nowa metoda `_sprawdz_nakladanie_okresow` obok)
+- Test: `src/import_pracownikow/tests/test_nadpisywanie_dat.py`
+
+**Interfaces:**
+- Consumes: `nadpisano_daty: bool` z `_integruj_daty_aj` (Task 4), `BPPDatabaseError` (istniejący, używany w `models.py:1383`).
+- Produces: `_sprawdz_nakladanie_okresow(aj)` — raise `BPPDatabaseError` przy kolizji; wołane TYLKO gdy `nadpisano_daty`.
+
+- [ ] **Step 1: Napisz failujące testy**
+
+Dopisz do `test_nadpisywanie_dat.py`:
+
+```python
+from import_common.exceptions import BPPDatabaseError
+
+
+@pytest.mark.django_db
+def test_nadpisanie_kolidujace_z_zamknietym_okresem_izolowane():
+    # Zamknięty okres 2019-2022 + otwarty od 2026. Plik cofa otwarty na
+    # 2021 → przedziały [2021,∞) i [2019-…-2022] nakładają się → błąd
+    # izolowany PRZED save (constraint w bazie zatrułby transakcję).
+    parent = baker.make(ImportPracownikow, nadpisuj_daty_zatrudnienia=True)
+    autor, jednostka = baker.make(Autor), baker.make(Jednostka)
+    baker.make(
+        Autor_Jednostka,
+        autor=autor,
+        jednostka=jednostka,
+        rozpoczal_prace=date(2019, 1, 1),
+        zakonczyl_prace=date(2022, 12, 31),
+    )
+    aj = baker.make(
+        Autor_Jednostka,
+        autor=autor,
+        jednostka=jednostka,
+        rozpoczal_prace=date(2026, 7, 19),
+        zakonczyl_prace=None,
+    )
+    row = _row_z_data(
+        parent, {"data_zatrudnienia": "2021-10-01"}, autor, jednostka, aj
+    )
+    with pytest.raises(BPPDatabaseError) as exc:
+        row.integrate()
+    assert "nakładają się" in str(exc.value)
+    aj.refresh_from_db()
+    assert aj.rozpoczal_prace == date(2026, 7, 19)  # nic nie zapisano
+
+
+@pytest.mark.django_db
+def test_wypelnienie_null_nie_odpala_precheku():
+    # Wypełnienie NULL-a to dzisiejsza ścieżka — bez pre-checku, nawet
+    # przy fladze ON (spec §3.4).
+    parent = baker.make(ImportPracownikow, nadpisuj_daty_zatrudnienia=True)
+    autor, jednostka = baker.make(Autor), baker.make(Jednostka)
+    aj = baker.make(
+        Autor_Jednostka,
+        autor=autor,
+        jednostka=jednostka,
+        rozpoczal_prace=None,
+        zakonczyl_prace=None,
+    )
+    row = _row_z_data(
+        parent, {"data_zatrudnienia": "2021-10-01"}, autor, jednostka, aj
+    )
+    row.integrate()
+    aj.refresh_from_db()
+    assert aj.rozpoczal_prace == date(2021, 10, 1)
+
+
+@pytest.mark.django_db
+def test_od_wieksze_rowne_do_po_nadpisaniu_odrzucone():
+    parent = baker.make(ImportPracownikow, nadpisuj_daty_zatrudnienia=True)
+    autor, jednostka = baker.make(Autor), baker.make(Jednostka)
+    aj = baker.make(
+        Autor_Jednostka,
+        autor=autor,
+        jednostka=jednostka,
+        rozpoczal_prace=date(2020, 1, 1),
+        zakonczyl_prace=date(2021, 1, 1),
+    )
+    row = _row_z_data(
+        parent, {"data_zatrudnienia": "2022-01-01"}, autor, jednostka, aj
+    )
+    with pytest.raises(BPPDatabaseError):
+        row.integrate()
+```
+
+(`BPPDatabaseError` jest zdefiniowany w
+`src/import_common/exceptions.py:90` — models.py importuje go z
+`import_common.exceptions`, linia 23.)
+
+- [ ] **Step 2: Uruchom testy — kolizja ma paść**
+
+```bash
+uv run pytest src/import_pracownikow/tests/test_nadpisywanie_dat.py -x -k "kolidujace or precheku or odrzucone" 2>&1 | tee /tmp/t5.log | tail -8
+```
+
+Oczekiwane: `test_nadpisanie_kolidujace...` FAIL (IntegrityError zamiast
+BPPDatabaseError albo brak wyjątku), pozostałe dwa PASS.
+
+- [ ] **Step 3: Dodaj pre-check**
+
+W `src/import_pracownikow/models.py`, nowa metoda bezpośrednio POD
+`_integruj_daty_aj`:
+
+```python
+    def _sprawdz_nakladanie_okresow(self, aj):
+        """Pythonowe lustro constraintu
+        ``bpp_autor_jednostka_okresy_bez_nakladan`` (bpp/models/autor.py):
+        przedziały DOMKNIĘTE ``[od, do]``, ``zakonczyl_prace IS NULL`` =
+        otwarty w prawo ``[od, ∞)``. Wołane TYLKO po nadpisaniu niepustej
+        daty (flaga „nadpisuj daty") — naruszenie constraintu w bazie
+        zatruwa transakcję i psuje izolację wiersza, więc kolizję łapiemy
+        PRZED save (jak niezmiennik od<do wyżej)."""
+        from bpp.models import Autor_Jednostka
+
+        if aj.rozpoczal_prace is None:
+            return
+        od = aj.rozpoczal_prace
+        do = aj.zakonczyl_prace or date.max
+        koliduje = (
+            Autor_Jednostka.objects.filter(
+                autor_id=aj.autor_id,
+                jednostka_id=aj.jednostka_id,
+                rozpoczal_prace__isnull=False,
+            )
+            .exclude(pk=aj.pk)
+            .filter(rozpoczal_prace__lte=do)
+        )
+        for inny in koliduje:
+            if od <= (inny.zakonczyl_prace or date.max):
+                raise BPPDatabaseError(
+                    self.dane_z_xls,
+                    self,
+                    f"nadpisane daty ({od} – "
+                    f"{aj.zakonczyl_prace or 'obecnie'}) nakładają się na "
+                    f"inny okres zatrudnienia w tej jednostce "
+                    f"({inny.rozpoczal_prace} – "
+                    f"{inny.zakonczyl_prace or 'obecnie'})",
+                )
+```
+
+W `_integrate_autor_jednostka`, PO istniejącym checku `od >= do`
+(za linią `raise BPPDatabaseError(... jest późniejsza ...)`, przed
+blokiem `funkcja_autora`):
+
+```python
+        if nadpisano_daty:
+            self._sprawdz_nakladanie_okresow(aj)
+```
+
+- [ ] **Step 4: Uruchom testy — mają przejść**
+
+```bash
+uv run pytest src/import_pracownikow/tests/test_nadpisywanie_dat.py 2>&1 | tee /tmp/t5b.log | tail -5
+```
+
+Oczekiwane: PASS wszystkie.
+
+- [ ] **Step 5: Commit**
+
+```bash
+git add src/import_pracownikow/models.py src/import_pracownikow/tests/test_nadpisywanie_dat.py
+git commit -m "feat(import_pracownikow): pre-check nakladania okresow przed nadpisaniem dat"
+```
+
+---
+
+### Task 6: Licznik nadpisań + callout + confirm w końcowym formularzu
+
+**Files:**
+- Modify: `src/import_pracownikow/models.py` (nowa metoda `ImportPracownikowRow.nadpisze_daty()`; nowa metoda `ImportPracownikow.liczba_nadpisan_dat()`)
+- Modify: `src/import_pracownikow/views.py` (`PodgladImportuView.get_context_data`, linie 978-1039)
+- Create: `src/import_pracownikow/templates/import_pracownikow/partials/_ostrzezenie_nadpisanie_dat.html`
+- Modify: `src/import_pracownikow/templates/import_pracownikow/przeglad.html` (linie 194-216)
+- Test: `src/import_pracownikow/tests/test_nadpisywanie_dat.py`
+
+**Interfaces:**
+- Consumes: `rozwiaz_okres_zatrudnienia` + `wstepnie_zaladuj_okresy` (`okresy.py`), `row._aj_lista()` (istniejące memo), wiersze parenta przez DOMYŚLNY related_name `importpracownikowrow_set` (FK `parent` nie ustawia własnego — `models.py:782-785`; tak samo robi reszta modelu, np. `models.py:400`).
+- Produces: `ImportPracownikow.liczba_nadpisan_dat() -> int`; kontekst `nadpisywanie_dat_wlaczone: bool` i `liczba_nadpisan_dat: int` w `przeglad.html`.
+
+- [ ] **Step 1: Napisz failujące testy**
+
+Dopisz do `test_nadpisywanie_dat.py`:
+
+```python
+@pytest.mark.django_db
+def test_liczba_nadpisan_dat_liczy_tylko_realne_nadpisania():
+    parent = baker.make(ImportPracownikow, nadpisuj_daty_zatrudnienia=True)
+    autor1, autor2, autor3 = baker.make(Autor), baker.make(Autor), baker.make(Autor)
+    jednostka = baker.make(Jednostka)
+    # 1) realne nadpisanie: baza 2026, plik 2021 → LICZY SIĘ
+    aj1 = baker.make(
+        Autor_Jednostka, autor=autor1, jednostka=jednostka,
+        rozpoczal_prace=date(2026, 7, 19),
+    )
+    _row_z_data(parent, {"data_zatrudnienia": "2021-10-01"}, autor1, jednostka, aj1)
+    # 2) wypełnienie NULL-a → NIE liczy się
+    aj2 = baker.make(
+        Autor_Jednostka, autor=autor2, jednostka=jednostka, rozpoczal_prace=None
+    )
+    _row_z_data(parent, {"data_zatrudnienia": "2021-10-01"}, autor2, jednostka, aj2)
+    # 3) zgodne daty → NIE liczy się
+    aj3 = baker.make(
+        Autor_Jednostka, autor=autor3, jednostka=jednostka,
+        rozpoczal_prace=date(2021, 10, 1),
+    )
+    _row_z_data(parent, {"data_zatrudnienia": "2021-10-01"}, autor3, jednostka, aj3)
+    assert parent.liczba_nadpisan_dat() == 1
+
+
+@pytest.mark.django_db
+def test_liczba_nadpisan_dat_zero_przy_fladze_off():
+    parent = baker.make(ImportPracownikow, nadpisuj_daty_zatrudnienia=False)
+    autor, jednostka = baker.make(Autor), baker.make(Jednostka)
+    aj = baker.make(
+        Autor_Jednostka, autor=autor, jednostka=jednostka,
+        rozpoczal_prace=date(2026, 7, 19),
+    )
+    _row_z_data(parent, {"data_zatrudnienia": "2021-10-01"}, autor, jednostka, aj)
+    assert parent.liczba_nadpisan_dat() == 0
+```
+
+- [ ] **Step 2: Uruchom testy — mają paść**
+
+```bash
+uv run pytest src/import_pracownikow/tests/test_nadpisywanie_dat.py -x -k liczba 2>&1 | tee /tmp/t6.log | tail -5
+```
+
+Oczekiwane: FAIL (`AttributeError: liczba_nadpisan_dat`).
+
+- [ ] **Step 3: Dodaj metody modelu**
+
+`ImportPracownikowRow` (obok `_plik_od`/`_plik_do`):
+
+```python
+    def nadpisze_daty(self):
+        """Czy zapis osób NADPISZE niepustą datę tego wiersza (flaga
+        ``nadpisuj_daty_zatrudnienia``). Liczy TYLKO realne nadpisania —
+        obie strony niepuste i różne; wypełnienia NULL-i i nowe okresy to
+        NIE nadpisania (spec §3.5, stan „zmienione" ze ``stany_pol`` byłby
+        zawyżony). Zasila licznik ostrzeżenia finalizacji."""
+        if not self.parent.nadpisuj_daty_zatrudnienia:
+            return False
+        if self.autor_id is None or self.jednostka_id is None:
+            return False
+        from import_pracownikow.okresy import rozwiaz_okres_zatrudnienia
+
+        rodzaj, aj = rozwiaz_okres_zatrudnienia(
+            self.autor, self.jednostka, self._plik_od(), aj_lista=self._aj_lista()
+        )
+        if rodzaj != "istniejacy":
+            return False
+        plik_od, plik_do = self._plik_od(), self._plik_do()
+        return bool(
+            plik_od and aj.rozpoczal_prace and aj.rozpoczal_prace != plik_od
+        ) or bool(
+            plik_do and aj.zakonczyl_prace and aj.zakonczyl_prace != plik_do
+        )
+```
+
+`ImportPracownikow` (obok `liczba_wierszy_do_pominiecia` — znajdź przez
+grep):
+
+```python
+    def liczba_nadpisan_dat(self):
+        """Ile wierszy przy zapisie osób NADPISZE istniejącą datę
+        zatrudnienia (flaga ``nadpisuj_daty_zatrudnienia``) — do calloutu
+        i confirmu finalizacji. Liczone LIVE (``stany_pol_snapshot`` bywa
+        NULL do backfillu i miesza wypełnienia NULL-i z nadpisaniami);
+        ``wstepnie_zaladuj_okresy`` + przypięcie ``parent`` chronią przed
+        N+1."""
+        if not self.nadpisuj_daty_zatrudnienia:
+            return 0
+        from import_pracownikow.okresy import wstepnie_zaladuj_okresy
+
+        rows = list(
+            self.importpracownikowrow_set.filter(
+                autor__isnull=False, jednostka__isnull=False
+            ).select_related("autor", "jednostka")
+        )
+        for row in rows:
+            row.parent = self
+        wstepnie_zaladuj_okresy(rows)
+        return sum(1 for row in rows if row.nadpisze_daty())
+```
+
+- [ ] **Step 4: Uruchom testy metod — mają przejść**
+
+```bash
+uv run pytest src/import_pracownikow/tests/test_nadpisywanie_dat.py -k liczba 2>&1 | tee /tmp/t6b.log | tail -5
+```
+
+- [ ] **Step 5: Kontekst widoku + testy renderu**
+
+W `PodgladImportuView.get_context_data` do `ctx.update({...})` dopisz:
+
+```python
+                # Ostrzeżenie finalizacji (spec nadpisywania dat §3.5):
+                # licznik liczony tylko w fazie osób — w Kroku 1 formularz
+                # zapisu osób i tak się nie renderuje.
+                "nadpisywanie_dat_wlaczone": parent.nadpisuj_daty_zatrudnienia,
+                "liczba_nadpisan_dat": (
+                    parent.liczba_nadpisan_dat() if parent.faza_osob else 0
+                ),
+```
+
+Dopisz testy renderu do `test_nadpisywanie_dat.py` — wzoruj klienta i
+setup na istniejącym `src/import_pracownikow/tests/test_przeglad.py`
+(fixture zalogowanego usera w grupie wprowadzania danych + parent w
+stanie fazy osób; skopiuj stamtąd konstrukcję, nie wymyślaj własnej):
+
+```python
+def test_przeglad_callout_nadpisywania_widoczny_przy_on(...):
+    # parent: stan fazy osób, nadpisuj_daty_zatrudnienia=True + 1 wiersz
+    # z realnym nadpisaniem (jak w test_liczba_nadpisan_dat...).
+    # GET przeglądu:
+    # assert "Włączono nadpisywanie dat zatrudnienia" in html
+    # assert "NADPISANE" in confirm-owym onsubmit formularza zapisu osób
+    ...
+
+
+def test_przeglad_bez_calloutu_przy_off(...):
+    # identyczny setup, flaga OFF:
+    # assert "Włączono nadpisywanie dat zatrudnienia" not in html
+    ...
+```
+
+(Ciała testów uzupełnij konkretami z `test_przeglad.py` — asercje jak w
+komentarzach powyżej; to jedyne dopuszczalne odstępstwo od gotowego kodu
+w tym planie, bo fixtures przeglądu są rozbudowane i już istnieją.)
+
+- [ ] **Step 6: Partial + przeglad.html**
+
+Utwórz `src/import_pracownikow/templates/import_pracownikow/partials/_ostrzezenie_nadpisanie_dat.html`:
+
+```django
+{# Ostrzeżenie finalizacji: flaga „nadpisuj daty zatrudnienia" — zapis #}
+{# osób NADPISZE istniejące daty wartościami z pliku (spec nadpisywania #}
+{# dat §3.5). Liczba = realne nadpisania (obie strony niepuste i różne). #}
+<div class="callout alert">
+    <p>
+        <span class="fi-alert"></span>
+        <strong>Włączono nadpisywanie dat zatrudnienia.</strong>
+        Wiersze z datami różnymi od bazy:
+        <strong>{{ liczba_nadpisan_dat }}</strong> — ich daty rozpoczęcia
+        / zakończenia pracy zostaną <strong>NADPISANE</strong>
+        wartościami z pliku. Ręcznie ustawione daty zostaną utracone.
+    </p>
+</div>
+```
+
+W `przeglad.html`:
+
+1. Po linii `{% include ".../_ostrzezenie_podstawowe_miejsce.html" %}`
+   (linia 196) dodaj:
+
+```django
+                    {% if nadpisywanie_dat_wlaczone %}
+                        {% include "import_pracownikow/partials/_ostrzezenie_nadpisanie_dat.html" %}
+                    {% endif %}
+```
+
+2. Zmień `onsubmit` formularza (linia 205) tak, by obejmował oba
+   ostrzeżenia (jedna linia — szablonowe `{% if %}` w atrybucie):
+
+```django
+                              {% if liczba_pominietych or liczba_nadpisan_dat %}onsubmit="return confirm('{% if liczba_pominietych %}Wiersze bez dopasowania zostaną pominięte: {{ liczba_pominietych }}. {% endif %}{% if liczba_nadpisan_dat %}Daty zatrudnienia zostaną NADPISANE wartościami z pliku dla wierszy: {{ liczba_nadpisan_dat }}. {% endif %}Kontynuować zapis?');"{% endif %}>
+```
+
+- [ ] **Step 7: Uruchom testy renderu + regresję przeglądu**
+
+```bash
+uv run pytest src/import_pracownikow/tests/test_nadpisywanie_dat.py src/import_pracownikow/tests/test_przeglad.py 2>&1 | tee /tmp/t6c.log | tail -5
+```
+
+Oczekiwane: PASS wszystkie.
+
+- [ ] **Step 8: Commit**
+
+```bash
+git add src/import_pracownikow/models.py src/import_pracownikow/views.py src/import_pracownikow/templates/ src/import_pracownikow/tests/
+git commit -m "feat(import_pracownikow): ostrzezenie finalizacji o nadpisaniu dat (callout + confirm)"
+```
+
+---
+
+### Task 7: Newsfragment + dokumentacja administratora
+
+**Files:**
+- Create: `src/bpp/newsfragments/nadpisywanie-dat-zatrudnienia.feature.rst`
+- Modify: `docs/administrator/import-pracownikow.md`
+
+- [ ] **Step 1: Newsfragment**
+
+Utwórz `src/bpp/newsfragments/nadpisywanie-dat-zatrudnienia.feature.rst`:
+
+```rst
+Import pracowników: nowa opcja „Nadpisuj daty zatrudnienia (od/do)
+wartościami z pliku" (szuflada „Opcje zaawansowane"). Pozwala skorygować
+daty istniejących okresów zatrudnienia — np. po wcześniejszym imporcie
+pliku bez dat, który ostemplował wszystkich datą importu. Puste komórki
+niczego nie kasują; przed zapisem system pokazuje liczbę nadpisań i prosi
+o potwierdzenie.
+```
+
+- [ ] **Step 2: Dokumentacja**
+
+W `docs/administrator/import-pracownikow.md` znajdź sekcję o opcjach
+formularza nowego importu (grep „przepnij" lub „Opcje"); dopisz PO niej
+podsekcję:
+
+```markdown
+### Nadpisywanie dat zatrudnienia wartościami z pliku
+
+Domyślnie import **nigdy nie nadpisuje** istniejących dat zatrudnienia —
+wypełnia tylko puste („data od"/„data do" bez wartości w bazie), a
+różnice pokazuje wyłącznie w podglądzie. Jeśli daty w bazie są błędne
+(np. poprzedni import pliku bez dat ostemplował wszystkich datą importu),
+zaznacz w szufladzie „Opcje zaawansowane" opcję **Nadpisuj daty
+zatrudnienia (od/do) wartościami z pliku**:
+
+- nadpisywane są tylko daty osób obecnych w pliku, tam gdzie plik niesie
+  datę różną od bazy;
+- puste komórki pliku niczego nie kasują;
+- przy zaznaczaniu opcji oraz przed końcowym zapisem system prosi o
+  potwierdzenie i pokazuje liczbę wierszy, których daty zostaną
+  nadpisane;
+- nadpisanie, które spowodowałoby nałożenie się dwóch okresów
+  zatrudnienia w tej samej jednostce, jest odrzucane per-wiersz z
+  czytelnym błędem.
+```
+
+(Dopasuj poziom nagłówka do konwencji pliku.)
+
+- [ ] **Step 3: Commit**
+
+```bash
+git add src/bpp/newsfragments/nadpisywanie-dat-zatrudnienia.feature.rst docs/administrator/import-pracownikow.md
+git commit -m "docs(import_pracownikow): newsfragment + dokumentacja nadpisywania dat"
+```
+
+---
+
+### Task 8: Jakość, pełne testy modułu, PR
+
+**Files:** bez nowych; poprawki z ruff/testów.
+
+- [ ] **Step 1: Ruff na zmienionych plikach**
+
+```bash
+ruff format src/import_pracownikow/ src/bpp/newsfragments/ && ruff check src/import_pracownikow/
+```
+
+Oczekiwane: brak błędów (poprawki ręcznie, pojedynczo — nie `--fix`).
+
+- [ ] **Step 2: Pełny przebieg testów modułu importu**
+
+```bash
+uv run pytest src/import_pracownikow/ -n auto 2>&1 | tee /tmp/import_all.log | tail -10
+```
+
+Oczekiwane: wszystkie PASS. Padnięte — napraw przed pushem (pamiętaj:
+17 testów `api_v1 test_autor` bywało RED na dev niezależnie od nas — to
+inny moduł; w `src/import_pracownikow/` ma być zielono).
+
+- [ ] **Step 3: Push + PR do dev**
+
+```bash
+git push -u origin feat/import-nadpisywanie-dat
+```
+
+PR przez `gh api` (gh pr create flakuje):
+
+```bash
+gh api repos/{owner}/{repo}/pulls -f title="feat(import_pracownikow): nadpisywanie dat zatrudnienia wartościami z pliku" -f head=feat/import-nadpisywanie-dat -f base=dev -f body="$(cat <<'EOF'
+Opcja per-import „Nadpisuj daty zatrudnienia (od/do) wartościami z pliku"
+(szuflada + confirm + ostrzeżenie finalizacji). Spec:
+docs/superpowers/specs/2026-07-26-import-nadpisywanie-dat-zatrudnienia-design.md
+
+- pole nadpisuj_daty_zatrudnienia (mig 0028) + checkbox w szufladzie z confirm-em
+- bramka zmiany_potrzebne świadoma flagi (bez tego wiersz z samą różnicą dat nie wchodził do integracji)
+- _integruj_daty_aj: nadpisywanie niepustych dat przy fladze ON, puste komórki nic nie kasują
+- pythonowy pre-check nakładania okresów (lustro ExclusionConstraint) przed save
+- callout + rozszerzony confirm w końcowym formularzu zapisu osób (licznik realnych nadpisań)
+
+UWAGA przy scalaniu: migracja 0028 → `make baseline-update` (raz, przy merge).
+
+🤖 Generated with [Claude Code](https://claude.com/claude-code)
+EOF
+)"
+```
+
+- [ ] **Step 4: Czekaj na REALNE gejty CI**
+
+`Build test-runner image` + `Tests (sharded)` (wszystkie shardy) muszą
+być zielone; równolegle można odpalić lokalnie
+`make tests-without-playwright` (wynik do `/tmp/tests_local.log`).
+
+---
+
+## Self-review planu
+
+- Pokrycie specu: §3.1→Task 1, §3.2→Task 2, §3.3→Task 4, §3.3a→Task 3,
+  §3.4→Task 5, §3.5→Task 6, §3.6→testy w Taskach 1-6, §3.7→Task 7. ✓
+- Typy/nazwy spójne między taskami (`nadpisuj_daty_zatrudnienia`,
+  `nadpisze_daty()`, `liczba_nadpisan_dat()`,
+  `_sprawdz_nakladanie_okresow(aj)`, zwrot bool z `_integruj_daty_aj`). ✓
+- Jedyny niedosłowny fragment: ciała 2 testów renderu w Task 6 Step 5
+  (fixtures kopiowane z istniejącego `test_przeglad.py` — wskazane
+  źródło). Import `BPPDatabaseError` i related_name zweryfikowane w
+  kodzie. ✓

From a13dc6b1face9d6b7076af952e3c4fa2c302bea5 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Micha=C5=82=20Pasternak?= <michal.dtz@gmail.com>
Date: Sun, 26 Jul 2026 17:54:19 +0200
Subject: [PATCH 03/11] feat(import_pracownikow): pole
 nadpisuj_daty_zatrudnienia (mig 0028)

---
 .../0028_nadpisuj_daty_zatrudnienia.py        | 22 +++++++++++++++++++
 src/import_pracownikow/models.py              | 14 ++++++++++++
 .../tests/test_nadpisywanie_dat.py            | 15 +++++++++++++
 3 files changed, 51 insertions(+)
 create mode 100644 src/import_pracownikow/migrations/0028_nadpisuj_daty_zatrudnienia.py
 create mode 100644 src/import_pracownikow/tests/test_nadpisywanie_dat.py

diff --git a/src/import_pracownikow/migrations/0028_nadpisuj_daty_zatrudnienia.py b/src/import_pracownikow/migrations/0028_nadpisuj_daty_zatrudnienia.py
new file mode 100644
index 000000000..91b37d20b
--- /dev/null
+++ b/src/import_pracownikow/migrations/0028_nadpisuj_daty_zatrudnienia.py
@@ -0,0 +1,22 @@
+# Generated by Django 5.2.16 on 2026-07-26 15:53
+
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ("import_pracownikow", "0027_profil_uczelnia"),
+    ]
+
+    operations = [
+        migrations.AddField(
+            model_name="importpracownikow",
+            name="nadpisuj_daty_zatrudnienia",
+            field=models.BooleanField(
+                default=False,
+                help_text="Gdy zaznaczone, daty rozpoczęcia i zakończenia pracy <strong>ISTNIEJĄCYCH</strong> okresów zatrudnienia zostaną <strong>NADPISANE</strong> wartościami z pliku — tam, gdzie plik niesie datę różną od bazy.<br>Użyj do KOREKTY dat (np. po wcześniejszym imporcie pliku bez dat, który ostemplował wszystkich datą importu).<br>Puste komórki pliku niczego nie kasują. Dotyczy wyłącznie osób obecnych w pliku.",
+                verbose_name="Nadpisuj daty zatrudnienia (od/do) wartościami z pliku",
+            ),
+        ),
+    ]
diff --git a/src/import_pracownikow/models.py b/src/import_pracownikow/models.py
index 7f68945e8..3f8638956 100644
--- a/src/import_pracownikow/models.py
+++ b/src/import_pracownikow/models.py
@@ -169,6 +169,20 @@ class ImportPracownikow(LiveOperation):
         "zostaw ODZNACZONE</strong> — przepięłoby to historyczne afiliacje.<br>"
         "Można korygować per wiersz przed zapisem osób.",
     )
+    nadpisuj_daty_zatrudnienia = models.BooleanField(
+        "Nadpisuj daty zatrudnienia (od/do) wartościami z pliku",
+        default=False,
+        # HTML w help_text (crispy renderuje przez |safe) — wzorzec jak w
+        # przepnij_wszystkie_prace. Ten sam string dosłownie w migracji 0028.
+        help_text="Gdy zaznaczone, daty rozpoczęcia i zakończenia pracy "
+        "<strong>ISTNIEJĄCYCH</strong> okresów zatrudnienia zostaną "
+        "<strong>NADPISANE</strong> wartościami z pliku — tam, gdzie plik "
+        "niesie datę różną od bazy.<br>"
+        "Użyj do KOREKTY dat (np. po wcześniejszym imporcie pliku bez "
+        "dat, który ostemplował wszystkich datą importu).<br>"
+        "Puste komórki pliku niczego nie kasują. Dotyczy wyłącznie osób "
+        "obecnych w pliku.",
+    )
     zakres_integracji = models.CharField(
         "Zakres integracji",
         max_length=20,
diff --git a/src/import_pracownikow/tests/test_nadpisywanie_dat.py b/src/import_pracownikow/tests/test_nadpisywanie_dat.py
new file mode 100644
index 000000000..60d2ef906
--- /dev/null
+++ b/src/import_pracownikow/tests/test_nadpisywanie_dat.py
@@ -0,0 +1,15 @@
+"""Opcja „Nadpisuj daty zatrudnienia (od/do) wartościami z pliku" —
+flaga modelu, bramka zmian, nadpisywanie w integracji, pre-check
+nakładania okresów, licznik ostrzeżenia finalizacji (spec
+2026-07-26-import-nadpisywanie-dat-zatrudnienia-design.md)."""
+
+import pytest
+from model_bakery import baker
+
+from import_pracownikow.models import ImportPracownikow
+
+
+@pytest.mark.django_db
+def test_flaga_nadpisywania_domyslnie_wylaczona():
+    parent = baker.make(ImportPracownikow)
+    assert parent.nadpisuj_daty_zatrudnienia is False

From 46bb94643a1e90224540492215d9026530b1e102 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Micha=C5=82=20Pasternak?= <michal.dtz@gmail.com>
Date: Sun, 26 Jul 2026 18:00:50 +0200
Subject: [PATCH 04/11] feat(import_pracownikow): checkbox nadpisywania dat w
 szufladzie + confirm

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
---
 src/import_pracownikow/forms.py               | 14 ++++++----
 .../importpracownikow_form.html               | 26 ++++++++++++++++---
 .../tests/test_nadpisywanie_dat.py            | 18 +++++++++++++
 3 files changed, 49 insertions(+), 9 deletions(-)

diff --git a/src/import_pracownikow/forms.py b/src/import_pracownikow/forms.py
index e89600b55..1587a985b 100644
--- a/src/import_pracownikow/forms.py
+++ b/src/import_pracownikow/forms.py
@@ -20,6 +20,7 @@ class Meta:
             "plik_xls",
             "data_zmian_personalnych",
             "przepnij_wszystkie_prace",
+            "nadpisuj_daty_zatrudnienia",
         ]
         widgets = {
             # natywny date-picker przeglądarki (bez JS-owej zależności)
@@ -42,18 +43,21 @@ def __init__(self, *args, **kwargs):
                 Row(
                     Column("data_zmian_personalnych", css_class="large-6 small-12"),
                 ),
-                # „Przepnij wszystkie prace" to opcja groźna i rzadko potrzebna —
-                # chowamy ją w domyślnie ZWINIĘTYM <details> (natywny collapsible,
-                # bez JS). Input zwiniętego <details> normalnie się wysyła, a
-                # confirm (po #id_przepnij_wszystkie_prace) dalej działa.
+                # Opcje groźne i rzadko potrzebne — chowamy je w domyślnie
+                # ZWINIĘTYM <details> (natywny collapsible, bez JS). Input
+                # zwiniętego <details> normalnie się wysyła, a confirm-y (po
+                # #id_...) dalej działają.
                 HTML(
                     '<details class="callout secondary">'
                     '<summary><span class="fi-widget"></span> '
-                    "Opcje zaawansowane — masowe przepięcie prac</summary>"
+                    "Opcje zaawansowane</summary>"
                 ),
                 Row(
                     Column("przepnij_wszystkie_prace", css_class="large-12 small-12"),
                 ),
+                Row(
+                    Column("nadpisuj_daty_zatrudnienia", css_class="large-12 small-12"),
+                ),
                 HTML("</details>"),
                 formdefaults_html_after(self),
             ),
diff --git a/src/import_pracownikow/templates/import_pracownikow/importpracownikow_form.html b/src/import_pracownikow/templates/import_pracownikow/importpracownikow_form.html
index bb44ead50..9ddc12338 100644
--- a/src/import_pracownikow/templates/import_pracownikow/importpracownikow_form.html
+++ b/src/import_pracownikow/templates/import_pracownikow/importpracownikow_form.html
@@ -22,10 +22,10 @@ <h1>Utwórz nowy import: <a class="button secondary" id="download-example-file"
     <div class="formdefaults-wrap">{% bpp_formdefaults_button form %}</div>
     {% crispy form %}
 
-    {# „Przepnij wszystkie prace" to opcja groźna na dojrzałej bazie — przy #}
-    {# ZAZNACZANIU wymuszamy świadome potwierdzenie (confirm). Anulowanie #}
-    {# cofa zaznaczenie. Vanilla JS na DOMContentLoaded (nie zależy od #}
-    {# kolejności $(document).ready). #}
+    {# Opcje z szuflady („przepnij wszystkie prace", „nadpisuj daty #}
+    {# zatrudnienia") są groźne — przy ZAZNACZANIU wymuszamy świadome #}
+    {# potwierdzenie (confirm). Anulowanie cofa zaznaczenie. Vanilla JS na #}
+    {# DOMContentLoaded (nie zależy od kolejności $(document).ready). #}
     <script>
     document.addEventListener('DOMContentLoaded', function () {
         var cb = document.getElementById('id_przepnij_wszystkie_prace');
@@ -43,6 +43,24 @@ <h1>Utwórz nowy import: <a class="button secondary" id="download-example-file"
             );
             if (!ok) { cb.checked = false; }
         });
+        var cbDaty = document.getElementById('id_nadpisuj_daty_zatrudnienia');
+        if (cbDaty) {
+            cbDaty.addEventListener('change', function () {
+                if (!cbDaty.checked) { return; }
+                var ok = confirm(
+                    "UWAGA — NADPISYWANIE DAT ZATRUDNIENIA\n\n" +
+                    "Zaznaczasz opcję, która przy zapisie osób NADPISZE " +
+                    "istniejące daty rozpoczęcia/zakończenia pracy " +
+                    "wartościami z pliku (dla osób obecnych w pliku).\n\n" +
+                    "Ręcznie ustawione daty w bazie zostaną UTRACONE tam, " +
+                    "gdzie plik niesie inną datę.\n\n" +
+                    "Użyj tylko, gdy plik jest źródłem prawdy o datach " +
+                    "(np. korekta po imporcie bez dat).\n\n" +
+                    "Czy na pewno wiesz, co robisz?"
+                );
+                if (!ok) { cbDaty.checked = false; }
+            });
+        }
     });
     </script>
 {% endblock %}
diff --git a/src/import_pracownikow/tests/test_nadpisywanie_dat.py b/src/import_pracownikow/tests/test_nadpisywanie_dat.py
index 60d2ef906..e403a1a72 100644
--- a/src/import_pracownikow/tests/test_nadpisywanie_dat.py
+++ b/src/import_pracownikow/tests/test_nadpisywanie_dat.py
@@ -6,6 +6,7 @@
 import pytest
 from model_bakery import baker
 
+from import_pracownikow.forms import NowyImportForm
 from import_pracownikow.models import ImportPracownikow
 
 
@@ -13,3 +14,20 @@
 def test_flaga_nadpisywania_domyslnie_wylaczona():
     parent = baker.make(ImportPracownikow)
     assert parent.nadpisuj_daty_zatrudnienia is False
+
+
+def test_formularz_ma_pole_nadpisywania_dat():
+    form = NowyImportForm()
+    assert "nadpisuj_daty_zatrudnienia" in form.fields
+    assert form.fields["nadpisuj_daty_zatrudnienia"].initial in (None, False)
+
+
+@pytest.mark.django_db
+def test_formularz_pole_nadpisywania_w_szufladzie():
+    from crispy_forms.utils import render_crispy_form
+
+    html = render_crispy_form(NowyImportForm())
+    assert "<details" in html
+    pozycja_details = html.index("<details")
+    pozycja_pola = html.index("nadpisuj_daty_zatrudnienia")
+    assert pozycja_pola > pozycja_details

From 204cfa6b4844ae5cc6742fc0ec642ace4ae441eb Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Micha=C5=82=20Pasternak?= <michal.dtz@gmail.com>
Date: Sun, 26 Jul 2026 18:09:56 +0200
Subject: [PATCH 05/11] feat(import_pracownikow): bramka zmiany_potrzebne
 swiadoma flagi nadpisywania dat

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
---
 src/import_pracownikow/models.py              | 13 ++++++
 .../tests/test_nadpisywanie_dat.py            | 45 ++++++++++++++++++-
 2 files changed, 57 insertions(+), 1 deletion(-)

diff --git a/src/import_pracownikow/models.py b/src/import_pracownikow/models.py
index 3f8638956..400a17298 100644
--- a/src/import_pracownikow/models.py
+++ b/src/import_pracownikow/models.py
@@ -1269,6 +1269,11 @@ def _check_autor_jednostka_needs_update(self, dane):
             # pracy. Guard przed dostępem do atrybutów None (checki niżej łapią
             # None dopiero przez short-circuit dane.get(...), a #4 primary nie).
             return False
+        # §3.3a specu nadpisywania dat: przy fladze „nadpisuj daty" różnica
+        # wobec ISTNIEJĄCEJ (niepustej) daty też wymaga integracji — bez
+        # flagi liczy się, jak dotąd, wyłącznie wypełnienie NULL-a.
+        nadpisywanie = self.parent.nadpisuj_daty_zatrudnienia
+        plik_od, plik_do = self._plik_od(), self._plik_do()
         checks = [
             # #4: rozpoczęcie stemplujemy TYLKO gdy puste (data z pliku / importu)
             # — integracja potrzebna, gdy plik niesie datę, a AJ jej nie ma.
@@ -1292,6 +1297,14 @@ def _check_autor_jednostka_needs_update(self, dane):
             # Stanowisko dydaktyczne — overwrite-if-different (mirror funkcja).
             self.stanowisko_dydaktyczne_id is not None
             and aj.stanowisko_id != self.stanowisko_dydaktyczne_id,
+            nadpisywanie
+            and plik_od is not None
+            and aj.rozpoczal_prace is not None
+            and aj.rozpoczal_prace != plik_od,
+            nadpisywanie
+            and plik_do is not None
+            and aj.zakonczyl_prace is not None
+            and aj.zakonczyl_prace != plik_do,
         ]
         return any(checks)
 
diff --git a/src/import_pracownikow/tests/test_nadpisywanie_dat.py b/src/import_pracownikow/tests/test_nadpisywanie_dat.py
index e403a1a72..19f77dba0 100644
--- a/src/import_pracownikow/tests/test_nadpisywanie_dat.py
+++ b/src/import_pracownikow/tests/test_nadpisywanie_dat.py
@@ -3,11 +3,14 @@
 nakładania okresów, licznik ostrzeżenia finalizacji (spec
 2026-07-26-import-nadpisywanie-dat-zatrudnienia-design.md)."""
 
+from datetime import date
+
 import pytest
 from model_bakery import baker
 
+from bpp.models import Autor, Autor_Jednostka, Jednostka
 from import_pracownikow.forms import NowyImportForm
-from import_pracownikow.models import ImportPracownikow
+from import_pracownikow.models import ImportPracownikow, ImportPracownikowRow
 
 
 @pytest.mark.django_db
@@ -31,3 +34,43 @@ def test_formularz_pole_nadpisywania_w_szufladzie():
     pozycja_details = html.index("<details")
     pozycja_pola = html.index("nadpisuj_daty_zatrudnienia")
     assert pozycja_pola > pozycja_details
+
+
+def _row_z_data(parent, dane, autor, jednostka, aj):
+    return baker.make(
+        ImportPracownikowRow,
+        parent=parent,
+        autor=autor,
+        jednostka=jednostka,
+        autor_jednostka=aj,
+        dane_znormalizowane=dane,
+    )
+
+
+def _scenariusz_tytulowy(nadpisuj):
+    """Otwarty okres od 2026-07-19 (fallback z poprzedniego importu),
+    plik niesie 2021-10-01. Jedyna różnica wiersza = data od."""
+    parent = baker.make(ImportPracownikow, nadpisuj_daty_zatrudnienia=nadpisuj)
+    autor, jednostka = baker.make(Autor), baker.make(Jednostka)
+    aj = baker.make(
+        Autor_Jednostka,
+        autor=autor,
+        jednostka=jednostka,
+        rozpoczal_prace=date(2026, 7, 19),
+        zakonczyl_prace=None,
+        podstawowe_miejsce_pracy=True,
+    )
+    row = _row_z_data(parent, {"data_zatrudnienia": "2021-10-01"}, autor, jednostka, aj)
+    return row, aj
+
+
+@pytest.mark.django_db
+def test_bramka_roznica_dat_flaga_on():
+    row, _ = _scenariusz_tytulowy(nadpisuj=True)
+    assert row.check_if_integration_needed() is True
+
+
+@pytest.mark.django_db
+def test_bramka_roznica_dat_flaga_off_jak_dzis():
+    row, _ = _scenariusz_tytulowy(nadpisuj=False)
+    assert row.check_if_integration_needed() is False

From 49fd9abcd3c8a53d0a701b6ea738f2529727ee9e Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Micha=C5=82=20Pasternak?= <michal.dtz@gmail.com>
Date: Sun, 26 Jul 2026 18:26:12 +0200
Subject: [PATCH 06/11] feat(import_pracownikow): nadpisywanie dat AJ
 wartosciami z pliku (flaga ON)

---
 src/import_pracownikow/models.py              | 67 +++++++++++++------
 .../tests/test_nadpisywanie_dat.py            | 64 ++++++++++++++++++
 2 files changed, 110 insertions(+), 21 deletions(-)

diff --git a/src/import_pracownikow/models.py b/src/import_pracownikow/models.py
index 400a17298..52fcf741b 100644
--- a/src/import_pracownikow/models.py
+++ b/src/import_pracownikow/models.py
@@ -1360,28 +1360,53 @@ def _ustaw_tytul_i_stopien_autora(self, a):
     def _integruj_daty_aj(self, aj, dane):
         """Ustawia daty zatrudnienia na powiązaniu z danych wiersza.
 
-        „Data od" (``rozpoczal_prace``) na ISTNIEJĄCYM AJ wypełniamy TYLKO gdy
-        baza ma ``NULL`` a plik NIESIE datę (§3: „wypełnienie NULL") — nie
-        nadpisujemy istniejącej daty. Pusty ``plik_od`` na istniejącym AJ →
-        NIC NIE ZMIENIAJ (§5), nawet gdy ``rozpoczal_prace`` jest ``NULL``.
-        Fallback ``data zmian → dziś`` dla NOWEGO okresu stemplujemy przy
-        MATERIALIZACJI (``integrate._materializuj_diff``), nie tutaj — świeży AJ
-        ma już ``rozpoczal_prace``, więc ta gałąź go nie dotyczy.
-
-        „Data do" (``zakonczyl_prace``) — wstaw-tylko-gdy-pusta (§3): różnicę
-        wobec istniejącej daty POKAZUJEMY w porównywarce, ale NIE nadpisujemy."""
-        if aj.rozpoczal_prace is None and dane.get("data_zatrudnienia"):
-            aj.rozpoczal_prace = dane["data_zatrudnienia"]
-            self.log_zmian["autor_jednostka"].append(
-                f"data rozpoczęcia pracy na {aj.rozpoczal_prace}"
-            )
+        Bez flagi „nadpisuj daty": „data od"/„data do" na ISTNIEJĄCYM AJ
+        wypełniamy TYLKO gdy baza ma ``NULL`` a plik NIESIE datę (§3:
+        „wypełnienie NULL") — istniejącej daty nie ruszamy; pusty plik →
+        nic nie zmieniaj (§5). Fallback ``data zmian → dziś`` dla NOWEGO
+        okresu stempluje materializacja (``integrate._materializuj_diff``).
+
+        Z flagą ``parent.nadpisuj_daty_zatrudnienia`` (spec §3.3): datę
+        różną od niepustej wartości w bazie NADPISUJEMY wartością z pliku
+        (osobno „od" i „do"); puste komórki nadal niczego nie kasują.
+
+        Zwraca ``True`` TYLKO gdy nadpisano niepustą wartość — sygnał dla
+        pre-checku nakładania okresów; wypełnienia NULL-i zwracają
+        ``False`` (idą dzisiejszą ścieżką, bez pre-checku)."""
+        nadpisywanie = self.parent.nadpisuj_daty_zatrudnienia
+        nadpisano = False
+
+        plik_od = dane.get("data_zatrudnienia")
+        if plik_od:
+            if aj.rozpoczal_prace is None:
+                aj.rozpoczal_prace = plik_od
+                self.log_zmian["autor_jednostka"].append(
+                    f"data rozpoczęcia pracy na {aj.rozpoczal_prace}"
+                )
+            elif nadpisywanie and aj.rozpoczal_prace != plik_od:
+                self.log_zmian["autor_jednostka"].append(
+                    f"data rozpoczęcia pracy: {aj.rozpoczal_prace} → "
+                    f"{plik_od} (nadpisano z pliku)"
+                )
+                aj.rozpoczal_prace = plik_od
+                nadpisano = True
 
         data_konca = dane.get("data_końca_zatrudnienia")
-        if data_konca and aj.zakonczyl_prace is None:
-            aj.zakonczyl_prace = data_konca
-            self.log_zmian["autor_jednostka"].append(
-                f"data końca zatrudnienia na {data_konca}"
-            )
+        if data_konca:
+            if aj.zakonczyl_prace is None:
+                aj.zakonczyl_prace = data_konca
+                self.log_zmian["autor_jednostka"].append(
+                    f"data końca zatrudnienia na {data_konca}"
+                )
+            elif nadpisywanie and aj.zakonczyl_prace != data_konca:
+                self.log_zmian["autor_jednostka"].append(
+                    f"data końca zatrudnienia: {aj.zakonczyl_prace} → "
+                    f"{data_konca} (nadpisano z pliku)"
+                )
+                aj.zakonczyl_prace = data_konca
+                nadpisano = True
+
+        return nadpisano
 
     def _integrate_autor_jednostka(self):
         aj = self.autor_jednostka
@@ -1392,7 +1417,7 @@ def _integrate_autor_jednostka(self):
             return
         dane = self.dane_bardziej_znormalizowane
 
-        self._integruj_daty_aj(aj, dane)
+        nadpisano_daty = self._integruj_daty_aj(aj, dane)
 
         # Niezmiennik rozpoczal < zakonczyl walidujemy PRZED jakimkolwiek zapisem.
         # Model.save() nie woła clean(), a ustaw_podstawowe_miejsce_pracy() niżej
diff --git a/src/import_pracownikow/tests/test_nadpisywanie_dat.py b/src/import_pracownikow/tests/test_nadpisywanie_dat.py
index 19f77dba0..5e3f7a684 100644
--- a/src/import_pracownikow/tests/test_nadpisywanie_dat.py
+++ b/src/import_pracownikow/tests/test_nadpisywanie_dat.py
@@ -44,6 +44,10 @@ def _row_z_data(parent, dane, autor, jednostka, aj):
         jednostka=jednostka,
         autor_jednostka=aj,
         dane_znormalizowane=dane,
+        # jawnie: `zmiany_potrzebne` to zwykły BooleanField (bez default),
+        # baker.make losowałby True/False — a `integrate()` niżej asertuje
+        # `self.zmiany_potrzebne` (musi być zdeterminowane, nie flaky).
+        zmiany_potrzebne=True,
     )
 
 
@@ -74,3 +78,63 @@ def test_bramka_roznica_dat_flaga_on():
 def test_bramka_roznica_dat_flaga_off_jak_dzis():
     row, _ = _scenariusz_tytulowy(nadpisuj=False)
     assert row.check_if_integration_needed() is False
+
+
+@pytest.mark.django_db
+def test_integracja_nadpisuje_date_od_flaga_on():
+    row, aj = _scenariusz_tytulowy(nadpisuj=True)
+    row.integrate()
+    aj.refresh_from_db()
+    assert aj.rozpoczal_prace == date(2021, 10, 1)
+    assert any("nadpisano z pliku" in wpis for wpis in row.log_zmian["autor_jednostka"])
+
+
+@pytest.mark.django_db
+def test_integracja_nie_nadpisuje_flaga_off():
+    row, aj = _scenariusz_tytulowy(nadpisuj=False)
+    # OFF: bramka nie przepuści wiersza; wołamy integrate() wprost, żeby
+    # potwierdzić, że nawet wtedy data NIE jest ruszana.
+    row.integrate()
+    aj.refresh_from_db()
+    assert aj.rozpoczal_prace == date(2026, 7, 19)
+
+
+@pytest.mark.django_db
+def test_integracja_nadpisuje_date_do():
+    parent = baker.make(ImportPracownikow, nadpisuj_daty_zatrudnienia=True)
+    autor, jednostka = baker.make(Autor), baker.make(Jednostka)
+    aj = baker.make(
+        Autor_Jednostka,
+        autor=autor,
+        jednostka=jednostka,
+        rozpoczal_prace=date(2020, 1, 1),
+        zakonczyl_prace=date(2026, 12, 31),
+    )
+    row = _row_z_data(
+        parent,
+        {"data_zatrudnienia": "2020-01-01", "data_końca_zatrudnienia": "2024-06-30"},
+        autor,
+        jednostka,
+        aj,
+    )
+    row.integrate()
+    aj.refresh_from_db()
+    assert aj.zakonczyl_prace == date(2024, 6, 30)
+
+
+@pytest.mark.django_db
+def test_pusta_komorka_nie_kasuje_daty_przy_on():
+    parent = baker.make(ImportPracownikow, nadpisuj_daty_zatrudnienia=True)
+    autor, jednostka = baker.make(Autor), baker.make(Jednostka)
+    aj = baker.make(
+        Autor_Jednostka,
+        autor=autor,
+        jednostka=jednostka,
+        rozpoczal_prace=date(2020, 1, 1),
+        zakonczyl_prace=date(2024, 6, 30),
+    )
+    row = _row_z_data(parent, {}, autor, jednostka, aj)
+    row.integrate()
+    aj.refresh_from_db()
+    assert aj.rozpoczal_prace == date(2020, 1, 1)
+    assert aj.zakonczyl_prace == date(2024, 6, 30)

From ba0e2143c4198c47d364084b0819ba7d7cba1cbc Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Micha=C5=82=20Pasternak?= <michal.dtz@gmail.com>
Date: Sun, 26 Jul 2026 18:42:56 +0200
Subject: [PATCH 07/11] feat(import_pracownikow): pre-check nakladania okresow
 przed nadpisaniem dat

---
 src/import_pracownikow/models.py              | 85 +++++++++++++++----
 .../tests/test_nadpisywanie_dat.py            | 70 +++++++++++++++
 2 files changed, 138 insertions(+), 17 deletions(-)

diff --git a/src/import_pracownikow/models.py b/src/import_pracownikow/models.py
index 52fcf741b..60c124670 100644
--- a/src/import_pracownikow/models.py
+++ b/src/import_pracownikow/models.py
@@ -1408,25 +1408,61 @@ def _integruj_daty_aj(self, aj, dane):
 
         return nadpisano
 
-    def _integrate_autor_jednostka(self):
-        aj = self.autor_jednostka
-        if aj is None:
-            # Ochrona: świeży okres mógł zostać scalony przez defragmentację i
-            # bez ocalałego AJ (`_przepnij_aj_po_defragmentacji`). Dane zatrudnienia
-            # niesie już scalony rekord — nie ma czego zapisywać.
-            return
-        dane = self.dane_bardziej_znormalizowane
+    def _sprawdz_nakladanie_okresow(self, aj):
+        """Pythonowe lustro constraintu
+        ``bpp_autor_jednostka_okresy_bez_nakladan`` (bpp/models/autor.py):
+        przedziały DOMKNIĘTE ``[od, do]``, ``zakonczyl_prace IS NULL`` =
+        otwarty w prawo ``[od, ∞)``. Wołane TYLKO po nadpisaniu niepustej
+        daty (flaga „nadpisuj daty") — naruszenie constraintu w bazie
+        zatruwa transakcję i psuje izolację wiersza, więc kolizję łapiemy
+        PRZED save (jak niezmiennik od<do wyżej)."""
+        from bpp.models import Autor_Jednostka
 
-        nadpisano_daty = self._integruj_daty_aj(aj, dane)
+        if aj.rozpoczal_prace is None:
+            return
+        od = aj.rozpoczal_prace
+        do = aj.zakonczyl_prace or date.max
+        koliduje = (
+            Autor_Jednostka.objects.filter(
+                autor_id=aj.autor_id,
+                jednostka_id=aj.jednostka_id,
+                rozpoczal_prace__isnull=False,
+            )
+            .exclude(pk=aj.pk)
+            .filter(rozpoczal_prace__lte=do)
+        )
+        for inny in koliduje:
+            if od <= (inny.zakonczyl_prace or date.max):
+                raise BPPDatabaseError(
+                    self.dane_z_xls,
+                    self,
+                    f"nadpisane daty ({od} – "
+                    f"{aj.zakonczyl_prace or 'obecnie'}) nakładają się na "
+                    f"inny okres zatrudnienia w tej jednostce "
+                    f"({inny.rozpoczal_prace} – "
+                    f"{inny.zakonczyl_prace or 'obecnie'})",
+                )
 
-        # Niezmiennik rozpoczal < zakonczyl walidujemy PRZED jakimkolwiek zapisem.
-        # Model.save() nie woła clean(), a ustaw_podstawowe_miejsce_pracy() niżej
-        # już utrwala aj (i zdejmuje flagę „podstawowe" z innych powiązań autora).
-        # Odwrócony zakres z XLS musi zostać odrzucony (BPPDatabaseError → izolacja
-        # wiersza) zanim cokolwiek trafi do bazy — inaczej przedwczesny save
-        # zderza się z DB-owym CHECK `poczatek_przed_koncem` (mig 0469) i daje
-        # nieizolowany CheckViolation. Reguły „koniec < dziś" celowo NIE
-        # egzekwujemy: import może nieść przyszłe (planowane) daty końca.
+    def _waliduj_daty_aj(self, aj, nadpisano_daty):
+        """Walidacja dat AJ PRZED jakimkolwiek zapisem (wydzielona z
+        ``_integrate_autor_jednostka``, żeby nie przekraczać limitu
+        złożoności cyklomatycznej — obie walidacje i tak są ze sobą
+        związane: obie muszą paść przed ``aj.save()`` niżej).
+
+        Niezmiennik rozpoczal < zakonczyl walidujemy PRZED jakimkolwiek
+        zapisem. Model.save() nie woła clean(), a
+        ustaw_podstawowe_miejsce_pracy() w wołającej metodzie już utrwala
+        aj (i zdejmuje flagę „podstawowe" z innych powiązań autora).
+        Odwrócony zakres z XLS musi zostać odrzucony (BPPDatabaseError →
+        izolacja wiersza) zanim cokolwiek trafi do bazy — inaczej
+        przedwczesny save zderza się z DB-owym CHECK
+        `poczatek_przed_koncem` (mig 0469) i daje nieizolowany
+        CheckViolation. Reguły „koniec < dziś" celowo NIE egzekwujemy:
+        import może nieść przyszłe (planowane) daty końca.
+
+        Pre-check nakładania okresów (lustro ExclusionConstraint) wołamy
+        TYLKO gdy nadpisano niepustą datę — patrz
+        ``_sprawdz_nakladanie_okresow``."""
         if (
             aj.rozpoczal_prace is not None
             and aj.zakonczyl_prace is not None
@@ -1439,6 +1475,21 @@ def _integrate_autor_jednostka(self):
                 f"lub równa dacie zakończenia ({aj.zakonczyl_prace})",
             )
 
+        if nadpisano_daty:
+            self._sprawdz_nakladanie_okresow(aj)
+
+    def _integrate_autor_jednostka(self):
+        aj = self.autor_jednostka
+        if aj is None:
+            # Ochrona: świeży okres mógł zostać scalony przez defragmentację i
+            # bez ocalałego AJ (`_przepnij_aj_po_defragmentacji`). Dane zatrudnienia
+            # niesie już scalony rekord — nie ma czego zapisywać.
+            return
+        dane = self.dane_bardziej_znormalizowane
+
+        nadpisano_daty = self._integruj_daty_aj(aj, dane)
+        self._waliduj_daty_aj(aj, nadpisano_daty)
+
         if self.funkcja_autora is not None and aj.funkcja != self.funkcja_autora:
             aj.funkcja = self.funkcja_autora
             self.log_zmian["autor_jednostka"].append(
diff --git a/src/import_pracownikow/tests/test_nadpisywanie_dat.py b/src/import_pracownikow/tests/test_nadpisywanie_dat.py
index 5e3f7a684..1ffe6dc55 100644
--- a/src/import_pracownikow/tests/test_nadpisywanie_dat.py
+++ b/src/import_pracownikow/tests/test_nadpisywanie_dat.py
@@ -9,6 +9,7 @@
 from model_bakery import baker
 
 from bpp.models import Autor, Autor_Jednostka, Jednostka
+from import_common.exceptions import BPPDatabaseError
 from import_pracownikow.forms import NowyImportForm
 from import_pracownikow.models import ImportPracownikow, ImportPracownikowRow
 
@@ -44,6 +45,11 @@ def _row_z_data(parent, dane, autor, jednostka, aj):
         jednostka=jednostka,
         autor_jednostka=aj,
         dane_znormalizowane=dane,
+        # jak w innych testach tej apki (np. test_stany_pol_materializacja.py):
+        # BPPDatabaseError.__str__ woła `self.elem.get(...)` bez guardu na
+        # None — bez tego str(exc.value) na ścieżce błędu wywala się
+        # AttributeError zamiast dać czytelny komunikat.
+        dane_z_xls={"__xls_loc_sheet__": 0, "__xls_loc_row__": 0},
         # jawnie: `zmiany_potrzebne` to zwykły BooleanField (bez default),
         # baker.make losowałby True/False — a `integrate()` niżej asertuje
         # `self.zmiany_potrzebne` (musi być zdeterminowane, nie flaky).
@@ -138,3 +144,67 @@ def test_pusta_komorka_nie_kasuje_daty_przy_on():
     aj.refresh_from_db()
     assert aj.rozpoczal_prace == date(2020, 1, 1)
     assert aj.zakonczyl_prace == date(2024, 6, 30)
+
+
+@pytest.mark.django_db
+def test_nadpisanie_kolidujace_z_zamknietym_okresem_izolowane():
+    # Zamknięty okres 2019-2022 + otwarty od 2026. Plik cofa otwarty na
+    # 2021 → przedziały [2021,∞) i [2019-…-2022] nakładają się → błąd
+    # izolowany PRZED save (constraint w bazie zatrułby transakcję).
+    parent = baker.make(ImportPracownikow, nadpisuj_daty_zatrudnienia=True)
+    autor, jednostka = baker.make(Autor), baker.make(Jednostka)
+    baker.make(
+        Autor_Jednostka,
+        autor=autor,
+        jednostka=jednostka,
+        rozpoczal_prace=date(2019, 1, 1),
+        zakonczyl_prace=date(2022, 12, 31),
+    )
+    aj = baker.make(
+        Autor_Jednostka,
+        autor=autor,
+        jednostka=jednostka,
+        rozpoczal_prace=date(2026, 7, 19),
+        zakonczyl_prace=None,
+    )
+    row = _row_z_data(parent, {"data_zatrudnienia": "2021-10-01"}, autor, jednostka, aj)
+    with pytest.raises(BPPDatabaseError) as exc:
+        row.integrate()
+    assert "nakładają się" in str(exc.value)
+    aj.refresh_from_db()
+    assert aj.rozpoczal_prace == date(2026, 7, 19)  # nic nie zapisano
+
+
+@pytest.mark.django_db
+def test_wypelnienie_null_nie_odpala_precheku():
+    # Wypełnienie NULL-a to dzisiejsza ścieżka — bez pre-checku, nawet
+    # przy fladze ON (spec §3.4).
+    parent = baker.make(ImportPracownikow, nadpisuj_daty_zatrudnienia=True)
+    autor, jednostka = baker.make(Autor), baker.make(Jednostka)
+    aj = baker.make(
+        Autor_Jednostka,
+        autor=autor,
+        jednostka=jednostka,
+        rozpoczal_prace=None,
+        zakonczyl_prace=None,
+    )
+    row = _row_z_data(parent, {"data_zatrudnienia": "2021-10-01"}, autor, jednostka, aj)
+    row.integrate()
+    aj.refresh_from_db()
+    assert aj.rozpoczal_prace == date(2021, 10, 1)
+
+
+@pytest.mark.django_db
+def test_od_wieksze_rowne_do_po_nadpisaniu_odrzucone():
+    parent = baker.make(ImportPracownikow, nadpisuj_daty_zatrudnienia=True)
+    autor, jednostka = baker.make(Autor), baker.make(Jednostka)
+    aj = baker.make(
+        Autor_Jednostka,
+        autor=autor,
+        jednostka=jednostka,
+        rozpoczal_prace=date(2020, 1, 1),
+        zakonczyl_prace=date(2021, 1, 1),
+    )
+    row = _row_z_data(parent, {"data_zatrudnienia": "2022-01-01"}, autor, jednostka, aj)
+    with pytest.raises(BPPDatabaseError):
+        row.integrate()

From 476cba038e5c4d760ade92c1a04f90f3dd992b04 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Micha=C5=82=20Pasternak?= <michal.dtz@gmail.com>
Date: Sun, 26 Jul 2026 19:01:21 +0200
Subject: [PATCH 08/11] feat(import_pracownikow): ostrzezenie finalizacji o
 nadpisaniu dat (callout + confirm)

---
 src/import_pracownikow/models.py              | 43 +++++++++
 .../partials/_ostrzezenie_nadpisanie_dat.html | 13 +++
 .../import_pracownikow/przeglad.html          |  5 +-
 .../tests/test_nadpisywanie_dat.py            | 91 +++++++++++++++++++
 src/import_pracownikow/views.py               |  7 ++
 5 files changed, 158 insertions(+), 1 deletion(-)
 create mode 100644 src/import_pracownikow/templates/import_pracownikow/partials/_ostrzezenie_nadpisanie_dat.html

diff --git a/src/import_pracownikow/models.py b/src/import_pracownikow/models.py
index 60c124670..7c0939ab2 100644
--- a/src/import_pracownikow/models.py
+++ b/src/import_pracownikow/models.py
@@ -583,6 +583,27 @@ def liczba_wierszy_do_pominiecia(self):
             autor__isnull=True, utworz_nowego=False
         ).count()
 
+    def liczba_nadpisan_dat(self):
+        """Ile wierszy przy zapisie osób NADPISZE istniejącą datę
+        zatrudnienia (flaga ``nadpisuj_daty_zatrudnienia``) — do calloutu
+        i confirmu finalizacji. Liczone LIVE (``stany_pol_snapshot`` bywa
+        NULL do backfillu i miesza wypełnienia NULL-i z nadpisaniami);
+        ``wstepnie_zaladuj_okresy`` + przypięcie ``parent`` chronią przed
+        N+1."""
+        if not self.nadpisuj_daty_zatrudnienia:
+            return 0
+        from import_pracownikow.okresy import wstepnie_zaladuj_okresy
+
+        rows = list(
+            self.importpracownikowrow_set.filter(
+                autor__isnull=False, jednostka__isnull=False
+            ).select_related("autor", "jednostka")
+        )
+        for row in rows:
+            row.parent = self
+        wstepnie_zaladuj_okresy(rows)
+        return sum(1 for row in rows if row.nadpisze_daty())
+
     @staticmethod
     def _liczniki_decyzji(queryset, tryb_brak, tryb_zgadywanie):
         """Rozkład NIEROZSTRZYGNIĘTYCH decyzji (jednostek/tytułów) po ``tryb``:
@@ -1046,6 +1067,28 @@ def _plik_do(self):
             return None
         return self.dane_bardziej_znormalizowane.get("data_końca_zatrudnienia") or None
 
+    def nadpisze_daty(self):
+        """Czy zapis osób NADPISZE niepustą datę tego wiersza (flaga
+        ``nadpisuj_daty_zatrudnienia``). Liczy TYLKO realne nadpisania —
+        obie strony niepuste i różne; wypełnienia NULL-i i nowe okresy to
+        NIE nadpisania (spec §3.5, stan „zmienione" ze ``stany_pol`` byłby
+        zawyżony). Zasila licznik ostrzeżenia finalizacji."""
+        if not self.parent.nadpisuj_daty_zatrudnienia:
+            return False
+        if self.autor_id is None or self.jednostka_id is None:
+            return False
+        from import_pracownikow.okresy import rozwiaz_okres_zatrudnienia
+
+        rodzaj, aj = rozwiaz_okres_zatrudnienia(
+            self.autor, self.jednostka, self._plik_od(), aj_lista=self._aj_lista()
+        )
+        if rodzaj != "istniejacy":
+            return False
+        plik_od, plik_do = self._plik_od(), self._plik_do()
+        return bool(
+            plik_od and aj.rozpoczal_prace and aj.rozpoczal_prace != plik_od
+        ) or bool(plik_do and aj.zakonczyl_prace and aj.zakonczyl_prace != plik_do)
+
     def _aj_lista(self):
         """Lista okresów ``Autor_Jednostka`` dla ``(autor, jednostka)`` wiersza —
         JEDNO zapytanie (memo na instancji), współdzielone przez resolver
diff --git a/src/import_pracownikow/templates/import_pracownikow/partials/_ostrzezenie_nadpisanie_dat.html b/src/import_pracownikow/templates/import_pracownikow/partials/_ostrzezenie_nadpisanie_dat.html
new file mode 100644
index 000000000..a03fc9566
--- /dev/null
+++ b/src/import_pracownikow/templates/import_pracownikow/partials/_ostrzezenie_nadpisanie_dat.html
@@ -0,0 +1,13 @@
+{# Ostrzeżenie finalizacji: flaga „nadpisuj daty zatrudnienia" — zapis #}
+{# osób NADPISZE istniejące daty wartościami z pliku (spec nadpisywania #}
+{# dat §3.5). Liczba = realne nadpisania (obie strony niepuste i różne). #}
+<div class="callout alert">
+    <p>
+        <span class="fi-alert"></span>
+        <strong>Włączono nadpisywanie dat zatrudnienia.</strong>
+        Wiersze z datami różnymi od bazy:
+        <strong>{{ liczba_nadpisan_dat }}</strong> — ich daty rozpoczęcia
+        / zakończenia pracy zostaną <strong>NADPISANE</strong>
+        wartościami z pliku. Ręcznie ustawione daty zostaną utracone.
+    </p>
+</div>
diff --git a/src/import_pracownikow/templates/import_pracownikow/przeglad.html b/src/import_pracownikow/templates/import_pracownikow/przeglad.html
index a476ecdab..760dac6be 100644
--- a/src/import_pracownikow/templates/import_pracownikow/przeglad.html
+++ b/src/import_pracownikow/templates/import_pracownikow/przeglad.html
@@ -194,6 +194,9 @@ <h3><span class="fi-alert"></span> Najpierw słowniki</h3>
                     {# Ostrzeżenie #4 (PR #531): zapis zmienia podstawowe #}
                     {# miejsce pracy — szczegółowy warning tuż przed przyciskiem. #}
                     {% include "import_pracownikow/partials/_ostrzezenie_podstawowe_miejsce.html" %}
+                    {% if nadpisywanie_dat_wlaczone %}
+                        {% include "import_pracownikow/partials/_ostrzezenie_nadpisanie_dat.html" %}
+                    {% endif %}
                     {# Ostrzeżenie finalizacji: wiersze bez decyzji zostaną #}
                     {# pominięte. {% with %} liczy COUNT raz (nie 2× w if+confirm). #}
                     {% with liczba_pominietych=parent_object.liczba_wierszy_do_pominiecia %}
@@ -202,7 +205,7 @@ <h3><span class="fi-alert"></span> Najpierw słowniki</h3>
                         {% endif %}
                         <form method="post"
                               action="{% url "import_pracownikow:zatwierdz" parent_object.pk %}"
-                              {% if liczba_pominietych %}onsubmit="return confirm('Wiersze bez dopasowania zostaną pominięte: {{ liczba_pominietych }}. Kontynuować zapis?');"{% endif %}>
+                              {% if liczba_pominietych or liczba_nadpisan_dat %}onsubmit="return confirm('{% if liczba_pominietych %}Wiersze bez dopasowania zostaną pominięte: {{ liczba_pominietych }}. {% endif %}{% if liczba_nadpisan_dat %}Daty zatrudnienia zostaną NADPISANE wartościami z pliku dla wierszy: {{ liczba_nadpisan_dat }}. {% endif %}Kontynuować zapis?');"{% endif %}>
                             {% csrf_token %}
                             <input type="hidden" name="zakres" value="pelny">
                             <label>
diff --git a/src/import_pracownikow/tests/test_nadpisywanie_dat.py b/src/import_pracownikow/tests/test_nadpisywanie_dat.py
index 1ffe6dc55..874adc018 100644
--- a/src/import_pracownikow/tests/test_nadpisywanie_dat.py
+++ b/src/import_pracownikow/tests/test_nadpisywanie_dat.py
@@ -3,9 +3,11 @@
 nakładania okresów, licznik ostrzeżenia finalizacji (spec
 2026-07-26-import-nadpisywanie-dat-zatrudnienia-design.md)."""
 
+import re
 from datetime import date
 
 import pytest
+from django.urls import reverse
 from model_bakery import baker
 
 from bpp.models import Autor, Autor_Jednostka, Jednostka
@@ -208,3 +210,92 @@ def test_od_wieksze_rowne_do_po_nadpisaniu_odrzucone():
     row = _row_z_data(parent, {"data_zatrudnienia": "2022-01-01"}, autor, jednostka, aj)
     with pytest.raises(BPPDatabaseError):
         row.integrate()
+
+
+@pytest.mark.django_db
+def test_liczba_nadpisan_dat_liczy_tylko_realne_nadpisania():
+    parent = baker.make(ImportPracownikow, nadpisuj_daty_zatrudnienia=True)
+    autor1, autor2, autor3 = baker.make(Autor), baker.make(Autor), baker.make(Autor)
+    jednostka = baker.make(Jednostka)
+    # 1) realne nadpisanie: baza 2026, plik 2021 → LICZY SIĘ
+    aj1 = baker.make(
+        Autor_Jednostka,
+        autor=autor1,
+        jednostka=jednostka,
+        rozpoczal_prace=date(2026, 7, 19),
+    )
+    _row_z_data(parent, {"data_zatrudnienia": "2021-10-01"}, autor1, jednostka, aj1)
+    # 2) wypełnienie NULL-a → NIE liczy się
+    aj2 = baker.make(
+        Autor_Jednostka, autor=autor2, jednostka=jednostka, rozpoczal_prace=None
+    )
+    _row_z_data(parent, {"data_zatrudnienia": "2021-10-01"}, autor2, jednostka, aj2)
+    # 3) zgodne daty → NIE liczy się
+    aj3 = baker.make(
+        Autor_Jednostka,
+        autor=autor3,
+        jednostka=jednostka,
+        rozpoczal_prace=date(2021, 10, 1),
+    )
+    _row_z_data(parent, {"data_zatrudnienia": "2021-10-01"}, autor3, jednostka, aj3)
+    assert parent.liczba_nadpisan_dat() == 1
+
+
+@pytest.mark.django_db
+def test_liczba_nadpisan_dat_zero_przy_fladze_off():
+    parent = baker.make(ImportPracownikow, nadpisuj_daty_zatrudnienia=False)
+    autor, jednostka = baker.make(Autor), baker.make(Jednostka)
+    aj = baker.make(
+        Autor_Jednostka,
+        autor=autor,
+        jednostka=jednostka,
+        rozpoczal_prace=date(2026, 7, 19),
+    )
+    _row_z_data(parent, {"data_zatrudnienia": "2021-10-01"}, autor, jednostka, aj)
+    assert parent.liczba_nadpisan_dat() == 0
+
+
+def _przeglad_url(parent):
+    return reverse("import_pracownikow:przeglad", kwargs={"pk": parent.pk})
+
+
+def _parent_faza_osob_z_nadpisaniem(owner, nadpisuj):
+    """Parent w stanie fazy osób (Krok 2 — struktura już zapisana) + 1
+    wiersz z realnym nadpisaniem (baza 2026-07-19, plik 2021-10-01), jak
+    w ``test_liczba_nadpisan_dat_liczy_tylko_realne_nadpisania``."""
+    parent = baker.make(
+        ImportPracownikow,
+        owner=owner,
+        stan=ImportPracownikow.STAN_STRUKTURA_ZINTEGROWANA,
+        nadpisuj_daty_zatrudnienia=nadpisuj,
+    )
+    autor, jednostka = baker.make(Autor), baker.make(Jednostka)
+    aj = baker.make(
+        Autor_Jednostka,
+        autor=autor,
+        jednostka=jednostka,
+        rozpoczal_prace=date(2026, 7, 19),
+    )
+    _row_z_data(parent, {"data_zatrudnienia": "2021-10-01"}, autor, jednostka, aj)
+    return parent
+
+
+@pytest.mark.django_db
+def test_przeglad_callout_nadpisywania_widoczny_przy_on(admin_client, admin_user):
+    parent = _parent_faza_osob_z_nadpisaniem(admin_user, nadpisuj=True)
+    resp = admin_client.get(_przeglad_url(parent))
+    html = resp.content.decode("utf-8")
+    assert "Włączono nadpisywanie dat zatrudnienia" in html
+    # NADPISANE musi być w confirm-owym onsubmit formularza zapisu osób —
+    # nie wystarczy, że jest gdzieś na stronie (np. sam callout).
+    match = re.search(r'onsubmit="([^"]*)"', html)
+    assert match is not None
+    assert "NADPISANE" in match.group(1)
+
+
+@pytest.mark.django_db
+def test_przeglad_bez_calloutu_przy_off(admin_client, admin_user):
+    parent = _parent_faza_osob_z_nadpisaniem(admin_user, nadpisuj=False)
+    resp = admin_client.get(_przeglad_url(parent))
+    html = resp.content.decode("utf-8")
+    assert "Włączono nadpisywanie dat zatrudnienia" not in html
diff --git a/src/import_pracownikow/views.py b/src/import_pracownikow/views.py
index 4c5e065b8..a28b79cca 100644
--- a/src/import_pracownikow/views.py
+++ b/src/import_pracownikow/views.py
@@ -1034,6 +1034,13 @@ def get_context_data(self, **kwargs):
                 # Item 6: ekran audytu (log zmian) ma sens po pełnej integracji
                 # osób — wtedy wiersze mają zapisany log_zmian.
                 "pokaz_audyt": parent.stan == ImportPracownikow.STAN_ZINTEGROWANY,
+                # Ostrzeżenie finalizacji (spec nadpisywania dat §3.5):
+                # licznik liczony tylko w fazie osób — w Kroku 1 formularz
+                # zapisu osób i tak się nie renderuje.
+                "nadpisywanie_dat_wlaczone": parent.nadpisuj_daty_zatrudnienia,
+                "liczba_nadpisan_dat": (
+                    parent.liczba_nadpisan_dat() if parent.faza_osob else 0
+                ),
             }
         )
         return ctx

From 7d03901697afee750b9a422f96ad39a20536e3e2 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Micha=C5=82=20Pasternak?= <michal.dtz@gmail.com>
Date: Sun, 26 Jul 2026 19:09:31 +0200
Subject: [PATCH 09/11] docs(import_pracownikow): newsfragment + dokumentacja
 nadpisywania dat

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
---
 docs/administrator/import-pracownikow.md      | 19 +++++++++++++++++++
 .../nadpisywanie-dat-zatrudnienia.feature.rst |  6 ++++++
 2 files changed, 25 insertions(+)
 create mode 100644 src/bpp/newsfragments/nadpisywanie-dat-zatrudnienia.feature.rst

diff --git a/docs/administrator/import-pracownikow.md b/docs/administrator/import-pracownikow.md
index 1dbb62251..cea6a8cc9 100644
--- a/docs/administrator/import-pracownikow.md
+++ b/docs/administrator/import-pracownikow.md
@@ -51,6 +51,25 @@ Aby uruchomić procedure importu danych, wystarczy dodać plik do systemu przy p
     rozpocznie się automatycznie.
 
 
+## Nadpisywanie dat zatrudnienia wartościami z pliku
+
+Domyślnie import **nigdy nie nadpisuje** istniejących dat zatrudnienia —
+wypełnia tylko puste („data od"/„data do" bez wartości w bazie), a
+różnice pokazuje wyłącznie w podglądzie. Jeśli daty w bazie są błędne
+(np. poprzedni import pliku bez dat ostemplował wszystkich datą importu),
+zaznacz w szufladzie „Opcje zaawansowane" opcję **Nadpisuj daty
+zatrudnienia (od/do) wartościami z pliku**:
+
+- nadpisywane są tylko daty osób obecnych w pliku, tam gdzie plik niesie
+  datę różną od bazy;
+- puste komórki pliku niczego nie kasują;
+- przy zaznaczaniu opcji oraz przed końcowym zapisem system prosi o
+  potwierdzenie i pokazuje liczbę wierszy, których daty zostaną
+  nadpisane;
+- nadpisanie, które spowodowałoby nałożenie się dwóch okresów
+  zatrudnienia w tej samej jednostce, jest odrzucane per-wiersz z
+  czytelnym błędem.
+
 ## Odpinanie nieaktualnych miejsc pracy
 
 Po zaimportowaniu listy pracowników system prezentuje raport z dokonanych zmian.
diff --git a/src/bpp/newsfragments/nadpisywanie-dat-zatrudnienia.feature.rst b/src/bpp/newsfragments/nadpisywanie-dat-zatrudnienia.feature.rst
new file mode 100644
index 000000000..2718ed02b
--- /dev/null
+++ b/src/bpp/newsfragments/nadpisywanie-dat-zatrudnienia.feature.rst
@@ -0,0 +1,6 @@
+Import pracowników: nowa opcja „Nadpisuj daty zatrudnienia (od/do)
+wartościami z pliku" (szuflada „Opcje zaawansowane"). Pozwala skorygować
+daty istniejących okresów zatrudnienia — np. po wcześniejszym imporcie
+pliku bez dat, który ostemplował wszystkich datą importu. Puste komórki
+niczego nie kasują; przed zapisem system pokazuje liczbę nadpisań i prosi
+o potwierdzenie.

From ee2d07ec1dae0e958adf1d9b9c50aa12ebf4e160 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Micha=C5=82=20Pasternak?= <michal.dtz@gmail.com>
Date: Sun, 26 Jul 2026 19:36:53 +0200
Subject: [PATCH 10/11] test(import_pracownikow): e2e przez _integruj_wiersz +
 granice domkniete przedzialow

Domyka luke z finalnego review feat/import-nadpisywanie-dat: dotychczasowe
testy wolaly row.integrate() wprost, wiec swiezy re-check w _integruj_wiersz
(pipeline/integrate.py), zamrozenie snapshotu i izolacja per-wiersz przez
_oznacz_wiersz_bledny nie mialy pokrycia z flaga nadpisywania dat wlaczona.
Dodano tez dwa testy granicy domknietych przedzialow w
_sprawdz_nakladanie_okresow (wspolny dzien koliduje, dzien-po-koncu nie).
---
 .../tests/test_nadpisywanie_dat.py            | 128 ++++++++++++++++++
 1 file changed, 128 insertions(+)

diff --git a/src/import_pracownikow/tests/test_nadpisywanie_dat.py b/src/import_pracownikow/tests/test_nadpisywanie_dat.py
index 874adc018..95b1b9c6c 100644
--- a/src/import_pracownikow/tests/test_nadpisywanie_dat.py
+++ b/src/import_pracownikow/tests/test_nadpisywanie_dat.py
@@ -14,6 +14,10 @@
 from import_common.exceptions import BPPDatabaseError
 from import_pracownikow.forms import NowyImportForm
 from import_pracownikow.models import ImportPracownikow, ImportPracownikowRow
+from import_pracownikow.pipeline.integrate import (
+    _integruj_wiersz,
+    _oznacz_wiersz_bledny,
+)
 
 
 @pytest.mark.django_db
@@ -177,6 +181,64 @@ def test_nadpisanie_kolidujace_z_zamknietym_okresem_izolowane():
     assert aj.rozpoczal_prace == date(2026, 7, 19)  # nic nie zapisano
 
 
+@pytest.mark.django_db
+def test_nakladanie_wspolny_dzien_koliduje():
+    # Granica przedziałów DOMKNIĘTYCH (_sprawdz_nakladanie_okresow lustrzy
+    # DATERANGE(..., '[]') z bpp/models/autor.py): inny okres zamknięty
+    # kończy się DOKŁADNIE w dniu, na który plik nadpisuje początek innego
+    # okresu → [2019-01-01, 2021-10-01] i [2021-10-01, ∞) dzielą wspólny
+    # dzień → kolidują (odwrotnie niż „dzień po końcu" niżej).
+    parent = baker.make(ImportPracownikow, nadpisuj_daty_zatrudnienia=True)
+    autor, jednostka = baker.make(Autor), baker.make(Jednostka)
+    baker.make(
+        Autor_Jednostka,
+        autor=autor,
+        jednostka=jednostka,
+        rozpoczal_prace=date(2019, 1, 1),
+        zakonczyl_prace=date(2021, 10, 1),
+    )
+    aj = baker.make(
+        Autor_Jednostka,
+        autor=autor,
+        jednostka=jednostka,
+        rozpoczal_prace=date(2026, 7, 19),
+        zakonczyl_prace=None,
+    )
+    row = _row_z_data(parent, {"data_zatrudnienia": "2021-10-01"}, autor, jednostka, aj)
+    with pytest.raises(BPPDatabaseError) as exc:
+        row.integrate()
+    assert "nakładają się" in str(exc.value)
+    aj.refresh_from_db()
+    assert aj.rozpoczal_prace == date(2026, 7, 19)  # nic nie zapisano
+
+
+@pytest.mark.django_db
+def test_nakladanie_dzien_po_koncu_nie_koliduje():
+    # Dzień PO końcu innego zamkniętego okresu — przedziały domknięte
+    # [2019-01-01, 2021-09-30] i [2021-10-01, ∞) SĄSIADUJĄ, ale się nie
+    # nakładają → brak kolizji, nadpisanie przechodzi.
+    parent = baker.make(ImportPracownikow, nadpisuj_daty_zatrudnienia=True)
+    autor, jednostka = baker.make(Autor), baker.make(Jednostka)
+    baker.make(
+        Autor_Jednostka,
+        autor=autor,
+        jednostka=jednostka,
+        rozpoczal_prace=date(2019, 1, 1),
+        zakonczyl_prace=date(2021, 9, 30),
+    )
+    aj = baker.make(
+        Autor_Jednostka,
+        autor=autor,
+        jednostka=jednostka,
+        rozpoczal_prace=date(2026, 7, 19),
+        zakonczyl_prace=None,
+    )
+    row = _row_z_data(parent, {"data_zatrudnienia": "2021-10-01"}, autor, jednostka, aj)
+    row.integrate()
+    aj.refresh_from_db()
+    assert aj.rozpoczal_prace == date(2021, 10, 1)
+
+
 @pytest.mark.django_db
 def test_wypelnienie_null_nie_odpala_precheku():
     # Wypełnienie NULL-a to dzisiejsza ścieżka — bez pre-checku, nawet
@@ -299,3 +361,69 @@ def test_przeglad_bez_calloutu_przy_off(admin_client, admin_user):
     resp = admin_client.get(_przeglad_url(parent))
     html = resp.content.decode("utf-8")
     assert "Włączono nadpisywanie dat zatrudnienia" not in html
+
+
+# Testy e2e przez pipeline'owy `_integruj_wiersz` (pipeline/integrate.py) —
+# w odróżnieniu od testów wyżej, które wołają `row.integrate()` wprost, te
+# przechodzą też przez świeży re-check (integrate.py ~L227), zamrożenie
+# `stany_pol_snapshot` PRZED materializacją i (w scenariuszu kolizji) tę samą
+# izolację per-wiersz co pętla integracji w `integruj()` (~L1005-1013).
+
+
+def _integruj_wiersz_izolowany(row):
+    """Odbicie pętli integracji w ``integruj()`` (pipeline/integrate.py
+    ~L1005-1013): łapie ``BPPDatabaseError`` per-wiersz i oznacza wiersz
+    ``_oznacz_wiersz_bledny`` — DOKŁADNIE tak, jak robi to prawdziwy
+    pipeline, żeby błąd JEDNEGO wiersza nie wywalił całej integracji."""
+    try:
+        return _integruj_wiersz(row)
+    except BPPDatabaseError as e:
+        _oznacz_wiersz_bledny(row, e.reason)
+        return None
+
+
+@pytest.mark.django_db
+def test_integruj_wiersz_scenariusz_tytulowy_nadpisuje_przez_pipeline():
+    """Spec §3.6 pkt 2: pełna ścieżka pipeline'u (nie sam
+    ``row.integrate()``) nadpisuje datę i NIE oznacza wiersza fałszywie
+    ``pominiety_bo_nieaktualny`` (świeży re-check widzi realną potrzebę
+    zmiany, więc bierze normalną gałąź integracji, nie gałąź driftu)."""
+    row, aj = _scenariusz_tytulowy(nadpisuj=True)
+    wynik = _integruj_wiersz(row)
+    assert wynik is False  # to NIE jest nowy okres — istniejący AJ
+    aj.refresh_from_db()
+    row.refresh_from_db()
+    assert aj.rozpoczal_prace == date(2021, 10, 1)
+    assert row.pominiety_bo_nieaktualny is False
+
+
+@pytest.mark.django_db
+def test_integruj_wiersz_kolizja_izolowana_oznacza_wiersz_bledny():
+    """Kolizja z zamkniętym okresem przechodząca przez ``_integruj_wiersz``
+    + izolację pętli integracji (``_integruj_wiersz_izolowany`` wyżej):
+    błąd JEDNEGO wiersza NIE propaguje się na zewnątrz (per-wiersz
+    izolacja), wiersz zostaje oznaczony ``log_zmian["blad"]`` przez
+    ``_oznacz_wiersz_bledny``, a data AJ zostaje nietknięta."""
+    parent = baker.make(ImportPracownikow, nadpisuj_daty_zatrudnienia=True)
+    autor, jednostka = baker.make(Autor), baker.make(Jednostka)
+    baker.make(
+        Autor_Jednostka,
+        autor=autor,
+        jednostka=jednostka,
+        rozpoczal_prace=date(2019, 1, 1),
+        zakonczyl_prace=date(2022, 12, 31),
+    )
+    aj = baker.make(
+        Autor_Jednostka,
+        autor=autor,
+        jednostka=jednostka,
+        rozpoczal_prace=date(2026, 7, 19),
+        zakonczyl_prace=None,
+    )
+    row = _row_z_data(parent, {"data_zatrudnienia": "2021-10-01"}, autor, jednostka, aj)
+    wynik = _integruj_wiersz_izolowany(row)  # nie podnosi — izolacja
+    assert wynik is None
+    row.refresh_from_db()
+    assert "nakładają się" in row.log_zmian["blad"][0]
+    aj.refresh_from_db()
+    assert aj.rozpoczal_prace == date(2026, 7, 19)  # nic nie zapisano

From de3a0297f68cc28b7b352352ed2a7e60b716b959 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Micha=C5=82=20Pasternak?= <michal.dtz@gmail.com>
Date: Sun, 26 Jul 2026 19:37:03 +0200
Subject: [PATCH 11/11] docs(import_pracownikow): doprecyzowanie licznika
 nadpisan

liczba_nadpisan_dat() nie wola pre-checku nakladania okresow, wiec wiersz
policzony jako "nadpisanie" moze pozniej zostac odrzucony przy realnej
integracji -- docstring dopisuje to zastrzezenie (orientacyjna, moze byc
zawyzona). Poprawiono tez opis w docs/administrator/import-pracownikow.md:
liczba wierszy jest znana i pokazywana dopiero przed koncowym zapisem, nie
juz przy zaznaczaniu checkboxa (tam jest tylko ostrzezenie bez liczby).
---
 docs/administrator/import-pracownikow.md | 6 +++---
 src/import_pracownikow/models.py         | 8 +++++++-
 2 files changed, 10 insertions(+), 4 deletions(-)

diff --git a/docs/administrator/import-pracownikow.md b/docs/administrator/import-pracownikow.md
index cea6a8cc9..ee1be6718 100644
--- a/docs/administrator/import-pracownikow.md
+++ b/docs/administrator/import-pracownikow.md
@@ -63,9 +63,9 @@ zatrudnienia (od/do) wartościami z pliku**:
 - nadpisywane są tylko daty osób obecnych w pliku, tam gdzie plik niesie
   datę różną od bazy;
 - puste komórki pliku niczego nie kasują;
-- przy zaznaczaniu opcji oraz przed końcowym zapisem system prosi o
-  potwierdzenie i pokazuje liczbę wierszy, których daty zostaną
-  nadpisane;
+- przy zaznaczaniu opcji system prosi o potwierdzenie ryzyka (bez liczby
+  wierszy — ta jeszcze nie jest znana), a dopiero przed końcowym zapisem
+  pokazuje liczbę wierszy, których daty zostaną nadpisane;
 - nadpisanie, które spowodowałoby nałożenie się dwóch okresów
   zatrudnienia w tej samej jednostce, jest odrzucane per-wiersz z
   czytelnym błędem.
diff --git a/src/import_pracownikow/models.py b/src/import_pracownikow/models.py
index 7c0939ab2..51e1982e2 100644
--- a/src/import_pracownikow/models.py
+++ b/src/import_pracownikow/models.py
@@ -589,7 +589,13 @@ def liczba_nadpisan_dat(self):
         i confirmu finalizacji. Liczone LIVE (``stany_pol_snapshot`` bywa
         NULL do backfillu i miesza wypełnienia NULL-i z nadpisaniami);
         ``wstepnie_zaladuj_okresy`` + przypięcie ``parent`` chronią przed
-        N+1."""
+        N+1.
+
+        Liczba jest orientacyjna i może być zawyżona: ``nadpisze_daty()``
+        porównuje wyłącznie daty, nie woła pre-checku nakładania okresów
+        (``_sprawdz_nakladanie_okresow``) — wiersz policzony tu jako
+        „nadpisanie" może przy faktycznej integracji zostać odrzucony
+        (kolizja z innym okresem) i mimo to zostać wliczony."""
         if not self.nadpisuj_daty_zatrudnienia:
             return 0
         from import_pracownikow.okresy import wstepnie_zaladuj_okresy