Nutzzz / GameCollector

.NET library for finding games.
Other
5 stars 0 forks source link

Add a list of reasons for (or instead of) the HasProblem flag #27

Open Nutzzz opened 7 months ago

Nutzzz commented 7 months ago

HasProblem usually indicates an expired trial (currently for Big Fish, Humble, and Oculus), but for MAME ROMs it could indicate that it didn't verify correctly or that the MachineStatus doesn't meet the minimum criteria (I was OK with it being equivocal since "DriverStatus" can be checked manually). Now, however, I decided to flip the bit to indicate a workaround for #23 a Legacy Games Launcher issue I ran into where the launcher lost track of an installed game.

I imagined the use-case for a consumer application would involve adding a yellow yield-sign pling next to the title to let the user know there was an issue that needed to be investigated. However, if this flag is going to indicate a lot of different conditions, it makes the application's job much harder to produce an appropriate message to the user.

I'll use an enum rather than a full error message, so it's still up to the application how to communicate/translate the issue.

EDIT: Here's my first pass at a list of problems for the enum; not all of them are currently used, but I think I've seen all of them so I can try to flag them where possible:

Nutzzz commented 7 months ago

Fixed in commit 70a31fd. Not all of the flags listed in the enum are currently used. I need to go through the handlers and see where I can flag some of the issues that I've seen and worked around.