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

Suppressed warning about banks with locales not matching setting, when they actually do #76

Open
wants to merge 2 commits into
base: 2.02-UE5.1
Choose a base branch
from

Conversation

gamecentric
Copy link

Banks with non-default locale are never recognized even if suffix is correct

@fmod-cameron
Copy link
Contributor

What do you mean by they are never recognized? The banks are stored by GUID and localized banks all share the same GUID.

@gamecentric
Copy link
Author

In the code affected by my PR, banks paths are being compared, not GUIDs.

@fmod-cameron
Copy link
Contributor

Here the bank strings are compared to determine which banks to add to the map of bank guids/paths, the strings are compared because localized banks all share the same GUID. The map uses the GUID as the key so you cannot have multiple entries using the same key, which is why we only want to use the locale that is marked as 'bDefault'.

Are you having any issues or errors related to this?

@fmod-cameron fmod-cameron self-assigned this Jun 2, 2023
@gamecentric
Copy link
Author

The issue is that the warning message "Ignoring bank %s as another bank with the same GUID is already being used.\nBank %s does not match any locales in the FMOD Studio plugin settings." (see line 280) is always output for all non-default banks even if the bank actually matches locales in the settings.

@gamecentric
Copy link
Author

Now I see, My original PR was wrong since the guid must be added to the map only for default locales. I updated the PR, so that only the warning is suppressed for non-default locales, while keeping the original logic.

@gamecentric gamecentric changed the title Fixed bug: banks with non-default locale are never recognized Suppressed warning about banks with locales not matching setting, when they actually do Jun 5, 2023
@fmod-cameron
Copy link
Contributor

Thanks, I have raised this a task that will be scheduled for a future release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants