Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bring back custom translation files for BuddyPress! #316

Closed
wants to merge 3 commits into from

Conversation

imath
Copy link
Member

@imath imath commented Jun 16, 2024

Use the 'load_translation_file' filter to check custom file locations for the buddypress text domain.

NB: this filter was introduced by WordPress in version 6.5.

If there's a buddypress-xx_XX.mo (where xx_XX is the locale code) file into one of the following directories, it will be used to replace the default one (WordPress.org Plugin Directory / GlotPress):

  1. /wp-content/languages/plugins/buddypress (added in 14.0.0)
  2. /wp-content/languages/buddypress
  3. /wp-content/languages

Trac ticket: https://buddypress.trac.wordpress.org/ticket/9187


This Pull Request is for code review only. Please keep all other discussion in the BuddyPress Trac ticket. Do not merge this Pull Request. See GitHub Pull Requests for Code Review in the WordPress Core Handbook for more details.

Copy link
Member

@renatonascalves renatonascalves left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🌵

src/bp-core/bp-core-functions.php Show resolved Hide resolved
imath and others added 3 commits June 23, 2024 23:30
Use the `'load_translation_file'` filter to check custom file locations for the `buddypress` text domain.

**NB**: this filter was introduced by WordPress in version 6.5.

If there's a `buddypress-xx_XX.mo` (where xx_XX is the locale code) file into one of the following directories, it will be used to replace the default one (WordPress.org Plugin Directory / GlotPress).
…nslation_file'` filter

This will be the case for WordPress 6.5 as `$locale` 3rd filter argument was introduced in WP 6.6.
@imath
Copy link
Member Author

imath commented Jun 23, 2024

FYI, I've also added a function to override default JavaScript translation files used in BP Blocks.

@imath imath deleted the patch/9187 branch June 24, 2024 02:08
dcavins pushed a commit to dcavins/buddypress-wp-svn that referenced this pull request Jul 3, 2024
Prior to WordPress 4.6, it was possible to completely override the translation file with a custom one from one of the `'buddypress_locale_locations'` filterable list items.

With WordPress 6.5 latest improvements about l10n, we are now able to bring these back. Making it easier for users to customize every BuddyPress strings should help them build match their community needs (e.g.: using sport/school/working/any other wording).

3 functions has been introduced:

-  `bp_get_custom_translation_file()` is looking for a file name into `'buddypress_locale_locations'` filterable list items.
- `bp_load_custom_translation_file()` is the `'load_translation_file'` filter callback for regular translation files.
- `bp_load_custom_script_translation_file()` is the `'load_script_translation_file'` filter callback for JavaScript translation files.

The `bp_core_load_buddypress_textdomain()` function has been deprecated as it was pretty useless since WordPress 4.6.

Props espellcaste

Fixes #9187
Closes buddypress/buddypress#316



git-svn-id: http:https://buddypress.svn.wordpress.org/trunk@13936 cdf35c40-ae34-48e0-9cc9-0c9da1808c22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants