Opened 8 years ago

Closed 8 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 8 years ago.

Download all attachments as: .zip

Change History (4)

Changed 8 years ago by maxmotovilov

Attachment: Standby.2.patch added

comment:1 Changed 8 years ago by Eugene Lazutkin

Description: modified (diff)
severity: normaltrivial
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 8 years ago by Eugene Lazutkin

Milestone: tbd1.7
Owner: changed from dante to Eugene Lazutkin
Priority: normalhigh
Status: newassigned

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

comment:3 Changed 8 years ago by Eugene Lazutkin

Resolution: fixed
Status: assignedclosed

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.