The patch attached fixes many bugs in translatable strings like context sensitive issues, double blanks and make some strings a bit more self speaking.

I'm running this code and all looks good to me.

CommentFileSizeAuthor
taxonomy_breadcrumb_translatable_review.patch10.72 KBhass

Comments

MGN’s picture

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

Thanks for fixing this up!

MGN’s picture

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

Committed to 6.x-1.x-dev.

hass’s picture

hass commented
Status: Fixed » Needs work

I may missed one string...

Buggy

'#description'    => t('If enabled and if viewing a node, the node\'s title will be shown as the last item in the breadcrumb trail.'),

Correct:

'#description'    => t("If enabled and if viewing a node, the node's title will be shown as the last item in the breadcrumb trail."),
MGN’s picture

MGN commented
Status: Needs work » Reviewed & tested by the community

Ok. I'll fix that on the next commit.

MGN’s picture

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

Committed.

Status: Fixed » Closed (fixed)

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