28 KiB
28 KiB
phase, plan, type, wave, depends_on, files_modified, files_deleted, autonomous, delegation
| phase | plan | type | wave | depends_on | files_modified | files_deleted | autonomous | delegation | ||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| 09-finalizacja | 03 | execute | 1 |
|
|
|
false | off |
Purpose
Po 09-02 plugin ma DWA równoległe mechanizmy synchronizacji z GCal: globalny iCal (używany przez klienta) i OAuth push (uśpiony, klient nie używa) + per-yacht iCal feedy/importy (relikt sprzed globalnego). To komplikuje model mentalny i potencjalnie powoduje duplikaty. Sprzątanie eliminuje confusing UI i tysiące linii martwego kodu, czyniąc plugin gotowym produkcyjnie do v1.0.
Output
- Usunięte 3 pliki OAuth/GCal (
class-sync-controller.php,class-gcal-service.php,class-oauth-handler.php) - Wyłączony cały OAuth flow (UI w settings, hooki cron, callback redirect, AJAX manual sync)
- Usunięty per-jacht iCal feed (
output_ics,get_feed_token, rewrite rule^yacht-ical/(\d+)/...) - Usunięty per-jacht iCal import (cron
yacht_booking_ical_import,run_import,import_for_yacht, poleiCal Import URLw yacht-edit, kolumna w yacht-list) - Usunięte pole "Google Calendar ID" w yacht-edit + metody
Yacht::get_gcal_id/update_gcal_id - Cleanup migration: jednorazowy
delete_post_meta(_yacht_gcal_id, _yacht_ical_import_url, _yacht_ical_token, yacht_ical_last_import) +delete_option(gcal*, gcal credentials, gcal_sync_enabled) - Zaktualizowany
uninstall.php
Source Files
@wp-content/plugins/yacht-booking-system/yacht-booking-system.php @wp-content/plugins/yacht-booking-system/includes/class-yacht-booking.php @wp-content/plugins/yacht-booking-system/admin/class-admin.php @wp-content/plugins/yacht-booking-system/integrations/ical/class-ical-import.php @wp-content/plugins/yacht-booking-system/integrations/ical/class-ical-feed.php @wp-content/plugins/yacht-booking-system/integrations/google-calendar/class-sync-controller.php @wp-content/plugins/yacht-booking-system/integrations/google-calendar/class-gcal-service.php @wp-content/plugins/yacht-booking-system/integrations/google-calendar/class-oauth-handler.php
- **GCal ID per-jacht** — Usunąć z UI + skasować meta z bazy. → Odpowiedź: pole `_yacht_gcal_id` było martwe (OAuth używał globalnej opcji). Usuwamy pole z yacht-edit, metody `Yacht::get_gcal_id`/`update_gcal_id`, kolumnę z yacht-list-table, oraz wywołujemy `delete_post_meta` w cleanup migration.-
Per-jacht iCal Import URL — Usunąć całkowicie (UI + cron + dane). → Odpowiedź: usuwamy pole
iCal Import URLz yacht-edit, kolumnę z yacht-list, hookyacht_booking_ical_import, metodyrun_import,import_for_yacht,get_import_url,set_import_url,get_last_import_time,get_existing_import_map,upsert_booking(per-yacht), stałąIMPORT_SOURCE. Per-yacht booking sourceical_importzostaje rozpoznany w cleanup migration jako kandydat do usunięcia (rozważymy w boundaries — patrz niżej). -
OAuth push do GCal — Wyłączyć całkowicie i usunąć. → Odpowiedź: usuwamy 3 pliki (
class-sync-controller.php,class-gcal-service.php,class-oauth-handler.php), wszystkie hooki w bootstrap, sekcję UI OAuth wrender_google_calendar_settings, AJAXwp_ajax_yacht_booking_manual_sync, ścieżkę OAuth callback wprocess_settings_save, notice connected/disconnected. Sekcja Settings → Google Calendar zawiera już teraz TYLKO globalną iCal sync. -
Migracja — Bez widocznej migracji, po cichu. → Odpowiedź: cleanup wykonuje się w
Installer::install()przy aktywacji nowej wersji (sprawdzamy zapisaną wersję pluginu vsYACHT_BOOKING_VERSION— jeśli upgrade z wersji <X.Y.Z, uruchamiamy cleanup). Brak admin notice. Brak żądań akcji od klienta. Klient po prostu zauważy że niepotrzebne pola znikły. -
Per-jacht booking source 'ical_import' — istniejące rezerwacje zaimportowane przez stary per-yacht mechanizm. → Decyzja techniczna (nie pytana): w cleanup migration usuwamy je razem z meta (clear_booking_availability + wp_delete_post). Bo bez per-yacht importu nie będą odświeżane, więc lepiej żeby nie blokowały dat na stałe. Klient po cleanup uruchomi globalny "Importuj teraz" i nowe dane się zaimportują.
<acceptance_criteria>
AC-1: Pliki OAuth/Sync usunięte i nie ładowane
Given plugin po nowym deploy
When sprawdzamy strukturę katalogu integrations/
Then nie istnieje plik class-sync-controller.php
And nie istnieje plik class-gcal-service.php
And nie istnieje plik class-oauth-handler.php
And bootstrap pluginu (yacht-booking-system.php + class-yacht-booking.php) nie odwołuje się do żadnej z tych klas
And `php -l` na bootstrap nie zgłasza błędów
And aktywacja pluginu w WP Admin nie powoduje fatal error
AC-2: Sekcja OAuth zniknęła z Settings, globalna iCal pozostała
Given admin otwiera WP Admin → Rezerwacje Jachtów → Ustawienia → zakładka Google Calendar
When strona się renderuje
Then NIE widzi "Krok 1: Dodaj dane OAuth"
And NIE widzi "Krok 2: Autoryzuj połączenie"
And NIE widzi przycisku "Synchronizuj teraz" (AJAX manual sync)
And NIE widzi pola "Client ID"/"Client Secret"
And widzi sekcję "Globalna synchronizacja iCal" (z 09-02) z polami import URL + export URL
And działa "Importuj teraz" + zapis URL + regeneracja tokenu
AC-3: Pola per-jacht zniknęły z yacht-edit i yacht-list
Given admin otwiera Edycję jachtu (yacht-bookings-add-yacht)
When strona się renderuje
Then NIE widzi pola "Google Calendar ID"
And NIE widzi pola "iCal Import URL"
And NIE widzi linku "iCal Feed URL" per-jacht
And widzi nadal pole "Alias dla Google Calendar" (z 09-02 — to jest aktywne dla globalnego importu)
And lista jachtów (yacht-bookings) NIE pokazuje kolumny GCal ID ani iCal status
AC-4: Per-jacht iCal cron i feed wyłączone
Given plugin po deploy nowej wersji
When sprawdzamy zarejestrowane crony WP
Then NIE jest zaplanowany hook 'yacht_booking_ical_import' (per-yacht)
And NIE jest zaplanowany hook 'yacht_booking_pull_from_gcal' (OAuth pull)
And NIE jest zaplanowany hook 'yacht_booking_sync_to_gcal' (OAuth push)
And JEST zaplanowany hook 'yacht_booking_ical_global_import' (z 09-02)
And URL `/yacht-ical/{yacht_id}/{token}.ics` zwraca 404 (per-yacht feed niedostępny)
And URL `/yacht-ical-global/{token}.ics` nadal działa (z 09-02)
AC-5: Cleanup migration — czysta baza
Given baza ma stare meta (_yacht_gcal_id, _yacht_ical_import_url, _yacht_ical_token, _yacht_ical_last_import)
And baza ma stare opcje (yacht_booking_gcal_calendar_id, yacht_booking_gcal_credentials, yacht_booking_gcal_oauth_*, yacht_booking_gcal_sync_enabled)
And baza ma rezerwacje z _booking_source = 'ical_import' (per-yacht)
When admin aktywuje nową wersję pluginu (lub uruchomi się hook upgrade)
Then wszystkie wymienione meta są usunięte (delete_post_meta)
And wszystkie wymienione opcje są usunięte (delete_option)
And rezerwacje z source 'ical_import' są usunięte (Availability::clear_booking_availability + wp_delete_post)
And opcja `yacht_booking_version` ma nową wartość pluginu
And cleanup wykonuje się TYLKO RAZ (kolejna aktywacja nie powtarza)
AC-6: Globalna iCal sync (z 09-02) działa bez regresji
Given po cleanup admin ma globalny iCal Import URL ustawiony
When kliknie "Importuj teraz"
Then plugin pobiera kalendarz Google
And eventy z prefiksem nazwy jachtu są importowane jako rezerwacje (source 'ical_import_global')
And frontend kalendarz per-jacht pokazuje tylko swoje rezerwacje
And export feed `/yacht-ical-global/{token}.ics` zwraca poprawny .ics
AC-7: uninstall.php zaktualizowany
Given admin usuwa plugin przez Plugins → Delete
When uruchamia się uninstall.php
Then NIE próbuje delete_option dla nieistniejących już kluczy (lub robi to bezpiecznie — delete_option jest no-op dla nieistniejących)
And usuwa wszystkie aktualne klucze: yacht_booking_global_ical_*, yacht_booking_default_status, yacht_booking_email_*, etc.
And usuwa wszystkie meta jachtów + bookingów + tabele wp_yacht_availability (jak dotąd)
</acceptance_criteria>
Task 1: Usunąć OAuth/Sync/GCal_Service — pliki i wszystkie odniesienia wp-content/plugins/yacht-booking-system/integrations/google-calendar/class-sync-controller.php (DELETE), wp-content/plugins/yacht-booking-system/integrations/google-calendar/class-gcal-service.php (DELETE), wp-content/plugins/yacht-booking-system/integrations/google-calendar/class-oauth-handler.php (DELETE), wp-content/plugins/yacht-booking-system/yacht-booking-system.php, wp-content/plugins/yacht-booking-system/includes/class-yacht-booking.php, wp-content/plugins/yacht-booking-system/admin/class-admin.php 1. **Usunąć 3 pliki** w `integrations/google-calendar/`: - `class-sync-controller.php` - `class-gcal-service.php` - `class-oauth-handler.php` Można usunąć cały katalog `integrations/google-calendar/` jeśli to jedyne pliki (sprawdź `ls integrations/google-calendar/` przed delete).2. **`yacht-booking-system.php` (bootstrap)**:
- Usunąć `require_once` dla 3 plików GCal (jeśli są)
- Usunąć wywołania `Sync_Controller::get_instance()`, `Sync_Controller::register_cron_actions()`, `Sync_Controller::setup_cron()`, `Sync_Controller::clear_cron()` w hookach activate/deactivate/plugins_loaded
- Zachować bez zmian wszystkie wywołania ICal_* (te zostają)
3. **`includes/class-yacht-booking.php`**:
- W metodzie `init()` lub `register()` usunąć linie tworzące/inicjalizujące `Sync_Controller`, `GCal_Service`, `OAuth_Handler`
- Usunąć `require_once` dla 3 plików GCal
4. **`admin/class-admin.php` — masywny refactor sekcji OAuth**:
- W `process_settings_save()`:
- Usunąć `require_once` dla `class-oauth-handler.php` i `class-gcal-service.php`
- Usunąć blok obsługujący `gcal_callback` (OAuth callback redirect)
- Usunąć blok `yacht_booking_disconnect_gcal`
- Usunąć blok `yacht_booking_save_gcal_credentials`
- **ZACHOWAĆ** bloki: globalny iCal save, regenerate token, run global import, save_settings (general), save_email_templates
- Usunąć metodę `handle_oauth_callback()`
- Usunąć metodę `save_gcal_credentials()`
- W `display_admin_notices()`: usunąć notice `connected`, `disconnected`. Pozostawić: saved, templates_saved, templates_reset, error, global_ical_saved, token_regenerated, global_import_done, global_import_failed
- W `render_google_calendar_settings()`: usunąć CAŁĄ sekcję `if ( ! $is_connected ) ... else ... endif` (Step 1: credentials, Step 2: authorize, Connected status, Manual sync, Instrukcja). **ZACHOWAĆ** sekcję "Globalna synchronizacja iCal" (z 09-02). Metoda po refactorze powinna mieć tylko: nagłówek `<h2>` + tę jedną sekcję `<div class="card">` z 09-02.
- Usunąć lokalne require_once `OAuth_Handler` i `GCal_Service` w tej metodzie (już niepotrzebne).
- Pozostawić bez zmian `process_yacht_save`, `save_yacht`, list table'y, eksport CSV, etc.
Avoid:
- Nie ruszać sekcji "Globalna synchronizacja iCal" (z 09-02) — to jest TO co zostaje
- Nie usuwać CSS/JS dla globalnego iCal
- Nie usuwać AJAX handler globalnego iCal (jeśli istnieje — w 09-02 jest tylko form POST, żaden AJAX)
2. **`class-ical-feed.php` — usuwamy per-yacht feed, zostaje tylko globalny**:
- W `add_rewrite_rules()`: usunąć rule `^yacht-ical/([0-9]+)/([a-zA-Z0-9]+)\.ics$` (zostaje tylko global). Zaktualizować flush check.
- W `add_query_vars()`: usunąć `yacht_ical_id` (zostają `yacht_ical_token` i `yacht_ical_global`)
- W `handle_feed_request()`: usunąć cały blok per-yacht (zachować tylko branch global)
- Usunąć metody: `get_feed_token()`, `regenerate_token()`, `get_feed_url()`, `output_ics()`
- Zachować: `register()`, `add_rewrite_rules()` (już poprawiony), `add_query_vars()` (poprawiony), `handle_feed_request()` (poprawiony), wszystkie metody global, `escape_ical()`
3. **`admin/views/yacht-edit.php`**:
- Usunąć całe wiersze tabeli z polami: `yacht_gcal_id`, `yacht_ical_import_url`, oraz blok wyświetlający per-yacht iCal Feed URL (z tokenem)
- Usunąć z PHP na górze pliku: `$gcal_id = ...`, `$ical_import_url = ...`, `$ical_feed_url = ...`, `$ical_last_import = ...`
- **ZACHOWAĆ** pole `yacht_gcal_alias` (z 09-02 — używane przez globalny import)
- Pozostawić bez zmian: title, description, capacity, price_per_day, features
4. **`admin/class-yacht-list-table.php`**:
- Usunąć z `get_columns()` kolumnę dla GCal i iCal status (jeśli istnieją)
- Usunąć metodę `column_gcal_id()` lub equivalent (linia ok. 173 — `Yacht::get_gcal_id`)
- Sprawdź czy są inne kolumny używające usuniętych meta — usuń ich rendering
5. **`admin/class-admin.php` — czyszczenie save_yacht**:
- W `save_yacht()`: usunąć linie:
- `$gcal_id = ...`
- `Yacht::update_gcal_id( $saved_id, $gcal_id );`
- `$ical_import_url = ...`
- `\YachtBooking\Integrations\ICal\ICal_Import::set_import_url( ... )`
- **ZACHOWAĆ** zapis aliasu (`Yacht::update_gcal_alias`)
6. **`includes/class-yacht.php`**:
- Usunąć metody: `get_gcal_id()`, `update_gcal_id()`
- **ZACHOWAĆ**: get_capacity, update_capacity, get_price_per_day, update_price_per_day, get_features, update_features, get_gcal_alias, update_gcal_alias
Avoid:
- Nie tknąć w żaden sposób mechanizmu globalnego iCal (run_global_import, output_global_ics, alias)
- Nie usuwać cron `yacht_booking_ical_global_import` (z 09-02)
- Nie usuwać `parse_ics` ani helperów iCal — używane przez globalny import
2. **`uninstall.php`**:
- Sprawdź obecną zawartość (`Read`).
- Usuń `delete_option` dla nieistniejących już kluczy GCal (te z punktu 1.b)
- **ZACHOWAJ** delete_option dla:
- `yacht_booking_default_status`, `yacht_booking_email_from_name`, `yacht_booking_email_from`, `yacht_booking_date_format`, `yacht_booking_currency_symbol`, `yacht_booking_terms_page_id`, `yacht_booking_enable_notifications`
- `yacht_booking_global_ical_import_url`, `yacht_booking_global_ical_token`, `yacht_booking_global_ical_last_import`
- `yacht_booking_email_templates`, `yacht_booking_version`, `yacht_booking_installed_at`, `yacht_booking_enabled`
- **ZACHOWAJ** delete `wp_yacht_availability` table + delete_post_meta dla wszystkich CPT meta + wp_delete_post dla CPT.
Avoid:
- Nie usuwać `_yacht_gcal_alias` (aktywny dla globalnego importu)
- Nie usuwać `_booking_source = 'ical_import_global'` (aktywne globalne importy z 09-02 — zostawić!)
- Nie usuwać innych user data (rezerwacji ze strony — `_booking_source != 'ical_import'`)
- Cleanup ma być idempotentny — uruchamia się TYLKO przy upgrade z wersji < CURRENT
DO NOT CHANGE
- Globalna synchronizacja iCal (09-02) —
output_global_ics,run_global_import,build_yacht_lookup_map,match_yacht_by_prefix,upsert_global_booking, settings UI sekcja "Globalna synchronizacja iCal",_yacht_gcal_alias, opcjeyacht_booking_global_ical_*, cronyacht_booking_ical_global_import Yacht::get_gcal_alias/update_gcal_alias— używane przez globalny import- CPT yacht_booking i yacht — schemat, capabilities, registration
wp_yacht_availability— tabela i logikaAvailability- Frontend widget i shortcode — działają dzięki
_booking_yacht_id - REST API (
Rest_Controller) — endpoint POST /bookings używa wewnętrznych funkcji booking::create, niezależny od GCal - Email templates + Inquiry CPT + CSV export — out of scope
SCOPE LIMITS
- Plan NIE dotyka security audit (to 09-04)
- Plan NIE dotyka tłumaczeń .pot/.po/.mo (to 09-05)
- Plan NIE refactoruje
parse_icsani helperów iCal (działają — zostają) - Plan NIE zmienia formatu prefiksu w SUMMARY (separator
" - "z 09-02 zostaje) - Brak nowych dependencies
- Brak admin notice o migracji (klient wybrał "po cichu")
- Plan NIE migruje rezerwacji typu 'ical_import' do 'ical_import_global' — usuwa je (klient ponownie zaimportuje przez globalny mechanizm)
<success_criteria>
- Plugin ma JEDEN mechanizm synchronizacji z GCal: globalny iCal feed (eksport + import)
- Zero martwego kodu (3 pliki usunięte, ~500+ linii zbędnego UI/cron usuniętych)
- Czysta baza po upgrade (klient nie widzi starych pól, baza nie zawiera nieaktywnych meta/opcji)
- Brak regresji w globalnej iCal sync ani frontendzie
- Plugin gotowy produkcyjnie w warstwie integracji GCal (pozostaje security audit + i18n + docs) </success_criteria>