ByTIC\Payments\Models\Methods\Traits\RecordTrait

Complexity

Total Complexity

19

Size/Duplication

Total Lines	140
Duplicated Lines	0 %

Test Coverage

Coverage

9.3%

Importance

Changes	4
Bugs	1	Features	0

11 Methods

Rating	Name	Duplication	Size	Complexity
A	getEntryDescription()	0	7	1
A	writeData()	0	7	2
A	getFileModelName()	0	6	2
A	getDescription()	0	3	1
A	getName()	0	6	2
A	checkConfirmRedirect()	0	7	2
A	getGateway()	0	7	2
A	canDelete()	0	7	2
A	getPaymentGatewayOptions()	0	5	1
A	setType()	0	8	2
A	setPaymentGatewayOptions()	0	5	2

<?php

namespace ByTIC\Payments\Models\Methods\Traits;

use ByTIC\Common\Payments\Models\Methods\Files\MobilpayFile;
use \ByTIC\Models\SmartProperties\RecordsTraits\HasTypes\RecordTrait as HasTypesRecordTrait;
use ByTIC\MediaLibrary\HasMedia\HasMediaTrait;
use ByTIC\MediaLibrary\HasMedia\Traits\AddMediaTrait;
use ByTIC\Payments\Gateways\Providers\AbstractGateway\Traits\GatewayTrait;
use ByTIC\Payments\Models\Methods\Types\AbstractType;
use ByTIC\Payments\Models\Methods\Types\CreditCards;
use Nip\Records\RecordManager;

/**
 * Class MethodTrait
 * @package ByTIC\Payments\Models\Methods\Traits
 *
 * @property string $name
 * @property string $internal_name
 * @property string $description
 * @property string $__notes
 *
 * @method AbstractType|CreditCards getType
 * @method RecordsTrait|RecordManager getManager()
 */
trait RecordTrait
{
    use HasTypesRecordTrait {
        setType as setTypeTrait;
    }
    use \ByTIC\Common\Records\Traits\HasSerializedOptions\RecordTrait;

    use HasMediaTrait;

    /**
     * @param array $data
     * @return mixed
     */
    public function writeData($data = [])
    {
        if (!$data['type']) {
            $data['type'] = 'bank-transfer';
        }

        return parent::writeData($data);
    }

    /**
     * @param bool $type
     * @return string
     */
    public function getName($type = false)
    {
        if ($type == 'internal') {
            return $this->internal_name;
        }
        return $this->name;
    }

    /**
     * @return bool
     */
    public function checkConfirmRedirect()
    {
        if ($this->getType()->checkConfirmRedirect()) {
            return true;
        }

        return false;
    }

    /**
     * @return bool|string
     */
    public function getEntryDescription()
    {
        $return = $this->getType()->getEntryDescription();
        $return .= $this->getDescription();
        $return .= $this->__notes;

        return $return;
    }

    /**
     * @return string
     */
    public function getDescription()
    {
        return $this->description;
    }

    /**
     * @return bool
     */
    public function canDelete()
    {
        if ($this->getPurchasesCount() > 0) {
            return $this->getManager()->getMessage('delete.denied.has-purchases');

The expression return $this->getManager()->getMessage('delete.denied.has-purchases') also could return the type ByTIC\Payments\Models\Methods\Traits\RecordsTrait|Nip\Records\Collections\Collection|Nip\Records\RecordManager which is incompatible with the documented return type boolean.

        }

        return true;
    }

    /**
     * @return int
     */
    abstract public function getPurchasesCount();

    /**
     * @return bool|GatewayTrait|null
     */
    public function getGateway()
    {
        if ($this->getType()->getName() == 'credit-cards') {
            return $this->getType()->getGateway();

The expression return $this->getType()->getGateway() also could return the type Omnipay\Common\GatewayInterface which is incompatible with the documented return type ByTIC\Payments\Gateways\Providers\AbstractGateway\Traits\GatewayTrait|boolean|null.

The method getGateway() does not exist on ByTIC\Payments\Models\Methods\Types\AbstractType. It seems like you code against a sub-type of ByTIC\Payments\Models\Methods\Types\AbstractType such as ByTIC\Payments\Models\Methods\Types\CreditCards.

        }

        return false;
    }

    /**
     * @inheritDoc
     */
    public function setType($type = null)
    {
        $paymentGatewaysNames = \ByTIC\Payments\Gateways\Manager::getCollection()->keys();
        if (in_array($type, $paymentGatewaysNames)) {
            $this->setOption('payment_gateway', $type);
            $type = 'credit-cards';
        }
        return $this->setTypeTrait($type);
    }

    /**
     * @return mixed
     */
    public function getPaymentGatewayOptions()
    {
        $gatewayName = $this->getOption('payment_gateway');

        return $this->getOption($gatewayName);
    }

    /**
     * @param array $options
     * @param null $gatewayName

Are you sure the doc-type for parameter $gatewayName is correct as it would always require null to be passed?

     * @return mixed
     */
    public function setPaymentGatewayOptions($options, $gatewayName = null)
    {
        $gatewayName = $gatewayName? $gatewayName : $this->getOption('payment_gateway');

$gatewayName is of type null, thus it always evaluated to false.

        return $this->setOption($gatewayName, $options);
    }

    /**
     * @param null $type

Are you sure the doc-type for parameter $type is correct as it would always require null to be passed?

     * @return string
     */
    public function getFileModelName($type = null)
    {
        if ($type == 'Mobilpay') {
            return MobilpayFile::class;
        }
        return $this->getFileModelNameAbstract($type);

The method getFileModelNameAbstract() does not exist on ByTIC\Payments\Models\Methods\Traits\RecordTrait. Did you maybe mean getFileModelName()?

    }
}