nsFileWidget::PutFile() should not return

VERIFIED FIXED

Status

SeaMonkey
General
P3
normal
VERIFIED FIXED
18 years ago
9 years ago

People

(Reporter: jefft, Assigned: Stuart Parmenter)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

18 years ago
nsFileWidget::PutFile() should not return with nsFileDlgResults_Cancel if file
name is a directory name or empty. It should remain in the gtk_main() loop until
we have a valid file name produced. As a client there is no way to tell whether
user hit cancel button or user selected a directory.
(Reporter)

Comment 1

18 years ago
Also, when switching directories the default file name string gets wipped out.
This is extremely bad.

Comment 2

18 years ago
reassigning to pavlov for triage
Assignee: trudelle → pavlov
(Reporter)

Updated

18 years ago
Depends on: 31096

Comment 3

18 years ago
One defect per bug report, please.  The second problem you mention is a dup, so 
let's just ignore it here.
(Assignee)

Comment 4

18 years ago
this is the case with nsIFilePicker on unix.  nsFileWidget is depriciated and
should not be used.
Status: NEW → RESOLVED
Last Resolved: 18 years ago
Resolution: --- → FIXED
(Reporter)

Comment 5

18 years ago
Closing this bug down as fix doesn't help me. I need more information on how to 
use nsIFilePicker and how to instantiate it. nsIFilePicker.h doesn't say much. 

Comment 6

18 years ago
moving from architecture product to Browser. Architecture product is going
away.
Component: Widget → Browser-General
Product: Architecture → Browser
Version: 5.0 → other

Comment 7

18 years ago
*spam* changing qa contact from nobody@mozilla.org to me (BlakeR1234@aol.com) 
on 121 open or resolved (but not verified) bugs.  sorry for the spam everybody, 
but most of these bugs would just remain dormant and not checked by QA 
otherwise.  I'm not sure how so many bugs have nobody as their QA contact, but 
I suspect this is the fault of some sort of bugzilla corruption that happened 
at some point (most of these bugs are in the 20000-26000 range, and I don't see 
where in the activity log that QA contact explicitly changed to 
nobody@mozilla.org)

Anyways, sorry again for spam.  If you really get annoyed, I'm usually 
available in #mozilla on IRC for torture.
QA Contact: nobody → BlakeR1234

Comment 8

18 years ago
QA assigning to doron to verify
Keywords: verifyme
QA Contact: blakeross → doronr
(Reporter)

Comment 9

17 years ago
mailnews is now client of nsIFilePicker. Marking this as verified.
Status: RESOLVED → VERIFIED
Product: Browser → Seamonkey

Updated

9 years ago
Keywords: verifyme
You need to log in before you can comment on or make changes to this bug.