FileReader icon indicating copy to clipboard operation
FileReader copied to clipboard

Wrong behavior after the first file read

Open bamarni opened this issue 11 years ago • 6 comments

Hi,

When I read from the input file the first time, everything is correct, but when the file is changed and I read again, the load event will be triggered 2 times, when I change the file again, it's getting triggered 3 times, and so on... This is obviously wrong, the real Filereader object doesn't behaves like this.

Here are some logs :

debugMode : ,true 
File Input add...,my_input_id,false,,, 
MouseOverEvent...,my_input_id 
FileInput Event click[object Object] 
Browsing...,my_input_id 
File Input change...,my_input_id 
FileInput Event change[object Object] 
FileReader read...,my_input_id,A.JPG,readAsDataURL 
FileReader Event loadstart[object Object]true 
FileReader Event progress[object Object]true 
FileReader Event progress[object Object]true 
FileReader Event progress[object Object]true 
FileReader Event progress[object Object]true 
FileReader Event progress[object Object]true 
FileReader Event progress[object Object]true 
FileReader Event load[object Object]true 
FileReader result...,my_input_id.A.JPG 
FileReader Event loadend[object Object]true 
FileInput Event click[object Object] 
Browsing...,my_input_id 
File Input change...,my_input_id 
FileInput Event change[object Object] 
FileReader read...,my_input_id,B.jpg,readAsDataURL 
FileReader Event loadstart[object Object]true 
FileReader Event progress[object Object]true 
FileReader Event progress[object Object]true 
FileReader Event progress[object Object]true 
FileReader Event progress[object Object]true 
FileReader Event load[object Object]true <====== WRONG FROM HERE
FileReader result...,my_input_id.A.JPG 
FileReader Event loadend[object Object]true <=== TO HERE
FileReader Event load[object Object]true 
FileReader result...,my_input_id.B.jpg 
FileReader Event loadend[object Object]true 

@Jahdrien if you don't have time to fix this please give me some pointers and I'd be happy to submit a PR, thanks

bamarni avatar Jan 29 '13 14:01 bamarni

ping @Jahdrien

bamarni avatar Feb 11 '13 09:02 bamarni

Occurs for me as well. Would love a fix.

itsmattking avatar Mar 07 '13 03:03 itsmattking

@mattking17 : here is the dirty hack I've done for now (https://github.com/bamarni/FileReader/commit/74546ba4019576878f62e1dc421131905088bd87).

I didn't go further in my investigation since it fixed my issue, I'm not sure though wether the issue is in this library or on the underlying swfObject it uses.

bamarni avatar Mar 07 '13 11:03 bamarni

thanks @bamarni! I'm trying to remove all dependencies on jquery from this plugin and ended up rewriting an implementation myself. I think what I'm going to do is refactor it to only support one file handler at a time. It seems that the reader result comes in multiple times, it ends up triggering load events for all file handlers ever registered. This works for my case since I only care about people uploading one file at a time.

itsmattking avatar Mar 07 '13 16:03 itsmattking

ping @Jahdrien again.

john-bai avatar Jul 30 '13 19:07 john-bai

ping @Jahdrien

encore1908 avatar Apr 09 '14 23:04 encore1908