search

marinsagovac / woocommerce-tcom-payway

[WORDPRESS] Woocommerce plugin for payment service using PayWay HT payment gateway service
MIT License
26 stars 16 forks source link

Neispravan broj narudžbe #13

Closed shaxxx closed 3 years ago

shaxxx commented 3 years ago

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."

https://github.com/marinsagovac/woocommerce-tcom-payway/blob/0c0764c457855107ec378dea715c9734dfefc3d3/classes/class-wc-tpayway.php#L482

shaxxx commented 3 years ago

Č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 https://github.com/marinsagovac/woocommerce-tcom-payway/pull/12