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

メールテンプレートのエンコードがJISでは「はしご高」など文字化けもしくは空白になってしまう #373

Merged
merged 2 commits into from
Mar 12, 2020

Conversation

nobuhiko
Copy link
Contributor

@nobuhiko nobuhiko commented Feb 6, 2020

ISO-2022-JP-MSを使用する
PHP 5.2.1 以降になったので使用可能

@nobuhiko nobuhiko changed the title JISでははしご高など文字化けもしくは空白になってしまう メールテンプレートのエンコードがJISでは「はしご高」など文字化けもしくは空白になってしまう Feb 6, 2020
@3tiles
Copy link
Contributor

3tiles commented Feb 6, 2020

$_name = mb_encode_mimeheader($_name, 'JIS', 'B', "\n");

の部分も対応の必要とかないでしょうか?

@3tiles
Copy link
Contributor

3tiles commented Feb 6, 2020

あと、

$this->tpl_body = mb_convert_encoding($objSendMail->body, CHAR_CODE, 'auto');

の部分も、
$this->tpl_body = mb_convert_encoding($objSendMail->body, CHAR_CODE, 'ISO-2022-JP-MS');
とか
$this->tpl_body = mb_convert_encoding($objSendMail->body, CHAR_CODE, 'ISO-2022-JP-MS,auto');
とか。

@nobuhiko
Copy link
Contributor Author

nobuhiko commented Feb 6, 2020

@3tiles
ありがとうございます
LC_Page_Admin_Order_Mail.php はUTF-8で元々化けないので問題ないです

SC_SendMail.php は追加で修正します

@okazy okazy added the bug label Feb 7, 2020
@okazy okazy added this to the 2.17.1 milestone Feb 7, 2020
@okazy okazy merged commit e97bcc7 into EC-CUBE:master Mar 12, 2020
@okazy
Copy link
Contributor

okazy commented Mar 12, 2020

ありがとうございます。取り込みました。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants