[CB-4263] The FileReader readyState is not working as expected (value in onload for readAsText)

• Type: Bug
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 2.9.0
• Component/s: cordova-plugin-file

Environment

Gingerbread emulator

• Assignee: Jesse MacFadyen
• Reporter: Peter
• Created: 16/Jul/13 01:53
• Updated: 23/Aug/18 07:19
• Resolved: 22/Jul/14 23:53

Description

onload is called when readAsText was successful, so the readyState should be FileReader.DONE. See http://www.w3.org/TR/FileAPI/#blobreader-state

But it is still set as FileReader.LOADING.

Looking at the v2.9.0 cordova.js code you can see that the setting of the readyState for readAsText is inconsistent with all the other read methods.

i.e.

// If onload callback
if (typeof me.onload === "function") {
	me.onload(new ProgressEvent("load", {target:me}));
}

// DONE state
me._readyState = FileReader.DONE;

It looks like that DONE state should be set before the onload invocation.