8/30/2012 3:29:52 PM
Pierre-Alexandre Meyer <pierre@ning.com>
|
8/30/2012 3:27:12 PM
Pierre-Alexandre Meyer <pierre@ning.com>
|
8/30/2012 2:49:38 PM
was failing to notice the difference between existing emails
and new ones because the underlying comparison is flawed (it expects the
same class). This was making server throw a 500 error when trying to
add an existing email address.
We really need to fix AuditedCollectionDaoBase eventually. For now, apply a
band-aid.
Signed-off-by: Pierre-Alexandre Meyer <pierre@ning.com>
|
8/30/2012 1:35:44 PM
for e.g. invoice item adjustments.
Signed-off-by: Pierre-Alexandre Meyer <pierre@ning.com>
|
8/29/2012 7:07:06 PM
due to bundle transfers.
Signed-off-by: Pierre-Alexandre Meyer <pierre@ning.com>
|
8/29/2012 3:01:41 PM
throws NPE if paymentId is null.
We want to ignore these events in analytics anyways as they are not real
attempts.
Signed-off-by: Pierre-Alexandre Meyer <pierre@ning.com>
|
8/29/2012 1:55:34 PM
Pierre-Alexandre Meyer <pierre@ning.com>
|
8/29/2012 1:49:37 PM
previously checking the overdue states by explicitly
calling refresh on the bundle (the configuration parameter
autoReevaluationInterval was so high, that the notification queue never
kicked in).
This wasn't fully testing overdue though. For instance, the test
testOverdueStateIfNoPaymentMethod() would pass even if we don't send
the payment errors on the bus when there is no default payment method
(because we were asking overdue to refresh its state manually).
Since there is no payment retry if the account doesn't have a default
payment method, overdue has to rely on the notification queue to refresh
its state (and make the bundle go through the various overdue stages
when we move the clock).
Signed-off-by: Pierre-Alexandre Meyer <pierre@ning.com>
|
8/29/2012 1:18:53 PM
method
If a payment cannot be processed because there is no default payment
method, we need to post an event on the bus for overdue to refresh the
overdue state.
Note that an event will be posted even if the invoice balance is zero.
Added a Beatrix test for this scenario.
Signed-off-by: Pierre-Alexandre Meyer <pierre@ning.com>
|
8/27/2012 8:25:33 PM
change.
Signed-off-by: Pierre-Alexandre Meyer <pierre@ning.com>
|
