Skip to content

Commit

Permalink
Merge pull request #41 from eileenmcnaughton/master
Browse files Browse the repository at this point in the history
#261 clarify meaning of transactionRef and add transactionId as a sta…
  • Loading branch information
greydnls committed Jul 23, 2015
2 parents 3d7b925 + ec845cc commit 0ee00e2
Show file tree
Hide file tree
Showing 2 changed files with 16 additions and 1 deletion.
5 changes: 5 additions & 0 deletions src/Omnipay/Common/Message/AbstractRequest.php
Original file line number Diff line number Diff line change
Expand Up @@ -455,6 +455,8 @@ public function setDescription($value)
/**
* Get the transaction ID.
*
* The transaction ID is the identifier generated by the merchant website.
*
* @return string
*/
public function getTransactionId()
Expand All @@ -476,6 +478,9 @@ public function setTransactionId($value)
/**
* Get the transaction reference.
*
* The transaction reference is the identifier generated by the remote
* payment gateway.
*
* @return string
*/
public function getTransactionReference()
Expand Down
12 changes: 11 additions & 1 deletion src/Omnipay/Common/Message/AbstractResponse.php
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@ abstract class AbstractResponse implements ResponseInterface
* @var RequestInterface
*/
protected $request;

/**
* The data contained in the response.
*
Expand Down Expand Up @@ -147,6 +147,16 @@ public function getTransactionReference()
return null;
}

/**
* Get the transaction ID as generated by the merchant website.
*
* @return string
*/
public function getTransactionId()
{
return null;
}

/**
* Automatically perform any required redirect
*
Expand Down

0 comments on commit 0ee00e2

Please sign in to comment.