Should display local times

Bug #1096252 reported by Jan Schneider on 2013-01-04
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
mythtv-status (Ubuntu)
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.

Thomas Mashos (tgm4883) on 2013-01-04
Changed in mythtv-status (Ubuntu):
importance: Undecided → Medium
status: New → Confirmed
Andrew Ruthven (andrew-etc) wrote :

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

Andrew Ruthven (andrew-etc) wrote :

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

Jan Schneider (yunosh) wrote :

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

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.

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
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.

Andrew Ruthven (andrew-etc) wrote :

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

To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers