725278 - Inconsistent submitted date between query comment tab and crash report

Status: VERIFIED FIXED

(Socorro :: Webapp, task)

Description

[Scoobidiver (away)]

If you take a look at the bottom comment, the submitted date is 2012-02-21 05:49:45+00 (in the future!) while it's 2012-02-07 05:52:06 in the crash report:
https://crash-stats.mozilla.com/report/list?range_value=7&range_unit=days&date=2012-02-08&signature=je_free%20|%20PL_DHashFreeTable&version=Firefox%3A9.0.1
bp-ea28ff90-1941-4cc0-9e91-14cc22120207

Comment 1

[Robert Kaiser]

Sounds to me like this date may come from the user's machine and this person has a wrong clock setting.

Comment 2

[[DEACTIVATED] Adrian Gaudebert]

Indeed, the SQL query generating this report uses the 'client_crash_date' field. I suppose we should change it to 'date_processed' so it makes more sense?

Comment 3

[Robert Kaiser]

Yes, it would. Thought I think it might be nice to actually have that client date available somewhere in the report details, but let's keep the comments section to the date we are also using elsewhere, and that's the processed one.

Comment 4

[[DEACTIVATED] Adrian Gaudebert]

Opened https://github.com/mozilla/socorro/pull/331 for this problem, and filed bug 725383 for displaying the client_crash_date somewhere in the UI.

Comment 5

[[DEACTIVATED] Adrian Gaudebert]

Steps to QA
-----------

1. Go to advanced search and run a search (any parameters)
2. Click on a signature to go to report/list
3. Open the Comments tab
4. Click on the datetime to go to report/index, and verify that the previously clicked date is the same as the one displayed in the Date Processed field of report/index. 
5. Repeat this process until you are satisfied.

Comment 6

[[github robot]]

Commit pushed to master at https://github.com/mozilla/socorro

https://github.com/mozilla/socorro/commit/82126387174a476065967145b5d4cf51c2064d43
Merge pull request #331 from AdrianGaudebert/725278-comment-date

Fixes bug 725278 - Used date_processed instead of client_crash_date when...

Comment 7

[Matt Brandt [:mbrandt]]

QA verified on dev. The reports are now using the date from 'date_processed' instead of 'client_crash_date.'