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
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

droplet’s picture

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

droplet’s picture

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

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

jbrown’s picture

Status: Active » Needs work

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

droplet’s picture

Status: Needs work » Needs review
FileSize
611 bytes

yes, @jbrown right.

hansfn’s picture

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

This patch is fine!

catch’s picture

Issue tags: +Needs backport to D7

.

Dries’s picture

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.