Błąd #544
zamknięte
23. [REFAKTOR-001] Scentralizować normalizację day_order
100%
Opis
Kolejność realizacji: 23/24
Problem:
Normalizacja jest powielona w trips.php i trip_move_ajax.php.
Skutek / ryzyko:
Możliwa jest niespójność danych, błędny przebieg, nieaktualny raport albo niejednoznaczna historia operacji.
Docelowe rozwiązanie:
Jedna wspólna funkcja używana przy edycji, usuwaniu i przesuwaniu.
Pliki:
public/trips.php; public/trip_move_ajax.php
Kryteria akceptacji:
Rozwiązanie działa atomowo, zachowuje ciągłość przebiegu, nie zmienia po cichu zamkniętych okresów i pozostawia czytelny ślad tam, gdzie wykonywana jest korekta.
Uaktualnione przez Michał Kurkiewicz 5 dni temu
- Zmieniono Status z Nowy na Zamknięty
- Zmieniono % Wykonania z 0 na 100
Normalizacja day_order została przeniesiona do jednego wspólnego miejsca:
web\app\mileage_sync.php
Centralna funkcja:
normalize_trip_day_order()
jest używana przez:
edycję przejazdu,
usuwanie przejazdu,
zmianę kolejności przez AJAX.
Usunąłem lokalne kopie z:
web\public\trips.php
web\public\trip_move_ajax.php
Końcowa kontrola wykazała dokładnie jedną definicję tej funkcji w całej przygotowanej paczce.