ByTIC / omnipay-twispay

src/Message/Traits/CompletePurchaseRequestTrait.php

<?php

namespace ByTIC\Omnipay\Twispay\Message\Traits;

use ByTIC\Omnipay\Common\Message\Traits\GatewayNotificationRequestTrait;
use ByTIC\Omnipay\Twispay\Helper;

/**
 * Trait CompletePurchaseRequestTrait
 * @package ByTIC\Omnipay\Twispay\Message\Traits
 */
trait CompletePurchaseRequestTrait
{
    use GatewayNotificationRequestTrait;

    /**
     * @return bool|mixed
     */
    protected function parseNotification()
    {
        $dataResult = $this->hasPOST('opensslResult')
            ? $this->httpRequest->request->get('opensslResult')
            : $this->httpRequest->request->get('result');

        $json = Helper::decrypt($dataResult, $this->getApiKey());

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

        $data = json_decode($json, true);
        return $data;
    }

    /**
     * @return mixed
     */
    protected function isValidNotification()
    {
        return $this->hasGet('id') && ($this->hasPOST('opensslResult') || $this->hasPOST('result'));
    }
}