dojo.io.bind error handling model needs change

[tjerk@…]

One calls dojo.io.bind with an object argument that defines a load and error function.

Now when the load function throws an error dojo sees this as a failed AjaxRequest? and calls the error function.

Dojo should treat errors thrown in the load function as normal program errors and it should not catch this error.

Example code:

dojo.io.bind({
    url: action,
    encoding: "utf-8",
    load: function(type, data, e) {
          
          throw "SomeError";
          
    },
    error: function(type, error) {
            alert(error.message);
    }
    mimetype: "text/xml"
  });

The error thrown in load is catched and the error function is called.

I think it should report the error to the browser. ( thus not catch it ). This is better because now we can do some debugging if some unespected error occured in a load function.

PS: In previous versions of dojo I had no problems

[jburke]

This is "fixed" in a way in Dojo 1.2. Now, with djConfig.isDebug = true, the errors are thrown and not caught. They are only caught now in the djConfig.isDebug false case.