商品名に「'」が入ると受注管理で商品を追加できない不具合の修正

[ryo-endo]

概要(Overview・Refs Issue)

開発コミュニティより
「商品名に「'」が入ると受注管理で商品を追加できない」
https://xoops.ec-cube.net/modules/newbb/viewtopic.php?topic_id=24166&forum=9

方針(Policy)

商品名をエスケープできていなかった。

実装に関する補足(Appendix)

エスケープには、他の箇所でも利用されている escape('js') を使いました。

テスト（Test)

・商品名に「'」が含まれていても、受注管理で商品を追加できることを確認。（指摘のケース）
・「'」が含まれていない商品も追加できることを確認。
・fnAddOrderItem関数でデバッグをして、商品名が正しく引数で渡っていることを確認。

相談（Discussion）

なし

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

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

レビュワー確認項目

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

[okazy]

動作確認して問題ありませんでした。修正内容も問題ないと思います。