iworks/omnibus

`get_price_including_tax` deprecated

kanlukasz opened this issue · 3 comments

Cześć,

W ostatniej wersji używasz metody get_price_including_tax(), która jest już deprecated.
Dokładnie w 3 miejscach:
image

To betonuje mi logi w tysiącach wpisów:
PHP Deprecated: Funkcja WC_Product::get_price_including_tax jest przestarzała od wersji 3.0! Zamiast jej proszę użyć wc_get_price_including_tax.

Pytanie:
Czy do momentu wyjścia nowej wersji, wystarczy zmiana tej metody na funkcje wc_get_price_including_tax()?
I czy tylko zakładasz pierwszy argument jako obiekt klasy WC_Product czy w drugim też przekazać jakieś dane?
Czy coś jeszcze do podmiany?

Po poprawce widzę to tak:
image

Hey @kanlukasz

Masz rację, dlatego aktualna wersja dev nie zawiera już funkcji get_price_including_tax().

Można ją wypróbować pobierając tutaj: https://github.com/iworks/omnibus/releases/tag/3.0.0.beta2

Tyle to ja wiem :) Bardziej pytałem jaki widzisz tymczasowy fix bo zanim 3.0 wyjdzie to jeszcze pewnie minie trochę, a logi w serwisach mi puchną oporowo

Pytanie: Czy do momentu wyjścia nowej wersji, wystarczy zmiana tej metody na funkcje wc_get_price_including_tax()? I czy tylko zakładasz pierwszy argument jako obiekt klasy WC_Product czy w drugim też przekazać jakieś dane? Czy coś jeszcze do podmiany?

Po poprawce widzę to tak: image

W poniedziałek wydam :D