day8 / re-frame-debux

A fork of debux for tracing re-frame code (for eventual consumption by re-frame-10x)
Eclipse Public License 1.0
43 stars 7 forks source link

Assert failed: indent was not correctly replaced for form #29

Closed mengu closed 4 years ago

mengu commented 5 years ago

hi, i had this message in my console when i wanted to trace an event:

the whole event looks like this:

(reg-event-fx
  :send-export
  (fn-traced [{:keys [db]} _]
    (let [params (-> db :inbox :filter-container)
          params (cond-> params
                         (= "custom_range" (:range params)) (dissoc :range)
                         (= "custom_range" (:range params)) (assoc :from (util/format-date-unix (:from params)))
                         (= "custom_range" (:range params)) (assoc :to (util/format-date-unix (:to params))))
          params (remove (fn [[k v]] (or (= k :page) (nil? v))) params)]
      {:http-xhrio (merge (util/create-request-map :get "/feedback/export_as_csv"
                                                   :send-export-result-ok
                                                   :send-export-result-error)
                          {:params (merge {:nodes (-> db :current-user :selectedAccountId)} params)})})))

here's the full stack trace:

router.cljc?rel=1563958455083:204 Uncaught Error: Assert failed: indent was not correctly replaced for form (= "custom_range" (:range params))

This is a bug, please report it to https://github.com/Day8/re-frame-debux
(integer? indent)
    at debux$common$util$spy_first (util.cljc?rel=1563958473380:343)
    at events.cljs?rel=1564041927698:527
    at events.cljs?rel=1564041927698:527
    at events.cljs?rel=1564041927698:527
    at std_interceptors.cljc?rel=1563958455257:148
    at re_frame$std_interceptors$fx_handler__GT_interceptor_$_fx_handler_before (std_interceptors.cljc?rel=1563958455257:145)
    at re_frame$interceptor$invoke_interceptor_fn (interceptor.cljc?rel=1563958454942:68)
    at re_frame$interceptor$invoke_interceptors (interceptor.cljc?rel=1563958454942:106)
    at re_frame$interceptor$execute (interceptor.cljc?rel=1563958454942:199)
    at re_frame$events$handle (events.cljc?rel=1563958455006:65)
    at Object.re_frame$router$IEventQueue$_process_1st_event_in_queue$arity$1 (router.cljc?rel=1563958455083:179)
    at re_frame$router$_process_1st_event_in_queue (router.cljc?rel=1563958455083:84)
    at Object.re_frame$router$IEventQueue$_run_queue$arity$1 (router.cljc?rel=1563958455083:198)
    at re_frame$router$_run_queue (router.cljc?rel=1563958455083:86)
    at router.cljc?rel=1563958455083:146
    at Object.re_frame$router$IEventQueue$_fsm_trigger$arity$3 (router.cljc?rel=1563958455083:169)
    at re_frame$router$_fsm_trigger (router.cljc?rel=1563958455083:80)
    at router.cljc?rel=1563958455083:187
    at run (antd.inc.js:136079)
    at runIfPresent (antd.inc.js:136108)
    at onGlobalMessage (antd.inc.js:136148)
debux$common$util$spy_first @ util.cljc?rel=1563958473380:343
(anonymous) @ events.cljs?rel=1564041927698:527
(anonymous) @ events.cljs?rel=1564041927698:527
(anonymous) @ events.cljs?rel=1564041927698:527
(anonymous) @ std_interceptors.cljc?rel=1563958455257:148
re_frame$std_interceptors$fx_handler__GT_interceptor_$_fx_handler_before @ std_interceptors.cljc?rel=1563958455257:145
re_frame$interceptor$invoke_interceptor_fn @ interceptor.cljc?rel=1563958454942:68
re_frame$interceptor$invoke_interceptors @ interceptor.cljc?rel=1563958454942:106
re_frame$interceptor$execute @ interceptor.cljc?rel=1563958454942:199
re_frame$events$handle @ events.cljc?rel=1563958455006:65
(anonymous) @ router.cljc?rel=1563958455083:179
re_frame$router$_process_1st_event_in_queue @ router.cljc?rel=1563958455083:84
(anonymous) @ router.cljc?rel=1563958455083:198
re_frame$router$_run_queue @ router.cljc?rel=1563958455083:86
(anonymous) @ router.cljc?rel=1563958455083:146
(anonymous) @ router.cljc?rel=1563958455083:169
re_frame$router$_fsm_trigger @ router.cljc?rel=1563958455083:80
(anonymous) @ router.cljc?rel=1563958455083:187
run @ antd.inc.js:136079
runIfPresent @ antd.inc.js:136108
onGlobalMessage @ antd.inc.js:136148
postMessage (async)
registerImmediate @ antd.inc.js:136159
setImmediate @ antd.inc.js:136066
goog.async.nextTick @ nexttick.js:74
(anonymous) @ router.cljc?rel=1563958455083:187
re_frame$router$_run_next_tick @ router.cljc?rel=1563958455083:85
(anonymous) @ router.cljc?rel=1563958455083:142
(anonymous) @ router.cljc?rel=1563958455083:169
re_frame$router$_fsm_trigger @ router.cljc?rel=1563958455083:80
(anonymous) @ router.cljc?rel=1563958455083:102
re_frame$router$push @ router.cljc?rel=1563958455083:74
re_frame$router$dispatch @ router.cljc?rel=1563958455083:247
dashboard$inbox$views$export_modal_send @ views.cljs?rel=1564041654637:191
dashboard$inbox$views$export_modal_action @ views.cljs?rel=1564041654637:205
(anonymous) @ views.cljs?rel=1564041654637:224
Button._this.handleClick @ antd.inc.js:4211
callCallback @ react-dom.inc.js:140
invokeGuardedCallbackDev @ react-dom.inc.js:178
invokeGuardedCallback @ react-dom.inc.js:227
invokeGuardedCallbackAndCatchFirstError @ react-dom.inc.js:241
executeDispatch @ react-dom.inc.js:604
executeDispatchesInOrder @ react-dom.inc.js:623
executeDispatchesAndRelease @ react-dom.inc.js:724
executeDispatchesAndReleaseTopLevel @ react-dom.inc.js:735
forEachAccumulated @ react-dom.inc.js:705
runEventsInBatch @ react-dom.inc.js:866
runExtractedEventsInBatch @ react-dom.inc.js:875
handleTopLevel @ react-dom.inc.js:4551
batchedUpdates @ react-dom.inc.js:12983
batchedUpdates @ react-dom.inc.js:2115
dispatchEvent @ react-dom.inc.js:4632
interactiveUpdates @ react-dom.inc.js:13038
interactiveUpdates @ react-dom.inc.js:2134
dispatchInteractiveEvent @ react-dom.inc.js:4609
Show 24 more frames
MarkN3DTI commented 4 years ago

Had the same error for the following form inside a let statement:

new-logo? (not
            (or (empty? logo)
                (= logo original-logo)))
stumitchell commented 4 years ago

@mengu this should be solved on master now

@MarkN3DTI sorry I could not replicate this with a let

mengu commented 4 years ago

hi @stumitchell,

thank you for letting me know. i'll give it a go.

mike-thompson-day8 commented 4 years ago

@mengu the fixes have migrated from being on master to being in the latest release which will make it even easier for you to test