Corrige chamadas diretas às propriedades do objeto order #130
Conversation
|
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 :) |
sfwill-dev
force-pushed
the
fix/direct-call-to-order-object-properties
branch
2 times, most recently
from
3b38f0b to
5ec3279
Compare
|
@claudiosanches consegue dar uma força com esse review? |
sfwill-dev
force-pushed
the
fix/direct-call-to-order-object-properties
branch
2 times, most recently
from
489a495 to
691e698
Compare
|
Valeu pelos comentários @claudiosanches! Já fiz as mudanças :) |
sfwill-dev
force-pushed
the
fix/direct-call-to-order-object-properties
branch
from
691e698 to
46ac70f
Compare
|
Boa @claudiosanches, feito! |
|
Oi @claudiosanches tudo bem? Consegue por favor dar uma força no review após minhas últimas alterações? |
includes/class-wc-pagarme-api.php
Outdated
| @@ -246,17 +246,17 @@ public function generate_transaction_data( $order, $posted ) { | |||
| 'amount' => $order->get_total() * 100, | |||
| 'postback_url' => WC()->api_request_url( get_class( $this->gateway ) ), | |||
| 'customer' => array( | |||
| 'name' => trim( $order->billing_first_name . ' ' . $order->billing_last_name ), | |||
| 'email' => $order->billing_email, | |||
| 'name' => trim( $order->get_billing_first_name() . ' ' . $order->get_billing_last_name() ), | |||
There was a problem hiding this comment.
| 'name' => trim( $order->get_billing_first_name() . ' ' . $order->get_billing_last_name() ), | |
| 'name' => $order->get_formatted_billing_full_name(), |
sfwill-dev
force-pushed
the
fix/direct-call-to-order-object-properties
branch
from
46ac70f to
0d19b64
Compare
|
Boa, obrigado @claudiosanches, fiz a alteração! |
|
Oi @claudiosanches tudo bem? Consegue por favor dar uma força nesse review / approve? |
sfwill-dev
force-pushed
the
fix/direct-call-to-order-object-properties
branch
2 times, most recently
from
1b829a0 to
0a18de5
Compare
|
Oi @claudiosanches tudo bem? Consegue por favor dar uma força nesse review / approve para a gente poder fazer o merge? |
1 similar comment
|
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 |
|
@murilohns consigo sim no final de semana. |
sfwill-dev
force-pushed
the
fix/direct-call-to-order-object-properties
branch
2 times, most recently
from
95e8e18 to
cac2ca7
Compare
sfwill-dev
force-pushed
the
fix/direct-call-to-order-object-properties
branch
from
cac2ca7 to
b161f7c
Compare
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->idfica:
$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.