openedx / openedx-wordpress-ecommerce

You can sell your Open edX courses with WooCommerce using this free and open-source WordPress plugin.
https://docs.openedx.org/projects/wordpress-ecommerce-plugin/en/latest/index.html
GNU General Public License v2.0
11 stars 8 forks source link

["BUG"] Internationalization: Text domain does not match plugin slug. #87

Closed MaferMazu closed 1 month ago

MaferMazu commented 2 months ago

Describe the bug Comes from the WordPress Plugin Directory.

In order to make a string translatable in your plugin you are using a set of special functions. These functions collectively are known as "gettext".

These functions have a parameter called "text domain", which is a unique identifier for retrieving translated strings.

This "text domain" must be the same as your plugin slug so that the plugin can be translated by the community using the tools provided by the directory. As for example, if this plugin slug is penfold-macrame the Internationalization functions should look like: esc_html__('Hello', 'penfold-macrame');

From your plugin, you have set your text domain as follows:

This plugin is using the domain "woocommerce" for 8 element(s).

However, the current plugin slug is this:

openedx-commerce

MaferMazu commented 2 months ago

The problem is we use this kind of thing: https://github.com/openedx/openedx-wordpress-ecommerce/blob/v2.0.4/admin/class-openedx-commerce-admin.php#L219