"update-end" event returns error but "onUpdateEnd" event doesn't

kenbuja

I have converted the code from using the "onUpdateEnd" event on an ArcGISDynamicMapServiceLayer to the class.on style event

 //before     connect.connect(layer, "onUpdateEnd", function (error) {         if (error) {             alert("Update complete with error: " & error);         }      });  //after     layer.on("update-end", function (error) {         if (error) {             alert("Update complete with error: " & error);         }      });

I am now getting an error returned for this event. I have created a Fiddle that shows this. Running the code with the "connect.connect" line (line 44) un-commented does not return an error, while the "layer.on" line (line 43) un-commented returns the error. Why is the error returned using this syntax?

ReneRubalcava

I think this has to do with how the error object is passed around using dojo/on.
Somewhere in the layer code there is now this.emit('update-end', errorObject);
But if you look at the error object itself, it's actually

{
  error: undefined,
  target: Object, // <- that target layer
  undefined: undefined // <- heh, probably has to do with some json and mixin stuff
}

Anyway, previously with connect.connect(layer, 'onUpdateEnd') it would pass nothing as an error object, so it looks like now you just need to check the returned error object to see if the error.error is undefined instead.

I hadn't run into this yet, but this is the behavior I'm seeing.
I'm guessing in the source for layer there is a method that handles this.emit('update-end', passedvariable) and it defaults to an object of some sort.

This is all best guess stuff since it isn't explicitly defined that this is the expected behavior in the docs/samples.