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...
Comment | File | Size | Author |
---|---|---|---|
#8 | remove-t-for-label-1672694-8.patch | 633 bytes | jibran |
#3 | remove-t-for-label-1672694-3.patch | 651 bytes | JulienD |
#1 | remove-t-for-label-1672694-1.patch | 653 bytes | Devin Carlson |
field_ui-remove-t-for-label.patch | 633 bytes | kalabro | |
Comments
Comment #1
Devin Carlson CreditAttribution: Devin Carlson commentedRerolled for Drupal 8.
Comment #2
Devin Carlson CreditAttribution: Devin Carlson commentedThis 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).
Comment #3
JulienD CreditAttribution: JulienD commentedI just regenerated your patch and I didn't find any other occurrences
Comment #4
webflo CreditAttribution: webflo commentedComment #5
webflo CreditAttribution: webflo commentedField labels should be localized in CMI and not through t().
Comment #6
JulienD CreditAttribution: JulienD commented#3: remove-t-for-label-1672694-3.patch queued for re-testing.
Comment #7
catchCommitted/pushed to 8.x, thanks!
Comment #8
jibranD7 patch
Comment #9
jibranstatus updated
Comment #10
michaelmol CreditAttribution: michaelmol commentedReviewed and tested. Patch looks fine.
Comment #11
David_Rothstein CreditAttribution: David_Rothstein commentedCommitted to 7.x - thanks! http://drupalcode.org/project/drupal.git/commit/d1f4ea3
Comment #13
colan