Skip to content

Commit 742c8b9

Browse files
committed
correct naming from Sender to Recipient, remove aft flag
fixed some formatting
1 parent 86c7eda commit 742c8b9

8 files changed

+61
-78
lines changed

src/Json/JsonGenerator.php

+19-20
Original file line numberDiff line numberDiff line change
@@ -151,27 +151,27 @@ public function generateSchedule($action, $scheduleData){
151151
protected function createDebit($transaction, $language){
152152
/** @var Debit $transaction */
153153
$data = [
154-
'referenceUuid' => $transaction->getReferenceUuid(),
155-
'amount' => (string)$transaction->getAmount(),
156-
'currency' => $transaction->getCurrency(),
157-
'successUrl' => $transaction->getSuccessUrl(),
158-
'cancelUrl' => $transaction->getCancelUrl(),
159-
'errorUrl' => $transaction->getErrorUrl(),
160-
'callbackUrl' => $transaction->getCallbackUrl(),
161-
'transactionToken' => $transaction->getTransactionToken(),
162-
'description' => $transaction->getDescription(),
163-
'items' => $this->createItems($transaction->getItems()),
164-
'splits' => $this->createSplits($transaction->getTransactionSplits()),
165-
'withRegister' => $transaction->isWithRegister(),
154+
'referenceUuid' => $transaction->getReferenceUuid(),
155+
'amount' => (string)$transaction->getAmount(),
156+
'currency' => $transaction->getCurrency(),
157+
'successUrl' => $transaction->getSuccessUrl(),
158+
'cancelUrl' => $transaction->getCancelUrl(),
159+
'errorUrl' => $transaction->getErrorUrl(),
160+
'callbackUrl' => $transaction->getCallbackUrl(),
161+
'transactionToken' => $transaction->getTransactionToken(),
162+
'description' => $transaction->getDescription(),
163+
'items' => $this->createItems($transaction->getItems()),
164+
'splits' => $this->createSplits($transaction->getTransactionSplits()),
165+
'withRegister' => $transaction->isWithRegister(),
166166
'transactionIndicator' => $transaction->getTransactionIndicator(),
167-
'customer' => $this->createCustomer($transaction->getCustomer()),
168-
'schedule' => $this->createSchedule($transaction->getSchedule()),
169-
'customerProfileData' => $this->createAddToCustomerProfile($transaction->getCustomerProfileData()),
170-
'threeDSecureData' => $this->createThreeDSecureData($transaction->getThreeDSecureData()),
171-
'language' => $language,
167+
'customer' => $this->createCustomer($transaction->getCustomer()),
168+
'schedule' => $this->createSchedule($transaction->getSchedule()),
169+
'customerProfileData' => $this->createAddToCustomerProfile($transaction->getCustomerProfileData()),
170+
'threeDSecureData' => $this->createThreeDSecureData($transaction->getThreeDSecureData()),
171+
'language' => $language,
172172
'referenceSchemeTransactionIdentifier' => $transaction->getReferenceSchemeTransactionIdentifier(),
173-
'surchargeAmount' => $transaction->getSurchargeAmount(),
174-
'senderAccountReferenceUuid' => $transaction->getSenderAccountReferenceUuid()
173+
'surchargeAmount' => $transaction->getSurchargeAmount(),
174+
'recipientAccountReferenceUuid' => $transaction->getRecipientAccountReferenceUuid()
175175
];
176176

177177
if ($transaction->getL2L3Data()) {
@@ -297,7 +297,6 @@ protected function createRegister($transaction, $language){
297297
'customerProfileData' => $this->createAddToCustomerProfile($transaction->getCustomerProfileData()),
298298
'threeDSecureData' => $this->createThreeDSecureData($transaction->getThreeDSecureData()),
299299
'language' => $language,
300-
'isAft' => $transaction->isAft(),
301300
];
302301

303302
if ($transaction->getL2L3Data()) {
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,12 @@
1+
<?php
2+
3+
namespace Ixopay\Client\Transaction\Base;
4+
5+
interface RecipientAccountPanInterface
6+
{
7+
/** @return string|null */
8+
public function getRecipientAccountReferenceUuid();
9+
10+
/** @param string $recipientAccountReferenceUuid */
11+
public function setRecipientAccountReferenceUuid($recipientAccountReferenceUuid);
12+
}
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,22 @@
1+
<?php
2+
3+
namespace Ixopay\Client\Transaction\Base;
4+
5+
trait RecipientAccountPanTrait
6+
{
7+
/** @var null|string */
8+
protected $recipientAccountReferenceUuid = null;
9+
10+
/** @return null|string */
11+
public function getRecipientAccountReferenceUuid()
12+
{
13+
return $this->recipientAccountReferenceUuid;
14+
}
15+
16+
/** @param string $recipientAccountReferenceUuid */
17+
public function setRecipientAccountReferenceUuid($recipientAccountReferenceUuid)
18+
{
19+
$this->recipientAccountReferenceUuid = $recipientAccountReferenceUuid;
20+
}
21+
22+
}

src/Transaction/Base/SenderAccountPanInterface.php

-9
This file was deleted.

src/Transaction/Base/SenderAccountPanTrait.php

-22
This file was deleted.

src/Transaction/Debit.php

+4-4
Original file line numberDiff line numberDiff line change
@@ -24,8 +24,8 @@
2424
use Ixopay\Client\Transaction\Base\ReferenceSchemeTransactionIdentifierTrait;
2525
use Ixopay\Client\Transaction\Base\ScheduleInterface;
2626
use Ixopay\Client\Transaction\Base\ScheduleTrait;
27-
use Ixopay\Client\Transaction\Base\SenderAccountPanInterface;
28-
use Ixopay\Client\Transaction\Base\SenderAccountPanTrait;
27+
use Ixopay\Client\Transaction\Base\RecipientAccountPanInterface;
28+
use Ixopay\Client\Transaction\Base\RecipientAccountPanTrait;
2929
use Ixopay\Client\Transaction\Base\SurchargeInterface;
3030
use Ixopay\Client\Transaction\Base\SurchargeTrait;
3131
use Ixopay\Client\Transaction\Base\TransactionSplitsInterface;
@@ -51,7 +51,7 @@ class Debit extends AbstractTransactionWithReference
5151
DccDataInterface,
5252
SurchargeInterface,
5353
ReferenceSchemeTransactionIdentifierInterface,
54-
SenderAccountPanInterface,
54+
RecipientAccountPanInterface,
5555
LevelTwoAndThreeDataInterface
5656
{
5757
use AddToCustomerProfileTrait;
@@ -67,7 +67,7 @@ class Debit extends AbstractTransactionWithReference
6767
use DccDataTrait;
6868
use SurchargeTrait;
6969
use ReferenceSchemeTransactionIdentifierTrait;
70-
use SenderAccountPanTrait;
70+
use RecipientAccountPanTrait;
7171
use LevelTwoAndThreeDataTrait;
7272

7373
const TRANSACTION_INDICATOR_SINGLE = 'SINGLE';

src/Transaction/Preauthorize.php

+4-4
Original file line numberDiff line numberDiff line change
@@ -24,8 +24,8 @@
2424
use Ixopay\Client\Transaction\Base\ReferenceSchemeTransactionIdentifierTrait;
2525
use Ixopay\Client\Transaction\Base\ScheduleInterface;
2626
use Ixopay\Client\Transaction\Base\ScheduleTrait;
27-
use Ixopay\Client\Transaction\Base\SenderAccountPanInterface;
28-
use Ixopay\Client\Transaction\Base\SenderAccountPanTrait;
27+
use Ixopay\Client\Transaction\Base\RecipientAccountPanInterface;
28+
use Ixopay\Client\Transaction\Base\RecipientAccountPanTrait;
2929
use Ixopay\Client\Transaction\Base\SurchargeInterface;
3030
use Ixopay\Client\Transaction\Base\SurchargeTrait;
3131
use Ixopay\Client\Transaction\Base\TransactionSplitsInterface;
@@ -51,7 +51,7 @@ class Preauthorize extends AbstractTransactionWithReference
5151
DccDataInterface,
5252
SurchargeInterface,
5353
ReferenceSchemeTransactionIdentifierInterface,
54-
SenderAccountPanInterface,
54+
RecipientAccountPanInterface,
5555
LevelTwoAndThreeDataInterface
5656
{
5757

@@ -68,7 +68,7 @@ class Preauthorize extends AbstractTransactionWithReference
6868
use DccDataTrait;
6969
use SurchargeTrait;
7070
use ReferenceSchemeTransactionIdentifierTrait;
71-
use SenderAccountPanTrait;
71+
use RecipientAccountPanTrait;
7272
use LevelTwoAndThreeDataTrait;
7373

7474
const TRANSACTION_INDICATOR_SINGLE = 'SINGLE';

src/Transaction/Register.php

-19
Original file line numberDiff line numberDiff line change
@@ -56,25 +56,6 @@ class Register extends AbstractTransaction
5656
*/
5757
protected $transactionIndicator;
5858

59-
/** @var bool */
60-
protected $isAft=false;
61-
62-
/**
63-
* @return bool
64-
*/
65-
public function isAft()
66-
{
67-
return $this->isAft;
68-
}
69-
70-
/**
71-
* @param bool $isAft
72-
*/
73-
public function setIsAft($isAft)
74-
{
75-
$this->isAft = $isAft;
76-
}
77-
7859
/**
7960
* @return string
8061
*/

0 commit comments

Comments
 (0)