ByTIC\Payments\Gateways\Providers\Payu\Message\Traits\CompletePurchaseRequestTrait

Complexity

Total Complexity

3

Size/Duplication

Total Lines	26
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	getData()	0	9	2
A	updateParametersFromGateway()	0	4	1

<?php

namespace ByTIC\Payments\Gateways\Providers\Payu\Message\Traits;

use ByTIC\Payments\Gateways\Providers\AbstractGateway\Message\Traits\HasGatewayRequestTrait;
use ByTIC\Payments\Gateways\Providers\AbstractGateway\Message\Traits\HasModelRequest;

/**
 * Trait CompletePurchaseRequestTrait
 * @package ByTIC\Payments\Gateways\Providers\Payu\Message\Traits
 */
trait CompletePurchaseRequestTrait
{
    use HasGatewayRequestTrait;
    use HasModelRequest;

    /**
     * @inheritdoc
     */
    public function getData()
    {
        $return = parent::getData();
        // Add model only if has data
        if (count($return)) {
            $return['model'] = $this->getModel();
        }

        return $return;
    }

    /**
     * @param \ByTIC\Payments\Gateways\Providers\Payu\Gateway $model
     */
    protected function updateParametersFromGateway($gateway)
    {
//        $this->setMerchant($gateway->getMerchant());
        $this->setSecretKey($gateway->getSecretKey());

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

    }
}