ByTIC\Omnipay\PlatiOnline\Message\Traits\RelayResponseTrait

Complexity

Total Complexity

7

Size/Duplication

Total Lines	64
Duplicated Lines	0 %

Importance

Changes	2
Bugs	0	Features	1

7 Methods

Rating	Name	Duplication	Size	Complexity
A	getTransactionId()	0	3	1
A	getMessage()	0	5	1
A	isPending()	0	3	1
A	getCode()	0	3	1
A	getNotification()	0	3	1
A	isSuccessful()	0	3	1
A	getTransactionReference()	0	3	1

<?php

namespace ByTIC\Omnipay\PlatiOnline\Message\Traits;

use SimpleXMLElement;

/**
 * Trait RelayResponseTrait
 * @package ByTIC\Omnipay\PlatiOnline\Message\Traits
 */
trait RelayResponseTrait
{
    /**
     * @inheritDoc
     * @noinspection PhpMissingReturnTypeInspection
     */
    public function isSuccessful()
    {
        return $this->getCode() == '2';
    }

    /**
     * @inheritDoc
     * @noinspection PhpMissingReturnTypeInspection
     */
    public function isPending()
    {
        return $this->getCode() == '13';
    }

    /**
     * @inheritDoc
     * @noinspection PhpMissingReturnTypeInspection
     */
    public function getMessage()
    {
        return
            'CODE: '. $this->getNotification()->x_response_reason_code
            .' '. $this->getNotification()->x_response_reason_text;
    }

    /**
     * @inheritDoc
     * @noinspection PhpMissingReturnTypeInspection
     */
    public function getCode()
    {
        return (string)$this->getNotification()->x_response_code;
    }

    /**
     * @inheritDoc
     * @noinspection PhpMissingReturnTypeInspection
     */
    public function getTransactionReference()
    {
        return (string)$this->getNotification()->x_trans_id;
    }

    /**
     * @inheritDoc
     * @noinspection PhpMissingReturnTypeInspection
     */
    public function getTransactionId()
    {
        return (string)$this->getNotification()->f_order_number;
    }

    /**
     * @return SimpleXMLElement
     */
    protected function getNotification()
    {
        return $this->getDataProperty('notification');

It seems like getDataProperty() must be provided by classes using this trait. How about adding it as abstract method to this trait?

    }
}