Opened 11 years ago

Closed 10 years ago

#5737 closed enhancement (wontfix)

upnp recording add subfolders supports

Reported by: Xavier Hervy <xavier.hervy@…> 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)

upnp-recordings-subfolders.diff (21.6 KB) - added by Xavier Hervy <xavier.hervy@…> 11 years ago.
upnp-recordings-subfolders-2.diff (23.9 KB) - added by Xavier Hervy <xavier.hervy@…> 11 years ago.
No change apart for coding convention and update description on top of the cpp file

Download all attachments as: .zip

Change History (9)

Changed 11 years ago by Xavier Hervy <xavier.hervy@…>

comment:1 Changed 11 years ago by Xavier Hervy <xavier.hervy@…>

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

comment:2 Changed 11 years ago by Xavier Hervy <xavier.hervy@…>

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 11 years ago by Xavier Hervy <xavier.hervy@…>

No change apart for coding convention and update description on top of the cpp file

comment:3 Changed 11 years ago by Xavier Hervy <xavier.hervy@…>

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 11 years ago by Dibblah

Status: newassigned

comment:5 Changed 10 years ago by stuartm

Component: upnpMythTV - UPnP

comment:6 Changed 10 years ago by robertm

Status: assignedinfoneeded

Xavier, you mentioned a patch to fix the UTF-8 and slash-escaping 19 months ago, still working on this?

comment:7 Changed 10 years ago by robertm

Resolution: wontfix
Status: infoneededclosed

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.

Note: See TracTickets for help on using tickets.