Should display local times

Bug #1096252 reported by Jan Schneider
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
mythtv-status (Ubuntu)
Fix Released
Medium
Unassigned

Bug Description

Since MythTV 0.26, all times are returned as UTC via the API. When displaying times, e.g. start times of upcoming recordings, they should be converted to the local time.

Tags: patch
Thomas Mashos (tgm4883)
Changed in mythtv-status (Ubuntu):
importance: Undecided → Medium
status: New → Confirmed
Revision history for this message
Andrew Ruthven (andrew-etc) wrote :

Ah, I'm looking into the best way to resolve this now.

Revision history for this message
Andrew Ruthven (andrew-etc) wrote :

This is now fixed in git. I'll push out a new release shortly.

Revision history for this message
Jan Schneider (yunosh) wrote :

Your commit only effects the status line, but not the (more important) schedule lines. Attached patch fixes this.

Revision history for this message
Jan Schneider (yunosh) wrote :

This patch changes the date and time format to really use the locale's format. For some reason, the script still doesn't pick up LC_TIME but uses the C locale for formatting. Not sure why this is happening.

Revision history for this message
Ubuntu Foundations Team Bug Bot (crichton) wrote :

The attachment "0001-Apply-ISO-date-processing-to-schedule-too.patch" of this bug report has been identified as being a patch. The ubuntu-reviewers team has been subscribed to the bug report so that they can review the patch. In the event that this is in fact not a patch you can resolve this situation by removing the tag 'patch' from the bug report and editing the attachment so that it is not flagged as a patch. Additionally, if you are member of the ubuntu-reviewers team please also unsubscribe the team from this bug report.

[This is an automated message performed by a Launchpad user owned by Brian Murray. Please contact him regarding any issues with the action taken in this bug report.]

tags: added: patch
Revision history for this message
Andrew Ruthven (andrew-etc) wrote :

Thanks Jan.

Sadly the Date::Manip stuff doesn't listen to locale for the date format, you must explicitly set US or non-US when you init it.

To avoid that silliness I've changed it to use %Y-%m-%d which is hopefully more generally acceptable.

I've also added another patch that handles the Ends time for in progress recordings.

Revision history for this message
Andrew Ruthven (andrew-etc) wrote :

This issue is resolved in 0.10.4 in Debian Unstable...

Changed in mythtv-status (Ubuntu):
status: Confirmed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

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