I found my fields' labels on 'admin/config/regional/translate/translate' page because they are outputed with t() in _add_existing_field select list.
The user finds field label on 'admin/config/regional/translate/translate' page, translates it and expects that this translation will appear in entity view but it doesn't according to #1157426: Field system uses t() incorrectly and inconsistently.
After all this translated label is set as default label for new instance of existing field which not always useful.

Came from here: http://drupal.stackexchange.com/questions/36065/interface-translation-of...

CommentFileSizeAuthor
#8 remove-t-for-label-1672694-8.patch633 bytesjibran
PASSED: [[SimpleTest]]: [MySQL] 39,574 pass(es). View
#3 remove-t-for-label-1672694-3.patch651 bytesJulienD
PASSED: [[SimpleTest]]: [MySQL] 47,854 pass(es). View
#1 remove-t-for-label-1672694-1.patch653 bytesDevin Carlson
PASSED: [[SimpleTest]]: [MySQL] 37,262 pass(es). View
field_ui-remove-t-for-label.patch633 byteskalabro
FAILED: [[SimpleTest]]: [MySQL] Unable to apply patch field_ui-remove-t-for-label.patch. Unable to apply patch. See the log in the details link for more information. View
Members fund testing for the Drupal project. Drupal Association Learn more

Comments

Devin Carlson’s picture

Version: 7.x-dev » 8.x-dev
Status: Active » Needs review
Issue tags: +needs backport to D7
FileSize
653 bytes
PASSED: [[SimpleTest]]: [MySQL] 37,262 pass(es). View

Rerolled for Drupal 8.

Devin Carlson’s picture

Issue tags: +Novice

This looks like a good novice issue (check if the old patch still applies, check if any translation issues occur as a result of the change, reroll or update as necessary).

JulienD’s picture

FileSize
651 bytes
PASSED: [[SimpleTest]]: [MySQL] 47,854 pass(es). View

I just regenerated your patch and I didn't find any other occurrences

webflo’s picture

Issue tags: +D8MI
webflo’s picture

Status: Needs review » Reviewed & tested by the community

Field labels should be localized in CMI and not through t().

JulienD’s picture

#3: remove-t-for-label-1672694-3.patch queued for re-testing.

catch’s picture

Version: 8.x-dev » 7.x-dev
Status: Reviewed & tested by the community » Patch (to be ported)

Committed/pushed to 8.x, thanks!

jibran’s picture

FileSize
633 bytes
PASSED: [[SimpleTest]]: [MySQL] 39,574 pass(es). View

D7 patch

jibran’s picture

Status: Patch (to be ported) » Needs review

status updated

michaelmol’s picture

Status: Needs review » Reviewed & tested by the community

Reviewed and tested. Patch looks fine.

David_Rothstein’s picture

Status: Reviewed & tested by the community » Fixed

Automatically closed -- issue fixed for 2 weeks with no activity.

colan’s picture