Opened 10 years ago

Closed 10 years ago

#13381 closed defect (fixed)

[CLA][Patch] Standby widget throws an exception in IE7 if not initialized completely

Reported by: maxmotovilov Owned by: Eugene Lazutkin
Priority: high Milestone: 1.7
Component: DojoX Widgets Version: 1.6.1
Keywords: Cc:
Blocked By: Blocking:

Description (last modified by Eugene Lazutkin)

IE7 only: if the startup() method has not been called for any reason -- e.g. startup() on other widgets terminating with an exception -- calling uninitialize() results in an exception due to missing this._ieFixNode.

The minimal fix is attached. Arguably this is not a proper sequence of calls but the fix seems simpler than ensuring uninitialize() is never called out of sequence by other components.

Attachments (1)

Standby.2.patch​ (410 bytes) - added by maxmotovilov 10 years ago.

Download all attachments as: .zip

Change History (4)

Changed 10 years ago by maxmotovilov

Attachment: Standby.2.patch​ added

comment:1 Changed 10 years ago by Eugene Lazutkin

Description: modified (diff)
severity: normal → trivial
Summary: Standby widget throws an exception in IE7 if not initialized completely → [CLA][Patch] Standby widget throws an exception in IE7 if not initialized completely

comment:2 Changed 10 years ago by Eugene Lazutkin

Milestone: tbd → 1.7
Owner: changed from dante to Eugene Lazutkin
Priority: normal → high
Status: new → assigned

Pete, if you don't mind I'll take care of this trivial one-liner.

comment:3 Changed 10 years ago by Eugene Lazutkin

Resolution: → fixed
Status: assigned → closed

In [26382]:

standby: trivial fix to check a node before deleting it, thx Max!, !strict, fixes #13381.

Note: See TracTickets for help on using tickets.