Ubuntu
qtubuntu package

clock forcefully switches from 24h to 12h AM/PM format once the panel clock is loaded

Bug #1256061 reported by Oliver Grawert
14
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Indicator Date and Time
Fix Released
High
Charles Kerr
indicator-datetime (Ubuntu)
Fix Released
Undecided
Unassigned
qtubuntu (Ubuntu)
Invalid
Critical
Unassigned
unity8 (Ubuntu)
Invalid
High
Unassigned

Bug Description

i initially had filed a bug against indicator-datetime but after checking all environment variables together with seb128 it appears there is no reason for this to happen and according to him it must be something unity8 or a QT component does to the LC_TIME or other locale settings.

Initially when booting in a german localized install i properly have a 24h display on teh lock screen ... a few seconds later (as soon as the panel clock is loaded) the clock switches to 12h display everywhere in the UI until recently that still showed AM/PM ... in one of the latest images (r27 and above) the AM/PM display is gone but the clock still switches to 12h display.

Tags: avengers

Related branches

lp:~saviq/unity8/fox-clock-formatting
lp:~charlesk/indicator-datetime/lp-1256061
PS Jenkins bot (community): Approve (continuous-integration)
Ted Gould (community): Approve
Diff: 383 lines (+271/-21)
6 files modified
debian/control (+3/-0)
src/CMakeLists.txt (+1/-1)
src/utils.c (+30/-17)
tests/CMakeLists.txt (+19/-3)
tests/glib-fixture.h (+120/-0)
tests/test-formatter.cc (+98/-0)
Michał Sawicz (saviq)
Changed in unity8 (Ubuntu):
status: New → Confirmed
Nicolas d'Offay (nicolas-doffay)
Changed in unity8 (Ubuntu):
assignee: nobody → Nicolas d'Offay (nicolas-doffay)
kevin gunn (kgunn72)
Changed in unity8 (Ubuntu):
importance: Undecided → High
Revision history for this message
Nicolas d'Offay (nicolas-doffay) wrote :

This is using Qt.formatTime().

Changed in unity8 (Ubuntu):
assignee: Nicolas d'Offay (nicolas-doffay) → nobody
affects: unity8 (Ubuntu) → qtubuntu (Ubuntu)
Revision history for this message
kevin gunn (kgunn72) wrote :

this is an image blocker (i didn't pick)

Changed in qtubuntu (Ubuntu):
importance: High → Critical
kevin gunn (kgunn72)
Changed in qtubuntu (Ubuntu):
assignee: nobody → Loïc Molinari (loic.molinari)
Michał Sawicz (saviq)
Changed in qtubuntu (Ubuntu):
status: Confirmed → Invalid
Changed in unity8:
status: New → In Progress
assignee: nobody → Michał Sawicz (saviq)
Revision history for this message
Michał Sawicz (saviq) wrote :

Did some more digging, and this is not a bug in qtubuntu at all. It was attributed to that, but in fact qtubuntu was only responsible for the *correct* time displayed at the beginning.

The indicator label is fully controlled by the indicator, though, and is always sent in the AM/PM 24h form. Whether it should at all be responsible for supplying this label is debatable (as opposed to the shell knowing this is a time label and updating it as needed), but still - fix for the greeter is attached, while the indicator still needs fixing.

Changed in qtubuntu (Ubuntu):
assignee: Loïc Molinari (loic.molinari) → nobody
Changed in unity8:
importance: Undecided → High
Revision history for this message
Michał Sawicz (saviq) wrote :

Oooohkay wait...

I just flashed my phone, installed the pl locale, selected it in the settings app, rebooted, and my clock is 24h...

Confirming now with bootstrap...

Revision history for this message
Michał Sawicz (saviq) wrote :

Ok, flashed with bootstrap, selected Deutsch, rebooted - 24h time for me everywhere.

Changed in indicator-datetime:
status: New → Invalid
Changed in unity8:
status: In Progress → Opinion
status: Opinion → Incomplete
Revision history for this message
Michał Sawicz (saviq) wrote :

Can anyone please confirm if this is not the case for them?

Revision history for this message
Michał Sawicz (saviq) wrote :

OK I misattributed missing AM/PM label to this actually being fixed.

Aaanyway. The time label for both the greeter and the panel comes pre-formatted from indicator-datetime directly. So no qtubuntu, no unity8 issue - indicator-datetime needs to fix the label it sends up.

Changed in indicator-datetime:
status: Invalid → New
Changed in unity8:
status: Incomplete → Invalid
Changed in indicator-datetime:
assignee: nobody → Charles Kerr (charlesk)
Changed in unity8:
assignee: Michał Sawicz (saviq) → nobody
Revision history for this message
Oliver Grawert (ogra) wrote :

i assume testing this bugs should only be done in the afternoon after 13:00 local time ;)

Revision history for this message
Ted Gould (ted) wrote :

Confirming that indicator-datetime is still sending the label. It's planned to change, but today it is:

$ gdbus call --session --dest com.canonical.indicator.datetime --object-path /com/canonical/indicator/datetime --method org.gtk.Actions.Describe phone-header
((true, signature '', [<{'title': <'Upcoming'>, 'visible': <true>, 'label': <' 8:07 AM'>, 'accessible-desc': <' 8:07 AM'>}>]),)

Changed in indicator-datetime:
importance: Undecided → High
status: New → Confirmed
Charles Kerr (charlesk)
Changed in indicator-datetime:
status: Confirmed → In Progress
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

Fix committed into lp:indicator-datetime at revision None, scheduled for release in indicator-datetime, milestone Unknown

Changed in indicator-datetime:
status: In Progress → Fix Committed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package indicator-datetime - 13.10.0+14.04.20131217-0ubuntu1

---------------
indicator-datetime (13.10.0+14.04.20131217-0ubuntu1) trusty; urgency=low

  [ Charles Kerr ]
  * Make the phone profile's header format sensitive to whether the
    phone is running in a 12h or 24h locale, and use a 12h or 24h
    notation accordingly. (LP: #1256061)

  [ Ubuntu daily release ]
  * Automatic snapshot from revision 291
 -- Ubuntu daily release <email address hidden> Tue, 17 Dec 2013 09:49:56 +0000

Changed in indicator-datetime (Ubuntu):
status: New → Fix Released
Charles Kerr (charlesk)
Changed in indicator-datetime:
status: Fix Committed → Fix Released
Michał Sawicz (saviq)
affects: unity8 → unity8 (Ubuntu)
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.