Project

General

Profile

Actions

Emulator Issues #10897

closed

Android: user game-specific settings overrides improperly handle DOL and ELF

Added by ryanebola16 over 6 years ago. Updated over 6 years ago.

Status:
Fixed
Priority:
Normal
Assignee:
-
% Done:

0%

Operating system:
Android
Issue type:
Bug
Milestone:
Regression:
No
Relates to usability:
No
Relates to performance:
No
Easy:
Yes
Relates to maintainability:
No
Regression start:
Fixed in:
5.0-8068

Description

What's the problem? Describe what went wrong.

On Android, DOL and ELF user game-specific settings overrides will be improperly named.

What steps will reproduce the problem?

  1. Create a user game-specific settings overrides for a DOL or ELF via Dolphin's UI
  2. Observe it isn't applied since it is improperly named.

Is the issue present in the latest development version? For future reference, please also write down the version number of the latest development version.

5.0-6346

Is the issue present in the latest stable version?

No, 5.0

If the issue isn't present in the latest stable version, which is the first broken version? (You can find the first broken version by bisecting. Windows users can use the tool https://forums.dolphin-emu.org/Thread-green-notice-development-thread-unofficial-dolphin-bisection-tool-for-finding-broken-builds and anyone who is building Dolphin on their own can use git bisect.)

5.0-6341 introduced Android: Implement user game-specific settings overrides

What are your PC specifications? (CPU, GPU, Operating System, more)

Win 10 Pro x64
Intel Core i7-4702MQ CPU @2.20GHz
GeForce GT 750M
2x 8GB Hynix 1600MHz
GeForce Game Ready Driver 390.77

Actions #1

Updated by JosJuice over 6 years ago

What does it get named?

Actions #2

Updated by ryanebola16 over 6 years ago

The name of the DOL or ELF

Actions #3

Updated by merauder75 over 6 years ago

It sets the name based on whatever the frontend passes it as the game ID.

Actions #4

Updated by JosJuice over 6 years ago

  • Status changed from New to Accepted
  • Easy changed from No to Yes
  • Operating system Android added
  • Operating system deleted (N/A)

While I haven't tested this myself, it sounds like a valid issue. It's possible to make a game INI apply to DOL/ELF files by naming it 00000000.ini, but it would probably be better to not let people edit game INIs for DOLs/ELFs in the GUI since you otherwise would get the confusing behavior of editing the settings for all DOLs/ELFs instead of just the one you selected.

Actions #5

Updated by ryanebola16 over 6 years ago

I was in the middle of typing the same suggestion JosJuice made but he beat me to it. I agree.

Actions #6

Updated by merauder75 over 6 years ago

Okay, I'll work on implementing that, simply by checking if the game is in DOL/ELF format and throwing up a message stating game-specific overrides aren't available for it.

Actions #7

Updated by JosJuice over 6 years ago

I would suggest not showing the option of editing per-game settings for DOLs/ELFs rather than showing a message if you select the option, since that's how we do it on PC. (Or is it not in a menu?)

Actions #8

Updated by merauder75 over 6 years ago

It's implemented by long-pressing the game in the list of games, so a message of some sort may be appropriate.

Actions #9

Updated by JosJuice over 6 years ago

Yeah, I think it would be good to go with a message then.

Actions #10

Updated by ryanebola16 over 6 years ago

On Windows Dolphin you don't have the option to open game properties for DOL or ELF. Since the only Android long press options are game specific overrides, I'd rather have a message than users being confused as to why their long press does nothing for DOL or ELF.

Actions #12

Updated by JosJuice over 6 years ago

  • Status changed from Accepted to Fixed
  • Fixed in set to 5.0-8068
Actions

Also available in: Atom PDF