Support » Plugin: Remove admin menus by roles » L10N Issue

  • Resolved Alex Lion

    (@alexclassroom)


    This plugin’s slug is remove-admin-menus-by-role, but the current Text Domain is remove-admin-menus-by-roles. Change the current Text Domain to equal to your slug and modify the text domain in all your source files. This change is needed because your “Requires at least” is below 4.6. (see https://developer.wordpress.org/plugins/internationalization/how-to-internationalize-your-plugin/#text-domains)

    The modifications are the following:

    remove-admin-menus-by-role.php
    Line 6
    Description: Remove admin menus by roles.
    Adding a dot can make the plugin’s description localizable because it is split from the plugin name.

    Line 10
    Text Domain: remove-admin-menus-by-role
    Fix text domain

    Line 198
    load_plugin_textdomain( 'remove-admin-menus-by-role', FALSE, basename( dirname( __FILE__ ) ) . '/languages/' );

    views\form.php
    Line 31, 43, 44, 74, 78
    Text domain should be remove-admin-menus-by-role.

Viewing 5 replies - 1 through 5 (of 5 total)
  • Plugin Author manu225

    (@manu225)

    Hi,

    Thank you for these tips 🙂
    I’ll change this in the next version.

    PS : this plugin is already translated in 4 languages. So changing the slug in the function load_plugin_textdomain will not broke these translations?

    Thread Starter Alex Lion

    (@alexclassroom)

    Correcting current PO and MO filename from remove-admin-menus-by-roles to remove-admin-menus-by-role is fine.

    I took a look for the current installation file, it contained fr_FR language pack, so these files should be renamed as the following:

    remove-admin-menus-by-role-fr_FR.mo and remove-admin-menus-by-role-fr_FR.po

    But I strongly recommend you can gain fr_FR PTE permission to upload and approve fr_FR translations so that you won’t need to package fr_FR language pack to the plugin installation file.

    You can also remove POT file, if this file is out of date, this means it is useless for localization.

    Whatever users use Poedit or Loco Translate, generating a POT file from source code is easy.

    I always generate the newest POT file from source code by myself, not use the attached POT file in a ZIP file.

    • This reply was modified 10 months ago by Alex Lion.
    Plugin Author manu225

    (@manu225)

    Hi,

    Thanks for your answer.
    I just released the v1.32 with these fixes.
    For the french translation, i don’t understand how it works, beacause long time ago some people and i have translated the strings here: https://translate.wordpress.org/locale/fr/default/wp-plugins/remove-admin-menus-by-role/
    But it stay at waiting/fuzzy state?

    • This reply was modified 10 months ago by manu225.
    Plugin Author manu225

    (@manu225)

    Closing topic

    Thread Starter Alex Lion

    (@alexclassroom)

    You need to get fr PTE permission, like zh_TW, and then you can approve fr translations by yourself.

Viewing 5 replies - 1 through 5 (of 5 total)
  • The topic ‘L10N Issue’ is closed to new replies.