woocommerce-pagarme
woocommerce-pagarme copied to clipboard
claudiosanches
Corrige chamadas diretas às propriedades do objeto order
Nas versões mais recentes do Woocommerce, ao realizar chamadas diretas às propriedades do objeto order, nós recebemos um aviso (notice). Exemplo:
Notice: id foi chamada incorretamente. Order properties should not be accessed directly.
Por mais que acabe funcionando, devido a funções de compatibilidade (link do repo do Woocommerce), alguns clientes já estão reclamando desse "erro".
Ao corrigir, o que antes era, por exemplo:
$order->id
fica:
$order->get_id()
Além disso, adicionei uma verificação de versão mínima do Woocommerce, já que pelo que entendi as funções mais recentes de get das propriedades foram incluídas na versão 3.0 (que é de 2017).
Obs.: Também tive que ajustar a url do teste de boleto, revertendo a alteração desse commit, devido ao "revert" que também rolou aqui na Pagar.me.
Oi @claudiosanches tudo bem? Como pode ver, a verificação de versão mínima acabei fazendo em 2 arquivos, no arquivo principal do plugin e também no de template. Se tiver uma sugestão melhor por favor me avisa :)
Oi @claudiosanches tudo bem? Consegue por favor dar uma força no review após minhas últimas alterações?
Oi @claudiosanches tudo bem? Consegue por favor dar uma força nesse review / approve?
Oi @claudiosanches tudo bem? Consegue por favor dar uma força nesse review / approve para a gente poder fazer o merge?
Oi @claudiosanches tudo bem? Consegue por favor dar uma força nesse review / approve para a gente poder fazer o merge?
@claudiosanches, você consegue dar uma força nesse PR? Seu review tá como Request changes ainda, mas o @willian-soaresferreira já alterou as coisas que você pediu
Além disso não seguimos em frente com ele por mexer em alguns pontos centrais do plugin, e gostaríamos muito do teu approve
