This string use the esc_attr() function and are not translated

Hi @itapress ,

Thank you for reaching out to us.

Could you please share with us which string/path you want to be translated?

Awaiting your reply.

Hello @buildwps

Thank you for the reply.

I am referring to the placeholder string “Enter one password per line” which uses the esc_attr() function and not the esc_attr__() function which allows translation. Then the text domain should also be corrected from ‘wp-protect-password’ to PPW_Constants::DOMAIN.

Thanks.

Hello @buildwps

Thank you for the reply.

I saw that the fix was made in version 1.9.2.

But I think it’s better to delete the unnecessary text domain reference where is ‘<?php?echo?esc_html(?$password,?PPW_Constants::DOMAIN?);??>‘ in this line:

https://plugins.trac.www.ads-software.com/browser/password-protect-page/tags/1.9.2/includes/views/meta-box/view-ppw-meta-box.php#L88

Thanks.

Hi @itapress ,

Sorry for the late response.

>>>I am referring to the placeholder string “Enter one password per line” which uses the?esc_attr()?function and not the?esc_attr__()?function which allows translation. Then the text domain should also be corrected from?‘wp-protect-password’?to?PPW_Constants::DOMAIN.

Yes, we have already fixed it in version 1.9.2 by replacing ‘wp-protect-password’ with PPW_Constants::DOMAIN.

>>>But I think it’s better to delete the unnecessary?text domain?reference where is ‘<?php?echo?esc_html(?$password,?PPW_Constants::DOMAIN?);??>‘ in this line:

Thank you again for your feedback. We truly appreciate it.

I have notified our development team, and they will remove any unnecessary text domains in the plugin in the next version.