Some notes. Will raise a ticket against this.

judgej · judgej · commit 6661d9b179a6 · 2015-03-07T12:30:16.000Z
diff --git a/src/Message/SIMCompleteAuthorizeRequest.php b/src/Message/SIMCompleteAuthorizeRequest.php
@@ -19,7 +19,9 @@ public function getData()
     }
 
     /**
-     * CHECKME: DPM uses the transactionReference in the hash, not the transactionID.
+     * CHECKME: should this be the transactionReference in the hash, not the transactionId?
+     * The transaction reference and the amount are both sent by the remote gateway (x_trans_id
+     * and x_amount) and it is those that should be checked against.
      */
     public function getHash()
     {

Original file line number	Diff line number	Diff line change
`@@ -19,7 +19,9 @@ public function getData()`
`19`	`19`	`}`
`20`	`20`
`21`	`21`	`/**`
`22`		`- * CHECKME: DPM uses the transactionReference in the hash, not the transactionID.`
	`22`	`+ * CHECKME: should this be the transactionReference in the hash, not the transactionId?`
	`23`	`+ * The transaction reference and the amount are both sent by the remote gateway (x_trans_id`
	`24`	`+ * and x_amount) and it is those that should be checked against.`
`23`	`25`	`*/`
`24`	`26`	`public function getHash()`
`25`	`27`	`{`