Opened 7 years ago

Closed 7 years ago

Last modified 7 years ago

#11236 closed Bug Report - General (fixed)

frontend's PlayRecording service api uses local time where UTC is expected

Reported by: George Nassas <gnassas@…> Owned by: danielk
Priority: minor Milestone: 0.26.1
Component: MythTV - UPnP Version: Unspecified
Severity: medium Keywords:
Cc: Ticket locked: no

Description

oddly enough the method's log message shows UTC so somebody definitely made an effort.

As an aside; when Frontend::PlayRecording? constructs its message it puts 12345 as the 5th parameter, presumably to effect a bookmark resume, but it doesn't. Replacing 12345 with resume or RESUME doesn't do anything either but I didn't look into what's wrong.

Attachments (1)

0004-send-UTC-to-network-control-play.patch (974 bytes) - added by George Nassas <gnassas@…> 7 years ago.

Download all attachments as: .zip

Change History (6)

Changed 7 years ago by George Nassas <gnassas@…>

comment:1 Changed 7 years ago by mythtv@…

The method worked before the internal change to UTC, but appears to have been broken then.

See also:

http://code.mythtv.org/trac/ticket/11152

comment:2 Changed 7 years ago by danielk

Owner: changed from dblain to danielk
Status: newaccepted

comment:3 Changed 7 years ago by Chris Pinkham <cpinkham@…>

Resolution: fixed
Status: acceptedclosed

In b62198ffcada256ebf3c40c9c6c4be1615ae17ea/mythtv:

Fix mythfrontend services API /Frontend/PlayRecording? call.

We don't need to convert the specified starttime toLocalTime() anymore.

Copies 3eef99e from master.

Fixes #11236 and #11152 I believe.

comment:4 Changed 7 years ago by cpinkham

Thanks to George for this patch, it was the same conclusion I came to while debugging the issue in IRC tonight without realizing there was an open ticket with a patch.

comment:5 Changed 7 years ago by Raymond Wagner

Milestone: unknown0.26.1
Note: See TracTickets for help on using tickets.