rodasistemas
commented
4 years ago Podem remove-la, pois ela só teve utilidade no débito do plugin.
Em qua, 21 de out de 2020 12:47, Laerte Guimarães notifications@github.com escreveu:
@laerte-guimaraes requested changes on this pull request.
O arquivo src/utils/PaymentProcessor.php possui diversos loggers. Eles são realmente necessários? Se sim, acho que poderia ser legal padronizar as comunicações. Por exemplo: Se for um log que o cliente pode usar para tirar alguma dúvida com a Vindi, esses poderiam deixar mais claro qual é a parte do fluxo em que ocorreram e incluir informações importantes para esse fluxo.
Faz sentido? 🤔
In src/utils/PaymentProcessor.php https://github.com/vindi/vindi-woocommerce/pull/16#discussion_r509394500 :
$vindi_customer_id = get_user_meta($current_user->ID, 'vindi_customer_id', true);
$vindi_customer = $this->routes->findCustomerById($vindi_customer_id);
$this->logger->log(sprintf('Vindi Customer ID : %s', $vindi_customer_id));
// $vindi_customer = $this->routes->findCustomerById($vindi_customer_id);
Essa linha é necessária? Se não podemos removê-la ? 🤔
In src/utils/RedirectCheckout.php https://github.com/vindi/vindi-woocommerce/pull/16#discussion_r509396236 :
@@ -21,8 +21,8 @@ function VindiRedirectToMyAccount() {
function VindiRedirectInfo() {
-
if(is_user_logged_in()) return;
/ (fix:issue:#6) /
⬇️ Suggested change
- / (fix:issue:#6) /
In src/templates/admin-settings.html.php https://github.com/vindi/vindi-woocommerce/pull/16#discussion_r509397620 :
<p><?php echo sprintf(__('Falha na conexão! <br><strong>%s</strong>', VINDI), $settings->api->last_error); ?></p></div>
<?php endif; ?>
<?php if ((isset($api_key) || strlen($api_key)) == 43 && !empty($api_key)): ?>
- Githubissues.
- Githubissues is a development platform for aggregating issues.
Motivação
O plugin estava retornando Internal Server 500 ao tentar realizar uma assinatura ou compra avulsa.
Solução Proposta
Foi corrigido os parametros e os Hooks responsáveis pela comunicação entre o plugin e a API e também tratado os tipos de retornos emitidos pela API para atualização de dados no Woocommerce.