While I completely agree that the skinner shouldn't have to care about this
stuff, the problem is the disconnect in that the built in function parser
receives only a string to parse. By the time it gets it the string is
already formed by piecing together the info labels that the skinner uses.
Thus, if the string isn't already suitably escaped, the built-in parser is
going to get it wrong.
I can't recall the exact details, but I'm fairly sure that there wasn't an
(easy) alternative, else ofcourse I would have done it. I'll take another
look - no promises on how long it'll take to do this, as I'm quite busy atm.
Cheers,
Jonathan
2012/3/1 Predrag Mitić <[email protected]>
> Hi,
>
> thanks for feedback. Few comments from my side
>
> $ESC moves parsing string logic to the last instance that has nothing to
> do with coding - UI design. I don't know what are the reasons why XBMC is
> parsing strings inside $INFO and not taking it verbatim, I don't know why
> comma is being used as parameter separator inside $INFO. The way I see it
> is that this all should be handled before reaching skin. I have no idea
> what's going in background and I don't really want to know. I would like to
> _design_ if possible leaving all non related things to programmers as I
> have no control and knowledge about them.
>
> I'm following commits through RSS ATM and yes it's far from ideal but
> bigger problem is that even if I see them I may interpreting them in a
> wrong way. For instance:
>
> https://github.com/xbmc/xbmc/pull/124 - I have read as not concerning me
> as I don't use skin.properties in that way.
>
> Let's be honest - there is almost a zero chance that any skinner would
> think as: hey I'm trying to solve recently added layout on home screen, but
> wait.. I've seen commit about $ESC some months ago. Hmm.. This I better use
> because some character could decide to use comma in his path and since
> that's probably being hidden in this
> <onclick>PlayMedia("$INFO[Window.Property(LatestMovie.5.Path)]")</onclick>
> I'll better use them.
>
> Last but not least I would like to be sure when skin related part ends and
> XBMC code begins for trouble shooting purposes. It is already very messy
> and confusing for user as they are often sent from skin thread to script
> and/or xbmc code threads and back again in order to find where their
> problems are coming from.
>
> Peca
>
> On 28. 2. 2012, at 20:52, ronie wrote:
>
> On 28-02-12 11:27, Predrag Mitić wrote:
> > Use $ESCINFO[] rather than $INFO[]
> >
> > When was this changed? Consider me dumb, but except for recently added
> example I have no idea where else should it be used?? I just don't get it.
> >
> > Do skins really need to think about this kind of issues?
> >
> > Why don't we post such things to forum? It is very hard to follow this
> list as info gets mixed with code.
> hi,
>
> $ESCINFO[] was added about 10 months ago:
>
> https://github.com/xbmc/xbmc/commit/7d77070d7e14aa7d1a225e705c9c28f8755a48f7
>
> some info on why it was added can be found on the PR:
> https://github.com/xbmc/xbmc/pull/124
>
> the usecase is not only passing an infolabel to playmedia, basically it
> should be used when you pass an infolabel
> that may contain either quotes or commas, to a built-in function.
>
> i stumbled upon this issue last year when i was trying to copy an
> infolabel to a windowproperty.
> for example: SetProperty(MoviePlot,$INFO[ListItem.Plot])
> since the movie description is likely to contain a comma, the built-in
> function will fail in this case
> as it will interpret it as multiple arguments being passed to it.
>
> it's likely this info never hit the forum, as i either only used trac to
> report the issue or discussed it
> with some of the devs on irc... can't remember :-)
>
> currently the best way to stay up-to-date with skin engine changes is to
> keep an eye on the commits on git
> or check the timeline on trac. i understand many skinners consider this to
> be far from ideal, so i'm planning
> to start a thread in the skin-dev forum after eden, where we can collect
> all skin related changes for frodo.
> that way, everyone should have an easy overview of what's new and which
> things have changed.
>
>
> cheers,
> ronie
>
>
>
> ------------------------------------------------------------------------------
> Virtualization & Cloud Management Using Capacity Planning
> Cloud computing makes use of virtualization - but cloud computing
> also focuses on allowing computing to be delivered as a service.
> http://www.accelacomm.com/jaw/sfnl/114/51521223/
> _______________________________________________
> Xbmc-addons mailing list
> [email protected]
> https://lists.sourceforge.net/lists/listinfo/xbmc-addons
>
------------------------------------------------------------------------------
Virtualization & Cloud Management Using Capacity Planning
Cloud computing makes use of virtualization - but cloud computing
also focuses on allowing computing to be delivered as a service.
http://www.accelacomm.com/jaw/sfnl/114/51521223/
_______________________________________________
Xbmc-addons mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/xbmc-addons
