Skip to content

Fix: use a unique temporary file for the error CSV export - #676

Merged
Rom1-B merged 1 commit into
10.0/bugfixesfrom
fix_injection_export_10
Oct 1, 2026
Merged

Rom1-B merged 1 commit into
10.0/bugfixesfrom
fix_injection_export_10

Conversation

@stonebuzz

@stonebuzz stonebuzz commented Oct 1, 2026 •

Copy link
Copy Markdown
Contributor

Checklist before requesting a review

Please delete options that are not relevant.

  • I have performed a self-review of my code.
  • I have added tests (when available) that prove my fix is effective or that my feature works.
  • I have updated the CHANGELOG with a short functional description of the fix or new feature.
  • This change requires a documentation update.

Description

Backport For #675

  • The "Export the log" action wrote the rejected lines to a file named after the uploaded CSV, so two exports using the same file name could overwrite and remove each other's file.
  • The export now writes to a unique temporary file in the plugin upload directory and is always downloaded as Errors.csv.

Screenshots (if appropriate):

@stonebuzz stonebuzz self-assigned this Oct 1, 2026
@stonebuzz
stonebuzz marked this pull request as draft October 1, 2026 08:07
@stonebuzz
stonebuzz force-pushed the fix_injection_export_10 branch from af50c22 to 0512dd0 Compare October 1, 2026 08:14
@stonebuzz
stonebuzz force-pushed the fix_injection_export_10 branch from 0512dd0 to f003a11 Compare October 1, 2026 08:28
@stonebuzz
stonebuzz requested a review from Rom1-B October 1, 2026 08:33
@stonebuzz
stonebuzz marked this pull request as ready for review October 1, 2026 08:33
@Rom1-B
Rom1-B merged commit a8129ec into 10.0/bugfixes Oct 1, 2026
7 checks passed
@Rom1-B
Rom1-B deleted the fix_injection_export_10 branch October 1, 2026 08:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants