[mythtv-commits] Ticket #4442: upnpcdsvideo doesn't handle directories correctly

[MythTV]

#4442: upnpcdsvideo doesn't handle directories correctly
----------------------------------------------------------+-----------------
 Reporter:  David Asher <david.asher at caviumnetworks.com>  |        Owner:  greg    
     Type:  defect                                        |       Status:  assigned
 Priority:  minor                                         |    Milestone:  unknown 
Component:  upnp                                          |      Version:  unknown 
 Severity:  medium                                        |   Resolution:          
  Mlocked:  0                                             |  
----------------------------------------------------------+-----------------

Comment(by David Asher <david.asher at caviumnetworks.com>):

 Wow.  Thanks for the fast response.

 As for item 1: The problem I had was GetDistinctCount is called, but that
 is not a virtual function, so changing it will change it for all
 extensions.  The SQL executed was:


 {{{
 select count(*) from upnpmedia;
 }}}


 what i think is desired is:


 {{{
 select count(parentid) from upnpmedia where parentid=1000000;
 }}}


 That CAN be done with GetCount, but upnpcds.cpp is making the call in
 UPnpCDSExtension::ProcessAll, which again is not a virtual function.  So,
 I couldn't decide what was the best fix.

 For item 3: I see the folder entry for Sub2, but when I enter it I get "no
 titles", and a new entry for Sub2 appears, which does have the titles in
 it.  Very odd.  If you need the XML result let me know and I'll try and
 get it for you.  I got stuck when I realized I didn't really know what it
 SHOULD look like.

 BTW, the client I'm using is a PS3 w/firmware v2.10, so ymmv.

-- 
Ticket URL: <http://svn.mythtv.org/trac/ticket/4442#comment:4>
MythTV <http://svn.mythtv.org/trac>
MythTV