Opened 5 years ago
Closed 2 years ago
#13451 closed Patch - Feature (Trac EOL)
Change UPNP Class for Recorded TV from videoItem to videoItem.videoBroadcast subclass
Reported by: | nerddtvg | Owned by: | dblain |
---|---|---|---|
Priority: | minor | Milestone: | needs_triage |
Component: | MythTV - UPnP | Version: | Unspecified |
Severity: | medium | Keywords: | UPnP ContentDirectory videoBroadcast |
Cc: | Ticket locked: | no |
Description
The Content Directory v4 Service lists the videoBroadcast subclass as "A videoBroadcast instance represents a continuous stream from a video broadcast (for example, a conventional TV channel or a Webcast)" which is the perfect class for recorded TV.
An example of the usefulness of this is Windows Media Player. It categories Music, Videos, Pictures, and Recorded TV in its interface. Currently MythTV lists all shows under Videos because they're generic videoItem classes. However, HDHomeRun devices categorize their channels as videoItem.videoBroadcast which places them under Recorded TV (even though those are live channels).
The change would be to change upnpcdstv.cpp.
https://github.com/MythTV/mythtv/blob/541281f5d674fbb52d8dc6f40333a6e02eafbf50/mythtv/programs/mythbackend/upnpcdstv.cpp#L1059 (latest commit at the time of submitting this)
upnpcdstv.cpp Line 1059: Old Constructor: CDSObject::CreateVideoItem?
New Constructor: CDSObject::CreateVideoBroadcast?
The function already exists in upnpcdsobjects.cpp and is setup for this purpose.
Reference: http://www.upnp.org/specs/av/UPnP-av-ContentDirectory-v4-Service.pdf
We have moved all bug tracking to github [1]
If you continue to have this issue, please open a new issue at github, referencing this ticket.
[1] - https://github.com/MythTV/mythtv/issues