Backported DPM getHash() to SIM getHash() to support a specified currency.

Author: judgej

src/DPMGateway.php

@@ -12,21 +12,33 @@ public function getName()
         return 'Authorize.Net DPM';
     }
 
+    /**
+     * Helper to generate the authorize direct-post form.
+     */
     public function authorize(array $parameters = array())
     {
         return $this->createRequest('\Omnipay\AuthorizeNet\Message\DPMAuthorizeRequest', $parameters);
     }
 
+    /**
+     * Get, validate, interpret and respond to the Authorize.Net callback.
+     */
     public function completeAuthorize(array $parameters = array())
     {
         return $this->createRequest('\Omnipay\AuthorizeNet\Message\DPMCompleteRequest', $parameters);
     }
 
+    /**
+     * Helper to generate the purchase direct-post form.
+     */
     public function purchase(array $parameters = array())
     {
         return $this->createRequest('\Omnipay\AuthorizeNet\Message\DPMPurchaseRequest', $parameters);
     }
 
+    /**
+     * Get, validate, interpret and respond to the Authorize.Net callback.
+     */
     public function completePurchase(array $parameters = array())
     {
         return $this->createRequest('\Omnipay\AuthorizeNet\Message\DPMCompleteRequest', $parameters);

src/Message/DPMAuthorizeRequest.php

@@ -5,35 +5,13 @@
 use Omnipay\AuthorizeNet\Message\SIMAbstractRequest;
 
 /**
- * Authorize.Net DPM Authorize Request
+ * Authorize.Net DPM Authorize Request.
+ * Takes the data that will be used to create the direct-post form.
  */
 class DPMAuthorizeRequest extends SIMAuthorizeRequest
 {
     protected $action = 'AUTH_ONLY';
 
-    public function getHash($data)
-    {
-        $fingerprint = implode(
-            '^',
-            array(
-                $this->getApiLoginId(),
-                $data['x_fp_sequence'],
-                $data['x_fp_timestamp'],
-                $data['x_amount']
-            )
-        ).'^';
-
-        // If x_currency_code is specified, then it must follow the final trailing carat.
-        // CHECKME: this may need to be back-ported to SIMAuthorizeRequest and AIMAuthorizeRequest
-        // in order to supprot multiple currencies.
-
-        if ($this->getCurrency()) {
-            $fingerprint .= $this->getCurrency();
-        }
-
-        return hash_hmac('md5', $fingerprint, $this->getTransactionKey());
-    }
-
     public function getData()
     {
         $data = parent::getData();

src/Message/SIMAuthorizeRequest.php

@@ -36,6 +36,12 @@ public function getData()
         return $data;
     }
 
+    /**
+     * This hash is put into the form to confirm the amount has not been
+     * modified en-route.
+     * It uses the TransactionKey, which is a shared secret between the merchant
+     * and Authorize.Net The sequence and timestamp provide additional salt.
+     */
     public function getHash($data)
     {
         $fingerprint = implode(
@@ -48,6 +54,11 @@ public function getHash($data)
             )
         ).'^';
 
+        // If x_currency_code is specified, then it must follow the final trailing carat.
+        if ($this->getCurrency()) {
+            $fingerprint .= $this->getCurrency();
+        }
+
         return hash_hmac('md5', $fingerprint, $this->getTransactionKey());
     }