Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Get final batch id from SEPA message #28

Merged
merged 2 commits into from
Feb 19, 2014
Merged

Get final batch id from SEPA message #28

merged 2 commits into from
Feb 19, 2014

Conversation

edwinv
Copy link
Contributor

@edwinv edwinv commented Feb 19, 2014

This pull request contains changes to provide the batch id of a certain transaction. The batch id is used in bank transactions to reference a certain batch in a SEPA message.

Added are two extra methods for a SEPA::Message:

  1. SEPA::Message#batch_id(transaction_reference): fetches the batch id of the transaction with the given reference
  2. SEPA::Message#batches: returns an array with the batch ids of the batch in the current file

For a successful implementation of these methods, a refactor was required. Transactions are now stored in groups right after adding a transaction with add_transaction. For backward compatibility, it is still possible to get all transactions across all batches by calling the transactions method.

@schorsch
Copy link
Member

Does not look to bad :-)
@ledermann what are your thoughts on it?

ledermann added a commit that referenced this pull request Feb 19, 2014
Get final batch id from SEPA message
@ledermann ledermann merged commit 0472875 into salesking:master Feb 19, 2014
@ledermann
Copy link
Member

Great, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants