Page MenuHomePhabricator

Add read and write implementations for bank transfer payments to PaymentRepository
Closed, ResolvedPublic1 Estimated Story Points

Description

Acceptance Criteria

  • The domain objects for bank transfer payments are refactored according to the data model. You can leave other payment objects in a semi-broken state as long as all remaining tests still pass and the next steps are clear (i.e. putting TODOs/commented-out code in classes, skipping tests).
  • The implementation of the PaymentRepository interface implements storing and retrieving payments with bank transfer. All other payment methods may throw "Not implemented" logic exceptions for now.

Implementation Details
Not much to do here, just some interface/inheritance/name adjusting of BankTransferPayment to conform with the data model.
Maybe add a non-empty check for the bank transfer code in the constructor to avoid domain-invalid payments (bank transfer payments must always have a transfer code).

Event Timeline

gabriel-wmde set the point value for this task to 5.
AbbanWMDE moved this task from Doing to Review on the WMDE-FUN-Sprint-2022-03-14 board.
AbbanWMDE subscribed.
kai.nissen changed the point value for this task from 5 to 1.