Opened 7 years ago
Closed 7 years ago
#17632 closed defect (worksforme)
Not possible to send file metadata using dojox.form.Uploader
Reported by: | Denis Rykov | Owned by: | Denis Rykov |
---|---|---|---|
Priority: | low | Milestone: | 1.10 |
Component: | DojoX Uploader | Version: | 1.9.2 |
Keywords: | Cc: | ||
Blocked By: | Blocking: |
Description
Not possible to send file's metadata (name, size, type) if use dojox.form.Uploader
with uploadOnSelect
option while uploading multiple files. Now only first file metadata sends to the server because _HTML5.js
file contains the following code:
if(this.uploadOnSelect){ this.connect(this, "onChange", function(data){ this.upload(data[0]); }); }
Where data
is list of objects which contains file's metadata.
Change History (6)
comment:1 Changed 7 years ago by
Component: | DojoX Form → DojoX Uploader |
---|
comment:2 Changed 7 years ago by
Milestone: | tbd → 1.10 |
---|---|
Owner: | set to dylan |
Priority: | undecided → low |
Status: | new → assigned |
comment:3 Changed 7 years ago by
comment:4 Changed 7 years ago by
Owner: | changed from dylan to Denis Rykov |
---|---|
Status: | assigned → pending |
I've gone through the various tests, and modified some of them to use the uploadOnSelect flag, and they seem to upload fine and post the relevant metadata.
Can you please provide a testcase that shows otherwise? https://github.com/dojo/dojox/pull/98 is a minor patch to clean-up the tests to try and solve this issue, but I'm not seeing the reported bug as I see all metadata sent across the wire.
comment:6 Changed 7 years ago by
Resolution: | → worksforme |
---|---|
Status: | pending → closed |
If I had time I would completely refactor this code to use modern Dojo best practices, instead of a sort of mix of pre-AMD and AMD.
That said, it looks like all that is needed here is:
Does that seem correct?