Merge branch 'issue-101' of https://github.com/samlitowitz/omnipay-authorizenet into issue101

Author: judgej

src/Message/AIMAuthorizeRequest.php

@@ -3,6 +3,7 @@
 namespace Omnipay\AuthorizeNet\Message;
 
 use Omnipay\Common\CreditCard;
+use Omnipay\Common\Exception\InvalidRequestException;
 
 /**
  * Authorize.Net AIM Authorize Request
@@ -20,6 +21,7 @@ public function getData()
         $this->addSolutionId($data);
         $this->addBillingData($data);
         $this->addCustomerIP($data);
+        $this->addRetail($data);
         $this->addTransactionSettings($data);
 
         return $data;
@@ -54,4 +56,53 @@ protected function addCustomerIP(\SimpleXMLElement $data)
             $data->transactionRequest->customerIP = $ip;
         }
     }
+
+    protected function addRetail(\SimpleXMLElement $data)
+    {
+        $deviceType = $this->getDeviceType();
+        $marketType = $this->getMarketType();
+
+        if (!isset($deviceType) && !isset($marketType)) {
+            return;
+        }
+
+        if (!isset($deviceType) && isset($marketType)) {
+            throw new InvalidRequestException("deviceType is required if marketType is set");
+        }
+
+        if (isset($deviceType) && !isset($marketType)) {
+            $marketType = "2";
+        }
+
+        if (!in_array($deviceType, [ "1", "2", "3", "4", "5", "6", "7", "8", "9", "10" ])) {
+            throw new InvalidRequestException("deviceType `{$deviceType}` is invalid");
+        }
+
+        if (!in_array($marketType, [ "0", "1", "2" ])) {
+            throw new InvalidRequestException("marketType `{$marketType}` is invalid");
+        }
+
+        $data->transactionRequest->retail->marketType = $marketType;
+        $data->transactionRequest->retail->deviceType = $deviceType;
+    }
+
+    public function getDeviceType()
+    {
+        return $this->getParameter('deviceType');
+    }
+
+    public function setDeviceType($value)
+    {
+        return $this->setParameter('deviceType', $value);
+    }
+    
+    public function getMarketType()
+    {
+        return $this->getParameter('marketType');
+    }
+
+    public function setMarketType($value)
+    {
+        return $this->setParameter('marketType', $value);
+    }
 }

tests/Message/AIMAuthorizeRequestTest.php

@@ -22,6 +22,8 @@ public function setUp()
                 'card' => $card,
                 'duplicateWindow' => 0,
                 'solutionId' => 'SOL12345ID',
+                'marketType' => '2',
+                'deviceType' => '1',
             )
         );
     }
@@ -35,6 +37,8 @@ public function testGetData()
         $this->assertEquals('cust-id', $data->transactionRequest->customer->id);
         $this->assertEquals('[email protected]', $data->transactionRequest->customer->email);
         $this->assertEquals('SOL12345ID', $data->transactionRequest->solution->id);
+        $this->assertEquals('2', $data->transactionRequest->retail->marketType);
+        $this->assertEquals('1', $data->transactionRequest->retail->deviceType);
 
         // Issue #38 Make sure the transactionRequest properties are correctly ordered.
         // This feels messy, but works.
@@ -51,6 +55,7 @@ public function testGetData()
             "billTo",
             "shipTo",
             "customerIP",
+            "retail",
             "transactionSettings"
         );
         $this->assertEquals($keys, $transactionRequestProperties);