1192744 - [Message]Set device time format to '24-hour', but the time format in message report screen still shows as '12-hour' format.

Status: VERIFIED FIXED

(Firefox OS Graveyard :: Gaia::SMS, defect)

Description

[Shine(Leave from Mozilla)]

Attachment: Aries_KK_2.5_video1.3gp

[1.Description]:
[Aries KK 2.5][Flame KK 2.5][Message]Set device time format to '24-hour', launch message to check the time format in every screen, but the time format in message report screen still show as '12-hour' format.
See attachment: Aries_KK_2.5_video1.3gp & logcat_Aries_KK_2.5.txt
Found time: 11:36

[2.Testing Steps]: 
Precondition:. Have some incoming and outgoing messages(SMS & MMS) in Messages app. 
1. Launch 'Settings' app -> 'Date & Time' -> 'Time Format', change to '24-hour'.
2. Launch Messages app
3. Open a message thread. 
4. Long tap a message and select 'View message report'.

[3.Expected Result]: 
4. The time format in message report screen could show as '24-hour' format.

[4.Actual Result]: 
4. But the time format in message report screen still shows as '12-hour' format.

[5.Reproduction build]: 
Device: Aries KK 2.5(Affected)
Build ID               20150806203229
Gaia Revision          7f387f859d48f9ad0761637c78447dc524747738
Gaia Date              2015-08-06 15:21:13
Gecko Revision         https://hg.mozilla.org/mozilla-central/rev/03e3d77d1b6b
Gecko Version          42.0a1
Device Name            aries
Firmware(Release)      4.4.2
Firmware(Incremental)  eng.worker.20150806.195600
Firmware Date          Thu Aug  6 19:56:08 UTC 2015
Bootloader             s1

Device: Flame KK 2.2(Unaffected)
Flame KK 2.2
Build ID               20150809032505
Gaia Revision          102f1299e9eafe3760e1deb44d556b5c4f36b5af
Gaia Date              2015-08-06 20:46:56
Gecko Revision         https://hg.mozilla.org/releases/mozilla-b2g37_v2_2/rev/da29b5af4232
Gecko Version          37.0
Device Name            flame
Firmware(Release)      4.4.2
Firmware(Incremental)  eng.cltbld.20150809.065320
Firmware Date          Sun Aug  9 06:53:31 EDT 2015
Firmware Version       v18D v4
Bootloader             L1TC000118D0

Device:  Flame KK 2.5 (Affected)
Build ID               20150809150206
Gaia Revision          3e5271663e7ef26290c29a45d2e42c0d3c20fe04
Gaia Date              2015-08-08 21:13:59
Gecko Revision         https://hg.mozilla.org/mozilla-central/rev/fd69d51a4068
Gecko Version          42.0a1
Device Name            flame
Firmware(Release)      4.4.2
Firmware(Incremental)  eng.cltbld.20150809.182413
Firmware Date          Sun Aug  9 18:24:24 EDT 2015
Firmware Version       v18D v4
Bootloader             L1TC000118D0

[6.Reproduction Frequency]: 
Always Recurrence,10/10

[7.TCID]: 
14359

Comment 1

[Shine(Leave from Mozilla)]

Attachment: logcat_Aries_KK_2.5.txt

Comment 2

[Johan Lorenzo [:jlorenzo] - On PTO, back on May 13th]

[Blocking Requested - why for this release]: Functional regression.

Comment 3

[Julien Wajsberg [:julienw] (PTO -> may 13)]

Hey gandalf, I think this is coming from the patch to move to Intl.
Do you think you can take this? Or at least give pointers?

Comment 4

[GitHub Autolander Bot]

Attachment: [gaia] zbraniecki:1192744-use-hour12-in-message-report > mozilla-b2g:master

Comment 5

[Zibi Braniecki [:zbraniecki][:gandalf]]

Comment on attachment 8652633 [details] [review]
[gaia] zbraniecki:1192744-use-hour12-in-message-report > mozilla-b2g:master

Yeah, we just need to use mozHour12 when formatting this.

Comment 6

[Julien Wajsberg [:julienw] (PTO -> may 13)]

Comment on attachment 8652633 [details] [review]
[gaia] zbraniecki:1192744-use-hour12-in-message-report > mozilla-b2g:master

Hey Zibi,

Thanks a lot for this patch.

However this change doesn't work in all situations, especially this STR doesn't work:

1. open the SMS app.
2. open the Settings app, go to Date and time panel, and change the Time format
3. return to the SMS app, go to a conversation, long press a message and display the report
=> the format is wrong

More information on Github :)

Please request review again once you're ready !

Comment 7

[Zibi Braniecki [:zbraniecki][:gandalf]]

Comment on attachment 8652633 [details] [review]
[gaia] zbraniecki:1192744-use-hour12-in-message-report > mozilla-b2g:master

Good recommendation :) I'll make sure to be suspicious next time I see tests green too soon ;)

Comment 8

[Julien Wajsberg [:julienw] (PTO -> may 13)]

Comment on attachment 8652633 [details] [review]
[gaia] zbraniecki:1192744-use-hour12-in-message-report > mozilla-b2g:master

as perfect as it can be !
r=me

Thanks again for this fix !

Comment 9

[Zibi Braniecki [:zbraniecki][:gandalf]]

Commit: https://github.com/mozilla-b2g/gaia/commit/4f4498e1a73f83598b1fe57952e2276f38281887

Comment 10

[Shine(Leave from Mozilla)]

Attachment: Verify_Pass_AriesKK2.5.3gp

This problem is verified as "pass" on latest build of Aries KK v2.5 and Flame KK v2.5 by the STR in comment 0.
Actual result: Set device time format to '24-hour', the time format in message report screen will be shown as '24-hour' format.
See attachment: Verify_Pass_AriesKK2.5.3gp
Rate: 0/5

Device information:
Aries KK v2.5: (Pass)
Build ID               20150901190039
Gaia Revision          c2582f4be03cd12124b96a263c8d14c774f0ffe4
Gaia Date              2015-09-01 17:03:45
Gecko Revision         https://hg.mozilla.org/mozilla-central/rev/e47423c019643792e6de894cfcee598dead4d3ba
Gecko Version          43.0a1
Device Name            aries
Firmware(Release)      4.4.2
Firmware(Incremental)  eng.worker.20150901.182105
Firmware Date          Tue Sep  1 18:21:13 UTC 2015
Bootloader             s1

Flame KK v2.5: (Pass)
Build ID               20150901150204
Gaia Revision          c2582f4be03cd12124b96a263c8d14c774f0ffe4
Gaia Date              2015-09-01 17:03:45
Gecko Revision         https://hg.mozilla.org/mozilla-central/rev/e47423c019643792e6de894cfcee598dead4d3ba
Gecko Version          43.0a1
Device Name            flame
Firmware(Release)      4.4.2
Firmware(Incremental)  eng.cltbld.20150901.182949
Firmware Date          Tue Sep  1 18:30:04 EDT 2015
Firmware Version       v18D v4
Bootloader             L1TC000118D0