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)

PowerPC
Mac System 8.5
defect

Tracking

(Not tracked)

VERIFIED FIXED

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
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
Sorry, this isn't me...I only get involved once you can select a message. - rhp
Adding Jeff to cc: list
Removing the Mozilla Registry file and MSF files did not help. The behavior is consistent on both of my test Macs.
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.
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.
I am still trying to build the tree on my Mac to take a look a this problem...
Assignee: putterman → tague
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.
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.
>multiple declarations of the variable res Checked in nsDateTimeFormatMac.cpp to fix the multiple declarations.
Target Milestone: M6
I'm marking this target milestone M6 since we can't test much of mail w/out seeing the threads in the thread pane.
Target Milestone: M6 → M7
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.
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.
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.
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)
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
checked in a fix -- 12:30pm 5/18/99.
QA Contact: 4080 → 4112
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.
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.
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.
Resolution: FIXED → ---
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.
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.
Target Milestone: M6 → M7
No, M6 isn't going to get held for this. It will be in M7.
<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 ago26 years ago
Resolution: --- → FIXED
checked i a new fix for this today (5/26/99) - should be in tomarrows build.
Blocks: 5934
No longer blocks: 5934
Status: RESOLVED → VERIFIED
OK using 1999052808 on Mac PPC OS 8.51
Product: MailNews → Core
Product: Core → MailNews Core
You need to log in before you can comment on or make changes to this bug.