Skip to content

Commit

Permalink
Merge pull request #26155 from frappe/mergify/bp/version-15-hotfix/pr…
Browse files Browse the repository at this point in the history
…-26142

fix: handle null in CSV preview (backport #26142)
  • Loading branch information
akhilnarang committed Apr 29, 2024
2 parents c8387f8 + 551503a commit 24e2886
Showing 1 changed file with 4 additions and 0 deletions.
4 changes: 4 additions & 0 deletions frappe/public/js/frappe/views/reports/report_utils.js
Expand Up @@ -284,6 +284,10 @@ frappe.report_utils = {
.map((row) => {
return row
.map((col) => {
if (col === null) {
return "";
}

if (typeof col == "string" && col.includes('"')) {
col = col.replace(/"/g, '""');
}
Expand Down

0 comments on commit 24e2886

Please sign in to comment.