[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 greg):

 > 1. When ProcessAll is called (for Videos/0), it gets ALL videos instead
 of the videos whose parentid == STARTING_VIDEO_OBJECTID.  Consequently the
 container


  This shouldn't be too much to fix. It's just a modification to the
 GetCount() routine.   The code doesn't actually return ALL Videos, it just
 returns the count of all videos and the info for the children of the
 requested parentid.

 So for example a call to Videos/0 in my setup returns

 <NumberReturned>12</NumberReturned>
 <TotalMatches>2795</TotalMatches>


 > 2. In UPnpCDSVideo::AddItem, the child count is never filled in when a
 container is created.  The attached patch fixes this.

  Thanks I'll look at that.



 > 3. It looks like the support for multiple levels of directories is
 broken.  A path like <root>/Sub1/Sub2/Vid1 doesn't produce the expected
 results.  The IDs get kinda


  What exactly are you seeing ? I'm using it at least 5 levels deep (just
 tested again) with everything returning what I expect as far as I can
 tell.   Are you seeing bad ID's or the wrong videos returned?

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