commerceguys / authnet

src/DecryptPaymentDataRequest.php

<?php

namespace CommerceGuys\AuthNet;

use GuzzleHttp\Client;
use CommerceGuys\AuthNet\Request\RequestInterface;
use CommerceGuys\AuthNet\DataTypes\OpaqueData;

/**
 * Use this method to decrypt the VISA Checkout data.
 *
 * @link http://developer.authorize.net/api/reference/index.html#payment-transactions-charge-a-credit-card
 */
class DecryptPaymentDataRequest extends BaseApiRequest
{
    protected $opaqueData;
    protected $refId = '';
    protected $callId = '';

    public function __construct(
        Configuration $configuration,
        Client $client,
        OpaqueData $opaqueData = null,
        $refId,
        $callId
    ) {
          parent::__construct($configuration, $client);
          $this->opaqueData = $opaqueData;
          $this->refId = $refId;
          $this->callId = $callId;
    }

    protected function attachData(RequestInterface $request)
    {
        $request->addDataType($this->opaqueData);

It seems like $this->opaqueData can also be of type null; however, parameter $data of CommerceGuys\AuthNet\Request\RequestInterface::addDataType() does only seem to accept CommerceGuys\AuthNet\DataTypes\DataTypeInterface, maybe add an additional type check?

        $request->addData('refId', $this->refId);
        $request->addData('callId', $this->callId);
    }
}