9.2 KiB
9.2 KiB
phase, plan, type, wave, depends_on, files_modified, autonomous, delegation
| phase | plan | type | wave | depends_on | files_modified | autonomous | delegation | |||
|---|---|---|---|---|---|---|---|---|---|---|
| 96-automation-payment-method | 01 | execute | 1 |
|
false | off |
Purpose
Uzytkownik chce triggerowac reguly automatyzacji w zaleznosci od metody platnosci (np. COD vs przelew), co jest ortogonalne do statusu platnosci (oplacone/nieoplacone). Pozwala np. na: "Gdy metoda = Pobranie AND status = Nieoplacone -> wyslij email z przypomnieniem".
Output
- Nowy typ warunku
payment_methodw automatyzacji - Predefiniowane opcje: Platnosc przy odbiorze (COD), Przelew, Karta, Inna
- Ewaluacja warunku porownuje
external_payment_type_idz wybranymi metodami (COD uzywa StringHelper::isCodPayment)
Source Files
@src/Modules/Automation/AutomationController.php (ALLOWED_CONDITION_TYPES, PAYMENT_STATUS_OPTIONS, parseConditions, renderForm) @src/Modules/Automation/AutomationService.php (evaluateSingleCondition, evaluatePaymentStatusCondition) @resources/views/automation/form.php (condition type select, checkbox-group pattern) @src/Core/Support/StringHelper.php (isCodPayment — normalizacja COD)
No specialized flows configured.<acceptance_criteria>
AC-1: Warunek payment_method dostepny w formularzu
Given uzytkownik tworzy/edytuje regule automatyzacji
When wybierze typ warunku "Metoda platnosci"
Then zobaczy checkboxy: "Platnosc przy odbiorze (COD)", "Przelew", "Karta/online", "Inna"
AC-2: Ewaluacja warunku COD
Given regula ma warunek payment_method = ["cod"]
When zdarzenie dotyczy zamowienia z external_payment_type_id rozpoznawanym jako COD (np. "CASH_ON_DELIVERY", "pobranie", "za pobraniem")
Then warunek jest spelniony (uzywa StringHelper::isCodPayment)
AC-3: Ewaluacja warunku nie-COD
Given regula ma warunek payment_method = ["transfer"]
When zdarzenie dotyczy zamowienia z external_payment_type_id = "przelew" lub "wire_transfer" lub "bank_transfer"
Then warunek jest spelniony
And zamowienie COD NIE spelnia tego warunku
AC-4: Zapis i odczyt warunku
Given uzytkownik zapisal regule z warunkiem payment_method
When edytuje te regule ponownie
Then checkboxy payment_method sa poprawnie zaznaczone (zachowanie stanu)
</acceptance_criteria>
Task 1: Backend — nowy warunek payment_method src/Modules/Automation/AutomationController.php, src/Modules/Automation/AutomationService.php **AutomationController.php:** 1. Dodac 'payment_method' do ALLOWED_CONDITION_TYPES 2. Dodac stala PAYMENT_METHOD_OPTIONS: ```php private const PAYMENT_METHOD_OPTIONS = [ 'cod' => 'Platnosc przy odbiorze (COD)', 'transfer' => 'Przelew bankowy', 'online' => 'Karta / platnosc online', 'other' => 'Inna', ]; ``` 3. W renderForm() przekazac 'paymentMethodOptions' => self::PAYMENT_METHOD_OPTIONS 4. W parseConditions() dodac obsluge typu 'payment_method': - Odczytac `$cond['payment_method_keys']` jako tablica - Zapisac jako `['method_keys' => [...]]` 5. W walidacji (validateConditionValue lub analogiczne) dodac obsluge 'payment_method': - Odczytac `$condition['payment_method_keys']`, filtrowac przez array_keys(PAYMENT_METHOD_OPTIONS) - Zwrocic `['method_keys' => [...]]` lub null jesli puste**AutomationService.php:**
1. W evaluateSingleCondition() dodac case 'payment_method':
```php
if ($type === 'payment_method') {
return $this->evaluatePaymentMethodCondition($value, $order);
}
```
2. Nowa metoda evaluatePaymentMethodCondition(array $value, array $order): bool
- Pobrac $methodKeys z $value['method_keys'] (tablica stringow)
- Pobrac $paymentType z $order['external_payment_type_id']
- Jesli $paymentType pusty — return false
- Dla kazdego wybranego klucza:
- 'cod': sprawdzic StringHelper::isCodPayment($paymentType)
- 'transfer': sprawdzic czy $paymentType zawiera 'PRZELEW', 'TRANSFER', 'WIRE' (case-insensitive)
- 'online': sprawdzic czy zawiera 'CARD', 'ONLINE', 'PAYU', 'PRZELEWY24', 'BLIK', 'TPAY'
- 'other': true jesli zaden z powyzszych nie pasowal
- Return true jesli JAKIKOLWIEK wybrany klucz pasuje (OR logic, jak payment_status)
2. W sekcji renderowania pol warunku (po bloku payment_status, linia ~115) dodac:
```php
<?php elseif ($conditionType === 'payment_method'): ?>
<div class="checkbox-group">
<?php foreach ($paymentMethodOptions as $methodKey => $methodLabel): ?>
<label class="checkbox-label">
<input type="checkbox" name="conditions[<?= $idx ?>][payment_method_keys][]" value="<?= $e((string) $methodKey) ?>"<?= in_array((string) $methodKey, $selectedMethodKeys, true) ? ' checked' : '' ?>>
<?= $e($methodLabel) ?>
</label>
<?php endforeach; ?>
</div>
```
3. W sekcji ustawiajacej $selectedStatusKeys (okolo linii ~75-80) dodac analogiczny blok dla payment_method:
```php
$selectedMethodKeys = [];
if (($cond['condition_type'] ?? '') === 'payment_method') {
$condValue = is_array($cond['condition_value'] ?? null) ? $cond['condition_value'] : [];
$selectedMethodKeys = is_array($condValue['method_keys'] ?? null) ? $condValue['method_keys'] : [];
}
```
4. W JS (onConditionTypeChange) upewnic sie ze payment_method jest obslugiwany jak inne typy z checkboxami — jesli JS generuje dynamicznie pola, dodac case 'payment_method' wzorowany na 'payment_status'
DO NOT CHANGE
- src/Core/Support/StringHelper.php (tylko czytac isCodPayment, nie modyfikowac)
- database/migrations/* (nie potrzeba nowej migracji — dane sa juz w external_payment_type_id)
- Istniejace warunki (payment_status, shipment_status, order_status, integration, days_in_status) — nie ruszac
SCOPE LIMITS
- Nie dodawac nowego eventu — warunek payment_method dziala z KAZDYM istniejacym eventem (jak integration)
- Nie dodawac dynamicznego pobierania metod z bazy — predefiniowane opcje wystarczaja
- Nie zmieniac pola external_payment_type_id ani logiki importu
<success_criteria>
- Nowy typ warunku payment_method w ALLOWED_CONDITION_TYPES
- Ewaluacja w AutomationService oparta na StringHelper::isCodPayment + pattern matching
- UI z checkboxami identyczne wzorcem jak payment_status
- Regula z warunkiem payment_method poprawnie triggeruje sie dla zamowien COD </success_criteria>