Login at Kodi Home

**Hitcher** · 2012-10-15, 17:07

(2012-10-14, 23:45)jmarshall Wrote: Will look into switching Container.TVShowThumb/SeasonThumb to Container.Art for consistency (the previous ones will still work).

Container.TVShowThumb from the video info dialog is problematic, as the video and music info windows aren't media windows, thus the information is being grabbed from the underlying window. Thus, when you go from the cast list of a movie to the a tvshow, the underlying window will still be movies, thus you won't see the container art. As far as I can tell, this has always been the case (or at least has been the case for some time before my art changes).

You should be able to use ListItem.Art(tvshowthumb) though.

Cheers,
Jonathan

Episode info via movie cast list -

jmarshall · 2012-10-16, 01:53

So this is not in DialogVideoInfo.xml, but rather in VideoLibrary.xml ?

**Hitcher** · (This post was last modified: 2012-10-16, 12:32 by Hitcher.)

(2012-10-16, 01:53)jmarshall Wrote: So this is not in DialogVideoInfo.xml, but rather in VideoLibrary.xml ?

EDIT: Sorry must have been too early for me as I read your post wrong.

This is in DialogVideoInfo.xml.

jmarshall · 2012-10-16, 11:48

Sure

**`Black** · (This post was last modified: 2012-10-16, 12:30 by `Black.)

The info labels are empty in DialogVideoInfo as well in that case.

**Hitcher** · 2012-10-16, 12:34

Edited my earlier post as I read it wrong.

Anyway code for testing DialogVideoInfo.xml in Confluence - http://pastebin.com/pFPKJJRP

As before find a movie, select actor who has appeared in an episode and select episode.

Thanks.

**`Black** · 2012-10-16, 14:24

It also works if you find a tv show and select an actor who appeared in an episode. Might be easier to find. Smile

**Hitcher** · 2012-10-16, 17:30

(2012-10-14, 23:45)jmarshall Wrote: As far as I can tell, this has always been the case (or at least has been the case for some time before my art changes).

It was until your recent fix - https://github.com/jmarshallnz/xbmc/comm...4f7e3d157b

**Hitcher** · 2012-10-16, 19:12

@JM Couldn't ListItem.DBID be used in DialogVideoInfo to get ListItem.Art(tvshowthumb)?

**BigNoid** · 2012-10-16, 22:56

I found out what the problem was. Apparently its case sensitive now and it should be lowercase. I did see some commits about 12 days ago where it was mentioned that system.getbool is now case sensitive, but i didnt think it would touch the infolabels.
So Player.Art(TvShowThumb) doesnt work, but Player.Art(tvshowthumb) does.

jmarshall · (This post was last modified: 2012-10-17, 01:20 by jmarshall.)

Yes, everything has to be lower case for properties and art. Properties has always been this way I believe. Reason is that the use a map lookup, which is case-sensitive.

System.GetBool is actually case-insensitive. I could probably make this the case here as well by always lower-casing the returned string. Thoughts?

jmarshall · 2012-10-17, 01:22

@Hitcher: Actually, that commit didn't allow Container.TVShowThumb - i.e. the commit message is wrong. Reason is that the Container.* stuff always comes from the parent window. It would have enabled ListItem.TVShowThumb, however.

**Hitcher** · 2012-10-17, 07:50

(2012-10-17, 01:18)jmarshall Wrote: Yes, everything has to be lower case for properties and art. Properties has always been this way I believe. Reason is that the use a map lookup, which is case-sensitive.

System.GetBool is actually case-insensitive. I could probably make this the case here as well by always lower-casing the returned string. Thoughts?

If it slows things down then I'm ok with it as it is.

(2012-10-17, 01:22)jmarshall Wrote: @Hitcher: Actually, that commit didn't allow Container.TVShowThumb - i.e. the commit message is wrong. Reason is that the Container.* stuff always comes from the parent window. It would have enabled ListItem.TVShowThumb, however.

Ah right, so will that still work now then?

jmarshall · (This post was last modified: 2012-10-17, 08:06 by jmarshall.)

It *should*, yes Smile

And no, allowing mixed case properties wouldn't slow things down - we'd just lowercase them all on first read.

butchabay · (This post was last modified: 2012-10-17, 09:32 by butchabay.)

(2012-10-17, 01:18)jmarshall Wrote: Yes, everything has to be lower case for properties and art. Properties has always been this way I believe. Reason is that the use a map lookup, which is case-sensitive.

System.GetBool is actually case-insensitive. I could probably make this the case here as well by always lower-casing the returned string. Thoughts?

That's kind of strange as there are a lot of properties working uppercase/lowercase mixed: For example recently added

<texture>$INFO[Container(8000).ListItem.Property(Fanart)]</texture>

or

<icon>$INFO[Window(Home).Property(LatestEpisode.1.ShowThumb)]</icon>

We really should set all properties to lowercase ? Huh