#17407 closed defect (fixed)
dojox/mobile/_StoreMIxin badly maintains its promise observation handle
Reported by: | Patrick Ruzand | Owned by: | |
---|---|---|---|
Priority: | undecided | Milestone: | 1.9.2 |
Component: | DojoX Mobile | Version: | 1.9.1 |
Keywords: | Cc: | ||
Blocked By: | Blocking: |
Description
A race condition in refresh() means that it is possible to create multiple observation handles at once: Call refresh() twice or more. The old handle (if existing) will be removed twice. (because the handler property is not reset when remove()is called) When the query returns, a handle will be created twice (and the one created first will be lost).
Attachments (1)
Change History (4)
Changed 7 years ago by
Attachment: | test_StoreMixinLeak.html added |
---|
comment:1 Changed 7 years ago by
isn't that suppose to be closed by https://github.com/dojo/dojox/commit/663178d0583302612922bf7e52b30e651e8a70ba ?
comment:2 Changed 7 years ago by
Resolution: | → fixed |
---|---|
Status: | new → closed |
yes. don't know why the ticket has not been closed automatically.. (the github commit msg contains the ticket ref (see: dojox/mobile/_StoreMixin badly maintains its promise observation handle, fixes #17407)
comment:3 Changed 7 years ago by
Milestone: | tbd → 1.9.2 |
---|
testcase