Dynamic\FoxyStripe\Foxy\Transaction

Complexity

Total Complexity

9

Size/Duplication

Total Lines	64
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	getDecryptedData()	0	3	1
A	exists()	0	3	2
A	__construct()	0	3	1
A	getTransaction()	0	3	1
A	setTransaction()	0	16	4

<?php

namespace Dynamic\FoxyStripe\Foxy;

use Dynamic\FoxyStripe\Model\FoxyCart;
use SilverStripe\Core\Injector\Injectable;

/**
 * Class Transaction
 * @package Dynamic\FoxyStripe\Foxy
 */
class Transaction
{
    use Injectable;

    /**
     * @var
     */
    private $transaction;

    /**
     * Transaction constructor.
     * @param $data
     */
    public function __construct($transactionID, $data)
    {
        $this->setTransaction($transactionID, $data);
    }

    /**
     * @param $data
     * @return $this
     */
    public function setTransaction($transactionID, $data)
    {
        $decryptedData = $this->getDecryptedData($data);

        foreach ($decryptedData->transactions->transaction as $transaction) {
            if ($transactionID == (int)$transaction->id) {
                $this->transaction = $transaction;
                break;
            }
        }

        if (!$this->transaction) {
            $this->transaction = false;
        }

        return $this;
    }

    /**
     * @return mixed
     */
    public function getTransaction()
    {
        return $this->transaction;
    }

    /**
     * @return bool
     */
    public function exists()
    {
        return $this->getTransaction() != false && !empty($this->getTransaction());
    }

    /**
     * @param $data
     * @return \SimpleXMLElement
     * @throws \SilverStripe\ORM\ValidationException
     */
    private function getDecryptedData($data)
    {
        return new \SimpleXMLElement(\rc4crypt::decrypt(FoxyCart::getStoreKey(), $data));

It seems like Dynamic\FoxyStripe\Model\FoxyCart::getStoreKey() can also be of type false; however, parameter $pwd of rc4crypt::decrypt() does only seem to accept string, maybe add an additional type check?

    }
}