Cancelling publish doesn't really cancel currently executing uploads

VERIFIED FIXED

Status

SeaMonkey
Composer
VERIFIED FIXED
15 years ago
14 years ago

People

(Reporter: jag (Peter Annema), Assigned: jag (Peter Annema))

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [adt2])

Attachments

(1 attachment, 1 obsolete attachment)

(Assignee)

Description

15 years ago
In testing for another bug I found that calling cancel while we're publising a
document doesn't really cancel it, the upload keeps going till it's done or
Mozilla gets closed (whichever happens first). There seem to be two problems.
One is that in the JS for the dialog we don't have access to the
webbrowserpersist object to call cancel on it, the other is that with that fixed
(patch coming up) the backend still doesn't cancel any channels open for this
(these) upload(s).
(Assignee)

Comment 1

15 years ago
Created attachment 120171 [details] [diff] [review]
Init gPersistObj so we can call cancelSave on it when cancel is pressed
(Assignee)

Comment 2

15 years ago
See bug 201610 for the backend problem.
(Assignee)

Comment 3

15 years ago
Created attachment 120175 [details] [diff] [review]
Alternate approach, return the persist obj from StartPublishing

Comment 4

15 years ago
Comment on attachment 120175 [details] [diff] [review]
Alternate approach, return the persist obj from StartPublishing

r=ssu
Attachment #120175 - Flags: review+
(Assignee)

Updated

15 years ago
Attachment #120171 - Attachment is obsolete: true
Attachment #120175 - Flags: superreview+

Comment 5

15 years ago
moa=brade
(Assignee)

Comment 6

15 years ago
Checked the patch in, waiting for bug 201610 to be fixed.
Depends on: 201610

Comment 7

15 years ago
Editor triage team: nsbeta1+/adt2

Marking fixed since jag fixed the front-end issue and has filed 201610 for the
back-end issue.
Status: NEW → RESOLVED
Last Resolved: 15 years ago
Keywords: nsbeta1 → nsbeta1+
Resolution: --- → FIXED
Whiteboard: [adt2]

Comment 8

15 years ago
I'm not sure how to verify this one since bug 201610 hasn't been resolved yet.

Comment 9

15 years ago
From my testing with the 2003-05-23-09 Win32 and 2003-05-23-07 Macho trunk
builds,  I was able to successfully cancel the ftp publishing process in Composer.
Status: RESOLVED → VERIFIED
Product: Browser → Seamonkey
You need to log in before you can comment on or make changes to this bug.