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
L10n #55
L10n #55
Conversation
Why is a global ini file necessary for the language configuration - shouldn't this be part of the ini files read from "Application Data/Uru Live"? |
That's precisely where it goes. I added a new one because the existing ones are specifically marked for audio or graphics. The discussion in IRC concluded that we may want to use additional global settings in the future, and this general.ini will allow that. |
I see... I guess I was confused because plClient runs all ini files from the Init directory, without any hard-coded names. But this one naturally has to be loaded earlier. |
Before this is merged, I'd like to be certain that none of the already existing code will not save localized strings to the vault. I seem to recall that someone (Paradox?) found some in the MOULa vault from someone enabling localization in their client. |
Since the consensus here seems to be that we want more review before merging this to master, could we at least pull it into a branch of H-uru/Plasma so that it gets a bit more visibility and more experienced users could test enabling it? |
I don't have time to do any work on it myself right now. Do I need to re-initiate a pull request into a new branch, or can that be done from this request? |
That can be done from here (or rather, from the command-line). |
That's what I thought, but I just wanted to be sure. :) |
Okay, now I'm a bit conflicted about this one after the discussion on IRC yesterday about the Vault nodes:
I think it's fairly likely that these are coming from the Python rather than from this code, but that Python doesn't cause problems unless this is enabled. |
Technically, these particular changes affect nothing as long as the client isn't set to another language; it still defaults to English. It would make it trivially easy for people to pollute the vault, however, as long as the cause in the Python isn't tracked down and fixed, but that's a separate issue. |
Has any progress been made on figuring out how internationalization interacts with the vault? This would be pretty cool to be able to enable, and it's a highly visible feature for users. |
Age names in the vault are translated because they use the same function that the KI uses for displaying them, which runs them through the xLocalization code. Ideally the names should be localized when they are read from the Vault, rather than storing them in localized form. |
The python fixes available now should address the concerns here. I believe it's ready for wider testing, if possible. |
Re-enable Localization. Fixes #12
Re-enable Localization. Fixes H-uru#12
Re-enable Localization. Fixes H-uru#12
Re-enable Localization. Fixes H-uru#12
Update ACA for Hor+
These changes re-enable the console command App.SetLanguage, allowing it to be set in an ini file. To make sure it's set early enough and to allow for future options being set via ini, I've added an optional general.ini for general settings to be read in before the login dialog is shown.
Also included is a fix for a crash that would occur when visiting Eder Tsogal using any language other than English due to an attempt to translate the name of a Scene Object in Tsogal with no name.
I've also removed the code which read from Windows' registry entry for the Uru uninstaller to determine a default language, as this is no longer necessary and is not cross-platform.