# Raport: polskie znaki w tekstach dla użytkownika

Gałąź `polskie-znaki`, 2026-08-23. Zmiana treści napisów widocznych dla
użytkownika — bez zmiany zachowania aplikacji.

## Zakres i liczby

Zmienione stringi (etykiety, komunikaty, tytuły, atrybuty `title`) w podziale
na obszar, na podstawie `git diff --stat`:

| Obszar | Pliki | Zmienione linie |
|---|---|---|
| Szablony Twig (`templates/**/*.twig`) | 13 | 84 (+42/-42) |
| Formularze (`src/Form/`) | 5 | 34 (+17/-17) |
| Domena `src/League/` (enumy, wyjątki, scorery) | 6 | 30 (+15/-15) |
| Kontrolery (`FormError`, `addFlash`, wyjątki 404) | 5 | 36 (+18/-18) |
| Komendy konsolowe (`src/Command/`) | 2 | 32 (+16/-16) |
| **Razem, kod aplikacji** | **31 plików** | **208 linii (104 wstawień/104 usunięć)** |
| Testy zaktualizowane do nowego brzmienia | 9 plików | 75 linii (+41/-34) |

`src/Entity/Element::POSITIONS` (`BR`, `OB`, `PO`, `NA`, `TR`) sprawdzony i
pozostawiony bez zmian — te dwuliterowe skróty nie zawierają liter z
diakrytykami. Kody statystyk (`G`, `A`, `CS`, `DC`, `BP`, `PM`, `OG`, `YC`,
`RC`) i nazwy chipów (BB, TC, FH, WC, Wildcard, Free Hit, Bench Boost, Triple
Captain) pozostawione bez zmian zgodnie z zamówieniem — zmienione zostały
wyłącznie pełne nazwy statystyk w atrybutach `title` (np. „Wkład defensywny",
„Gol samobójczy", „Żółta kartka").

Komentarze w kodzie (`//`, `/* */`, docbloki PHP i Twig `{# #}`) celowo
pozostawione bez zmian — zweryfikowane przeglądem każdego pliku, w którym
występowały słowa bez ogonków pasujące do wzorców wyszukiwania (m.in.
`SquadManager.php`, `SumScorer.php`, `MatchFormat.php`, `DifferentialReport.php`,
`PlayerDifferential.php`, `FplTeamImportCommand.php`, kontrolery) — wszystkie
trafienia leżały w komentarzach albo w identyfikatorach (np. nazwa parametru
zapytania `odswiez` w adresach odświeżania), nie w tekście dla użytkownika.

Wyjątki z `src/Entity/Squad.php` i `src/Entity/LeagueFixture.php`
(`InvalidArgumentException` o nieprawidłowym numerze slotu) świadomie
pozostawione bez zmian — to kontrole niezmienników wewnętrznych wywoływane
zawsze z twardo zakodowanymi wartościami, nieosiągalne z poziomu formularza, a
zadanie explicite ograniczało zakres wyjątków do `src/League/`.

## Dlaczego testy wymagały aktualizacji

Bardzo duża część testów funkcjonalnych i integracyjnych asercjonuje treść
komunikatów przez `assertSelectorTextContains`/`assertStringContainsString`.
Po zmianie pisowni komunikatów w źródle te same asercje przestały trafiać w
nowy tekst — dopasowano je do nowego brzmienia, nie osłabiając zakresu
sprawdzenia (żadna asercja nie została usunięta ani zamieniona na luźniejszą).

Zaktualizowane testy:
- `tests/Unit/League/MatchFormatTest.php` — etykieta „Trójki".
- `tests/Integration/FplSyncCommandTest.php` — cztery komunikaty konsoli
  (`Brak meczu na kolejkę`, `Nie dodano jeszcze żadnego meczu`, `Nie udało się
  odświeżyć słowników`, `Synchronizacja zakończona z błędami`).
- `tests/Integration/FplTeamImportCommandTest.php` — cztery warianty komunikatu
  podsumowania importu („Dodano X nowych drużyn, zaktualizowano Y
  istniejących").
- `tests/Functional/AccessControllerTest.php` — „Nieprawidłowy kod".
- `tests/Functional/HomeControllerTest.php` — „Brak bieżącej kolejki".
- `tests/Functional/GameweekControllerTest.php` — „Brak meczów".
- `tests/Functional/MatchControllerTest.php` — „Różnice w składach".
- `tests/Functional/SettingsControllerTest.php` — „Moja czwórka", „FPL nie zna
  drużyny", „muszą być różne".
- `tests/Functional/FixtureNewControllerTest.php` — najwięcej zmian: komunikaty
  walidacji formularza nowego meczu („skonfiguruj swoją czwórkę", „Format
  […] wymaga N drużyn/drużyny", „Pojedynki dotyczą wyłącznie trójek", „Mecz
  4x4/mecz na tę kolejkę już istnieje", „muszą być różne", „FPL nie zna
  drużyny", „Nie znaleziono drużyny" / „Możesz podać numer ID", „Znaleziono
  kilka drużyn", „dokładnie jednym pojedynku") oraz naprawa dwóch testów opisana
  niżej.

## Naprawa dwóch testów z fałszywą zielenią

`testRejectsEntryAppearingOnBothSides` i `testRejectsOverlapWithMySquad` w
`tests/Functional/FixtureNewControllerTest.php` sprawdzały obecność frazy „po
obu stronach" przez `assertSelectorTextContains('body', 'po obu stronach')`.
Ta sama fraza występuje też w statycznej podpowiedzi nad formularzem
(`templates/fixture/new.html.twig`: „…numer albo nazwę każdej z drużyn po obu
stronach.") — więc test przechodził niezależnie od tego, czy reguła kolizji
stron (`FixtureController::resolveSides()`, sprawdzenie `array_intersect`)
faktycznie zadziałała.

Naprawa: asercja celuje teraz w `form > ul` zamiast w `body`. `form_errors(form)`
renderuje się w `new.html.twig` jako pierwszy (i jedyny na tym etapie
walidacji) element `<ul>` bezpośrednio wewnątrz `<form>` — błędy pól
poszczególnych (`form_row`) trafiają do zagnieżdżonych `<div>`, więc `form >
ul` wybiera wyłącznie listę błędów globalnych formularza, nigdy podpowiedzi
statycznej znajdującej się przed `{{ form_start(form) }}`.

Weryfikacja naprawy: tymczasowo wyłączyłem warunek `array_intersect` w
`FixtureController::resolveSides()` i uruchomiłem oba testy — z wyłączoną
regułą oba faile'ują (odpowiedź to 302 redirect, czyli mecz zostałby
utworzony), co potwierdza, że nowa asercja rzeczywiście testuje działanie
reguły kolizji, a nie treść podpowiedzi. Zmianę w kontrolerze cofnąłem od razu
po weryfikacji i zdiffowałem plik z kopią zapasową, żeby potwierdzić brak
resztek.

## Weryfikacja

- `vendor/bin/phpunit` — 267 testów, 922 asercje, zielono.
- `php bin/console lint:twig templates/` — 14 plików, składnia poprawna.
- `php -l` na wszystkich zmienionych plikach PHP — bez błędów składni.
- Sprawdzenie kodowania (UTF-8, brak BOM) na wszystkich zmienionych plikach.

## Wątpliwości / decyzje graniczne

- W kilku miejscach zmieniłem nie tylko ogonki, ale i drobną gramatykę, żeby
  zdanie brzmiało naturalnie po dodaniu diakrytyków (np. „ID musi być liczbą
  dodatnią" zamiast dosłownego „byc liczba dodatnia" z dodanymi ogonkami,
  „Mecz 4x4 na tę kolejkę już istnieje" z odmienionym „tę/tą"). Komunikaty
  pozostały rozpoznawalne, zgodnie z poleceniem — nie przepisywałem ich od
  nowa.
- `docs/do-zrobienia.md` zaktualizowany: dopisana notatka o nowej konwencji na
  górze pliku oraz usunięty wpis o dwóch fałszywie zielonych testach z sekcji
  „Znalezione przez recenzję, odłożone" (zadanie zostało wykonane w ramach tej
  gałęzi, więc nie jest już otwarte).
- Nie ruszałem pozostałych pozycji z `docs/do-zrobienia.md` — poza wskazaną
  naprawą dwóch testów, zgodnie z zakresem zlecenia.