Open
Bug 512507
Opened 15 years ago
Updated 14 years ago
Port Bug 484329 - Upgrading to 3.0 causes "archives" folder disappears
Categories
(SeaMonkey :: MailNews: General, defect)
SeaMonkey
MailNews: General
Tracking
(Not tracked)
NEW
People
(Reporter: InvisibleSmiley, Unassigned)
References
Details
Attachments
(1 file, 1 obsolete file)
(deleted),
patch
|
InvisibleSmiley
:
review-
|
Details | Diff | Splinter Review |
Fixing this bug should make sure that an Archives folder with a different case is used if present instead of hiding it from the UI. Final patch for the TB bug to be ported: <http://hg.mozilla.org/comm-central/rev/998e17e4acd6>
Reporter | ||
Comment 1•15 years ago
|
||
Straight port (only adjusted comments and bracing style).
Updated•15 years ago
|
Attachment #397656 -
Flags: review?(mnyromyr) → review-
Comment 2•15 years ago
|
||
Comment on attachment 397656 [details] [diff] [review]
proposed patch
>diff --git a/suite/mailnews/mailWindowOverlay.js b/suite/mailnews/mailWindowOverlay.js
>+ let identity;
Initialize identity.
>+ identity = getIdentityForHeader(msgs[0]);
My shell says "ReferenceError: getIdentityForHeader is not defined", because ours is named GetIdentityForHeader. Furthermore, the function takes a second parameter, which you should pass in null for here.
>+ let ciArchive = null;
>+ ciArchive = archiveFolder.server.rootFolder
>+ .getChildWithURI(archiveFolderUri, true, true);
Why assign null first and kill that right away again?!
Furthermore, all my tests regarding the problem claimed to be fixed failed...
Let's see how the discussion in bug 484329 comment 10 and following turns out.
Reporter | ||
Comment 3•15 years ago
|
||
(In reply to comment #2)
> Furthermore, all my tests regarding the problem claimed to be fixed failed...
> Let's see how the discussion in bug 484329 comment 10 and following turns out.
I don't want to forget this here so I'm attaching the updated patch, feel free to delay the review until you have an answer.
BTW: You may consider adding your nick (e.g. in parentheses with leading colon as seen elsewhere), maybe it helps people like WADA to recognize you. ;-)
Attachment #397656 -
Attachment is obsolete: true
Attachment #400382 -
Flags: review?(mnyromyr)
Reporter | ||
Updated•14 years ago
|
Assignee: jh → nobody
Status: ASSIGNED → NEW
Reporter | ||
Comment 4•14 years ago
|
||
Comment on attachment 400382 [details] [diff] [review]
patch v1a
Effectively, the last review result was r-. Updating attachment details to
reflect that, and unassigning myself since I'm not actively working on this.
Attachment #400382 -
Flags: review?(mnyromyr) → review-
You need to log in
before you can comment on or make changes to this bug.
Description
•