killbill-memoizeit

Implement notifyPendingTransactionOfStateChanged through …

state machine.

We are not creating new callbacks but instead are replaying with the existing callbacks
and relying on some extra context info to specify that underlying plugin operation should be skipped,
and final state should be read from context. This is fairly elegant, but also pollutes a little more the context.

Remove STEPH in the payment code (Fix issue, create bug, remove …

invalid test)

Implement soft delete for invoice_payment_control_plugin_auto_pay_off Minor …

refactoring of code.

Fix wrong group status Remove invalid test (createPurchase …

needs to have a paymentMethodId not null)

Fixes #207

Fix processedAmount in MockPaymentProviderPlugin and remove …

hack in the test TestPaymentProcessor

Fixes #208

Revert enum value NOTIFY_STATE_CHANGE in TransactionType …

(code review from Pierre)

See #206 (not sure this is the issue reported though, the line …

number in the bug report does not seem to match current code)

Add missing Nullable for externalKeys
Add specific null constant NULL_ATTEMPT_ID (to avoid arbitrary null all over api calls)
Check for null transactionExternalKey when checking for uniquness

Change the way we handle null paymentMethodId AND missing default …

paymentMethodId on the account, so that payment state machine will still run
and will end up generating a payment bus event (That can be caught by the system, such as the overdue module to move overdue state).