Opened 12 years ago

Closed 11 years ago

Last modified 11 years ago

#6167 closed patch (fixed)

LiveTV does not exit when "prompt on exit" is enabled.

Reported by: Roo Owned by: Isaac Richards
Priority: minor Milestone: unknown
Component: MythTV - Video Playback Version: head
Severity: medium Keywords: exit livetv
Cc: Ticket locked: no

Description

I suspect the -vid merge is the cause of the following issue.

When exiting LiveTV with the prompt on exit setting enabled the video pauses but no exit prompt is shown, playback can be resumed by pressing play.

The attached patch seems to fix the issue for me but it has only had a short amount of testing.

Attachments (2)

6167-v1.diff (917 bytes) - added by Roo 12 years ago.
Exit livetv with prompt enabled fix, plus trivial typo in logged message.
6167-v2.diff (1.2 KB) - added by Roo 12 years ago.
Don't show delete when promtping to exit LiveTV

Download all attachments as: .zip

Change History (7)

Changed 12 years ago by Roo

Attachment: 6167-v1.diff added

Exit livetv with prompt enabled fix, plus trivial typo in logged message.

comment:1 Changed 12 years ago by otto at kolsi dot fi

See also #6054.

With this patch applied the exit dialog works okay but it has an extra "Delete this recording" option that probably should not be there for LiveTV.

Changed 12 years ago by Roo

Attachment: 6167-v2.diff added

Don't show delete when promtping to exit LiveTV

comment:2 in reply to:  1 Changed 12 years ago by Roo

Replying to otto at kolsi dot fi:

With this patch applied the exit dialog works okay but it has an extra "Delete this recording" option that probably should not be there for LiveTV.

V2 should remove the delete from exit LiveTv? prompt.

comment:3 Changed 11 years ago by shane

(In [20194]) Refs #6167 Do not show the delete menu item when in livetv.

Thanks to Roo for the original patch.

comment:4 Changed 11 years ago by shane

Resolution: fixed
Status: newclosed

(In [20195]) Closes #6167 Exit livetv after the exit prompt is shown.

Thanks to Roo for the patch.

comment:5 Changed 11 years ago by shane

(In [20196]) Refs #6167 Fix typo. Thanks again to Roo.

Note: See TracTickets for help on using tickets.