linknacional / whmcs-boleto-pix Goto Github PK

O código não identificar se é PIX ou boleto, gerando confusão pro usuário, seria interessante corrigir o texto para mostrar aonde esta PIX, o método de pagamento selecionado: PIX ou BOLETO.

Olá,
após atualizar o módulo da versão original para o fork da linknacional (v2.5.3) visualizei no log a seguinte mensagem no primeiro update de status (através da opção "Log de Transações do Gateway" do WHMCS):

"Não foi possível atualizar o banco de dados da Paghiper para a versão 1.4. Por favor cheque se o usuário MySQL tem permissões para alterar a tabela mod_paghiper"

Ao invés do usuário digitar o ID do campo personalizado. Adicionar um select para o usuário selecionar o campo personalizado correspondiente na configurações do modulo.

@brunoferreiralkn Precisa adicionar verificação de acesso direto de arquivo?

if (!defined('WHMCS')) {
    die('This file cannot be accessed directly');
}

Originally posted by @emanuellopess in #2 (comment)

É necessário refatorar por completo o módulo para tornar viável novas implementações, sem aumentar o nível de complexidade.

• Aplicar orientação a objetos
• Aplicar PSRs
• Suporte ao PHP 8

Olá!

Não sou desenvolvedor WHMCS mas com a ajuda do ChatGPT consegui identificar esse problema no código de vocês, então vou ser bem objetivo e direto.

Eu notei que o módulo de boletos se chama "paghiper" e o de PIX "paghiper_pix". O problema ocorre quando alguns usuários, como eu, não querem dar a opção de pagamento por boleto bancário para os seus clientes, por isso só ativei "paghiper_pix".

No entanto, sempre que eu tentava cancelar uma fatura me deparava com um erro. Analisando os hooks notei que o problema está neste arquivo:
https://github.com/LinkNacional/whmcs-boleto-pix/blob/470100c1017c8a7ac0511889a7c9994e1f8b9276/src/includes/hooks/cancel_paghiper_slip.php

Levando em consideração o código parece que é enviado uma requisição HTTP para o PagHiper, informando que o boleto foi cancelado pelo cliente. O problema é que como o módulo "paghiper" de boleto não está ativo no meu WHMCS, me é retornado o erro "Module not found".

O que eu fiz foi comentar a adição de hook no final deste arquivo:

// add_hook('AddInvoicePayment', 1, 'paghiper_cancel_paghiper_slips');
// add_hook('InvoiceCancelled', 1, 'paghiper_cancel_paghiper_slips');
// add_hook('InvoicePaid', 1, 'paghiper_cancel_paghiper_slips');

Poderia apenas ter excluído o arquivo também...
Acredito que vocês devem adicionar uma verificação se o módulo "paghiper" está ativado antes de continuar a execução do código.

TypeError: array_shift(): Argument #1 ($array) must be of type array, bool given in /modules/gateways/paghiper/inc/helpers/gateway_functions.php:772
Stack trace:
#0 /modules/gateways/paghiper/inc/helpers/gateway_functions.php(772): array_shift()
#1 /modules/gateways/paghiper/inc/helpers/process_payment.php(323): generate_paghiper_billet()
#2 /modules/gateways/paghiper.php(299): require_once('...')
#3 {main}