marinsagovac/woocommerce-tcom-payway

Neispravan broj narudžbe

shaxxx opened this issue · 1 comments

shaxxx commented

Mail koji se pošalje kao broj narudžbe uvijek daje 0, pa sve narudžbe imaju isti naslov mail-a.
Primjerice: "Payment for order no. 0 was sucessful."

woocommerce-tcom-payway/classes/class-wc-tpayway.php

Line 482 in 0c0764c

__('Payment for order no. %s was sucessful.', 'tcom-payway-wc'),

shaxxx commented

Čini se da je problem malo veći, sanitacija POST parametara vrati prazan string za ShoppingCartID, što onda dovede do nepostojećeg ordera pa se onda ne ažurira ni status narudžbe. Osim toga isti parametar se iznova parsira na više mjesta umjesto korištenja postojećeg $order_id.
Kada se sanitacija parametra makni i koristi se sirova vrijednost sve radi očekivano.

Ispravljeno sa #12