ByTIC\Payments\Mobilpay\Message\Traits\CompletePurchaseRequestTrait

Complexity

Total Complexity

3

Size/Duplication

Total Lines	28
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	parseNotification()	0	8	2
A	updateParametersFromGateway()	0	5	1

<?php

namespace ByTIC\Payments\Mobilpay\Message\Traits;

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

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


    /**
     * @return bool|mixed
     * @throws \Exception
     */
    protected function parseNotification()
    {
        if ($this->validateModel()) {
            $model = $this->getModel();
            $this->updateParametersFromPurchase($model);
        }

        return parent::parseNotification();
    }

    /**
     * @param Gateway $modelGateway
     */
    protected function updateParametersFromGateway($modelGateway)
    {
        $this->setSignature($modelGateway->getSignature());

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

        $this->setCertificate($modelGateway->getCertificate());

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

        $this->setPrivateKey($modelGateway->getPrivateKey());

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

    }
}