Page MenuHomePhabricator

Export "date" and "remarkup" custom fields to Excel + "zip" extension check
ClosedPublic

Authored by epriestley on Jul 17 2019, 10:57 PM.
Tags
None
Referenced Files
F13050945: D20658.id49282.diff
Fri, Apr 19, 3:17 AM
F13050944: D20658.id49280.diff
Fri, Apr 19, 3:17 AM
F13050943: D20658.id.diff
Fri, Apr 19, 3:17 AM
Unknown Object (File)
Thu, Apr 11, 4:26 AM
Unknown Object (File)
Tue, Apr 9, 6:40 AM
Unknown Object (File)
Sun, Apr 7, 6:14 AM
Unknown Object (File)
Mon, Apr 1, 3:01 AM
Unknown Object (File)
Sat, Mar 30, 8:26 AM
Subscribers
None

Details

Summary

Fixes T13342. This does a few different things, although all of them seem small enough that I didn't bother splitting it up:

  • Support export of "remarkup" custom fields as text. There's some argument here to export them in some kind of structure if the target is JSON, but it's hard for me to really imagine we'll live in a world some day where we really regret just exporting them as text.
  • Support export of "date" custom fields as dates. This is easy except that I added null support.
  • If you built PHP from source without "--enable-zip", as I did, you can hit the TODO in Excel exports about "ZipArchive". Since I had a reproduction case, test for "ZipArchive" and give the user a better error if it's missing.
  • Add a setup check for the "zip" extension to try to avoid getting there in the first place. This is normally part of PHP so I believe users generally won't hit it, I just hit it because I built from source. See also T13232.
Test Plan
  • Added a custom "date" field. On tasks A and B, set it to null and some non-null value. Exported both tasks to Excel/JSON/text, saw null and a date, respectively.
  • Added a custom "remarkup" field, exported some values, saw the values in Excel.

Diff Detail

Repository
rP Phabricator
Branch
custom1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 23147
Build 31787: Run Core Tests
Build 31786: arc lint + arc unit