OctopusDeploy release: 2.4.4

CHANGELOG.md

@@ -7,6 +7,14 @@
 
 ## Latest version
 
+#### Enhancements:
+- Added avs/cvv mapping and support for findTransaction method
+- Added batch Close response to return GSAP-specific data
+- Added support for split tender GiftCardSale transactions
+- Fix some GP-API unit tests
+
+## v2.4.3 (11/17/2021)
+
 #### Enhancements:
 - Add Unified Payments Application support
 

metadata.xml

@@ -1,3 +1,3 @@
 <xml>
-  <releaseNumber>2.4.3</releaseNumber>
+  <releaseNumber>2.4.4</releaseNumber>
 </xml>

src/Entities/BatchSummary.php

@@ -42,4 +42,24 @@ class BatchSummary
      * @var string
      */
     public $batchReference;
+
+    /**
+     * @var integer
+     */
+    public $hostBatchNbr;
+
+    /**
+     * @var integer
+     */
+    public $hostTotalCnt;
+
+    /**
+     * @var decimal
+     */
+    public $hostTotalAmt;
+
+    /**
+     * @var integer
+     */
+    public $processedDeviceId;
 }

src/Entities/Reporting/TransactionSummary.php

@@ -41,6 +41,12 @@ class TransactionSummary
      */
     public $authCode;
 
+    /** @var string */
+    public $avsResponseCode;
+
+    /** @var string */
+    public $avsResponseMessage;
+
     /**
      * @var DateTime
      */
@@ -76,6 +82,26 @@ class TransactionSummary
      */
     public $cardHolderLastName;
 
+    /**
+     * @var string
+     */
+    public $cardHolderAddr;
+
+    /**
+     * @var string
+     */
+    public $cardHolderCity;
+
+    /**
+     * @var string
+     */
+    public $cardHolderState;
+
+    /**
+     * @var string
+     */
+    public $cardHolderZip;
+
     /**
      * @var string
      */
@@ -145,6 +171,12 @@ class TransactionSummary
      */
     public $customerLastName;
 
+    /** @var string */
+    public $cvnResponseCode;
+
+    /** @var string */
+    public $cvnResponseMessage;
+
     /**
      * @var bool
      */
@@ -482,12 +514,6 @@ class TransactionSummary
     /** @var string */
     public $orderId;
 
-    /** @var string */
-    public $avsResponseCode;
-
-    /** @var string */
-    public $cvnResponseCode;
-
     /** @var string */
     public $eciIndicator;
 

src/Entities/Transaction.php

@@ -172,6 +172,9 @@ class Transaction
      */
     public $responseMessage;
 
+    /** @var array */
+    public $splitTenderBalanceDueAmt;
+
     /** @var array */
     public $responseValues;
 

src/Gateways/PorticoConnector.php

@@ -1269,8 +1269,15 @@ protected function mapResponse($rawResponse, BaseBuilder $builder, $request)
             ? (string)$item->RspText
             : $gatewayRspText;
 
-        if (isset($item) && isset($item->AuthAmt)) {
-            $result->authorizedAmount = (string)$item->AuthAmt;
+        if (isset($item) && (isset($item->AuthAmt) || isset($item->SplitTenderCardAmt))) {
+            $result->authorizedAmount = 
+                isset($item->SplitTenderCardAmt)
+                ? (string)$item->SplitTenderCardAmt
+                : (string)$item->AuthAmt;
+        }
+
+        if (isset($item) && isset($item->SplitTenderBalanceDueAmt)) {
+            $result->splitTenderBalanceDueAmt = (string)$item->SplitTenderBalanceDueAmt;
         }
 
         if (isset($item) && isset($item->AvailableBalance)) {
@@ -1360,6 +1367,19 @@ protected function mapResponse($rawResponse, BaseBuilder $builder, $request)
             $result->batchSummary->transactionCount = (string)$item->TxnCnt;
             $result->batchSummary->totalAmount = (string)$item->TotalAmt;
             $result->batchSummary->sequenceNumber = (string)$item->BatchSeqNbr;
+
+            if (isset($item->HostBatchNbr)) {
+                $result->batchSummary->hostBatchNbr = (string)$item->HostBatchNbr;
+            }
+            if (isset($item->HostTotalCnt)) {
+                $result->batchSummary->hostTotalCnt = (string)$item->HostTotalCnt;
+            }
+            if (isset($item->HostTotalAmt)) {
+                $result->batchSummary->hostTotalAmt = (string)$item->HostTotalAmt;
+            }
+            if (isset($item->ProcessedDeviceId)) {
+                $result->batchSummary->processedDeviceId = (string)$item->ProcessedDeviceId;
+            }
         }
 
         if (isset($item) && isset($item->CardBrandTxnId)) {
@@ -1371,7 +1391,7 @@ protected function mapResponse($rawResponse, BaseBuilder $builder, $request)
             $result->payFacData->transactionId = !empty($root->PaymentFacilitatorTxnId) ? (string) $root->PaymentFacilitatorTxnId : '';
             $result->payFacData->transactionNumber = !empty($root->PaymentFacilitatorTxnNbr) ? (string) $root->PaymentFacilitatorTxnNbr : '';
         }
-        
+
         return $result;
     }
 
@@ -1391,7 +1411,8 @@ protected function mapReportResponse($rawResponse, ReportBuilder $builder)
         }
 
         if ($builder->reportType === ReportType::TRANSACTION_DETAIL) {
-            return $this->hydrateTransactionSummary($doc->Transactions);
+            if(isset($doc->Data))
+                return $this->hydrateTransactionSummary($doc->Data);
         }
 
         return null;
@@ -1417,6 +1438,14 @@ protected function hydrateTransactionSummary($item)
             $summary->authCode = (string)$item->AuthCode;
         }
 
+        if (isset($item) && isset($item->AVSRsltCode)) {
+            $summary->avsResponseCode = (string)$item->AVSRsltCode;
+        }
+
+        if (isset($item) && isset($item->AVSRsltText)) {
+            $summary->avsResponseMessage = (string)$item->AVSRsltText;
+        }
+
         if (isset($item) && isset($item->BatchCloseDT)) {
             $summary->batchCloseDate = (string)$item->BatchCloseDT;
         }
@@ -1427,10 +1456,43 @@ protected function hydrateTransactionSummary($item)
 
         if (isset($item) && isset($item->CardHolderData)) {
             if (isset($item->CardHolderData->CardHolderFirstName)) {
-                $summary->cardHolderFirstName = $item->CardHolderData->CardHolderFirstName;
+                $summary->cardHolderFirstName = (string)$item->CardHolderData->CardHolderFirstName;
             }
             if (isset($item->CardHolderData->CardHolderLastName)) {
-                $summary->cardHolderLastName = $item->CardHolderData->CardHolderLastName;
+                $summary->cardHolderLastName = (string)$item->CardHolderData->CardHolderLastName;
+            }
+            if (isset($item->CardHolderData->CardHolderAddr)) {
+                $summary->cardHolderAddr = (string)$item->CardHolderData->CardHolderAddr;
+            }
+            if (isset($item->CardHolderData->CardHolderCity)) {
+                $summary->cardHolderCity = (string)$item->CardHolderData->CardHolderCity;
+            }
+            if (isset($item->CardHolderData->CardHolderState)) {
+                $summary->cardHolderState = (string)$item->CardHolderData->CardHolderState;
+            }
+            if (isset($item->CardHolderData->CardHolderZip)) {
+                $summary->cardHolderZip = (string)$item->CardHolderData->CardHolderZip;
+            }
+        }
+        else
+        {
+            if (isset($item->CardHolderFirstName)) {
+                $summary->cardHolderFirstName = (string)$item->CardHolderFirstName;
+            }
+            if (isset($item->CardHolderLastName)) {
+                $summary->cardHolderLastName = (string)$item->CardHolderLastName;
+            }
+            if (isset($item->CardHolderAddr)) {
+                $summary->cardHolderAddr = (string)$item->CardHolderAddr;
+            }
+            if (isset($item->CardHolderCity)) {
+                $summary->cardHolderCity = (string)$item->CardHolderCity;
+            }
+            if (isset($item->CardHolderState)) {
+                $summary->cardHolderState = (string)$item->CardHolderState;
+            }
+            if (isset($item->CardHolderZip)) {
+                $summary->cardHolderZip = (string)$item->CardHolderZip;
             }
         }
 
@@ -1454,6 +1516,14 @@ protected function hydrateTransactionSummary($item)
             $summary->convenienceAmount = (string)$item->ConvenienceAmtInfo;
         }
 
+        if (isset($item) && isset($item->CVVRsltCode)) {
+            $summary->cvnResponseCode = (string)$item->CVVRsltCode;
+        }
+
+        if (isset($item) && isset($item->CVVRsltText)) {
+            $summary->cvnResponseMessage = (string)$item->CVVRsltText;
+        }
+
         if (isset($item) && isset($item->DeviceId)) {
             $summary->deviceId = (string)$item->DeviceId;
         }
@@ -1860,9 +1930,10 @@ protected function mapReportType(ReportBuilder $builder)
     {
         switch ($builder->reportType) {
             case ReportType::ACTIVITY:
-            case ReportType::TRANSACTION_DETAIL:
             case ReportType::FIND_TRANSACTIONS:
                 return 'FindTransactions';
+            case ReportType::TRANSACTION_DETAIL:
+                return 'ReportTxnDetail';
             default:
                 throw new UnsupportedTransactionException();
         }

src/Mapping/GpApiMapping.php

@@ -297,7 +297,7 @@ public static function mapDisputeSummary($response)
         $summary = new DisputeSummary();
         $summary->caseId = $response->id;
         $summary->caseIdTime = !empty($response->time_created) ? new \DateTime($response->time_created) :
-            (!empty($response->stage_time_created) ? $response->stage_time_created : '');
+            (!empty($response->stage_time_created) ? new \DateTime($response->stage_time_created) : '');
         $summary->caseStatus = $response->status;
         $summary->caseStage = $response->stage;
         $summary->caseAmount = StringUtils::toAmount($response->amount);

test/Integration/Gateways/GpApiConnector/CreditCardPresentTest.php

@@ -85,6 +85,17 @@ public function testRefundOnCardPresentChipCard()
         $this->assertTransactionResponse($response, 'SUCCESS', TransactionStatus::CAPTURED);
     }
 
+    public function testCardPresentWithManualEntryModeTransaction()
+    {
+        $card = $this->initCreditCardData();
+
+        $response = $card->charge($this->amount)
+            ->withCurrency($this->currency)
+            ->execute();
+
+        $this->assertTransactionResponse($response, 'SUCCESS', TransactionStatus::CAPTURED);
+    }
+
     public function testCreditVerification_CardPresent()
     {
         $card = $this->initCreditTrackData();

test/Integration/Gateways/GpApiConnector/GpApi3DS2Test.php

@@ -30,6 +30,8 @@
 use GlobalPayments\Api\Utils\GenerationUtils;
 use GlobalPayments\Api\Entities\Enums\Secure3dStatus;
 use PHPUnit\Framework\TestCase;
+use GlobalPayments\Api\Utils\Logging\Logger;
+use GlobalPayments\Api\Utils\Logging\SampleRequestLogger;
 
 class GpApi3DS2Test extends TestCase
 {
@@ -109,6 +111,7 @@ public function setUpConfig()
         $config->challengeNotificationUrl = 'https://ensi808o85za.x.pipedream.net/';
         $config->methodNotificationUrl = 'https://ensi808o85za.x.pipedream.net/';
         $config->merchantContactUrl = 'https://enp4qhvjseljg.x.pipedream.net/';
+        $config->requestLogger = new SampleRequestLogger(new Logger("logs"));
 
         return $config;
     }
@@ -1000,7 +1003,11 @@ public function testCardHolderEnrolled_ChallengeRequired_v2_Initiate_AllPreferen
 
     public function testCardHolderEnrolled_ChallengeRequired_v2_Initiate_AllSourceValues()
     {
-        $source = array(AuthenticationSource::BROWSER, AuthenticationSource::MERCHANT_INITIATED, AuthenticationSource::MOBILE_SDK);
+        $source = [
+            AuthenticationSource::BROWSER,
+            AuthenticationSource::MERCHANT_INITIATED,
+            AuthenticationSource::MOBILE_SDK
+        ];
         foreach ($source as $value) {
             $secureEcom = Secure3dService::checkEnrollment($this->card)
                 ->withCurrency($this->currency)