Products.cmfplone: Wrong default values for msgid label_email

Created on 8 Mar 2019  路  2Comments  路  Source: plone/Products.CMFPlone

Most helpful comment

I guess you saw the i18ndude warnings. :) Indeed Plone 5.2 currently gives

Warning: msgid 'label_email' in plone.app.users/plone/app/users/browser/membersearch.py:29 already exists with a different default (bad: E-mail, should be: Email)
The references for the existent value are:
Products.CMFPlone/Products/CMFPlone/browser/login/login_help.py:61
Products.CMFPlone/Products/CMFPlone/browser/login/login.py:109
Warning: msgid 'label_email' in plone.app.users/plone/app/users/schema.py:67 already exists with a different default (bad: E-mail, should be: Email)
The references for the existent value are:
Products.CMFPlone/Products/CMFPlone/browser/login/login_help.py:61
plone.app.users/plone/app/users/browser/membersearch.py:29
Products.CMFPlone/Products/CMFPlone/browser/login/login.py:109
Warning: msgid 'label_email' in plone.app.users/plone/app/users/schema.py:82 already exists with a different default (bad: E-mail, should be: Email)
The references for the existent value are:
Products.CMFPlone/Products/CMFPlone/browser/login/login_help.py:61
plone.app.users/plone/app/users/browser/membersearch.py:29
Products.CMFPlone/Products/CMFPlone/browser/login/login.py:109
plone.app.users/plone/app/users/schema.py:67
Warning: msgid 'label_email' in Products.CMFPlone/Products/CMFPlone/browser/login/login.py:109
Products.CMFPlone/Products/CMFPlone/browser/login/login_help.py:61
plone.app.users/plone/app/users/browser/membersearch.py:29
plone.app.users/plone/app/users/schema.py:67
plone.app.users/plone/app/users/schema.py:82 already exists with a different default (bad: Email, should be: E-mail)
The references for the existent value are:
plone.app.portlets/plone/app/portlets/portlets/login.pt:46

The message extracted is label_email, default="E-mail".

It would be great if the default message could be the same everywhere.
From the discussion on https://github.com/plone/Products.CMFPlone/pull/2791 with
@tkimnguyen @MatthewWilkes @jensens @thet
it seems we're in favor for Email.
Feel free to create a PR for each package to change it.

All 2 comments

I guess you saw the i18ndude warnings. :) Indeed Plone 5.2 currently gives

Warning: msgid 'label_email' in plone.app.users/plone/app/users/browser/membersearch.py:29 already exists with a different default (bad: E-mail, should be: Email)
The references for the existent value are:
Products.CMFPlone/Products/CMFPlone/browser/login/login_help.py:61
Products.CMFPlone/Products/CMFPlone/browser/login/login.py:109
Warning: msgid 'label_email' in plone.app.users/plone/app/users/schema.py:67 already exists with a different default (bad: E-mail, should be: Email)
The references for the existent value are:
Products.CMFPlone/Products/CMFPlone/browser/login/login_help.py:61
plone.app.users/plone/app/users/browser/membersearch.py:29
Products.CMFPlone/Products/CMFPlone/browser/login/login.py:109
Warning: msgid 'label_email' in plone.app.users/plone/app/users/schema.py:82 already exists with a different default (bad: E-mail, should be: Email)
The references for the existent value are:
Products.CMFPlone/Products/CMFPlone/browser/login/login_help.py:61
plone.app.users/plone/app/users/browser/membersearch.py:29
Products.CMFPlone/Products/CMFPlone/browser/login/login.py:109
plone.app.users/plone/app/users/schema.py:67
Warning: msgid 'label_email' in Products.CMFPlone/Products/CMFPlone/browser/login/login.py:109
Products.CMFPlone/Products/CMFPlone/browser/login/login_help.py:61
plone.app.users/plone/app/users/browser/membersearch.py:29
plone.app.users/plone/app/users/schema.py:67
plone.app.users/plone/app/users/schema.py:82 already exists with a different default (bad: Email, should be: E-mail)
The references for the existent value are:
plone.app.portlets/plone/app/portlets/portlets/login.pt:46

The message extracted is label_email, default="E-mail".

It would be great if the default message could be the same everywhere.
From the discussion on https://github.com/plone/Products.CMFPlone/pull/2791 with
@tkimnguyen @MatthewWilkes @jensens @thet
it seems we're in favor for Email.
Feel free to create a PR for each package to change it.

Was this page helpful?
0 / 5 - 0 ratings