Page MenuHomePhorge
Differential D25054

Replace PHPExcel with PhpSpreadsheet
Needs RevisionPublic
Actions

Authored by • jacques on Sep 26 2022, 09:28.
Tags
None
Referenced Files
F4055789: D25054.1746777550.diff
Thu, May 8, 07:59
F4042440: D25054.1746725418.diff
Wed, May 7, 17:30
F4042438: D25054.1746725417.diff
Wed, May 7, 17:30
F4042434: D25054.1746725416.diff
Wed, May 7, 17:30
F4024968: D25054.1746682259.diff
Wed, May 7, 05:30
F3966734: D25054.1746516424.diff
Mon, May 5, 07:27
F3966733: D25054.1746516392.diff
Mon, May 5, 07:26
F3945630: D25054.1746445074.diff
Sun, May 4, 11:37
View All 53 Files
Subscribers
avivey
Cigaryno
jeremy.norris
Matthew
speck
Sten
tobiaswiese
valerio.bozzolan
Tokens
"Baby Tequila" token, awarded by dcog.

Details

Reviewers
avivey
Group Reviewers
O1: Blessed Committers
Maniphest Tasks
T15140: switch from PHPExcel to PHPSpreadsheet for Maniphest task export
Summary

WIP: Update PHPExcel to PhpSpreadsheet

Test Plan

Test that exporting works with PhpSpreadsheet and that one can download Excel spreadsheets.

Diff Detail

Repository
rP Phorge
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

• jacques created this revision.Sep 26 2022, 09:28
Owners added a reviewer: O1: Blessed Committers.Sep 26 2022, 09:28
Herald added subscribers: Cigaryno, Matthew, valerio.bozzolan and 2 others. · View Herald TranscriptSep 26 2022, 09:28
• jacques requested review of this revision.Sep 26 2022, 09:28
dcog awarded a token.Sep 26 2022, 09:57
avivey subscribed.Oct 28 2022, 10:55

The change should also include updating the installation instructions, and a ticket.

Cigaryno added a comment.Oct 29 2022, 21:41
In D25054#1871, @avivey wrote:

The change should also include updating the installation instructions, and a ticket.

Sorry, @jacques is not a member of Trusted Contributors.

jeremy.norris subscribed.Nov 11 2022, 16:23
avivey requested changes to this revision.Feb 3 2023, 07:40

This change should include updating the installation instructions (in the manual and in the error message in PhabricatorExcelExportFormat)

This revision now requires changes to proceed.Feb 3 2023, 07:40
avivey added a task: T15140: switch from PHPExcel to PHPSpreadsheet for Maniphest task export.Feb 3 2023, 07:42
Sten subscribed.Aug 12 2023, 07:56

Revision Contents
Changeset List

Diff 278

View Options

src/infrastructure/export/field/PhabricatorDoubleExportField.php

Loading...
View Options

src/infrastructure/export/field/PhabricatorIntExportField.php

Loading...
View Options

src/infrastructure/export/format/PhabricatorExcelExportFormat.php

Loading...
Content licensed under Creative Commons Attribution-ShareAlike 4.0 (CC-BY-SA) unless otherwise noted; code licensed under Apache 2.0 or other open source licenses. · CC BY-SA 4.0 · Apache 2.0