Opened 13 years ago

Closed 12 years ago

Last modified 12 years ago

#2425 closed defect (wontfix)

dojo tree RPCErrorHandler NOT getting invoked in firefox 2.0.0.1

Reported by: JS Owned by: anonymous
Priority: high Milestone: 0.9
Component: Dijit Version: 0.4.1
Keywords: Cc:
Blocked By: Blocking:

Description

In TreeLoadingController?.js, the runRPC call will prepare the handler in the way that when there is an error, RPCErrorHandler will be called. In my situation server times out, this handler should invoke RPCErrorHandler, which it indeed does in any browser other than firefox 2.0.0.1. With firefox 2.0.0.1, when server times out, when I click on tree node to expand it, it hangs with the blue clock icon showing up; same time the firebug shows me that a uncaught exception: Security Error. The handler seems never get executed.

Change History (3)

comment:1 Changed 13 years ago by JS

I just managed to change the TreeLoadingController?.js of the runRPC method to make it work for me on firefox 2.0.0.1:

I replace the line in dojo.io.bind():

handle: dojo.lang.hitch(this, handle),

to (get rid of the hitch):

dojo.io.bind( {

url: kw.url, /* I hitch to get this.loadOkHandler */ handle: dojo.lang.hitch(this, handle), handle: function(type, data, evt) {

unlock BEFORE any processing is done so errorHandler may apply locking if (kw.lock) {

dojo.lang.forEach(kw.lock,

function(t) { t.unlock() }

);

} if(type == "load"){

kw.load.call(this, data);

}else{

_this.RPCErrorHandler(type, data, evt);

}

}, mimetype: "text/json", preventCache: _this.preventCache, sync: true, sync: kw.sync, content: { data: dojo.json.serialize(kw.params) }

});

seems hitch has to do with the scope of the method, not familiar with it at all... so not sure my change will break anything else, but at least it works for FF 2.0.0.1.

Any comments/advice? thanks

comment:2 Changed 12 years ago by bill

Milestone: 0.9
Resolution: wontfix
Status: newclosed

All this code is gone in 0.9; loads from dojo.data instead and this is no longer an issue.

comment:3 Changed 12 years ago by bill

Component: GeneralDijit
Note: See TracTickets for help on using tickets.