killbill-memoizeit

analytics: fix bug in SYSTEM_CANCEL event in BST The time

10/18/2012 6:51:03 PM

Details

diff --git a/analytics/src/main/java/com/ning/billing/analytics/BusinessSubscriptionTransitionDao.java b/analytics/src/main/java/com/ning/billing/analytics/BusinessSubscriptionTransitionDao.java
index 6c4a834..929164f 100644
--- a/analytics/src/main/java/com/ning/billing/analytics/BusinessSubscriptionTransitionDao.java
+++ b/analytics/src/main/java/com/ning/billing/analytics/BusinessSubscriptionTransitionDao.java
@@ -133,7 +133,9 @@ public class BusinessSubscriptionTransitionDao {
                                     bundle.getAccountId(),
                                     account.getExternalKey(),
                                     subscription.getId(),
-                                    event.getRequestedTransitionTime(),
+                                    // Note! The system cancel event requested time is the effective time when the subscription
+                                    // is cancelled, which is the effective time of the cancel event
+                                    event.getEffectiveTransitionTime(),
                                     new BusinessSubscriptionEvent(BusinessSubscriptionEvent.EventType.SYSTEM_CANCEL, businessEvent.getCategory()),
                                     prevSubscription,
                                     nextSubscription