Skip to content

[コア]CSVの項目にダブルクォーテーションが含まれると正しく出力されない事象を修正しました#2116

Merged
masaton0216 merged 3 commits into
masterfrom
escape-double-quote-in-csv
Feb 6, 2025
Merged

[コア]CSVの項目にダブルクォーテーションが含まれると正しく出力されない事象を修正しました#2116
masaton0216 merged 3 commits into
masterfrom
escape-double-quote-in-csv

Conversation

@gakigaki

@gakigaki gakigaki commented Feb 5, 2025

Copy link
Copy Markdown
Contributor

概要

CSV出力時、項目にダブルクォーテーションが含まれる場合、正しくエスケープするよう修正しました。

変更の目的

ExcelなどのスプレッドシートソフトでCSVを開いた際に、ダブルクォーテーションが正しく表示されない問題を解決するため。

変更内容

  • ダブルクォーテーションをエスケープする処理を追加しました。

テスト

  • ダブルクォーテーションを含むCSVを出力し、Excelで開いて正しく表示されることを確認しました。

影響範囲

CSVを出力する以下の機能に適応されます。

  • コード管理
  • ユーザ管理
  • フォームプラグイン
  • カウンタープラグイン
    • 項目にダブルクォーテーションが入ることはない
  • データベースプラグイン
  • 課題管理プラグイン
  • OPACプラグイン

レビュー完了希望日

関連Pull requests/Issues

参考

DB変更の有無

無し

チェックリスト

@gakigaki
gakigaki requested a review from masaton0216 February 5, 2025 03:57
@gakigaki
gakigaki marked this pull request as ready for review February 5, 2025 06:14

@masaton0216 masaton0216 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

対応ありがとうございます。
不幸なことにフォームプラグインがCsvUtilsを介していないようで、
スコープ広がっちゃって申し訳ないのですが、追加対応をお願いできますか?

@gakigaki

gakigaki commented Feb 5, 2025

Copy link
Copy Markdown
Contributor Author

@masaton0216
気付かなかったのでご指摘ありがとうございます!
修正しました!

@masaton0216 masaton0216 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ご対応ありがとうございます。マージします。

@masaton0216
masaton0216 merged commit ce9ecd8 into master Feb 6, 2025
@masaton0216
masaton0216 deleted the escape-double-quote-in-csv branch February 6, 2025 03:33
@akagane99 akagane99 added the bug バグ・不具合連絡 label Feb 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug バグ・不具合連絡

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants