Page MenuHomePhabricator

Mangle cells that look a little bit like formulas in CSV files
ClosedPublic

Authored by epriestley on Jan 31 2018, 12:25 AM.
Tags
None
Referenced Files
F21881984: D18974.diff
Thu, Aug 13, 6:07 PM
Unknown Object (File)
Jul 5 2026, 11:11 AM
Unknown Object (File)
Jun 25 2026, 7:49 PM
Unknown Object (File)
Apr 22 2026, 5:33 PM
Unknown Object (File)
Apr 16 2026, 11:12 AM
Unknown Object (File)
Apr 10 2026, 10:47 PM
Unknown Object (File)
Mar 17 2026, 9:16 PM
Unknown Object (File)
Mar 15 2026, 12:04 PM
Subscribers
None

Details

Summary

Fixes T12800. See that task for discussion. When a cell in a CSV begins with "=", "+", "-", or "@", mangle the content to discourage Excel from executing it.

This is clumsy, but we support other formats (e.g., JSON) which preserve the data faithfully and you should probably be using JSON if you're going to do anything programmatic with it.

We could add two formats or a checkbox or a warning or something but cells with these symbols are fairly rare anyway.

Some possible exceptions I can think of are "user monograms" (but we don't export those right now) and "negative numbers" (but also no direct export today). We can add exceptions for those as they arise.

Test Plan

Exported a task named =cmd|'/C evil.exe'!A0, saw the title get mangled with "(!)" in front.

Diff Detail

Repository
rP Phabricator
Branch
export10
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 19294
Build 26072: Run Core Tests
Build 26071: arc lint + arc unit