Skip to content

Commit e131e31

Browse files
committed
Merge pull request #108 from aderuwe/patch-4
Fix MultiSafepay signature generation
2 parents fb02659 + bb1884c commit e131e31

File tree

2 files changed

+9
-5
lines changed

2 files changed

+9
-5
lines changed

src/Omnipay/MultiSafepay/Message/PurchaseRequest.php

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -161,7 +161,11 @@ public function send()
161161
protected function generateSignature()
162162
{
163163
return md5(
164-
$this->getAmount().$this->getCurrency().$this->getAccountId().$this->getSiteId().$this->getTransactionId()
164+
$this->getAmountInteger().
165+
$this->getCurrency().
166+
$this->getAccountId().
167+
$this->getSiteId().
168+
$this->getTransactionId()
165169
);
166170
}
167171
}

tests/Omnipay/MultiSafepay/Message/PurchaseRequestTest.php

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -135,7 +135,7 @@ public function testGenerateSignature()
135135
$method->setAccessible(true);
136136

137137
$signature = $method->invoke($this->request);
138-
$this->assertEquals('bb886caff589f17e81b21097a39e47c2', $signature);
138+
$this->assertEquals('ad447bab87b8597853432c891e341db1', $signature);
139139
}
140140

141141
public function allDataProvider()
@@ -178,7 +178,7 @@ public function allDataProvider()
178178
<var2>extra 2</var2>
179179
<var3>extra 3</var3>
180180
</transaction>
181-
<signature>bb886caff589f17e81b21097a39e47c2</signature>
181+
<signature>ad447bab87b8597853432c891e341db1</signature>
182182
</redirecttransaction>
183183
184184
EOF;
@@ -225,7 +225,7 @@ public function noIssuerDataProvider()
225225
<var2>extra 2</var2>
226226
<var3>extra 3</var3>
227227
</transaction>
228-
<signature>bb886caff589f17e81b21097a39e47c2</signature>
228+
<signature>ad447bab87b8597853432c891e341db1</signature>
229229
</redirecttransaction>
230230
231231
EOF;
@@ -275,7 +275,7 @@ public function specialCharsDataProvider()
275275
<var2>extra 2</var2>
276276
<var3>extra 3</var3>
277277
</transaction>
278-
<signature>bb886caff589f17e81b21097a39e47c2</signature>
278+
<signature>ad447bab87b8597853432c891e341db1</signature>
279279
</redirecttransaction>
280280
281281
EOF;

0 commit comments

Comments
 (0)