in reviewing the 5.x port (http://drupal.org/node/99759) chx pointed out a line that was wrapped in t() which has no translatable parts. ;)

$link = l(t('%name (%count)', array('%name' => $term->name, '%count' => $term->count)), check_url("$path/$term->tid"));

furthermore, l() validates both its args (check_plain() on the title, check_url() on the link) unless you go out of your way to specify TRUE for the last argument ($html).

so, this just needs to be:

$link = l($term->name .' ('. $term->count .')', $path/$term->tid);

can i get an amen? ;)

CommentFileSizeAuthor
#3 project_bogus_t.patch_1.txt847 bytesdww
project_bogus_t.patch.txt845 bytesdww

Comments

heine’s picture

Status: Needs review » Reviewed & tested by the community

amen.

heine’s picture

Status: Reviewed & tested by the community » Needs work

Needs quotes: $path/$term->tid

dww’s picture

Status: Needs work » Needs review
StatusFileSize
new847 bytes
heine’s picture

Status: Needs review » Reviewed & tested by the community

amen II.

dww’s picture

Status: Reviewed & tested by the community » Fixed

committed to HEAD, DRUPAL-4-7--2 and DRUPAL-4-7.
thanks, heine. ;)

Anonymous’s picture

Status: Fixed » Closed (fixed)