CSVインポートの時に文字列に含まれる改行コードを残すようにする

[yshirai-joolen]

概要(Overview・Refs Issue)

商品CSV登録時に商品説明等のCSVフィールド内改行が消えるのを修正します
#4647

方針(Policy)

csvファイル読み出しのためのSplFileObjectを作る時にDROP_NEW_LINEフラグを付けないようにします。修正箇所は以下の二箇所です。

• Admin/Product/CsvImportController::splitCsv()で前処理としてアップロードされたcsvファイルを保存し直す時
• CsvImportServiceのコンストラクタで設定する時

また、以下のテストを修正しています。

• Eccube\Service\CsvImportService\testReadCsvFileWithTrailingBlankLines

テスト（Test)

空行が含まれるCSVインポートを行うテストに関して、空行を検知して無視する処理を追加しています。

相談（Discussion）

CsvImportServiceはOrderのcsvインポートでも使用していますので、問題がないかを気にしています

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

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

レビュワー確認項目

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

[chihiro-adachi]

@yshirai-joolen
途中に空行があるCSV(セル内の改行ではなく)はエラーになる、という認識であってますでしょうか？

[chihiro-adachi]

確認しました。
従来からの変更点として、空行がある場合に入力エラーとして扱うようになる仕様変更が入ります。

ただ、
・途中に空行があるCSVはあまり頻繁に発生するものではなさそう
・本修正で改善される内容のほうが有益
かと思いますのでマージしたいと思います

[yshirai-joolen]

@chihiro-adachi レスポンス遅れてしまってすみません。ありがとうございました。

[chihiro-adachi]

@yshirai-joolen ありがとうございます、マージしました。

[okazy]

以下の認識です。

修正前

csvの空行

"id","comment"
"1","hoge"

"2","piyo"

⬇️

"id","comment"
"1","hoge"
"2","piyo"

セル内の空行

"id","comment"
"1","hoge"
"2","piyo

piyp"

⬇️

"id","comment"
"1","hoge"
"2","piyo
piyp"

修正後

csvの空行

"id","comment"
"1","hoge"

"2","piyo"

⬇️

エラー

セル内の空行

"id","comment"
"1","hoge"
"2","piyo

piyp"

⬇️

"id","comment"
"1","hoge"
"2","piyo

piyp"