Closed Bug 40360 Opened 24 years ago Closed 24 years ago

Filter UI: Filter rules dialog missing New Folder button/functionality.

Categories

(MailNews Core :: Filters, defect, P2)

defect

Tracking

(Not tracked)

VERIFIED FIXED
mozilla0.9

People

(Reporter: laurel, Assigned: gayatrib)

Details

(Whiteboard: [nsbeta1+])

Attachments

(1 file)

Using may23 m16 commercial build

The filter rules dialog is missing the New Folder button in its Action section.
The button should appear enabled only when Move To Folder is selected in the
action field.  When Action is set to some other value, we should disable the
button (this is what 4.x does).
Oh, and, when user engages the enable New Folder button we should launch to New
Folder dialog and underlying functionality.  If user creates a new folder
through this dialog it should immediately appear in the list of available
selections for the Move to Folder destinations.
QA Contact: lchiang → laurel
doh! fix is easy.
Status: NEW → ASSIGNED
Target Milestone: --- → M17
fixed.
Status: ASSIGNED → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
This isn't in jun7 m17 builds... will check again in a later build.
wow, I have NO idea why I marked this fixed.
I must have though it was another bug.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
mass moving to M18 and adding nsbeta3 keyword
Keywords: nsbeta3
Target Milestone: M17 → M18
Whiteboard: [nsbeta3-]
Target Milestone: M18 → Future
cc jglick for possible note in spec.
reassigning my filter bugs to gayatrib..
Assignee: alecf → gayatrib
Status: REOPENED → NEW
Keywords: nsbeta1
Keywords: nsbeta3
Whiteboard: [nsbeta3-]
Related problem if this is implemented: new folders don't appear in menus, see
bug 57257 .
Priority: P3 → P2
Whiteboard: [nsbeta1+]
Target Milestone: Future → mozilla0.8
marking nsbeta1+ and moving to mozilla0.9
moving to mozilla0.9
Target Milestone: mozilla0.8 → mozilla0.9
fix in hand. will post patch soon.
Status: NEW → ASSIGNED
Fix in hand. Attaching patch. Adding patch, review keywords. Adding racham for 
review and bienvenu for sr.
Keywords: patch, review
Attached patch patchSplinter Review
The patch does the following:
1) Add New Folder button to the filter editor dialog--so a person can create new 
subfolders in a folder while creating/editing filters.
2) As I attempt to use the already existing functionality provided by 
MsgNewFolder and CreateNewSubfolder methods... but run into scope resolution 
problems as I attempt to call a different callback function on "ok" in the new 
folder dialog, I modified these functions to take the callBackFunction to be 
called as a parameter--instead of using it the hard coded way it is now.
3)So i modify all the calls to these functions too.

Patch has been well tested.
Bhuvan, could you please take a look and review it?
David, can I get a sr?
you should add the newlines at the ends of those files to get rid of that diff
warning. Also, cc'ing Seth about the passing of a callback function name - is
that the right way of doing things? If it's OK with Seth, it's OK with me.
thanks david. I will add the newlines. Jean-Francois suggested to me to pass the 
name of the callback function to resolve the scope resolution. Adding him also 
to the cc list.
yes, it is ok to use the function name like that for the callback.
what happens in the case where the new folder creation fails?
(say a folder with that name already exists)
is the error handled properly?
Error handling is done properly. The New Folder dialog takes care of that error 
handling part. So when i try to create a folder with the same name, it comes 
back saying that mailbox already exists. etc.

Can I get an r/sr on this?
sr=bienvenu
r=racham
review done (by me) at gayatrib's desk. sorry for the confusion.

-racham
fix checked in.
Status: ASSIGNED → RESOLVED
Closed: 24 years ago24 years ago
Resolution: --- → FIXED
Button present and functional.
Any specific issues with new folder button functionality in filters will be
logged separately.

OK using feb26 commercial trunk builds: linux rh6.0, mac OS 9.0, win98
Status: RESOLVED → VERIFIED
Product: MailNews → Core
Product: Core → MailNews Core
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: