On admin/structure/types, the machine name of a content type is displayed as "(Machine name: page)". The "Machine name" string isn't translatable. Trivial patch attached.

CommentFileSizeAuthor
#5 node.patch611 bytesdroplet
hansfn-2011-05-06.patch608 byteshansfn

Comments

droplet’s picture

droplet commented

can it include all "(Machine name: page)" in t function. because some lang is using different "()" and ":"
thanks.

droplet’s picture

droplet commented
Version: 7.x-dev » 8.x-dev
hansfn’s picture

hansfn commented

droplet, I don't understand your comment. (My patch just translates "Machine name".)

jbrown’s picture

jbrown commented
Status: Active » Needs work

The brackets and colon should be in the translatable string. Use replacements.

droplet’s picture

droplet commented
Status: Needs work » Needs review
StatusFileSize
new node.patch611 bytes

yes, @jbrown right.

hansfn’s picture

hansfn commented
Status: Needs review » Reviewed & tested by the community

OK, next time I'll use replacements. Tested node.patch and it worked fine (of course). Thx.

PS! Do we need to tag it so the fix is backported to 7.x dev?

jbrown’s picture

jbrown commented

This patch is fine!

catch’s picture

catch
he/him
Primary language English
commented
Issue tags: +Needs backport to D7

.

dries’s picture

dries commented
Status: Reviewed & tested by the community » Fixed

Committed to 7.x and 8.x.

(Working from the road so can't review big complex issues right now -- picking away some smaller patches that are unlikely to break bigger patches.)

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