Opened 16 years ago
Closed 15 years ago
#5737 closed enhancement (wontfix)
upnp recording add subfolders supports
Reported by: | Owned by: | dblain | |
---|---|---|---|
Priority: | minor | Milestone: | unknown |
Component: | MythTV - UPnP | Version: | head |
Severity: | medium | Keywords: | upnp recordings grouping |
Cc: | Ticket locked: | no |
Description
This patch all the multi-grouping via upnp for RecTv?: Example for ObjectID: RecTV
All Recordings By Title By Genre By Date By Channel By Group
RecTV/0 all recordings video item
RecTV/2
Animation Comedy ....
RecTV/2/key=Animation
All Recordings By Title By Date By Channel By Group
RecTV/2/key=Animation/0 => All animation
RecTV/2/key=Animation/1
Family Guy The Simpsons
RecTV/2/key=Animation/1/key=Family Guy/0 all Family Guy which are an animation
RecTV/4/key=101/2/key=Comedy/0 All comedy on channel 101 ... ...
Attachments (2)
Change History (9)
Changed 16 years ago by
Attachment: | upnp-recordings-subfolders.diff added |
---|
comment:1 Changed 16 years ago by
comment:2 Changed 16 years ago by
In fact, ProcessItem? is needed but the other are very specific to the organization of the upnp extension and we can not generalize Xavier
Changed 16 years ago by
Attachment: | upnp-recordings-subfolders-2.diff added |
---|
No change apart for coding convention and update description on top of the cpp file
comment:3 Changed 16 years ago by
This patch does not take care of utf8 yet and might not work if a key contain a / I will send a new patch later one
comment:4 Changed 16 years ago by
Status: | new → assigned |
---|
comment:5 Changed 15 years ago by
Component: | upnp → MythTV - UPnP |
---|
comment:6 Changed 15 years ago by
Status: | assigned → infoneeded |
---|
Xavier, you mentioned a patch to fix the UTF-8 and slash-escaping 19 months ago, still working on this?
comment:7 Changed 15 years ago by
Resolution: | → wontfix |
---|---|
Status: | infoneeded → closed |
Closing as stale, Xavier, if you still have interest and want to work through the issues you mentioned above, please feel free to reopen this ticket with a new patch and I will do my best to corral someone to look at it.
BTW, it seems unneeded to declare ProcessRoot?, ProcessAll?, ProcessItem?, ProcessKey?, ProcessContainer? in class UPnpCDSExtensionResults as virtual as they are never user in child class
Due to the sub grouping I have only 3 case: processAvailableGroup (title, genre ...) processGroup (genre=Animation ...) processResult which return a list of recordings
Xavier