zach-morris / plugin.program.iagl

The Internet Archive Game Launcher (IAGL) will launch Games from the Internet using Kodi
GNU General Public License v3.0
344 stars 49 forks source link

User Experience Issues Related to Caching and Downloading #284

Open CFTBL opened 1 year ago

CFTBL commented 1 year ago

Testing IAGL 3.0.9 on LibreELEC-RR on RPi4. So far, in IAGL. I've only been looking at this on FB NEO, but there are three things about IAGL caching and downloading that are problematic. If you specify a download folder in a game list's settings, there are two undesirable side-effects: 1) Every game launched from that list is retained in that download folder (regardless of whether or not the user clicked the Download button). This causes unnecessary clutter (and is problematic if your download folder is also your Emulation Station content folder). 2) If a download folder has been specified, the cache size setting is ignored and the game will not be cached in IAGL's default game cache folder. The third thing is this: If a download folder has not been specified, and the user clicks the Download button, the game is downloaded to IAGL's game cache folder. That's OK, but the expectation is that "download" refers to a permanent copy whereas "cache" refers to a temporary copy.

I think the following changes would result in expected/desired behavior:

1) Each list should have it's own game cache folder and its own download folder (the latter can be a user-specified path) 2) If the above change is made, the game cache size setting shouldn't be a global IAGL setting. It should be a list-specific setting. 3) Launched games should be retained in the list's game cache based on the list's game cache size setting. 4) Games are never downloaded to the download folder unless the user clicks the Download button.

zach-morris commented 1 year ago

There is an option to organize the cache folder in advanced settings. Probably not very good documentation, but see 'Organize Cache Folder using System Folders'. However - if the overall cache folder is too large, then the entire folder is purged. I dont think I'll change this functionally.

I understand what your saying with the download button - it might be best actually to hide the button if the default cache folder is used, or bring up another menu to ask where it should be downloaded to.

Another possibility is that I can add a context menu for an ad-hoc 'download this to ..'