Activity Tracker : update boxart after the game has been added to the database

[schmurtzm]

Currently adding a box art (with Scraper or else) doesn't actualize the game cover in Activity Tracker.

This is due to the fact that Activity Tracker stores the image path in a db instead of generating the image path dynamically.

[Aemiii91]

I think we should just get the box art image dynamically (by checking if the png exist) - instead of having to keep a db up-to-date

[theUnBurn]

@schmurtzm I made a fix for this issue here: https://github.com/OnionUI/Onion/pull/1305

Would love feedback! Particularly around on if we should update the db to no longer have an image path (and, if so, how to do that).

[nenjack]

@schmurtzm I made a fix for this issue here: #1305

Would love feedback! Particularly around on if we should update the db to no longer have an image path (and, if so, how to do that).

I love the mr https://github.com/OnionUI/Onion/pull/1305 please merge it to next onion dev release please

[nenjack]

@Aemiii91 how can I support MR #1305

[nenjack]

Would love feedback! Particularly around on if we should update the db to no longer have an image path (and, if so, how to do that).

I wouldn't remove the field in db or update it, this way:

if someone installs newer onionos, has some activities in db, then decides to downgrade, his db will still work with older onion (backwards compatibility)

[schmurtzm]

Hi, I have a problem with this PR... Curiously I'm not able to triggers the initial problem . Hum...😅 Quickly tested the actual Activity Tracker and I don't see any problem : changing the cover of a game impacts Activity Tracker too. I don't remember in which case the fact that Activity Tracker keeps the path of the image is a problem anymore. 😶

[nenjack]

Hi, I have a problem with this PR... Curiously I'm not able to triggers the initial problem . Hum...😅 Quickly tested the actual Activity Tracker and I don't see any problem : changing the cover of a game impacts Activity Tracker too. I don't remember in which case the fact that Activity Tracker keeps the path of the image is a problem anymore. 😶

in that case @schmurtzm at least please reopen https://github.com/OnionUI/Onion/issues/1419 because that was a bit different (missing images on games, dont update, dont show, nothing, in activity tracker, but they are scraped prior, more details in linked issue)

[nenjack]

the first example I recall is Tony Hawk's Pro Skater 4 on PS

[nenjack]

@Aemiii91 any progress on this one if this is a duplicate of 1441 as you said there?

[nenjack]

@schmurtzm did you try this on any PSX games like I said in my issue? did you try this with THPS4?

also, see:

https://github.com/OnionUI/Onion/assets/3052357/4ebe536c-aa32-4fcb-8f52-5d324136828a

[XK9274]

Thanks for the feedback, Lets keep it constructive.