Closed Bug 303737 Opened 19 years ago Closed 19 years ago

Feedview: The date is incorrect

Categories

(Firefox Graveyard :: RSS Discovery and Preview, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED INVALID
Firefox1.5

People

(Reporter: erik, Assigned: myk)

References

Details

Attachments

(1 file)

The shown date is incorrect in some case because the day part is done in UTC but the hours and minutes are done using local time I found this bug because a post had this GMT string: Wed, 13 Jul 2005 23:25:20 +0000 parsing this using js Date gives: Thu Jul 14 2005 01:25:20 GMT+0200 so the Feedview displays: Wed, 13 Jul @ 01:25 I guess this bug will be marked as invalid because the date time needs to be localized anyway. But until the date time is is localized this is indeed a valid bug.
Dupe of bug 303647, no?
(In reply to comment #0) can you give the url or attach a testcase Erik ? (In reply to comment #1) > Dupe of bug 303647, no? I'm not sure it is
> (In reply to comment #1) > > Dupe of bug 303647, no? > I'm not sure it is It's not - bug 303647 is about the format used to display the date/time, not the actual value of the date/time.
Attached file Test case (deleted) —
This is not a duplicate 303647 since 303647 is about improving the display of the date. This bug is that it shows an incorrect date. Hopefully, by fixing the localization issues of the date time display this bug can be closed.
Component: General → RSS Discovery and Preview
Version: unspecified → Trunk
Flags: blocking1.8b4?
OS: Windows Server 2003 → All
Hardware: PC → All
Target Milestone: --- → Firefox1.1
Assignee: nobody → myk
Flags: blocking1.8b4? → blocking1.8b4+
Does the checkin from bug 304362 comment 10 improve or fix this?
minusing, we're backing away from this feature now.
Flags: blocking1.8b4+ → blocking1.8b4-
(In reply to comment #5) No, the fix for bug 304362 does not fix this. The following code is in there and it is logically flawed because it uses GMT to get the day but local time to get the hours and minutes and this leads to errors as I've explained before 70 var day = d.toGMTString(); 71 day = day.substring(0, 11); 72 73 function padZeros(num) { 74 return num < 10 ? "0" + num : num; 75 } 76 divs[i].getElementsByTagName("span")[0].textContent = 77 day + " @ " + padZeros(d.getHours()) + ":" + padZeros(d.getMinutes()); This needs to either switch to use d.toString() or go the whole way and localize the date.
I have http://charter.msn.com/ as a homepage. The date still reads Wed Sept 28th I have to use IE to see current home page. (ick:) Thanks Teri
Closing: Son of Feedview will have a real parser that shouldn't have this particular mistake.
Status: NEW → RESOLVED
Closed: 19 years ago
Resolution: --- → INVALID
Resetting QA Contact to default.
QA Contact: general → rss.preview
Product: Firefox → Firefox Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: