Merge pull request thephpleague#7 from sachinsudheendra/XOL-2533

XOL-2533 Adding extraOptions payload to request XML

Author: rushi

.gitignore

@@ -2,3 +2,4 @@
 composer.lock
 composer.phar
 phpunit.xml
+.idea

src/Message/AIMAbstractRequest.php

@@ -53,6 +53,27 @@ public function setCustomerId($value)
         return $this->setParameter('customerId', $value);
     }
 
+    public function setDuplicateWindow($value)
+    {
+        $this->setParameter('duplicateWindow', $value);
+    }
+
+    private function getDuplicateWindow()
+    {
+        return $this->getParameter('duplicateWindow'); // Maps x_duplicate_window
+    }
+
+    protected function addExtraOptions(\SimpleXMLElement $data)
+    {
+        if (!is_null($this->getDuplicateWindow())) {
+            $extraOptions = $data->addChild('extraOptions');
+            $node = dom_import_simplexml($extraOptions);
+            $nodeOwner = $node->ownerDocument;
+            $node->appendChild($nodeOwner->createCDATASection(sprintf("x_duplicate_window=%s", $this->getDuplicateWindow())));
+        }
+        return $data;
+    }
+
     /**
      * @return mixed|\SimpleXMLElement
      * @throws \Omnipay\Common\Exception\InvalidRequestException

src/Message/AIMAuthorizeRequest.php

@@ -31,6 +31,7 @@ public function getData()
 
         $this->addBillingData($data);
         $this->addTestModeSetting($data);
+        $this->addExtraOptions($data);
 
         return $data;
     }

src/Message/CIMAuthorizeRequest.php

@@ -14,10 +14,9 @@ class CIMAuthorizeRequest extends CIMAbstractRequest
     public function getData()
     {
         $this->validate('cardReference', 'amount');
-
         $data = $this->getBaseData();
-
         $this->addTransactionData($data);
+        $this->addExtraOptions($data);
         return $data;
     }
 
@@ -45,7 +44,6 @@ protected function addTransactionData(\SimpleXMLElement $data)
         if (!empty($desc)) {
             $action->order->description = $desc;
         }
-
         return $data;
     }
 

tests/Message/AIMAuthorizeRequestTest.php

@@ -18,6 +18,7 @@ public function setUp()
                 'amount' => '12.00',
                 'customerId' => 'cust-id',
                 'card' => $this->getValidCard(),
+                'duplicateWindow' => 0
             )
         );
     }
@@ -45,4 +46,10 @@ public function testGetDataTestMode()
         $this->assertEquals('testRequest', $setting->settingName);
         $this->assertEquals('true', $setting->settingValue);
     }
+
+    public function testShouldReturnExtraOptionsToDisableDuplicateWindowPeriod()
+    {
+        $data = $this->request->getData();
+        $this->assertEquals('x_duplicate_window=0', strip_tags($data->extraOptions));
+    }
 }

tests/Message/CIMAuthorizeRequestTest.php

@@ -16,7 +16,8 @@ public function setUp()
             array(
                 'cardReference' => '{"customerProfileId":"28972085","customerPaymentProfileId":"26317841","customerShippingAddressId":"27057151"}',
                 'amount' => '12.00',
-                'description' => 'Test authorize transaction'
+                'description' => 'Test authorize transaction',
+                'duplicateWindow' => '0'
             )
         );
     }
@@ -31,4 +32,10 @@ public function testGetData()
         $this->assertEquals('27057151', $data->transaction->profileTransAuthOnly->customerShippingAddressId);
         $this->assertEquals('Test authorize transaction', $data->transaction->profileTransAuthOnly->order->description);
     }
+
+    public function testShouldReturnExtraOptionsToDisableDuplicateWindowPeriod()
+    {
+        $data = $this->request->getData();
+        $this->assertEquals('x_duplicate_window=0', strip_tags($data->extraOptions));
+    }
 }