Pass the eventbus to on-open callbacks [fixes #53]

ChangeLog.md

@@ -1,5 +1,9 @@
 # ChangeLog
 
+## v0.3.0 - unreleased
+
+* BREAKING CHANGE: Pass the eventbus to on-open callbacks in the ClojureScript client [#53](/../../issues/53)
+
 ## [v0.2.0](/../../tree/0.2.0) - 2013-09-17
 
 * add `with-vertx` convenience macro to embed ns

api/src/main/clojure/vertx/client/eventbus.cljs

@@ -47,24 +47,25 @@
   [eb]
   (= js/vertx.EventBus.CLOSED (ready-state eb)))
 
-
 (defn- extend-callback [old-f new-f]
   (fn []
     (and old-f (old-f))
     (new-f)))
 
 (defn on-open
   "Registers a fn to be called when the eventbus is opened.
-   If the eventbus is currently open, the fn is called immediately.
-   If the eventbus is closing or is closed, and error is thrown.
-   Can be called multiple times to register multiple fns."
-  [eb f]
+   The function should take one argument, and will be passed the
+   eventbus. If the eventbus is currently open, the fn is called
+   immediately. If the eventbus is closing or is closed, and error is
+   thrown. Can be called multiple times to register multiple fns."
+   [eb f]
   (if (or (closing? eb)
           (closed? eb))
     (throw (js/Error. "EventBus is closing or has closed.")))
-  (if (open? eb)
-    (f)
-    (set! (.-onopen eb) (extend-callback (.-onopen eb) f))))
+  (let [with-eb #(f eb)]
+    (if (open? eb)
+      (with-eb)
+      (set! (.-onopen eb) (extend-callback (.-onopen eb) with-eb)))))
 
 (defn on-close
   "Registers a fn to be called when the eventbus is closed.