dotpay / WooCommerce2

WooCommerce2
MIT License
11 stars 11 forks source link

ID zamówienia w dotpay gdy w ID jest zmienione w WC #53

Closed 12maup closed 5 years ago

12maup commented 5 years ago

Witam,

Klient zmienił defaultowe id zamówień na cykliczne. A więc post id !== id zamówienia w panelu admina i mejlach do klientów. Jest to o tyle kłopotliwe że klient dostaje mejla z dotpay z innym numerem zamówienia niż to które ma w mejlu potwierdzającym zakup.

Sprawę udało mi się rozwiązać:

C:\Users\Lenovo\Documents\RebelWeb\mumla\mumla.pl\wp-content\plugins\woocommerce-dotpay\Dotpay\Payment.php

    public function getDescription()
    {
        return __('Order ID: ', 'dotpay-payment-gateway') . esc_attr($this->getLegacyOrderId_2($this->getOrder()));
    }

gdzie getLegacyOrderId_2 to:

    private function getLegacyOrderId_2($orderObject)
    {
        if (method_exists($orderObject, 'get_id')) {
      //     return $orderObject->get_id();
            return $orderObject->get_order_number();
        } else {
            return $orderObject->id;
        }
    }

A więc get_id() zamieniłem na get_order_number();

Działa, ale może warto by było uwzględnić to w kolejnej aktualizacji?

Pozdrawiam, Piotr Wiktor.

wpawel commented 5 years ago

Dzięki za sugestię. Dodano w nowej wersji: v3.2.5