status-im/re-frame
2
0
Fork 0
mirror of https://github.com/status-im/re-frame.git synced 2025-02-24 07:48:14 +00:00
Code Issues Projects Releases Wiki Activity

Fix never-ending warn about not implementing IReactiveAtom

Browse Source
This commit is contained in:
Oliver Hine 2016-07-22 19:44:05 +01:00
parent 16585b2722
commit 5daafa7db4
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/re_frame/subs.cljs
View File

@ -79,7 +79,7 @@
(let [query-id (first-in-vector v) (let [query-id (first-in-vector v)
handler-fn (get @qid->fn query-id)] handler-fn (get @qid->fn query-id)]
(when ^boolean js/goog.DEBUG (when ^boolean js/goog.DEBUG
(when-let [not-reactive (remove #(implements? reagent.ratom/IReactiveAtom %) dynv)] (when-let [not-reactive (not-empty (remove #(implements? reagent.ratom/IReactiveAtom %) dynv))]
(console :warn "re-frame: your subscription's dynamic parameters that don't implement IReactiveAtom: " not-reactive))) (console :warn "re-frame: your subscription's dynamic parameters that don't implement IReactiveAtom: " not-reactive)))
(if (nil? handler-fn) (if (nil? handler-fn)
(console :error "re-frame: no subscription handler registered for: \"" query-id "\". Returning a nil subscription.") (console :error "re-frame: no subscription handler registered for: \"" query-id "\". Returning a nil subscription.")