File Upload broken (SeaMonkey part)

RESOLVED FIXED in seamonkey2.0b1

Status

RESOLVED FIXED
10 years ago
9 years ago

People

(Reporter: neil, Assigned: neil)

Tracking

({regression})

Trunk
seamonkey2.0b1
regression
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

10 years ago
Since bug 412822 file upload is broken because it expects an nsIFileURL object.
(Assignee)

Comment 1

10 years ago
Created attachment 380219 [details] [diff] [review]
Proposed patch
[Checkin: Comment 2]
Assignee: nobody → neil
Status: NEW → ASSIGNED
Attachment #380219 - Flags: review?(timeless)

Updated

10 years ago
Attachment #380219 - Flags: review?(timeless) → review+
(Assignee)

Comment 2

10 years ago
Pushed changeset fbd1bea81d2b to comm-central.

Note that Upload File is probably still broken, but for other reasons.
Version: unspecified → Trunk
Blocks: 467524
Comment on attachment 380219 [details] [diff] [review]
Proposed patch
[Checkin: Comment 2]

>-  var leafName = fileURL.file.leafName;
>+  var leafName = fileURL.QueryInterface(CI.nsIFileURL).file.leafName;

Fwiw, you may have wanted to update the previous comment too:

2434   // generate the target URI.  we use fileURL.file.leafName to get the

Comment 4

9 years ago
Is this connected with bug 467524?

Could you either describe specifically what is still broken or close this bug and file a separate one for the remaining issue(s)?
(Assignee)

Comment 5

9 years ago
File Upload was broken for two reasons (both mentioned in bug 467524, and one fixed here) and I'll leave bug 457524 open for the other half of the bug.
Status: ASSIGNED → RESOLVED
Last Resolved: 9 years ago
Resolution: --- → FIXED
Attachment #380219 - Attachment description: Proposed patch → Proposed patch [Checkin: Comment 2]
Target Milestone: --- → seamonkey2.0b1
Summary: File Upload broken → File Upload broken (SeaMonkey part)
You need to log in before you can comment on or make changes to this bug.