Closed
Bug 6680
Opened 26 years ago
Closed 26 years ago
[PP] Mac only - date is not displayed in date column of Messenger
Categories
(MailNews Core :: MIME, defect, P3)
Tracking
(Not tracked)
VERIFIED
FIXED
M7
People
(Reporter: pmock, Assigned: tague)
Details
(Whiteboard: fix ready to check in as soon as the tree reopens)
Build:
Mac May 18 Seamonkey build (1999051812) installed on PPC 8500/180 and Duo 2300c
running Mac OS 8.5.1
Problem: Failed smoke test. When you select a mail folder to open it, it fails
to displays threads in the Thread pane. You can not see or selects messages to
be displayed
This problem occurs in both the mozilla and commercial builds
Note: This problem did not occur in the May 17 Mac build (1999051708-m6)
Steps to reproduce problem:
1) Launch todays 1999051812-m6 build
2) Open Messaenger by selecting the Task|Messenger menu option
3) Double click on the mail server name
I was using "nsmail-2"
The folders are expanded.
4) Select the Inbox or folder that contains messages
No messages are displayed in the Thread Window. You can select any messages
to be displayed.
Sorry, typo in step 4). It should read, you can NOT select any messages to be
displayed
5/18 Win32 and Linux builds are fine.
Priority: P3 → P1
Summary: Regression-smoke test-threads do not display in the threads pane → [PP] Regression-smoke test-threads do not display in the threads pane
Updated•26 years ago
|
Assignee: rhp → putterman
Priority: P1 → P3
Summary: [PP] Regression-smoke test-threads do not display in the threads pane → Regression-smoke test-threads do not display in the threads pane
Comment 3•26 years ago
|
||
Sorry, this isn't me...I only get involved once you can select a message.
- rhp
Removing the Mozilla Registry file and MSF files did not help. The behavior is
consistent on both of my test Macs.
Comment 6•26 years ago
|
||
If I had to take a guess,I'd start looking at rdf and changes that may effect
how we build up that view. I'm scanning through the checkin logs now to see if
I see something suspicious.
Comment 7•26 years ago
|
||
Peter is the problem just for local mail folders? can you display an imap
folder? I'm wondering if it is a local specific or more generic problem.
Comment 8•26 years ago
|
||
I am still trying to build the tree on my Mac to take a look a this problem...
Updated•26 years ago
|
Assignee: putterman → tague
Comment 9•26 years ago
|
||
We know what the problem is and we checked in a fix to fix the crash.
The Mac version of FormatTMTime is returning an failure code and that was
causing me to return a wrong value to RDF. I made it so that if I get a failure
code I print an empty date string. So now the date won't show up, but at least
it won't crash. The failure is because of:
res = nsComponentManager::CreateInstance(kMacLocaleFactoryCID, NULL,
kIMacLocaleIID, (void**)&macLocale);
in nsDateTimeFormatMac.cpp
It also looks like the date will be incorrect. Reassigning to tague who I think
wrote the factory code and cc'ing nhotta.
Comment 10•26 years ago
|
||
Also, in FormatTMTime in nsDateTimeFormatMac.cpp, we noticed that there are
multiple declarations of the variable res. This probably doesn't affect this
bug, but it may be a problem later on.
Comment 11•26 years ago
|
||
>multiple declarations of the variable res
Checked in nsDateTimeFormatMac.cpp to fix the multiple declarations.
Comment 12•26 years ago
|
||
I'm marking this target milestone M6 since we can't test much of mail w/out
seeing the threads in the thread pane.
Comment 13•26 years ago
|
||
Who marked this M7? We can't test the Mail on Mac very well when we can't
display a list of messages? Pls change this back to M6 or state a reason why
this is M7 target milestone. Thanks.
Comment 14•26 years ago
|
||
With my changes you'll actually be able to view the thread pane. No date will
show up. However, I think it would be good to make the date show up for M6.
Comment 15•26 years ago
|
||
Thanks for the clarification. (Note: there was never any crash on this bug.)
Once we verify this "fix" for tomorrow's build (5/19), we'll change the severity
and the summary of this bug to reflect the date missing from the thread pane's
list of messages.
Comment 16•26 years ago
|
||
the Date missing issue should be add to bug 5492 as it seems to be related (http://bugzilla.mozilla.org/
show_bug.cgi?id=5492)
Comment 17•26 years ago
|
||
If this is a fix approved for M6, then please change the Target Milestone to M6,
not M7. Thanks!
Status: NEW → RESOLVED
Closed: 26 years ago
Resolution: --- → FIXED
Whiteboard: fix checked in for M6 build
Target Milestone: M7 → M6
Assignee | ||
Comment 18•26 years ago
|
||
checked in a fix -- 12:30pm 5/18/99.
Comment 19•26 years ago
|
||
Laurel - can you help verify this bug using tomorrow's 5/21 builds? This bug
has turned into verifying that you can see the date of messages in the thread
pane. Thanks.
Reporter | ||
Comment 20•26 years ago
|
||
In the Mac May 21 Seamonkey build (1999052108), dates are still not being
displayed in the thread pane.
I do see threads in the thread pane so my initial report of this bug has been
corrected.
Comment 21•26 years ago
|
||
Tague,
It doesn't look like you checked anything in for this bug. It looks like you
just added a couple of newlines.
Severity: blocker → normal
Status: RESOLVED → REOPENED
Summary: Regression-smoke test-threads do not display in the threads pane → [PP] Mac only - date is not displayed in date column of Messenger
Whiteboard: fix checked in for M6 build → Date is still not displaying in the Mac May 21 build.
Reporter | ||
Comment 22•26 years ago
|
||
Per Scott's last comment, I am re-openning this bug.
Updating summary to reflect actual bug:
Old summary: Regression-smoke test-threads do not display in the threads pane
New summary: [PP] Mac only - date is not displayed in date column of Messenger
In a e-mail exchange, Putterman wrote:
"I added code to use the I18N DateTimeFormat. It turns out that this interface
could not be created on the Mac due to factory problems. We couldn't see
messages in the thread pane because of this. I made it so that messages showed
up, but the date wouldn't get shown because we couldn't access this interface.
Tague's fix was supposed to make it so that this interface could be gotten from
the nsLocaleFactory. Either his could still isn't working, or there is another
problem ... ... Although this is annoying, I wouldn't call it an M6 stopper.
But we should get it fixed. "
Changing the Target Milestone from M6 to M7.
Changing the Severity from Blocker to Normal.
Comment 23•26 years ago
|
||
Tague - are you going to fix this for M6? If not, let me know. I'd like to ask
verah to release note this for M6.
Assignee | ||
Comment 24•26 years ago
|
||
No, M6 isn't going to get held for this. It will be in M7.
Comment 25•26 years ago
|
||
<Updated verah on release notes for M6.>
Status: REOPENED → ASSIGNED
Whiteboard: Date is still not displaying in the Mac May 21 build. → fix ready to check in as soon as the tree reopens
Status: ASSIGNED → RESOLVED
Closed: 26 years ago → 26 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 26•26 years ago
|
||
checked i a new fix for this today (5/26/99) - should be in tomarrows build.
Comment 27•26 years ago
|
||
OK using 1999052808 on Mac PPC OS 8.51
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
•