Dodanie odwołań do api commerca po zakończeniu płatności

Pierwsza próba. Wrzucam patcha, ale wydaje mi się że to jeszcze nie jest to zaimplementowane tak jak powinno.

As of Drupal Commerce 1.0, when you receive a payment notification and create the associated payment transaction, you should also be calling one of two API functions designed to move the related order forward or backward in the checkout process. The checkout router was updated to permit customers returning to the checkout/%commerce_order/payment page to be redirected to the appropriate checkout page upon their eventual return.

Writing a payment method module

I dalej opisują wspomniane wyżej funkcje.

Brak odwołań objawił się przy pokazywaniu błędu związanego z brakiem billing information. Po wykonaniu się kodu

     	drupal_set_message(t('Please enable Billing information in CHECKOUT PANE.'), 'error');
	drupal_goto('checkout');
	return '';

Zrzucało z checkout na główną; a samo zamówienie zostawało zablokowane. Zamówienie nie wracało ze stanu "w trakcie płatności", i użytkownik nie ma do niego dostępu.

To jest drobiazg, nikt poza developerem nie powinien zobaczyć takiego błędu, ale myślę że przy okazji można to też obsłużyć. Zmiany w pliku commerce_payu.module są wg mnie ok.

Z kolei moje zmiany w includes/commerce_payu.inc są w tym momencie prawdopodobnie błędne. Jak testowałem kod nie było widać żadnego wpływu obecności funkcji na działanie.

Z tego co czytałem błąd może być nie deterministyczny, lub występować kiedy ktoś po skończeniu płatności nie otworzy strony na którą będzie przekierowany. Potrzebuje jeszcze o tym doczytać; potestuję i podzielę się informacjami.

Myślę że rozumiem: przekierowania z 'commerce_payu_close_transaction' załatwiają sprawę - o ile 'commerce_payu/close_transaction' będzie otwarte przez usera.

Jeśli przekierowanie nie będzie otwarte - informacja o wyniku płatności trafia do nas na UrlOnline. Jeśli nie wywołamy w takiej sytuacji jednej z wyżej wymienionych funkcji - zlecenie zostanie w stanie 'w trakcie płatności' kiedy jest zablokowane dla usera i ukryte przed adminem. Rzadki, ale brzydki błąd.

w załączonym patchu:

1. commerce_payu.module - kosmetyczna zmiana opisana w #3
2. includes/commerce_payu.inc - wyeliminowanie przypadku opisanego wyżej

Przetestowałem: nie otworzyłem url od payu => a informacja o zakończeniu płatności trafiła do system. Patch zadziałał jak trzeba.

Zostaje wg. mnie umiejscowienia logiki podejmowania decyzji na podstawie danych przekazanych przez payu do urlpositive/urlnegative. Wg dokumentacji:

Adresy powrotu do Sklepu mają tylko charakter informacyjny, nie można na ich podstawie podejmować żadnych decyzji.

Myślę się tym zająć. Jak będę mieć coś gotowego dam znać.