It is not possible to localize the title of the forum administration page (admin/content/forum). This is caused by a drupal_set_title() without a call to t() in forum_overview(). AFAICT this call to drupal_set_title() is unnecessary.

The attached patch fixes this.

This problem exists in Drupal 6.x as well.

Also, this doesn't seem to be a duplicate of #148145 (which deals with the userfacing part of the forum).

CommentFileSizeAuthor
#4 forum-admin-D6.patch707 bytesandypost
#2 forum-460420-admin-l10n.patch695 byteswulff
forum.admin_.inc_.patch610 byteswulff
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Status: Needs review » Needs work

The last submitted patch failed testing.

wulff’s picture

wulff CreditAttribution: wulff commented
Status: Needs work » Needs review
FileSize
forum-460420-admin-l10n.patch695 bytes

...and now with a patch made according to spec.

Dries’s picture

Dries CreditAttribution: Dries commented
Version: 7.x-dev » 6.x-dev

Committed to CVS HEAD. Updating the version number.

andypost’s picture

andypost
he/him
Primary language Russian
CreditAttribution: andypost commented
FileSize
forum-admin-D6.patch707 bytes

Here 6-dev patch

Ilya1st’s picture

Ilya1st CreditAttribution: Ilya1st commented

drupal_set_title call is not necessary there.
Title goes from hook_menu() already.

Need backport to 6.x

andypost’s picture

andypost
he/him
Primary language Russian
CreditAttribution: andypost commented
Issue tags: +Needs backport to D6, +Quick fix

Just new tags

danSamara’s picture

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

Easy and useful patch.

pvasili’s picture

pvasili CreditAttribution: pvasili commented

This patch (#4) should be used in the 6 versions to!
Drupal forum without it looks no good :(

andypost - thank you for your work.

VladSavitsky’s picture

VladSavitsky CreditAttribution: VladSavitsky commented

This problem should be sovled in 6.x too.

Gábor Hojtsy’s picture

Gábor Hojtsy
he/him
Primary language Hungarian
Location Hungary
CreditAttribution: Gábor Hojtsy commented
Status: Reviewed & tested by the community » Fixed

Thanks, committed to Drupal 6. The change will be out with the next Drupal core release.

Status: Fixed » Closed (fixed)
Issue tags: -Needs backport to D6, -Quick fix, -l10n

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

Issue tags: +Needs backport to D6, +Quick fix, +l10n

Restoring issue tags, see #2125755: System messages removed all issue tags during D7 upgrade.