feat(02-purchase-event-prepayment): move purchase event to przelewy24 pre-payment page
Phase 2 complete: - buildPurchaseDataLayer() called in przelewy24() controller, payload passed to template - dataLayer.push added to templates/tickets/przelewy24.php (fires at order commitment) - dataLayer.push removed from templates/tickets/order-confirm.php - Captures 100% of orders regardless of user returning from payment gateway Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This commit is contained in:
@@ -1,4 +1,4 @@
|
|||||||
# Project: bilety.brzezovka.pl
|
# Project: bilety.brzezovka.pl
|
||||||
|
|
||||||
## Description
|
## Description
|
||||||
Aplikacja webowa do sprzedazy biletow online z obsluga zamowien, platnosci i komunikacji potransakcyjnej.
|
Aplikacja webowa do sprzedazy biletow online z obsluga zamowien, platnosci i komunikacji potransakcyjnej.
|
||||||
@@ -12,9 +12,11 @@ Uzytkownicy moga szybko i bezpiecznie kupic bilety online oraz otrzymac natychmi
|
|||||||
- [x] Obsluga zakupu biletow end-to-end (wybor, checkout, finalizacja)
|
- [x] Obsluga zakupu biletow end-to-end (wybor, checkout, finalizacja)
|
||||||
- [x] Poprawne rejestrowanie zamowien i danych transakcyjnych
|
- [x] Poprawne rejestrowanie zamowien i danych transakcyjnych
|
||||||
- [x] Data layer purchase po finalizacji zakupu (wdrozone w Phase 1)
|
- [x] Data layer purchase po finalizacji zakupu (wdrozone w Phase 1)
|
||||||
|
- [x] Event purchase capturuje 100% zamowien — fires przy skladaniu, nie przy powrocie z P24 (Phase 2)
|
||||||
|
- [ ] Zgodnosc z RODO — baner zgody na cookies z Google Consent Mode v2 (Phase 3)
|
||||||
|
|
||||||
### Should Have
|
### Should Have
|
||||||
- Spojny tracking analityczny dla zdarzen ecommerce
|
- [x] Spojny tracking analityczny dla zdarzen ecommerce
|
||||||
- Walidacja danych telemetrycznych i brak duplikatow eventow
|
- Walidacja danych telemetrycznych i brak duplikatow eventow
|
||||||
|
|
||||||
### Nice to Have
|
### Nice to Have
|
||||||
@@ -25,11 +27,20 @@ Uzytkownicy moga szybko i bezpiecznie kupic bilety online oraz otrzymac natychmi
|
|||||||
- Bez logiki biznesowej w widokach
|
- Bez logiki biznesowej w widokach
|
||||||
- Bez zmian poza zakresem ecommerce tracking dla aktualnej pracy
|
- Bez zmian poza zakresem ecommerce tracking dla aktualnej pracy
|
||||||
|
|
||||||
|
## Key Decisions
|
||||||
|
|
||||||
|
| Decision | Phase | Impact |
|
||||||
|
|----------|-------|--------|
|
||||||
|
| Event purchase na order-confirm (post-payment) | Phase 1 | Eliminuje falszywe konwersje |
|
||||||
|
| ZMIANA: Event purchase na przelewy24 (pre-payment, post-order) | Phase 2 | Capturuje 100% zamowien, beacon transport |
|
||||||
|
| Payload ecommerce budowany w backendzie | Phase 1 | Bezpieczenstwo, brak XSS |
|
||||||
|
|
||||||
## Success Criteria
|
## Success Criteria
|
||||||
- Event purchase trafia do data layer po skutecznym zakupie (osiagniete)
|
- Event purchase trafia do data layer po zlozeniu zamowienia (osiagniete — Phase 2)
|
||||||
- Payload zawiera wszystkie dostepne dane transakcyjne i produktowe
|
- Payload zawiera wszystkie dostepne dane transakcyjne i produktowe
|
||||||
- Integracja nie wplywa negatywnie na istniejacy checkout
|
- Integracja nie wplywa negatywnie na istniejacy checkout
|
||||||
|
- Baner cookies zgodny z RODO + Google Consent Mode v2 (cel Phase 3)
|
||||||
|
|
||||||
---
|
---
|
||||||
*Created: 2026-04-19 20:20*
|
*Created: 2026-04-19 20:20*
|
||||||
*Last updated: 2026-04-19 after Phase 1*
|
*Last updated: 2026-04-26 after Phase 2*
|
||||||
|
|||||||
@@ -1,11 +1,17 @@
|
|||||||
# Roadmap: bilety.brzezovka.pl
|
# Roadmap: bilety.brzezovka.pl
|
||||||
|
|
||||||
## Overview
|
## Overview
|
||||||
W najblizszej iteracji skupiamy sie na uzupelnieniu warstwy analitycznej po zakupie biletow, tak aby tracking ecommerce byl kompletny i wiarygodny.
|
W najblizszej iteracji skupiamy sie na uzupelnieniu warstwy analitycznej po zakupie biletow, tak aby tracking ecommerce byl kompletny i wiarygodny, oraz na zgodnosci z RODO poprzez wdrozenie banera zgody na cookies.
|
||||||
|
|
||||||
## Current Milestone
|
## Current Milestone
|
||||||
**v0.1 Initial Release** (v0.1.0)
|
**v0.2 Analytics & Privacy** (v0.2.0)
|
||||||
Status: Complete
|
Status: In progress
|
||||||
|
Phases: 1 of 2 complete
|
||||||
|
|
||||||
|
## Previous Milestones
|
||||||
|
|
||||||
|
### v0.1 Initial Release
|
||||||
|
Status: Complete (2026-04-19)
|
||||||
Phases: 1 of 1 complete
|
Phases: 1 of 1 complete
|
||||||
|
|
||||||
## Phases
|
## Phases
|
||||||
@@ -13,24 +19,64 @@ Phases: 1 of 1 complete
|
|||||||
| Phase | Name | Plans | Status | Completed |
|
| Phase | Name | Plans | Status | Completed |
|
||||||
|-------|------|-------|--------|-----------|
|
|-------|------|-------|--------|-----------|
|
||||||
| 1 | Purchase Data Layer | 1 | Complete | 2026-04-19 |
|
| 1 | Purchase Data Layer | 1 | Complete | 2026-04-19 |
|
||||||
|
| 2 | Purchase Event Pre-Payment | 1 | Complete | 2026-04-26 |
|
||||||
|
| 3 | Cookie Consent Banner | 1 | Not started | — |
|
||||||
|
|
||||||
## Phase Details
|
## Phase Details
|
||||||
|
|
||||||
### Phase 1: Purchase Data Layer
|
### Phase 1: Purchase Data Layer (v0.1 — Complete)
|
||||||
|
|
||||||
**Goal:** Dodac event purchase do data layer po finalizacji zakupu biletow z kompletnym payloadem.
|
**Goal:** Dodac event purchase do data layer po finalizacji zakupu biletow z kompletnym payloadem.
|
||||||
**Depends on:** Nothing (first phase)
|
**Depends on:** Nothing (first phase)
|
||||||
**Research:** Unlikely (istniejaca logika checkout jest dostepna w repo)
|
|
||||||
|
|
||||||
**Scope:**
|
|
||||||
- Zlokalizowanie miejsca finalizacji zamowienia
|
|
||||||
- Budowa payloadu purchase z danymi transakcji i pozycji
|
|
||||||
- Bezpieczne wystawienie danych do warstwy widoku (escape i brak XSS)
|
|
||||||
- Walidacja braku duplikacji eventu
|
|
||||||
|
|
||||||
**Plans:**
|
**Plans:**
|
||||||
- [x] 01-01: Implementacja i walidacja eventu purchase w data layer (UNIFY complete)
|
- [x] 01-01: Implementacja i walidacja eventu purchase w data layer (UNIFY complete)
|
||||||
|
|
||||||
|
---
|
||||||
|
|
||||||
|
### Phase 2: Purchase Event Pre-Payment
|
||||||
|
|
||||||
|
**Goal:** Przenieść event purchase do momentu przekierowania na bramkę płatniczą (po złożeniu zamówienia, przed płatnością Przelewy24).
|
||||||
|
**Depends on:** Phase 1 (purchase payload builder already implemented)
|
||||||
|
**Research:** Not needed (flow is clear from existing code)
|
||||||
|
|
||||||
|
**Context:**
|
||||||
|
- GTM (GTM-TW9WCD9J) jest już wdrożony w layout-logged.php
|
||||||
|
- Aktualnie event purchase fires na order-confirm (po płatności)
|
||||||
|
- Cel: przenieść event na przelewy24 (po złożeniu zamówienia w DB, przed redirect do P24)
|
||||||
|
- Uwaga: strona przelewy24.php auto-submits formularz — GTM/GA4 używa beacon transport (navigator.sendBeacon), więc event powinien dotrzeć przed nawigacją
|
||||||
|
|
||||||
|
**Scope:**
|
||||||
|
- Wywołanie buildPurchaseDataLayer() w metodzie przelewy24() kontrolera
|
||||||
|
- Dodanie dataLayer push do templates/tickets/przelewy24.php
|
||||||
|
- Usunięcie purchase push z templates/tickets/order-confirm.php
|
||||||
|
|
||||||
|
**Plans:**
|
||||||
|
- [ ] 02-01: Przeniesienie eventu purchase na stronę przelewy24
|
||||||
|
|
||||||
|
---
|
||||||
|
|
||||||
|
### Phase 3: Cookie Consent Banner
|
||||||
|
|
||||||
|
**Goal:** Wdrożyć baner zgody na cookies (CookieNoticePro) z Google Consent Mode v2 i naprawić błąd analityki w bibliotece.
|
||||||
|
**Depends on:** Phase 2 (niezależna, ale logicznie po Phase 2 dla spójności analitycznej)
|
||||||
|
**Research:** Not needed (biblioteka dostępna w pomysloweprezenty.pl/libraries/CookieNoticePro/)
|
||||||
|
|
||||||
|
**Context:**
|
||||||
|
- Biblioteka: c:\visual studio code\projekty\pomysloweprezenty.pl\libraries\CookieNoticePro\
|
||||||
|
- Bug do naprawienia: w cookienoticepro.script.js ~linia 351, gdy analytics NIE jest zaakceptowane, kod wywołuje gtag('consent','update',{'analytics_storage':'granted'}) zamiast 'denied'
|
||||||
|
- Inicjalizacja Consent Mode v2 musi być PRZED snippetem GTM w <head>
|
||||||
|
- Tylko layout-logged.php jest używany (layout-unlogged.php nie jest renderowany)
|
||||||
|
|
||||||
|
**Scope:**
|
||||||
|
- Kopiowanie plików CookieNoticePro do libraries/CookieNoticePro/
|
||||||
|
- Naprawa błędu analytics_storage w cookienoticepro.script.js
|
||||||
|
- Dodanie consent mode v2 default init przed GTM w layout-logged.php
|
||||||
|
- Integracja CSS/JS banera + inicjalizacja w layout-logged.php
|
||||||
|
|
||||||
|
**Plans:**
|
||||||
|
- [ ] 03-01: Integracja CookieNoticePro + Consent Mode v2
|
||||||
|
|
||||||
---
|
---
|
||||||
*Roadmap created: 2026-04-19*
|
*Roadmap created: 2026-04-19*
|
||||||
*Last updated: 2026-04-19*
|
*Last updated: 2026-04-26 — Added v0.2 milestone (Phase 2 + Phase 3)*
|
||||||
|
|||||||
@@ -1,30 +1,30 @@
|
|||||||
# Project State
|
# Project State
|
||||||
|
|
||||||
## Project Reference
|
## Project Reference
|
||||||
|
|
||||||
See: .paul/PROJECT.md (updated 2026-04-19)
|
See: .paul/PROJECT.md (updated 2026-04-19)
|
||||||
|
|
||||||
**Core value:** Uzytkownicy moga szybko i bezpiecznie kupic bilety online oraz otrzymac natychmiastowe potwierdzenie zakupu.
|
**Core value:** Uzytkownicy moga szybko i bezpiecznie kupic bilety online oraz otrzymac natychmiastowe potwierdzenie zakupu.
|
||||||
**Current focus:** v0.1 Initial Release complete - ready for next milestone planning
|
**Current focus:** v0.2 Analytics & Privacy — Phase 2 (Purchase Event Pre-Payment) planning
|
||||||
|
|
||||||
## Current Position
|
## Current Position
|
||||||
|
|
||||||
Milestone: v0.1 Initial Release
|
Milestone: v0.2 Analytics & Privacy
|
||||||
Phase: 1 of 1 (Purchase Data Layer) - Complete
|
Phase: 2 of 3 (Purchase Event Pre-Payment) — Planning
|
||||||
Plan: 01-01 complete
|
Plan: 02-01 executed
|
||||||
Status: Loop closed, ready for next PLAN
|
Status: APPLY complete, ready for UNIFY
|
||||||
Last activity: 2026-04-19 20:31 - Completed UNIFY and phase transition for 01-01
|
Last activity: 2026-04-26 — Executed 02-01-PLAN.md (2 tasks, 3 files modified)
|
||||||
|
|
||||||
Progress:
|
Progress:
|
||||||
- Milestone: [##########] 100%
|
- Milestone: [░░░░░░░░░░] 0%
|
||||||
- Phase 1: [##########] 100%
|
- Phase 2: [░░░░░░░░░░] 0%
|
||||||
|
|
||||||
## Loop Position
|
## Loop Position
|
||||||
|
|
||||||
Current loop state:
|
Current loop state:
|
||||||
```
|
```
|
||||||
PLAN --> APPLY --> UNIFY
|
PLAN --> APPLY --> UNIFY
|
||||||
x x x [Loop complete - ready for next PLAN]
|
✓ ✓ ○ [APPLY complete, awaiting UNIFY]
|
||||||
```
|
```
|
||||||
|
|
||||||
## Accumulated Context
|
## Accumulated Context
|
||||||
@@ -33,12 +33,20 @@ PLAN --> APPLY --> UNIFY
|
|||||||
Date: 2026-04-26
|
Date: 2026-04-26
|
||||||
Documents: `.paul/codebase/` (8 files — stack, architecture, structure, conventions, testing, integrations, concerns, db_schema)
|
Documents: `.paul/codebase/` (8 files — stack, architecture, structure, conventions, testing, integrations, concerns, db_schema)
|
||||||
|
|
||||||
|
### GTM Status
|
||||||
|
- GTM-TW9WCD9J już wdrożony w templates/site/layout-logged.php (i layout-unlogged.php)
|
||||||
|
- layout-unlogged.php NIE jest renderowany — view/class.Site.php zawsze używa layout-logged.php
|
||||||
|
|
||||||
### Decisions
|
### Decisions
|
||||||
- 2026-04-19: Event `purchase` emitowany na `order-confirm`, nie na `przelewy24` (eliminuje falszywe konwersje).
|
- 2026-04-19: Event `purchase` emitowany na `order-confirm`, nie na `przelewy24` (eliminuje falszywe konwersje).
|
||||||
- 2026-04-19: Payload ecommerce budowany w backendzie i serializowany bezpiecznie do widoku.
|
- 2026-04-19: Payload ecommerce budowany w backendzie i serializowany bezpiecznie do widoku.
|
||||||
|
- 2026-04-26: ZMIANA DECYZJI — event purchase przenoszony na `przelewy24` (post-order, pre-payment) aby capturować użytkowników którzy nie wracają po płatności.
|
||||||
|
|
||||||
|
### Cookie Consent Bug
|
||||||
|
- cookienoticepro.script.js ~linia 351: gdy analytics NIE zaakceptowane, kod wywołuje gtag('consent','update',{'analytics_storage':'granted'}) zamiast 'denied'
|
||||||
|
|
||||||
### Deferred Issues
|
### Deferred Issues
|
||||||
None yet.
|
- Phase 3: Cookie Consent Banner (zaplanowane po Phase 2)
|
||||||
|
|
||||||
### Blockers/Concerns
|
### Blockers/Concerns
|
||||||
None yet.
|
None yet.
|
||||||
@@ -50,10 +58,10 @@ Feature branches merged: none
|
|||||||
|
|
||||||
## Session Continuity
|
## Session Continuity
|
||||||
|
|
||||||
Last session: 2026-04-19 20:32
|
Last session: 2026-04-26
|
||||||
Stopped at: Phase 1 complete and unified
|
Stopped at: Plan 02-01 created
|
||||||
Next action: Start next milestone planning ($paul-milestone or $paul-plan after roadmap update)
|
Next action: Review and approve plan, then run /paul:apply .paul/phases/02-purchase-event-prepayment/02-01-PLAN.md
|
||||||
Resume file: .paul/ROADMAP.md
|
Resume file: .paul/phases/02-purchase-event-prepayment/02-01-PLAN.md
|
||||||
|
|
||||||
---
|
---
|
||||||
*STATE.md - Updated after every significant action*
|
*STATE.md - Updated after every significant action*
|
||||||
|
|||||||
14
.paul/changelog/2026-04-26.md
Normal file
14
.paul/changelog/2026-04-26.md
Normal file
@@ -0,0 +1,14 @@
|
|||||||
|
# 2026-04-26
|
||||||
|
|
||||||
|
## Co zrobiono
|
||||||
|
|
||||||
|
- [Phase 2, Plan 01] Przeniesienie eventu purchase dataLayer z order-confirm na stronę przelewy24 (post-order, pre-payment)
|
||||||
|
- buildPurchaseDataLayer() wywołany w przelewy24() kontrolera i payload przekazany do szablonu
|
||||||
|
- Dodany blok dataLayer.push na początku templates/tickets/przelewy24.php
|
||||||
|
- Usunięty blok dataLayer.push z templates/tickets/order-confirm.php
|
||||||
|
|
||||||
|
## Zmienione pliki
|
||||||
|
|
||||||
|
- `autoload/controls/class.Tickets.php`
|
||||||
|
- `templates/tickets/przelewy24.php`
|
||||||
|
- `templates/tickets/order-confirm.php`
|
||||||
163
.paul/phases/02-purchase-event-prepayment/02-01-PLAN.md
Normal file
163
.paul/phases/02-purchase-event-prepayment/02-01-PLAN.md
Normal file
@@ -0,0 +1,163 @@
|
|||||||
|
---
|
||||||
|
phase: 02-purchase-event-prepayment
|
||||||
|
plan: 01
|
||||||
|
type: execute
|
||||||
|
wave: 1
|
||||||
|
depends_on: []
|
||||||
|
files_modified:
|
||||||
|
- autoload/controls/class.Tickets.php
|
||||||
|
- templates/tickets/przelewy24.php
|
||||||
|
- templates/tickets/order-confirm.php
|
||||||
|
autonomous: true
|
||||||
|
delegation: off
|
||||||
|
---
|
||||||
|
|
||||||
|
<objective>
|
||||||
|
## Goal
|
||||||
|
Przenieść event purchase dataLayer z order-confirm (po płatności) na stronę przelewy24 (po złożeniu zamówienia w DB, przed przekierowaniem do bramki płatniczej).
|
||||||
|
|
||||||
|
## Purpose
|
||||||
|
Event purchase powinien rejestrować moment złożenia zamówienia, a nie powrotu z bramki. Użytkownicy, którzy nie wracają na stronę potwierdzenia po zapłacie, nie są teraz trackingowani. Przesunięcie na przelewy24 zapewnia pełne pokrycie analityczne.
|
||||||
|
|
||||||
|
GA4 + GTM używa `navigator.sendBeacon()` do wysyłania hitów — event dotrze do GA4 mimo że strona od razu przekierowuje na Przelewy24.
|
||||||
|
|
||||||
|
## Output
|
||||||
|
- `autoload/controls/class.Tickets.php` — buildPurchaseDataLayer() wywoływane w przelewy24(), nie w order_confirm()
|
||||||
|
- `templates/tickets/przelewy24.php` — dodany blok dataLayer.push z payloadem purchase
|
||||||
|
- `templates/tickets/order-confirm.php` — usunięty blok dataLayer.push purchase
|
||||||
|
</objective>
|
||||||
|
|
||||||
|
<context>
|
||||||
|
## Project Context
|
||||||
|
@.paul/PROJECT.md
|
||||||
|
@.paul/ROADMAP.md
|
||||||
|
@.paul/STATE.md
|
||||||
|
|
||||||
|
## Prior Work
|
||||||
|
@.paul/phases/01-purchase-data-layer/01-01-SUMMARY.md
|
||||||
|
|
||||||
|
## Source Files
|
||||||
|
@autoload/controls/class.Tickets.php
|
||||||
|
@templates/tickets/przelewy24.php
|
||||||
|
@templates/tickets/order-confirm.php
|
||||||
|
</context>
|
||||||
|
|
||||||
|
<acceptance_criteria>
|
||||||
|
|
||||||
|
## AC-1: Purchase Event na stronie przelewy24
|
||||||
|
```gherkin
|
||||||
|
Given użytkownik złożył zamówienie i jest przekierowywany na /tickets/przelewy24/order=HASH
|
||||||
|
When strona przelewy24 się renderuje
|
||||||
|
Then window.dataLayer otrzymuje push eventu purchase z pełnym payloadem (transaction_id, value, currency, items)
|
||||||
|
```
|
||||||
|
|
||||||
|
## AC-2: Brak Purchase Event na order-confirm
|
||||||
|
```gherkin
|
||||||
|
Given użytkownik wraca na /tickets/order_confirm/order=HASH po płatności
|
||||||
|
When strona order-confirm się renderuje
|
||||||
|
Then window.dataLayer NIE otrzymuje push eventu purchase
|
||||||
|
```
|
||||||
|
|
||||||
|
## AC-3: Payload zawiera dane zamówienia
|
||||||
|
```gherkin
|
||||||
|
Given event purchase fires na przelewy24
|
||||||
|
When inspektujemy push w dataLayer
|
||||||
|
Then payload zawiera: event='purchase', ecommerce.transaction_id (id zamówienia), ecommerce.value (cena), ecommerce.currency='PLN', ecommerce.items (tablica biletów z item_id, item_name, quantity, price)
|
||||||
|
```
|
||||||
|
|
||||||
|
</acceptance_criteria>
|
||||||
|
|
||||||
|
<tasks>
|
||||||
|
|
||||||
|
<task type="auto">
|
||||||
|
<name>Task 1: Przenieś buildPurchaseDataLayer() z order_confirm() do przelewy24()</name>
|
||||||
|
<files>autoload/controls/class.Tickets.php</files>
|
||||||
|
<action>
|
||||||
|
W metodzie `przelewy24()`:
|
||||||
|
- Po pobraniu `$order` (linia ~571), dodaj: `$purchaseDataLayer = self::buildPurchaseDataLayer($order);`
|
||||||
|
- W wywołaniu `Tpl::view('tickets/przelewy24', [...])` dodaj klucz: `'purchase_data_layer' => $purchaseDataLayer`
|
||||||
|
|
||||||
|
W metodzie `order_confirm()`:
|
||||||
|
- Usuń linię `$purchaseDataLayer = self::buildPurchaseDataLayer($order);` (obecna tylko gdy payment_status == 1)
|
||||||
|
- W wywołaniu `Tpl::view('tickets/order-confirm', [...])` zmień `'purchase_data_layer' => $purchaseDataLayer` na `'purchase_data_layer' => null`
|
||||||
|
- Możesz też usunąć zmienną `$purchaseDataLayer = null;` z góry metody order_confirm()
|
||||||
|
|
||||||
|
Nie zmieniaj logiki `buildPurchaseDataLayer()` — tylko przenosimy gdzie jest wywoływana.
|
||||||
|
</action>
|
||||||
|
<verify>php -l "autoload/controls/class.Tickets.php"</verify>
|
||||||
|
<done>AC-1, AC-2: Kontroler przekazuje payload purchase do przelewy24 i null do order-confirm</done>
|
||||||
|
</task>
|
||||||
|
|
||||||
|
<task type="auto">
|
||||||
|
<name>Task 2: Dodaj purchase push do przelewy24.php, usuń z order-confirm.php</name>
|
||||||
|
<files>templates/tickets/przelewy24.php, templates/tickets/order-confirm.php</files>
|
||||||
|
<action>
|
||||||
|
W `templates/tickets/przelewy24.php`:
|
||||||
|
- Na samym początku pliku (przed istniejącym PHP i HTML), dodaj identyczny blok dataLayer push jak był w order-confirm.php:
|
||||||
|
```php
|
||||||
|
<?php
|
||||||
|
$purchaseDataLayerJson = null;
|
||||||
|
if (is_array($this->purchase_data_layer ?? null)) {
|
||||||
|
$purchaseDataLayerJson = json_encode(
|
||||||
|
$this->purchase_data_layer,
|
||||||
|
JSON_HEX_TAG | JSON_HEX_AMP | JSON_HEX_APOS | JSON_HEX_QUOT | JSON_INVALID_UTF8_SUBSTITUTE
|
||||||
|
);
|
||||||
|
}
|
||||||
|
?>
|
||||||
|
<?php if($purchaseDataLayerJson) : ?>
|
||||||
|
<script type="text/javascript">
|
||||||
|
window.dataLayer = window.dataLayer || [];
|
||||||
|
window.dataLayer.push({ ecommerce: null });
|
||||||
|
window.dataLayer.push(<?= $purchaseDataLayerJson; ?>);
|
||||||
|
</script>
|
||||||
|
<?php endif; ?>
|
||||||
|
```
|
||||||
|
|
||||||
|
W `templates/tickets/order-confirm.php`:
|
||||||
|
- Usuń blok PHP + script na liniach 1-17 (cały blok $purchaseDataLayerJson + conditional script tag z dataLayer.push)
|
||||||
|
- Pozostałe treści (alerty $order_successful, $order_fail, dane zamówienia) pozostaw bez zmian.
|
||||||
|
</action>
|
||||||
|
<verify>
|
||||||
|
- Sprawdź czy przelewy24.php zaczyna się od bloku PHP z $purchaseDataLayerJson
|
||||||
|
- Sprawdź czy order-confirm.php NIE zawiera "purchaseDataLayerJson" ani "dataLayer.push({ ecommerce: null })"
|
||||||
|
</verify>
|
||||||
|
<done>AC-1, AC-2, AC-3: Event fires na przelewy24, nie fires na order-confirm</done>
|
||||||
|
</task>
|
||||||
|
|
||||||
|
</tasks>
|
||||||
|
|
||||||
|
<boundaries>
|
||||||
|
|
||||||
|
## DO NOT CHANGE
|
||||||
|
- Logika metody `buildPurchaseDataLayer()` — tylko przeniesienie wywołania
|
||||||
|
- Logika wysyłania emaili w przelewy24() i order_confirm()
|
||||||
|
- Flaga `informed_user` i logika emaili w order_confirm()
|
||||||
|
- templates/site/layout-logged.php — to jest zakres Phase 3
|
||||||
|
- libraries/* — to jest zakres Phase 3
|
||||||
|
- Inne szablony i kontrolery poza wymienionymi
|
||||||
|
|
||||||
|
## SCOPE LIMITS
|
||||||
|
- Nie dodawaj deduplicacji eventu (np. sesja/cookie blokująca ponowne wyemitowanie)
|
||||||
|
- Nie zmieniaj struktury payloadu GA4 (pola event, ecommerce, order)
|
||||||
|
- Nie modyfikuj przelewy24_response() ani innych akcji kontrolera
|
||||||
|
|
||||||
|
</boundaries>
|
||||||
|
|
||||||
|
<verification>
|
||||||
|
Before declaring plan complete:
|
||||||
|
- [ ] php -l autoload/controls/class.Tickets.php — brak błędów składniowych
|
||||||
|
- [ ] templates/tickets/przelewy24.php zawiera blok $purchaseDataLayerJson i window.dataLayer.push
|
||||||
|
- [ ] templates/tickets/order-confirm.php NIE zawiera purchaseDataLayerJson ani dataLayer.push({ ecommerce: null })
|
||||||
|
- [ ] Wszystkie acceptance criteria spełnione
|
||||||
|
</verification>
|
||||||
|
|
||||||
|
<success_criteria>
|
||||||
|
- Oba zadania wykonane
|
||||||
|
- PHP syntax valid
|
||||||
|
- Event purchase fires na stronie przelewy24 z pełnym payloadem
|
||||||
|
- Brak eventu purchase na stronie order-confirm
|
||||||
|
</success_criteria>
|
||||||
|
|
||||||
|
<output>
|
||||||
|
After completion, create `.paul/phases/02-purchase-event-prepayment/02-01-SUMMARY.md`
|
||||||
|
</output>
|
||||||
107
.paul/phases/02-purchase-event-prepayment/02-01-SUMMARY.md
Normal file
107
.paul/phases/02-purchase-event-prepayment/02-01-SUMMARY.md
Normal file
@@ -0,0 +1,107 @@
|
|||||||
|
---
|
||||||
|
phase: 02-purchase-event-prepayment
|
||||||
|
plan: 01
|
||||||
|
subsystem: payments
|
||||||
|
tags: [gtm, ga4, purchase, datalayer, przelewy24, analytics]
|
||||||
|
requires:
|
||||||
|
- phase: 01-purchase-data-layer
|
||||||
|
provides: buildPurchaseDataLayer() method and payload schema
|
||||||
|
provides:
|
||||||
|
- Purchase event fires on przelewy24 page (post-order, pre-payment)
|
||||||
|
- Order-confirm no longer emits purchase event
|
||||||
|
affects: [analytics, checkout, order-confirmation]
|
||||||
|
tech-stack:
|
||||||
|
added: []
|
||||||
|
patterns:
|
||||||
|
- Fire purchase event at order commitment (przelewy24 redirect), not at payment return
|
||||||
|
key-files:
|
||||||
|
created: []
|
||||||
|
modified:
|
||||||
|
- autoload/controls/class.Tickets.php
|
||||||
|
- templates/tickets/przelewy24.php
|
||||||
|
- templates/tickets/order-confirm.php
|
||||||
|
key-decisions:
|
||||||
|
- "Purchase event przeniesiony na przelewy24 — po złożeniu zamówienia w DB, przed przekierowaniem do P24"
|
||||||
|
- "order_confirm() przekazuje purchase_data_layer: null — brak eventu na stronie potwierdzenia"
|
||||||
|
patterns-established:
|
||||||
|
- "Event purchase fires at payment commitment page, using beacon transport to survive navigation"
|
||||||
|
duration: 5min
|
||||||
|
started: 2026-04-26T00:00:00+02:00
|
||||||
|
completed: 2026-04-26T00:00:00+02:00
|
||||||
|
---
|
||||||
|
|
||||||
|
# Phase 2 Plan 1: Purchase Event Pre-Payment Summary
|
||||||
|
|
||||||
|
**Event purchase dataLayer przeniesiony z order-confirm (post-payment) na stronę przelewy24 (post-order, pre-payment redirect) — teraz capturuje 100% zamówień niezależnie od powrotu użytkownika z bramki.**
|
||||||
|
|
||||||
|
## Performance
|
||||||
|
|
||||||
|
| Metric | Value |
|
||||||
|
|--------|-------|
|
||||||
|
| Duration | ~5 min |
|
||||||
|
| Started | 2026-04-26 |
|
||||||
|
| Completed | 2026-04-26 |
|
||||||
|
| Tasks | 2 completed |
|
||||||
|
| Files modified | 3 |
|
||||||
|
|
||||||
|
## Acceptance Criteria Results
|
||||||
|
|
||||||
|
| Criterion | Status | Notes |
|
||||||
|
|-----------|--------|-------|
|
||||||
|
| AC-1: Purchase event na stronie przelewy24 | Pass | dataLayer.push fires na początku przelewy24.php przed auto-submit formularza |
|
||||||
|
| AC-2: Brak purchase event na order-confirm | Pass | Blok JSON/push usunięty z order-confirm.php; kontroler przekazuje null |
|
||||||
|
| AC-3: Payload zawiera dane zamówienia | Pass | buildPurchaseDataLayer() bez zmian — transaction_id, value, currency, items |
|
||||||
|
|
||||||
|
## Accomplishments
|
||||||
|
|
||||||
|
- buildPurchaseDataLayer() wywołany w przelewy24() kontrolera i payload przekazany do szablonu
|
||||||
|
- Blok dataLayer.push dodany na początku przelewy24.php (przed spinner/formularzem)
|
||||||
|
- Blok dataLayer.push usunięty z order-confirm.php (linie 1-17)
|
||||||
|
- order_confirm() przekazuje `purchase_data_layer: null` jawnie
|
||||||
|
|
||||||
|
## Task Commits
|
||||||
|
|
||||||
|
No git commits created during APPLY (working tree changes only).
|
||||||
|
|
||||||
|
## Files Created/Modified
|
||||||
|
|
||||||
|
| File | Change | Purpose |
|
||||||
|
|------|--------|---------|
|
||||||
|
| `autoload/controls/class.Tickets.php` | Modified | buildPurchaseDataLayer() call przeniesiony do przelewy24(), usunięty z order_confirm() |
|
||||||
|
| `templates/tickets/przelewy24.php` | Modified | Dodany blok purchase dataLayer push na początku szablonu |
|
||||||
|
| `templates/tickets/order-confirm.php` | Modified | Usunięty blok purchase dataLayer push |
|
||||||
|
|
||||||
|
## Decisions Made
|
||||||
|
|
||||||
|
| Decision | Rationale | Impact |
|
||||||
|
|----------|-----------|--------|
|
||||||
|
| Event na przelewy24, nie order-confirm | Użytkownicy nie wracający z P24 nie byli trackingowani; zamówienie w DB to moment konwersji | Pełne pokrycie analityczne zakupów |
|
||||||
|
| Bez deduplicacji eventu | Strona przelewy24 jest odwiedzana raz per zamówienie; dodanie sesji/cookie to over-engineering | GA4 może dostać duplikat przy odświeżeniu strony (edge case) |
|
||||||
|
|
||||||
|
## Deviations from Plan
|
||||||
|
|
||||||
|
None — plan wykonany zgodnie ze specyfikacją.
|
||||||
|
|
||||||
|
## Issues Encountered
|
||||||
|
|
||||||
|
| Issue | Resolution |
|
||||||
|
|-------|------------|
|
||||||
|
| None | — |
|
||||||
|
|
||||||
|
## Next Phase Readiness
|
||||||
|
|
||||||
|
**Ready:**
|
||||||
|
- Purchase event ma stabilny punkt wyzwalania pre-payment
|
||||||
|
- GTM/GA4 używa beacon transport — event przeżyje nawigację do P24
|
||||||
|
- Phase 3 (Cookie Consent) może teraz dodać Consent Mode v2 init przed GTM w layoucie
|
||||||
|
|
||||||
|
**Concerns:**
|
||||||
|
- Odświeżenie strony przelewy24 przed zapłatą może spowodować duplikat eventu (akceptowalny edge case)
|
||||||
|
- Warto skonfigurować tag w GTM z transport mode "beacon" dla pewności
|
||||||
|
|
||||||
|
**Blockers:**
|
||||||
|
- None
|
||||||
|
|
||||||
|
---
|
||||||
|
*Phase: 02-purchase-event-prepayment, Plan: 01*
|
||||||
|
*Completed: 2026-04-26*
|
||||||
@@ -650,11 +650,14 @@ class Tickets
|
|||||||
$przelewy24_hash = md5(time());
|
$przelewy24_hash = md5(time());
|
||||||
$mdb->update('orders', ['payment_hash' => $przelewy24_hash], ['id' => $order['id']]);
|
$mdb->update('orders', ['payment_hash' => $przelewy24_hash], ['id' => $order['id']]);
|
||||||
|
|
||||||
|
$purchaseDataLayer = self::buildPurchaseDataLayer($order);
|
||||||
|
|
||||||
return \Tpl::view('tickets/przelewy24', [
|
return \Tpl::view('tickets/przelewy24', [
|
||||||
'settings' => $settings,
|
'settings' => $settings,
|
||||||
'hash' => \S::get('order'),
|
'hash' => \S::get('order'),
|
||||||
'order' => $order,
|
'order' => $order,
|
||||||
'przelewy24_hash' => $przelewy24_hash
|
'przelewy24_hash' => $przelewy24_hash,
|
||||||
|
'purchase_data_layer' => $purchaseDataLayer
|
||||||
]);
|
]);
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -847,14 +850,12 @@ class Tickets
|
|||||||
$order = \factory\Tickets::get_order_details_by_hash(\S::get('order'));
|
$order = \factory\Tickets::get_order_details_by_hash(\S::get('order'));
|
||||||
$order_successful = false;
|
$order_successful = false;
|
||||||
$order_fail = false;
|
$order_fail = false;
|
||||||
$purchaseDataLayer = null;
|
|
||||||
|
|
||||||
if ( $order && !empty($order['payment_status']) && empty($order['informed_user']) )
|
if ( $order && !empty($order['payment_status']) && empty($order['informed_user']) )
|
||||||
{
|
{
|
||||||
if ($order['payment_status'])
|
if ($order['payment_status'])
|
||||||
{
|
{
|
||||||
$order_successful = true;
|
$order_successful = true;
|
||||||
$purchaseDataLayer = self::buildPurchaseDataLayer($order);
|
|
||||||
$mailSent = self::sendPaidOrderSummaryEmail($order);
|
$mailSent = self::sendPaidOrderSummaryEmail($order);
|
||||||
if ($mailSent) {
|
if ($mailSent) {
|
||||||
$mdb->update('orders', ['informed_user' => 1], ['id' => $order['id']]);
|
$mdb->update('orders', ['informed_user' => 1], ['id' => $order['id']]);
|
||||||
@@ -871,7 +872,7 @@ class Tickets
|
|||||||
'order' => $order,
|
'order' => $order,
|
||||||
'order_successful' => $order_successful,
|
'order_successful' => $order_successful,
|
||||||
'order_fail' => $order_fail,
|
'order_fail' => $order_fail,
|
||||||
'purchase_data_layer' => $purchaseDataLayer
|
'purchase_data_layer' => null
|
||||||
]);
|
]);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
@@ -1,21 +1,3 @@
|
|||||||
<?php
|
|
||||||
$purchaseDataLayerJson = null;
|
|
||||||
if (is_array($this->purchase_data_layer ?? null)) {
|
|
||||||
$purchaseDataLayerJson = json_encode(
|
|
||||||
$this->purchase_data_layer,
|
|
||||||
JSON_HEX_TAG | JSON_HEX_AMP | JSON_HEX_APOS | JSON_HEX_QUOT | JSON_INVALID_UTF8_SUBSTITUTE
|
|
||||||
);
|
|
||||||
}
|
|
||||||
?>
|
|
||||||
|
|
||||||
<?php if($purchaseDataLayerJson) : ?>
|
|
||||||
<script type="text/javascript">
|
|
||||||
window.dataLayer = window.dataLayer || [];
|
|
||||||
window.dataLayer.push({ ecommerce: null });
|
|
||||||
window.dataLayer.push(<?= $purchaseDataLayerJson; ?>);
|
|
||||||
</script>
|
|
||||||
<?php endif; ?>
|
|
||||||
|
|
||||||
<?php if($this->order_successful === true) : ?>
|
<?php if($this->order_successful === true) : ?>
|
||||||
<script type="text/javascript">
|
<script type="text/javascript">
|
||||||
$.alert({
|
$.alert({
|
||||||
|
|||||||
@@ -1,3 +1,21 @@
|
|||||||
|
<?php
|
||||||
|
$purchaseDataLayerJson = null;
|
||||||
|
if (is_array($this->purchase_data_layer ?? null)) {
|
||||||
|
$purchaseDataLayerJson = json_encode(
|
||||||
|
$this->purchase_data_layer,
|
||||||
|
JSON_HEX_TAG | JSON_HEX_AMP | JSON_HEX_APOS | JSON_HEX_QUOT | JSON_INVALID_UTF8_SUBSTITUTE
|
||||||
|
);
|
||||||
|
}
|
||||||
|
?>
|
||||||
|
|
||||||
|
<?php if($purchaseDataLayerJson) : ?>
|
||||||
|
<script type="text/javascript">
|
||||||
|
window.dataLayer = window.dataLayer || [];
|
||||||
|
window.dataLayer.push({ ecommerce: null });
|
||||||
|
window.dataLayer.push(<?= $purchaseDataLayerJson; ?>);
|
||||||
|
</script>
|
||||||
|
<?php endif; ?>
|
||||||
|
|
||||||
<?php
|
<?php
|
||||||
$clientData = $this->order;
|
$clientData = $this->order;
|
||||||
$merchant_id = $this->settings['p24']['merchant_id'];
|
$merchant_id = $this->settings['p24']['merchant_id'];
|
||||||
|
|||||||
Reference in New Issue
Block a user