機種依存文字が文字化けする不具合を修正

[chihiro-adachi]

概要(Overview・Refs Issue)

#4781 機種依存文字が文字化けする不具合の対応

方針(Policy)

本来はStringUtil::characterEncodingのデフォルトの検出順位を修正したほうがよいかと思いましたが、全体に影響が及ぶため該当箇所のみの修正としています。

テスト（Test)

• ユニットテストを追加しました。

マイナーバージョン互換性保持のための制限事項チェックリスト

• 既存機能の仕様変更
• フックポイントの呼び出しタイミングの変更
• フックポイントのパラメータの削除・データ型の変更
• twigファイルに渡しているパラメータの削除・データ型の変更
• Serviceクラスの公開関数の、引数の削除・データ型の変更
• 入出力ファイル(CSVなど)のフォーマット変更

レビュワー確認項目

• 動作確認
• コードレビュー
• E2E/Unit テスト確認(テストの追加・変更が必要かどうか)
• 互換性が保持されているか
• セキュリティ上の問題がないか

[kiy0taka]

優先順位は sjis-win を決め打ちで先にするのではなく、configの eccube_csv_export_encoding を先にするとしたほうがいいと思います。

[chihiro-adachi]

@kiy0taka
ありがとうございます。eccube_csv_export_encodingは出力用のエンコーディング指定なので、別途configにパラメータを設ける形になりますね。
ただ、sjis-winから変更したいというユースケースもあまり考えにくく、決め打ちでもいいのかなと思った次第です。

[kiy0taka]

元のIssue #4781 の、ダウンロードしたCSVをアップロードして文字化けするというのに対する対応だと思うので、eccube_csv_export_encoding に合わせるでいいかと思います。この設定を変えたときにロジックも変更しないといけなくなるので、決め打ちではまずいと思います。
アップロード用に別の設定を設けるかは必要あれば別途検討したいと思います。

[chihiro-adachi]

@kiy0taka

元のIssue #4781 の、ダウンロードしたCSVをアップロードして文字化けするというのに対する対応だと思うので、eccube_csv_export_encoding に合わせるでいいかと思います。この設定を変えたときにロジックも変更しないといけなくなるので、決め打ちではまずいと思います。

そういうことですね。理解しました。

[chihiro-adachi]

@kiy0taka
修正しました。