greguly / cielo-woocommerce

Adiciona a Cielo como opção de recebimento no WooCommerce
GNU General Public License v2.0
31 stars 21 forks source link

Abstrair a finalização #16

Closed claudiosanches closed 9 years ago

claudiosanches commented 9 years ago

Agora como temos várias formas de checkout precisamos abstrair o checkout no método:

public function process_payment( $order_id )

Agora esse método esta tentando tratar tudo de uma vez, precisamos dividir isso em métodos menores para manter organizado e simplificar a manutenção.

gopaulo commented 9 years ago

Claudio - estou pensando em fazer algo assim

    public function process_payment( $order_id ) {

        $payment_function = 'process_payment_'.$this->store_contract;
        return call_user_func($payment_function,$order_id);
    }

e aí ter uma função para cada solução process_payment_webservice, process_payment_buypage_cielo e no futuro process_payment_checkout_cielo

mas todas protected.

gopaulo commented 9 years ago

@claudiosmweb - Teria como você dar uma check na abstração que eu fiz.

claudiosanches commented 9 years ago

Pode trocar por outros métodos e dentro de process_payment apenas usar if, não precisa de muita coisa. De qualquer forma, final de semana vou tirar para terminar os detalhes, assim qualquer coisa dou uma arrumada.

claudiosanches commented 9 years ago

Resolvido em 8a0642cad21491d989120011af6b42861d25afb7