There are two issues with the usage of drupal_set_title in the forum overview function. The first is that it's already set to "Forum" by forum_menu, the second is that it's not using the t() function.

Patch removed the drupal_set_title call from the function.

CommentFileSizeAuthor
forum_overview.patch626 bytesbeeradb

Comments

pwolanin’s picture

pwolanin commented

Actually 3rd issue is that drupal_set_title is being called in a form builder, which is not supposed to have such side-effects.

beeradb’s picture

beeradb commented

Just a note, this is present in 6.x as well. As trivial as this patch is, it should get backported if committed.

Anonymous’s picture

Anonymous (not verified) commented
Status: Needs review » Needs work

The last submitted patch failed testing.

casey’s picture

casey commented
Status: Needs work » Fixed

http://api.drupal.org/api/function/forum_overview/7

casey’s picture

casey commented
Version: 7.x-dev » 6.x-dev
Status: Fixed » Active
andypost’s picture

andypost
he/him
Primary language Russian
commented
Status: Active » Fixed

D6 is not affected

Related issue #148145: "Forums" title is not localized

Status: Fixed » Closed (fixed)

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