Re: Patch for kaa.xmlutils. Add support for more than one text element inside an element

John Molohan <[email protected]>
Newsgroups gmane.comp.video.freevo.devel
Message-ID <[email protected]>
  On 11/08/2010 7:12, Jason Tackaberry wrote:
> Hi John,
Hi,
> On Tue, 2010-08-10 at 22:05 +0100, John Molohan wrote:
>> This patch addresses this and allows elements to contain more than one
>> text element so that tags such as in this example are handled correctly:
>>
>> <url mplayer-options="-user-agent
>> QuickTime7.6.2">http://trailers.apple.com/movies/universal/adjustmentbureau/adjustmentbureau-tlr1_a720p.m4v</url>
>
> But the url element above still has only one child element.  It has one
> attribute (mplayer-options) and one child node.
My wording here was probably incorrect. I know next to nothing about xml 
formats and as you already know programming in general :)

Just so I'm clear, url is the element, mplayer-options is the attribute 
and is the trailer url the child node?
> I can't reproduce the problem you describe.  I have a file called
> test.xml:
>
>          <fxd>
>              <url mplayer-options="-user-agent
>          QuickTime7.6.2">http://trailers.apple.com/movies/universal/adjustmentbureau/adjustmentbureau-tlr1_a720p.m4v</url>
>          </fxd>
>
> And:
>
>          >>>  import kaa.xmlutils
>          >>>  print kaa.xmlutils.create('test.xml').toxml()
>          <?xml version="1.0" encoding="UTF-8"?>
>          <fxd>
>              <url mplayer-options="-user-agent
>          QuickTime7.6.2">http://trailers.apple.com/movies/universal/adjustmentbureau/adjustmentbureau-tlr1_a720p.m4v</url>
>          </fxd>
>
>
> This looks right to me.  Can you tell me how to reproduce the problem?
I'll post my code so far for the appletrailers plugin in a new email 
(just to avoid cluttering this one) which hopefully will allow you to 
recreate the problem (or simply determine my ignorance :) ).
> (I've not actually looked this xmlutils API before.  I'm tempted to
> deprecate __getattr__ in favor of __getitem__ and implement
> __setitem__.)
>
> Cheers,
> Jason.
Thanks,

John

------------------------------------------------------------------------------
This SF.net email is sponsored by 

Make an app they can't live without
Enter the BlackBerry Developer Challenge
http://p.sf.net/sfu/RIM-dev2dev
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.