Opened 6 years ago

Closed 6 years ago

Last modified 6 years ago

#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)

test_StoreMixinLeak.html (2.4 KB) - added by Patrick Ruzand 6 years ago.
testcase

Download all attachments as: .zip

Change History (4)

Changed 6 years ago by Patrick Ruzand

Attachment: test_StoreMixinLeak.html added

testcase

comment:2 Changed 6 years ago by Patrick Ruzand

Resolution: fixed
Status: newclosed

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 6 years ago by Patrick Ruzand

Milestone: tbd1.9.2
Note: See TracTickets for help on using tickets.