Ticket #10190 (open feature)
"upload" property of the XMLHttpRequest
| Reported by: | superbulldeng@… | Owned by: | |
|---|---|---|---|
| Priority: | high | Milestone: | 1.next |
| Component: | ajax | Version: | 1.6.3 |
| Keywords: | 1.8-discuss | Cc: | jaubourg |
| Blocking: | Blocked by: |
Description (last modified by rwaldron) (diff)
There is a "upload" property in the xhr object returned by new XMLHttpRequest() in supported browser. But the object returned by $.ajax() seems didn't contain this property. This property is important to get the state of file upload progress.
Change History
comment:1 Changed 21 months ago by addyosmani
- Cc jaubourg added
- Priority changed from undecided to low
- Component changed from unfiled to ajax
- Type changed from feature to enhancement
comment:2 Changed 20 months ago by timmywil
- Status changed from new to open
- Milestone changed from None to 1.next
Marking as valid for 1.next
comment:3 Changed 18 months ago by malsup
- Priority changed from low to high
This is really a fairly significant omission. As stated above, it's impossible to track FormData file upload progress without this property exposed. The jqXHR object is documented as a "superset of the browser's native XMLHttpRequest object". The upload property is but one example where this is simply not true, with respect to XMLHttpRequest Level 2. Either all properties should be exposed or there must be a way to access to the native XHR.
comment:5 Changed 18 months ago by jaubourg
- Description modified (diff)
-1, There are ways to access the native xhr (by overriding options.xhr in a prefilter). If we want upload in jQuery, then we'd better implement an upload method rather than try and have ajax be even more bloated.
comment:6 Changed 18 months ago by jzaefferer
+1, "overriding options.xhr in a prefilter" is not good enough - at least document how to access non-exposed properties in a sane way as part of the existing docs (see also Mike's comment)
comment:7 Changed 18 months ago by dmethvin
- Description modified (diff)
+1, I'd like a better/simpler way to do this as well
comment:12 Changed 9 months ago by dmethvin
- Type changed from enhancement to feature
Bulk change from enhancement to feature.
comment:13 Changed 7 months ago by timmywil
- Keywords 1.8-discuss, needsdocs added; 1.8-discuss removed
comment:14 Changed 7 months ago by mikesherov
- Keywords 1.8-discuss added; 1.8-discuss, needsdocs removed
F you, Timmy
Please follow the bug reporting guidlines and use jsFiddle when providing test cases and demonstrations instead of pasting the code in the ticket.