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
Translation issue send-friend in sendphtml #18779
Comments
Hi @Franzld. Thank you for your report.
Please make sure that the issue is reproducible on the vanilla Magento instance following Steps to reproduce. To deploy vanilla Magento instance on our environment, please, add a comment to the issue:
where @Franzld do you confirm that you was able to reproduce the issue on vanilla Magento instance following steps to reproduce?
|
To reproduce the issue without install a translation, you may edit
replace
|
Hi @hardik-krish. Thank you for working on this issue.
|
I'm sorry, but I'm not a developer and have only rudimentary programming skills. |
@Franzld - i checked this and there is no issue. You need to have utf-8 encoding in the csv file. |
Thanks for your answer! please try to replace in if there's no issue at your place, something with my server-config may be wrong. |
@Franzld - Yes, I am able to reproduced the issue even with UTF-8 encoding. |
#SQUASHTOBERFEST |
Hi @rahulwebkul. Thank you for working on this issue.
|
#SQUASHTOBERFEST |
Hi @Franzld. Thank you for your report.
The fix will be available with the upcoming 2.3.1 release. |
Preconditions (*)
Steps to reproduce (*)
Set shop-> config->catalog->"Email to a friend" -> yes
Expected result (*)
Actual result (*)
Fixing the issue (*)
/vendor/magento/module-send-friend/view/frontend/templates/send.phtml
remove "escapeJs" in line 34 :
<label for="recipients-email<%- data._index_ %>" class="label"><span><?= $block->escapeJs($block->escapeHtml(__('Email'))) ?></span></label>
change to:
"<label for="recipients-email<%- data._index_ %>" class="label"><span><?= $block->escapeHtmlAttr(__('Email')) ?></span></label>"
The text was updated successfully, but these errors were encountered: