Code review comment for lp:~banking-addons-drivers/banking-addons/direct-debit-refactoring-fosdem

Revision history for this message
Stefan Rijnhart (Opener) (stefan-opener) wrote :

Oh, almost missed the files that you move from account_banking_payment to account_direct_debit. This includes the definition of the fields that are used in the integration that the former module accomplishes between the payment order and invoice workflows, and I'm afraid that this causes a severe breakage. If I now have account_banking_payment installed (but not account_direct_debit) I cannot even select invoices on a payment order because "Field 'payment_term_ids' does not exist in object 'browse_record(payment.mode, 1)". And that is just the start.

Any idea how to solve this?

review: Needs Fixing

« Back to merge proposal