On Thu, Oct 19, 2017 at 6:08 AM, Alexander Lakhin <exclusion@gmail.com> wrote:
Hello,
I would like to to add Russian translation to pgAdmin 4, but having complete messages.po, I get incomplete UI translation. (See ss1.png attached.) I found two issues with the i18n in the current master. The first one is with web/pgadmin/tools/templates/js/translations.js. I don't understand why, but commit 4a91bcde makes the function return an empty array instead of translations. Patch for fixing it is attached.
The second one is more complex and related to get_locale() method. This method is called when first gettext() invoked and it occurs on early stage, where the Preference module 'miscellaneous' is not registered yet. To be precise, first call to gettext() occurs when registering the 'Browser' module. But even if MiscModule will register first, it had to call _('Miscellaneous') before it gets registered. So I propose to implement some way to get current language when the misc preference module is not registered. Please look at the proposed patch.
After these two fixes I get (almost) complete Russian interface (See ss2.png attached.) (May be some more strings should be made localizeable, but that is another question.)
