Opened 13 years ago

Closed 13 years ago

#1223 closed patch (fixed)

TV listings should group by channum, not chanid

Reported by: Robert Tsai <rtsai1111> Owned by: xris
Priority: minor Milestone: unknown
Component: mythweb Version: head
Severity: low Keywords:
Cc: Ticket locked: no

Description

Grouping by chanid is strictly the most correct way to display TV listings, since it implicitly depicts which source will be recording which programs. However, this is overly verbose when different sources have overlapping channels (which I have assigned the same channum and callsign). For example, I have two separate antenna sources that overlap with some channels. The attached patch groups the TV listing by channum instead to help reduce the verbosity.

Grouping by channum effectively picks a random chanid for each channel, however, which means that some scheduled recordings will be hidden, since the $Scheduled_Recordings structure is keyed by chanid. So this patch changes $Scheduled_Recordings to be keyed by channum.

Upcoming Recordings, TV listings both now work as expected; clicking on a recording or timeslot will still direct the user to a page that references the true chanid.

I chose channum as the new key rather than callsign; I think that is slightly more unique (and appropriately so). For example, in my area, there is an analog "WBZ" and a digital "WBZ-DT" (for broadcast HDTV).

This patch supersedes #1054 and references [8873].

Attachments (1)

mythweb.channum.diff (7.5 KB) - added by Robert Tsai <rtsai1111> 13 years ago.

Download all attachments as: .zip

Change History (4)

Changed 13 years ago by Robert Tsai <rtsai1111>

Attachment: mythweb.channum.diff added

comment:1 Changed 13 years ago by Robert Tsai <rtsai1111>

To clarify my comment; "channum" is more likely to be different between the digital and non-digital versions of a channel. The callsigns could be the same (I've stripped out the "DT" suffixes from all my digital channels).

comment:2 Changed 13 years ago by anonymous

Component: mythtvmythweb

comment:3 Changed 13 years ago by xris

Resolution: fixed
Status: newclosed

(In [8887]) Apply patch to close #1223 and #1054

Note: See TracTickets for help on using tickets.