[FIXED] No CD cover displayed in the HomeBasic (1 Viewer)

[ltfearme]

I think most of mine came from fanart handler.

In basic home there is logic to switch the album and artist images in an animation, this logic (skin condition) is failing most likely.

 

[ltfearme]

Found the issue, for some reason this fails now when being evaluated by the skin engine:

string.equals(#Play.Current.thumb,)

This means check if '#Play.Current.thumb' is empty (because the value after the comma is an empty.string. Now I have seen some wacky expressions omitting the comma when checking for an empty string:

string.equals(#Play.Current.thumb)

What do you know, it works! It should be an epic fail in my opinion as that is not a valid expression. I have no idea why the above expression no longer works in MediaPortal 1.6...and you know what's even weirder? It only fails when used in BasicHome, it works in other overlays !!!!!.

Solution:
1. Edit the file common.basichome.overlay.xml
2. Replace "string.equals(#Play.Current.thumb,)" with "string.equals(#Play.Current.thumb)"

 

[ltfearme]

I had to re-work the conditions all together and throw aways checks using 'string.equals(#Play.Current.thumb)', because that fails under some scenarios and my old one worked! So for consistency everywhere I removed them, they were there for a reason but im not going to get into that so in next version it should all just work although not the best implementation.

 

[ltfearme]

Please test in latest release:
https://forum.team-mediaportal.com/...-2-0-28-jan-2014-supports-mp-1-6-only.124305/

 

[axlt2002]

You have changed the thread title in the correct way ltfearme!!!! I have just updated the skin and the bug has disappeared!!!!

Thanks a lot for your kind help!!!