mithril.js icon indicating copy to clipboard operation
mithril.js copied to clipboard

The internal redirect to the default route on failure isn't invalidating the current updater function.

Open dead-claudia opened this issue 4 years ago • 0 comments

It's all caused by this line which isn't directly delegating to m.route.set, but is instead using an internal call skipping the reset logic.

It's not causing any apparent bugs, just taking up more space than necessary. The relevant variable set, lastUpdate could probably just be made a simple object token, so the closure isn't itself being directly persisted.

I've conservatively marked it as a bug, in case there is actual observably buggy behavior here.

dead-claudia avatar Aug 07 '19 13:08 dead-claudia