Closed
Bug 107598
Opened 23 years ago
Closed 23 years ago
Filter UI: newly created folder not selected in filter action(IMAP) - Reopened for prefill case
Categories
(MailNews Core :: Filters, defect, P2)
MailNews Core
Filters
Tracking
(Not tracked)
VERIFIED
FIXED
mozilla1.0
People
(Reporter: laurel, Assigned: naving)
References
Details
(Keywords: regression)
Attachments
(1 file, 1 obsolete file)
15.80 KB,
patch
|
Bienvenu
:
review+
|
Details | Diff | Splinter Review |
Using oct29 commercial trunk When creating a new folder through the filter rules dialog, it used to be (in 0.9.4 branch) that the newly created folder was displayed as the selected destination folder in the filter action (when action set to Move to Folder). Now the selection doesn't change to reflect the new folder. 1. Edit|Message Filters, click New to open new filter rules dialog. 2. (Filter action is by default set to Move to Folder.) Click New Folder, select a parent and create a new folder. Result: After successful folder creation, filter action dropdown still shows default (account level) as selection. The user must select the new folder from the dropdown (adds an extra step). Expected: Filter action dropdown should change to show the new folder as the selection.
Keywords: regression
Assignee | ||
Comment 1•23 years ago
|
||
*** Bug 107996 has been marked as a duplicate of this bug. ***
Assignee | ||
Comment 2•23 years ago
|
||
It works fine for pop3. For imap, the folder is not immediately created , it takes a while to create the folder on the server and when the server notifies us then we create it locally. That is why it doesn't show up in the picker.
Assignee | ||
Updated•23 years ago
|
Summary: Filter UI: newly created folder not selected in filter action → Filter UI: newly created folder not selected in filter action(IMAP)
Comment 3•23 years ago
|
||
1. Creating folders is only possible on the first stage of the folder hierarchy (IMAP) while selecting a folder is possibple throughout the whole hierarchy.. 2. In Filter-Panel, folder list is not updated (build 2001112013). First you have to leave filter-creation-panel, then the new folder appears in the folder-list in mail-window, then starting filter-creation again shows the new folder in the move-to-list. Are you sure the filter-creation-panel is listening to the imap-server and the imap-server sends a message to all connection clients, that a new folder was created by itself? (I'm no expert in imap, but perhaps you have to query the IMAP-Server if there are new folders?)
Assignee | ||
Comment 4•23 years ago
|
||
IMAP folders are not created immediately so it does not stick to the picker. I think we can just have this as release note.
Keywords: relnote
Updated•23 years ago
|
Status: NEW → ASSIGNED
Target Milestone: --- → mozilla1.2
Assignee | ||
Updated•23 years ago
|
Assignee | ||
Updated•23 years ago
|
Assignee | ||
Comment 7•23 years ago
|
||
The fix is to createClient subfolder on the ui thread so that we can immediately set it on the picker. It doesn't hurt us because chances of folder creation are pretty high and also we take care of duplication problem before sending command to the server (this also fixes bug 45041). If folder creation failed on server for some reason we delete the client folder and if it succeeds we set the flag correctly in PossibleImapMailbox.
Assignee | ||
Comment 8•23 years ago
|
||
cc bienvenu for review
Assignee | ||
Comment 9•23 years ago
|
||
Also I cleaned duplicate error msgs code and added one duplicate msg to base.
Comment 10•23 years ago
|
||
I don't see us freeing this: + char* uriStr = ToNewCString(uri); As you can imagine, I'm not crazy about this approach of assuming success and trying to clean up in the case of failure. "chances of folder creation are pretty high" scares me a bit, as does the fact that we might be ignoring the whole mod-utf7 encoding issue both when creating the folder, and handling the case where the folder creation fails. It could all work fine, but it would require lots of testing, and I'm not sure there isn't some little hack that could fix the filter ui without changing the imap folder creation backend. The main things to test are folders where the online name and the folder name end up being different, i.e., the folders that require mod-utf7 encoding. The simplest case is folders with '&''s in them, although any non 7-bit ascii character will work as well. If you try to create a folder with a name like that, can we still load the folder correctly? If the creation of such a folder fails, do we find the right sub-folder name in the oncreatefailed method? Let me know if you need more info about mod utf7 encoding and what it means.
Assignee | ||
Comment 11•23 years ago
|
||
Good point, I forgot about modutf7. Actually it could be done in this way but the problem becomes real complicated with other servers hierarchyDelimiter and '/'. we just keep changing back and forth !!! I have thought about another way to fix it by notifying FE that folder has been created.
Assignee | ||
Comment 12•23 years ago
|
||
Add folderCreateFailed/Completed event that will tell us when imap folder creation is done and so we can set the name on the picker. We need to add folderListener to filterEditor and I copied SetBusyCursor from mail3Pane js code because i didn't want to include entire js script file. patch also includes alert clean-up changes.
Attachment #68980 -
Attachment is obsolete: true
Comment 13•23 years ago
|
||
Comment on attachment 69163 [details] [diff] [review] proposed fix r OR (not AND) sr = bienvenu, whichever one you need more. Seems like marginally more trouble than the original approach, but seems like the right thing to do.
Attachment #69163 -
Flags: review+
Comment 14•23 years ago
|
||
Comment on attachment 69163 [details] [diff] [review] proposed fix I don't think you need the following in SetBusyCursor() as they are not relevant here. + var numFrames = window.frames.length; + for(var i = 0; i < numFrames; i++) + SetBusyCursor(window.frames[i], enable); Remove those lines and try your patch again. r=bhuvan
Assignee | ||
Comment 15•23 years ago
|
||
fix checked in w/ bhuvan's suggestion
Reporter | ||
Comment 16•23 years ago
|
||
Using feb22 commercial trunk build: OK when using new folder in filter ui launched via Edit|Message Filters. Not OK when using new folder in filter rules launched via prefill/"create filters" (prefill case does indeed work for pop/local folders). Reopening for prefill/"create filters" case, appended to summary to reflect current issue.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Summary: Filter UI: newly created folder not selected in filter action(IMAP) → Filter UI: newly created folder not selected in filter action(IMAP) - Reopened for prefill case
Assignee | ||
Comment 17•23 years ago
|
||
I am also seeing it for "Create Filter.. " case.
Status: REOPENED → ASSIGNED
Target Milestone: mozilla0.9.9 → mozilla1.0
Reporter | ||
Comment 18•23 years ago
|
||
Navin talked to me and decided to close this report and open a separate one for the create filter/prefill case...see bug 127370. marking this one verified
Status: ASSIGNED → RESOLVED
Closed: 23 years ago → 23 years ago
Resolution: --- → FIXED
Updated•20 years ago
|
Product: MailNews → Core
Updated•16 years ago
|
Product: Core → MailNews Core
You need to log in
before you can comment on or make changes to this bug.
Description
•