Opened 14 years ago
Closed 14 years ago
#2040 closed defect (fixed)
[patch][cla] ContentPane examines wrong property on nested exceptions
Reported by: | Owned by: | mumme | |
---|---|---|---|
Priority: | high | Milestone: | 0.9 |
Component: | Dojox | Version: | 0.4.1rc1 |
Keywords: | Cc: | ||
Blocked By: | Blocking: |
Description
In the code below from ContentPane?.js, the caught exception "e" is queried for e.description. This should probably be e.message. Could I also suggest that st[i] is moved to the end of the string, as it will expand into the whole function definition?
_runStack: function(stName){ var st = this[stName]; var err = ""; var scope = this.scriptScope || window; for(var i = 0;i < st.length; i++){ try{ st[i].call(scope); }catch(e){ err += " "+st[i]+" failed: "+e.description; } }
Best regards Mike Wilson
Change History (5)
comment:1 Changed 14 years ago by
Milestone: | → 0.5 |
---|---|
Owner: | changed from bill to mumme |
comment:2 Changed 14 years ago by
Component: | Widgets → Dojox |
---|
comment:3 Changed 14 years ago by
Milestone: | 0.9 → 1.0 |
---|---|
Status: | new → assigned |
comment:4 Changed 14 years ago by
comment:5 Changed 14 years ago by
Milestone: | 1.0 → 0.9 |
---|---|
Resolution: | → fixed |
Status: | assigned → closed |
This code didn't survive dojox ContentPane? port, but I tried to make use of error.message where appropriate.
Note: See
TracTickets for help on using
tickets.
Hum, I don't think this code will survive into the dojox ContentPane?, but I leave it open just in case.