Closed Bug 562079 Opened 14 years ago Closed 14 years ago

Update dates shown in the diggest and expanded diggest view are one day in advance

Categories

(Toolkit :: Add-ons Manager, defect, P1)

Product:
Toolkit
Component:
Add-ons Manager
Type:
defect

Tracking

()

Status:
VERIFIED FIXED
Milestone:
mozilla1.9.3a5

People

(Reporter: whimboo, Assigned: Unfocused)

References

Details

(Whiteboard: [rewrite]) 

Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.5; en-US; rv:1.9.3a5pre) Gecko/20100426 Minefield/3.7a5pre

When you check the date when an addon has been installed or updated, you will notice that the date which is shown is one day in advance. This should probably an API related bug.

Steps:
1. Install an add-on
2. Check the date right beside the rating (diggest view) or at the top-right (expanded diggest view)
UI issue in the formatting code.
Assignee: nobody → bmcbride
Blocks: 550048
No longer blocks: 461973
Wow, that was stupid of me.
http://hg.mozilla.org/projects/addonsmgr/rev/aad24fcf0863
Whiteboard: [rewrite] → [rewrite][fixed-in-addonsmgr]
Whiteboard: [rewrite][fixed-in-addonsmgr] → [rewrite][fixed-in-addonsmgr][needs-review]
r=me
Whiteboard: [rewrite][fixed-in-addonsmgr][needs-review] → [rewrite][fixed-in-addonsmgr]
Landed on trunk as a part of bug 554007
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Whiteboard: [rewrite][fixed-in-addonsmgr] → [rewrite]
Target Milestone: --- → mozilla1.9.3a5
Verified fixed with Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.6; en-US; rv:1.9.3a5pre) Gecko/20100514 Minefield/3.7a5pre ID:20100514030623

Blair, I think this is wonderful testable with an automated test. I would like to leave out the Litmus test. What do you think?
Status: RESOLVED → VERIFIED
Flags: in-testsuite?
Flags: in-litmus?
Hmm, yes - should be able to automate this with the mocking provider, and setting the update date of an addon to something that we know the formatted string of.
Flags: in-litmus? → in-litmus-
browser_list.js and browser_details.js verify this
Flags: in-testsuite? → in-testsuite+
You need to log in before you can comment on or make changes to this bug.