Opened 13 years ago

Closed 13 years ago

#2206 closed defect (fixed)

LiveTV browse should show channels on other inputs on current card

Reported by: danielk Owned by: danielk
Priority: minor Milestone: 0.20
Component: mythtv Version: head
Severity: medium Keywords:
Cc: Ticket locked: no

Description


Attachments (1)

channelbase_browse_mode.diff (1005 bytes) - added by Isaac Richards 13 years ago.

Download all attachments as: .zip

Change History (10)

comment:1 Changed 13 years ago by danielk

Owner: changed from Isaac Richards to danielk

comment:2 Changed 13 years ago by cpinkham

Summary: LiveTV browse should show channels on other inputsLiveTV browse should show channels on other inputs on current card

Changing description so people don't get any funny ideas. If people want browse mode to show all channels on all inputs they need to read the comments in #978 pertaining to why it is not desired and what work would have to be done to fully support that functionality.

comment:3 Changed 13 years ago by mythtv-dev (at) dwilga-linux1.amherst.edu

When addressing this issue, please also keep in mind the other part of my original post:

Unlike previous versions, the "visibile" flag is being ignored. All channels for an input are currently being shown, regardless of this flag.

comment:4 Changed 13 years ago by danielk

Resolution: fixed
Status: newclosed

(In [10885]) Fixes #2206. Show channels on other inputs on current card in browse mode.

comment:5 Changed 13 years ago by danielk

(In [10933]) Refs #2206. Fixes signature of DBChannel assignment operator (it returned a const ref).

comment:6 in reply to:  4 Changed 13 years ago by gnome42@…

Resolution: fixed
Status: closedreopened

Replying to danielk:

(In [10885]) Fixes #2206. Show channels on other inputs on current card in browse mode.

I can't browse channels across inputs on the same card. I did some digging and came up with a patch that works for me, but I doubt it fixes the problem correctly.

Please see comments in patch for explanation.

Regards,

Shane

comment:7 Changed 13 years ago by cpinkham

Might help if you attach the patch.

Changed 13 years ago by Isaac Richards

comment:8 Changed 13 years ago by Isaac Richards

I just attached the patch for him - was my fault, trac was down when he went to attach it earlier.

comment:9 Changed 13 years ago by Isaac Richards

Resolution: fixed
Status: reopenedclosed

(In [11091]) In GetNextProgram?(), if we have access to a valid chanid, use that instead of a channum. Fixes #2206.

Add some sanity if we fail insert_program in programinfo.cpp.

Note: See TracTickets for help on using tickets.