Omnipay\Myfatoorah\Message\CompletePurchase

Complexity

Total Complexity

10

Size/Duplication

Total Lines	32
Duplicated Lines	0 %

Importance

Changes	2
Bugs	0	Features	0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	getPaymentId()	0	2	2
A	setPaymentId()	0	2	1
A	getEndpoint()	0	3	2
A	getInvoiceId()	0	2	2
A	getHttpMethod()	0	2	1
A	getData()	0	9	2

<?php

namespace Omnipay\Myfatoorah\Message;

class CompletePurchase extends AbstractRequest {

    public function getPaymentId() {
        return isset($_GET['paymentId']) ? $_GET['paymentId'] : '';
    }

    public function getInvoiceId() {
        return isset($_GET['invoiceid']) ? $_GET['invoiceid'] : '';
    }

    public function setPaymentId($value) {
        return $this->setParameter('paymentId', $value);
    }

    public function getData() {
        if($this->getPaymentId()){
            $data['Key']     = $this->getPaymentId();

$data was never initialized. Although not strictly required by PHP, it is generally a good practice to add $data = array(); before regardless.

            $data['KeyType'] = "paymentId";
        }else{
            $data['Key']     = $this->getInvoiceId();
            $data['KeyType'] = "invoiceid";
        }
        return $data;
    }

    public function getHttpMethod() {
        return 'POST';
    }

    public function getEndpoint() {
        $endpoint = $this->getTestMode() ? $this->sandboxEndpoint : $this->productionEndpoint;
        return $endpoint . '/v2/getpaymentstatus';
    }

}