killbill-aplcache

Details

diff --git a/osgi-bundles/libs/killbill/src/main/java/com/ning/killbill/osgi/libs/killbill/KillbillActivatorBase.java b/osgi-bundles/libs/killbill/src/main/java/com/ning/killbill/osgi/libs/killbill/KillbillActivatorBase.java
index ec729ca..bd2056f 100644
--- a/osgi-bundles/libs/killbill/src/main/java/com/ning/killbill/osgi/libs/killbill/KillbillActivatorBase.java
+++ b/osgi-bundles/libs/killbill/src/main/java/com/ning/killbill/osgi/libs/killbill/KillbillActivatorBase.java
@@ -21,20 +21,20 @@ import org.osgi.framework.BundleContext;
 
 import com.ning.killbill.osgi.libs.killbill.OSGIKillbillEventDispatcher.OSGIKillbillEventHandler;
 
-public abstract class KillbillActivatorBase implements BundleActivator, OSGIKillbillEventHandler {
+public abstract class KillbillActivatorBase implements BundleActivator {
 
 
-    protected OSGIKillbillAPI api;
+    protected OSGIKillbillAPI killbillAPI;
     protected OSGIKillbillLogService logService;
     protected OSGIKillbillRegistrar registrar;
     protected OSGIKillbillDataSource dataSource;
     protected OSGIKillbillEventDispatcher dispatcher;
 
     @Override
-    public void start(final BundleContext context) {
+    public void start(final BundleContext context) throws Exception {
 
         // Tracked resource
-        api = new OSGIKillbillAPI(context);
+        killbillAPI = new OSGIKillbillAPI(context);
         logService = new OSGIKillbillLogService(context);
         dataSource = new OSGIKillbillDataSource(context);
         dispatcher = new OSGIKillbillEventDispatcher(context);
@@ -45,15 +45,15 @@ public abstract class KillbillActivatorBase implements BundleActivator, OSGIKill
         // Killbill events
         final OSGIKillbillEventHandler handler = getOSGIKillbillEventHandler();
         if (handler != null) {
-            dispatcher.registerEventHandler(this);
+            dispatcher.registerEventHandler(handler);
         }
     }
 
     @Override
-    public void stop(final BundleContext context) {
+    public void stop(final BundleContext context) throws Exception {
 
         // Close trackers
-        api.close();
+        killbillAPI.close();
         dispatcher.close();
         dataSource.close();
         logService.close();
diff --git a/osgi-bundles/tests/beatrix/src/test/java/com/ning/billing/osgi/bundles/test/TestActivator.java b/osgi-bundles/tests/beatrix/src/test/java/com/ning/billing/osgi/bundles/test/TestActivator.java
index 78559e6..53a7ea4 100644
--- a/osgi-bundles/tests/beatrix/src/test/java/com/ning/billing/osgi/bundles/test/TestActivator.java
+++ b/osgi-bundles/tests/beatrix/src/test/java/com/ning/billing/osgi/bundles/test/TestActivator.java
@@ -48,7 +48,7 @@ public class TestActivator extends KillbillActivatorBase implements OSGIKillbill
     private TestDao testDao;
 
     @Override
-    public void start(final BundleContext context) {
+    public void start(final BundleContext context) throws Exception {
 
         final String bundleName = context.getBundle().getSymbolicName();
         System.out.println("TestActivator starting bundle = " + bundleName);
@@ -64,7 +64,7 @@ public class TestActivator extends KillbillActivatorBase implements OSGIKillbill
     }
 
     @Override
-    public void stop(final BundleContext context) {
+    public void stop(final BundleContext context) throws Exception {
         super.stop(context);
         System.out.println("Good bye world from TestActivator!");
     }
@@ -75,7 +75,6 @@ public class TestActivator extends KillbillActivatorBase implements OSGIKillbill
     }
 
     private void registerPaymentApi(final BundleContext context, final TestDao dao) {
-
         final Dictionary props = new Hashtable();
         props.put(OSGIPluginProperties.PLUGIN_NAME_PROP, "test");
         registrar.registerService(context, PaymentPluginApi.class, new TestPaymentPluginApi("test", dao), props);
@@ -99,12 +98,11 @@ public class TestActivator extends KillbillActivatorBase implements OSGIKillbill
         };
 
         try {
-            Account account = api.getAccountUserApi().getAccountById(killbillEvent.getAccountId(), tenantContext);
+            Account account = killbillAPI.getAccountUserApi().getAccountById(killbillEvent.getAccountId(), tenantContext);
             testDao.insertAccountExternalKey(account.getExternalKey());
 
         } catch (AccountApiException e) {
             logService.log(LogService.LOG_ERROR, e.getMessage());
         }
     }
-
 }