Re: pgAdmin4 Chinese (Simplified) translation Patch

Thanks - applied.

On Wed, Nov 16, 2016 at 4:40 AM, Khushboo Vashi
<khushboo(dot)vashi(at)enterprisedb(dot)com> wrote:
> Hi Dave,
>
> I have fixed the issue regarding the nodes are not translated. Please find
> the attached patch for the same.
>
> Issue: We were translating the nodes label while registering the module,
> but at that time the translator does not get the context, so it returns the
> null translation. Now I have put the translation function at the collection
> property. So, whenever required, it will be translated.
>
>
> Thanks,
> Khushboo
>
>
>
>
>
>
>
> On Mon, Nov 14, 2016 at 11:03 PM, lambda <googletoday009(at)163(dot)com> wrote:
>>
>>
>> Hi,
>> this is the new updated version,
>> thank you!
>> :)
>>
>>
>>
>>
>>
>> At 2016-11-14 23:41:21, "Dave Page" <dpage(at)pgadmin(dot)org> wrote:
>> >Hi
>> >
>> >On Mon, Nov 14, 2016 at 3:22 PM, lambda <googletoday009(at)163(dot)com> wrote:
>> >>
>> >>
>> >> with web/config.py add
>> >>
>> >> ```
>> >> LANGUAGES = {
>> >> 'zh_cn' : 'zh_CN',
>> >> 'en' : 'English'
>> >> }
>> >> ```
>> >> ok ?
>> >
>> >Hmm, I'm sure that failed yesterday - I remember debugging it! It's
>> >working now though, so let's use zh_cn.
>> >
>> >BTW; the second field above is just a label, so I'd probably put
>> >"Chinese (Simplified)" in there.
>> >
>> >Can you update your catalog with the latest changes please?
>> >
>> >> 在 2016-11-14 20:16:47，"Khushboo Vashi"
>> >> <khushboo(dot)vashi(at)enterprisedb(dot)com> 写道：
>> >>
>> >>
>> >>
>> >> On Mon, Nov 14, 2016 at 5:37 PM, Dave Page <dpage(at)pgadmin(dot)org> wrote:
>> >>>
>> >>> Hi
>> >>>
>> >>> (Khushboo, please see below)
>> >>>
>> >>> On Sun, Nov 13, 2016 at 1:26 PM, lambda <googletoday009(at)163(dot)com>
>> >>> wrote:
>> >>> > for the first question, pybabel support Chinese with zh_Hans_CN
>> >>> >
>> >>> > #pybabel --list-locales |grep zh
>> >>> > b'zh Chinese'
>> >>> > b'zh_Hans Chinese (Simplified)'
>> >>> > b'zh_Hans_CN Chinese (Simplified, China)'
>> >>> > b'zh_Hans_HK Chinese (Simplified, Hong Kong SAR China)'
>> >>> > b'zh_Hans_MO Chinese (Simplified, Macau SAR China)'
>> >>> > b'zh_Hans_SG Chinese (Simplified, Singapore)'
>> >>> > b'zh_Hant Chinese (Traditional)'
>> >>> > b'zh_Hant_HK Chinese (Traditional, Hong Kong SAR China)'
>> >>> > b'zh_Hant_MO Chinese (Traditional, Macau SAR China)'
>> >>> > b'zh_Hant_TW Chinese (Traditional, Taiwan)'
>> >>>
>> >>> Right, but if I try to match with 'zh-CN' (which is what Chrome offers
>> >>> for Chinese Simplified), then I get:
>> >>>
>> >>> 2016-11-14 11:53:52,919: INFO werkzeug: 127.0.0.1 - - [14/Nov/2016
>> >>> 11:53:52] "GET / HTTP/1.1" 500 -
>> >>> Traceback (most recent call last):
>> >>> File
>> >>>
>> >>> "/Users/dpage/.virtualenvs/pgadmin4/lib/python2.7/site-packages/flask/app.py",
>> >>> line 2000, in __call__
>> >>> return self.wsgi_app(environ, start_response)
>> >>> File
>> >>>
>> >>> "/Users/dpage/.virtualenvs/pgadmin4/lib/python2.7/site-packages/flask/app.py",
>> >>> line 1991, in wsgi_app
>> >>> response = self.make_response(self.handle_exception(e))
>> >>> File
>> >>>
>> >>> "/Users/dpage/.virtualenvs/pgadmin4/lib/python2.7/site-packages/flask/app.py",
>> >>> line 1567, in handle_exception
>> >>> reraise(exc_type, exc_value, tb)
>> >>> File
>> >>>
>> >>> "/Users/dpage/.virtualenvs/pgadmin4/lib/python2.7/site-packages/flask/app.py",
>> >>> line 1988, in wsgi_app
>> >>> response = self.full_dispatch_request()
>> >>> File
>> >>>
>> >>> "/Users/dpage/.virtualenvs/pgadmin4/lib/python2.7/site-packages/flask/app.py",
>> >>> line 1634, in full_dispatch_request
>> >>> self.try_trigger_before_first_request_functions()
>> >>> File
>> >>>
>> >>> "/Users/dpage/.virtualenvs/pgadmin4/lib/python2.7/site-packages/flask/app.py",
>> >>> line 1660, in try_trigger_before_first_request_functions
>> >>> func()
>> >>> File "/Users/dpage/git/pgadmin4/web/pgadmin/utils/__init__.py", line
>> >>> 42, in create_module_preference
>> >>> self.register_preferences()
>> >>> File "/Users/dpage/git/pgadmin4/web/pgadmin/browser/__init__.py",
>> >>> line 192, in register_preferences
>> >>> gettext("Show system objects?"), 'boolean', False,
>> >>> File
>> >>>
>> >>> "/Users/dpage/.virtualenvs/pgadmin4/lib/python2.7/site-packages/flask_babel/__init__.py",
>> >>> line 539, in gettext
>> >>> t = get_translations()
>> >>> File
>> >>>
>> >>> "/Users/dpage/.virtualenvs/pgadmin4/lib/python2.7/site-packages/flask_babel/__init__.py",
>> >>> line 216, in get_translations
>> >>> catalog = support.Translations.load(dirname, [get_locale()])
>> >>> File
>> >>>
>> >>> "/Users/dpage/.virtualenvs/pgadmin4/lib/python2.7/site-packages/flask_babel/__init__.py",
>> >>> line 248, in get_locale
>> >>> locale = Locale.parse(rv)
>> >>> File
>> >>>
>> >>> "/Users/dpage/.virtualenvs/pgadmin4/lib/python2.7/site-packages/babel/core.py",
>> >>> line 270, in parse
>> >>> parts = parse_locale(identifier, sep=sep)
>> >>> File
>> >>>
>> >>> "/Users/dpage/.virtualenvs/pgadmin4/lib/python2.7/site-packages/babel/core.py",
>> >>> line 1096, in parse_locale
>> >>> raise ValueError('expected only letters, got %r' % lang)
>> >>> ValueError: expected only letters, got 'zh-cn'
>> >>>
>> >>> zh_Hans_CN isn't offered by the browser at all, so that doesn't work.
>> >>>
>> >>> Maybe the answer is to not use the browser's language negotiation at
>> >>> all, but make it a configurable setting?
>> >>>
>> >>> > 2.for the compile error, it seems an bug in pybabel choose
>> >>> > ignore
>> >>> > it,
>> >>> > it well ok.in my vim , line 7283 nearly it highlights '% u' in " 20%
>> >>> > use"
>> >>> > i do not known why both '%' and 'u' high here.
>> >>> > in my laptop, although it make an error massage as you show,
>> >>> > it
>> >>> > generate the po file ok!
>> >>> > now i update the 7283 line with '20% use' with '20%\
>> >>> > use'
>> >>> > it will compile without errror,
>> >>> > attatchment is updated version.
>> >>> > Thank you!
>> >>>
>> >>> In testing I found a couple of issues:
>> >>>
>> >>> 1) We weren't extracting strings from SQL templates for translation
>> >>>
>> >>> 2) Strings in some of the dashboards weren't marked for translation
>> >>>
>> >>> I've fixed those two issues and updated the template. Can you please
>> >>> update your translation?
>> >>>
>> >>> Khushboo; I also noticed that the treeview collection labels aren't
>> >>> getting translated. Can you look to see why please?
>> >>>
>> >>
>> >> Sure. Will look into this.
>> >>
>> >>>
>> >>> Thanks.
>> >>>
>> >>> --
>> >>> Dave Page
>> >>> Blog: http://pgsnake.blogspot.com
>> >>> Twitter: @pgsnake
>> >>>
>> >>> EnterpriseDB UK: http://www.enterprisedb.com
>> >>> The Enterprise PostgreSQL Company
>> >>
>> >>
>> >>
>> >>
>> >>
>> >
>> >
>> >
>> >--
>> >Dave Page
>> >Blog: http://pgsnake.blogspot.com
>> >Twitter: @pgsnake
>> >
>> >EnterpriseDB UK: http://www.enterprisedb.com
>> >The Enterprise PostgreSQL Company
>> >
>> >
>> >--
>> >Sent via pgadmin-hackers mailing list (pgadmin-hackers(at)postgresql(dot)org)
>> >To make changes to your subscription:
>> >http://www.postgresql.org/mailpref/pgadmin-hackers
>>
>>
>>
>>
>
>

--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake

EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company