Skip to:

Opened 7 years ago

Closed 7 years ago

Last modified 7 years ago

#2555 closed enhancement (fixed)

Texts are not (properly) translated (bp-core-templatetags.php)

Reported by: helpy Owned by: sorich87
Milestone: 1.5 Priority: normal
Severity: Version:
Component: Core Keywords: needs-patch
Cc: gigalinux@…, Bountin



Have a look at the lines 926 and 928 in bp-core-templatetags.php:

The german translation depending on the loaded page is:

+ “Members Verzeichnis” … instead of: “Mitgliederverzeichnis” (engl.: “Members Directory”)
+ “Groups Verzeichnis” … instead of: “Gruppenverzeichnis” (engl.: “Groups Directory”)

For a proper translation, you should think about changing the source code! In german language “Mitgliderverzeichnis” (Members Directory) is ONE word! Therefore with the present code line …

$title = sprintf( __( ‘%s Directory’, ‘buddypress’ ), ucwords( BP_MEMBERS_SLUG ) );

Besides the that the words "groups, members, ..." will not be translated, it ist not possible to translate it properly! It is the same with other directory pages!

cu, guido

Attachments (3)

2555.patch (1.3 KB) - added by sorich87 7 years ago.
2555.diff (1.5 KB) - added by sorich87 7 years ago.
Removes "Directory"
patch_bp_gettext.txt (1.1 KB) - added by Bountin 7 years ago.

Download all attachments as: .zip

Change History (16)

#1 follow-up: @helpy
7 years ago

  • Milestone changed from 1.2.6 to 1.2.5

#2 in reply to: ↑ 1 @helpy
7 years ago

Sorry! I thought that "milestone" is the version where I found this issue.

@developers ... please change it to whatever fits your plans!

#3 @r-a-y
7 years ago

  • Milestone changed from 1.2.5 to 1.2.6

7 years ago

#4 @sorich87
7 years ago

  • Keywords has-patch added; I18N L10N Internationalization localization removed
  • Owner set to sorich87
  • Status changed from new to accepted

#5 @johnjamesjacoby
7 years ago

  • Milestone changed from 1.2.6 to 1.3

This works for core components, but not for external root components like Events, etc...

Bumping to 1.3 because this function needs a sweep anyhow.

#6 @paulhastings0
7 years ago

  • Summary changed from Texts are not (properly) translated (bp-core-templatetags.php) to [patch] Texts are not (properly) translated (bp-core-templatetags.php)

#7 @GIGALinux
7 years ago

I already had reported that bug and provided a patch that worked with plugins, too, but was marked as invalid :/ see #1364

#8 @GIGALinux
7 years ago

  • Cc gigalinux@… added

#9 @DJPaul
7 years ago

  • Keywords needs-patch added; has-patch removed
  • Summary changed from [patch] Texts are not (properly) translated (bp-core-templatetags.php) to Texts are not (properly) translated (bp-core-templatetags.php)

We need to think about this again for BP 1.3. The components now exists as a Page in the posts table. We need to be pulling in those pages' name from the database to accomodate users who want to rename the pages via the wp-admin. This will unfortunately lead back to the two-word translation problem that this ticket and #1364 has highlighted.

Using "groups directory" as an example, as that phrase only appears on the page template itself and in the page title, what about just using "groups" in the title, i.e. dropping "... directory"? That'd take care of both issues.

#10 @boonebgorges
7 years ago

+1 on removing the word 'Directory' altogether.

7 years ago

Removes "Directory"

#11 @Bountin
7 years ago

  • Cc Bountin added

Some lines above there is a similar problem because $bp->current_component isn't gettext'd (see patch suggestion)

Last edited 7 years ago by Bountin (previous) (diff)

#12 @DJPaul
7 years ago

  • Resolution set to duplicate
  • Status changed from accepted to closed

Closing this ticket in favour of #2595, as the two are effectively the same.

#13 @djpaul
7 years ago

  • Resolution changed from duplicate to fixed

(In [4194]) Improve page title i18n. Fixes #2595, #2555. Props for initial patches to SergeyBiryukov and sorich87.

Also fixes a couple of possible PHP notices.

Note: See TracTickets for help on using tickets.