Skip to content

[stable5] fix(export): escape CSV export for spreadsheet applications#2807

Merged
susnux merged 1 commit into
stable5from
backport/2804/stable5
Jul 7, 2025
Merged

[stable5] fix(export): escape CSV export for spreadsheet applications#2807
susnux merged 1 commit into
stable5from
backport/2804/stable5

Conversation

@backportbot
Copy link
Copy Markdown

@backportbot backportbot Bot commented Jul 7, 2025

Backport of PR #2804

While in theory CSV is just plain data most users do not care and open
it anyways in applications like Excel or LibreOffice.
This can cause unwanted behavior if the cell value is evaluated like a
formula.
So for this we manually escape suche values.

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
@backportbot backportbot Bot requested review from Chartman123 and susnux July 7, 2025 13:48
@backportbot backportbot Bot added bug Something isn't working 3. to review Waiting for reviews labels Jul 7, 2025
@backportbot backportbot Bot added this to the 5.2 milestone Jul 7, 2025
@codecov
Copy link
Copy Markdown

codecov Bot commented Jul 7, 2025

Codecov Report

Attention: Patch coverage is 90.00000% with 1 line in your changes missing coverage. Please review.

Project coverage is 43.70%. Comparing base (264ab0f) to head (d0f5fb9).
Report is 29 commits behind head on stable5.

Additional details and impacted files
@@              Coverage Diff              @@
##             stable5    #2807      +/-   ##
=============================================
+ Coverage      43.54%   43.70%   +0.16%     
- Complexity       908      911       +3     
=============================================
  Files             78       78              
  Lines           3392     3400       +8     
=============================================
+ Hits            1477     1486       +9     
+ Misses          1915     1914       -1     
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@susnux susnux merged commit 3f2dfee into stable5 Jul 7, 2025
53 checks passed
@susnux susnux deleted the backport/2804/stable5 branch July 7, 2025 13:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant