Opened 18 years ago
Closed 18 years ago
#1232 closed patch (fixed)
"Recorded Programs" needs to load all channel data.
Reported by: | Robert Tsai <rtsai1111> | Owned by: | xris |
---|---|---|---|
Priority: | minor | Milestone: | 0.19 |
Component: | mythtv | Version: | head |
Severity: | low | Keywords: | |
Cc: | Ticket locked: | no |
Description
The "Recorded Programs" page uses QUERY_RECORDINGS from the mythbackend; that picks up chanids used to make the recordings. The routine load_all_channels performs a "GROUP BY channel.channum"; this can end up hiding chanids that are referenced by the backend results, which results in the "Recorded Programs" page sometimes being unable to fill in the "channum" column.
This patch:
- Removes the GROUP BY from load_all_channels. Rather than re-key the $Channels array by channum, we simply make sure that all chanids are represented in the $Channels array (which means there could be object copies if two chanids refer to the same channum/callsign).
- Adds a call to load_all_channels in modules/tv/recorded.php, like many of the other modules.
There are a few other places that might need calls to load_one_channel (" No channel data? Load it"), but I don't address those.
Attachments (3)
Change History (10)
comment:1 Changed 18 years ago by
Owner: | changed from Isaac Richards to xris |
---|
Changed 18 years ago by
Attachment: | mythweb.recorded.diff added |
---|
comment:2 Changed 18 years ago by
comment:3 Changed 18 years ago by
This new patch loads all channels from the channel data (no more GROUP BY). The old GROUP BY effect is now explicitly implemented in the channel listings by keeping track of which channum's have already been displayed and not displaying any repeats.
comment:4 Changed 18 years ago by
this works, but after applying it, I'm also seeing everything without a program/seriesid show up as already-recorded. Did you have a fix for that one? Would be nice to get it in before .19 comes out.
comment:5 Changed 18 years ago by
Milestone: | → 0.19 |
---|
Changed 18 years ago by
Attachment: | mythweb.recorded.3.diff added |
---|
More group-by-channum fixes. Re-expose "nodups" UI for searches. Fix some calls to load_all_channels.
This patch is wrong; it re-doubles all the channels in the TV listings.