Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ListItem.Icon does not fallback to the default icon #17034

Closed
ronie opened this issue Dec 12, 2019 · 6 comments · Fixed by #17127
Closed

ListItem.Icon does not fallback to the default icon #17034

ronie opened this issue Dec 12, 2019 · 6 comments · Fixed by #17127
Assignees
Labels
Component: GUI engine Resolution: Fixed issue was resolved by a code change Triage: Confirmed issue has been reproduced by a team member v19 Matrix

Comments

@ronie
Copy link
Member

ronie commented Dec 12, 2019

when no thumbnail is available, ListItem.Icon should fallback to the default icon (ie. DefaultVideo.png)
this is currently broken for all video items (movies/tvshows/seasons/episodes/musicvideos).
most likely due to the same issue, ListItem.ActualIcon returns empty as well.

it works fine for music items.

Leia:
leia

Matrix:
matrix

pinging @enen92 / @DaveTBlake , iirc you have been making code changes in this area..

@DaveTBlake
Copy link
Member

This could be a consequence of #16444 moving the default icon into art array. I did a follow-up fix for music as there were regressions where Kodi was using an empty art test, and art was now not empty. But video was not effected in the same way so I didn't touch that.

Is this change to Estuary #17035 intended as a fix for this, or just a work around, or something else?

@ronie
Copy link
Member Author

ronie commented Dec 14, 2019

thanx for looking into it.
the Estuary change is unrelated to this issue.

@ronie
Copy link
Member Author

ronie commented Jan 2, 2020

@enen92 any ideas?

@enen92 enen92 self-assigned this Jan 3, 2020
@enen92 enen92 added Triage: Confirmed issue has been reproduced by a team member and removed Triage: Confirmed issue has been reproduced by a team member labels Jan 3, 2020
@enen92
Copy link
Member

enen92 commented Jan 3, 2020

@ronie can you provide some info on how to easily reproduce the issue? I'll definitely try to fix it if caused by my PR.

Added a .strm file to the library (The Social Network) and removed the items from the art table in MyVideos116.db but default icon is correctly filled:

alt text
alt text

Sorry for the delay, I need to enable email notifications for mentions.

Maybe send me a database where this issue is verified and let me know the item?

@ronie
Copy link
Member Author

ronie commented Jan 4, 2020

here's the database, it has just 1 tv show:
MyVideos116.db.zip

as mentioned on slack, the issue is noticeable in the 'widelist' view of estuary.
widelist

@enen92 enen92 added the Triage: Confirmed issue has been reproduced by a team member label Jan 4, 2020
@enen92
Copy link
Member

enen92 commented Jan 4, 2020

Ok I confirm the issue. I was not able to reproduce before because I removed all art items in the art table specific to the tvshow or movie. In your case you have an empty thumb/poster but still have a valid fanart.
I'll come up with a fix.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component: GUI engine Resolution: Fixed issue was resolved by a code change Triage: Confirmed issue has been reproduced by a team member v19 Matrix
Projects
None yet
4 participants