#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 |
|---|
Note: See
TracTickets for help on using
tickets.
testcase