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 <googletoday...@163.com> wrote:

>
> Hi,
>     this is the new updated version,
>     thank you!
>     :)
>
>
>
>
>
> At 2016-11-14 23:41:21, "Dave Page" <dp...@pgadmin.org> wrote:
> >Hi
> >
> >On Mon, Nov 14, 2016 at 3:22 PM, lambda <googletoday...@163.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.va...@enterprisedb.com> 
> >> 写道:
> >>
> >>
> >>
> >> On Mon, Nov 14, 2016 at 5:37 PM, Dave Page <dp...@pgadmin.org> wrote:
> >>>
> >>> Hi
> >>>
> >>> (Khushboo, please see below)
> >>>
> >>> On Sun, Nov 13, 2016 at 1:26 PM, lambda <googletoday...@163.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@postgresql.org)
> >To make changes to your subscription:
> >http://www.postgresql.org/mailpref/pgadmin-hackers
>
>
>
>
>
diff --git a/web/pgadmin/browser/collection.py b/web/pgadmin/browser/collection.py
index ed36e85..e4a83a2 100644
--- a/web/pgadmin/browser/collection.py
+++ b/web/pgadmin/browser/collection.py
@@ -126,11 +126,11 @@ class CollectionNodeModule(PgAdminModule, PGChildModule):
         Label to be shown for the collection node, do not forget to set the
         class level variable COLLECTION_LABEL.
         """
-        return self.COLLECTION_LABEL
+        return gettext(self.COLLECTION_LABEL)
 
     @property
     def label(self):
-        return self.COLLECTION_LABEL
+        return gettext(self.COLLECTION_LABEL)
 
     @property
     def collection_icon(self):
-- 
Sent via pgadmin-hackers mailing list (pgadmin-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgadmin-hackers

Reply via email to