Page MenuHomePhabricator

Update Figlet implementation to be PHP8 compatible

Authored by cspeckmim on Apr 25 2023, 2:53 PM.
Referenced Files
F13313478: D21860.diff
Tue, Jun 11, 7:58 AM
F13300735: D21860.diff
Fri, Jun 7, 9:31 AM
F13295184: D21860.diff
Wed, Jun 5, 11:28 PM
F13270507: D21860.id52105.diff
Wed, May 29, 9:49 AM
F13270506: D21860.id52104.diff
Wed, May 29, 9:49 AM
Wed, May 29, 9:49 AM
F13259509: D21860.id52105.diff
Sun, May 26, 9:32 PM
F13238705: D21860.diff
Tue, May 21, 8:50 PM



As of PHP ~v8 the zip_open and associated functions have been deprecated and
removed. The replacement is the ZipArchive API. This updates the figlet
implementation to use this API which has been present in PHP since 5.2.

Additionally in PHP 8 the use of squiggly brackets for indexing into arrays is
also deprecated. This updates to remove two uses of squiggly brackets and
replace with square brackets.

These two deprecations would result in being unable to load differential
revisions in which someone had commented using figlet remarkup.

Refs T13588

Test Plan

Applied these changes to an install and loaded a revision that had comments
where someone utilized figlet remarkup. The revision loaded properly and the
figlet comment rendered properly.

Diff Detail

rP Phabricator
Lint Not Applicable
Tests Not Applicable

Event Timeline

This revision is now accepted and ready to land.Apr 25 2023, 2:59 PM

(It would also probably be fine to just remove support for compressed figlet fonts, they're probably saving like 5KB?)

Oh whoops. I didn't realize figlet font file format wasn't compressed