Page MenuHomePhorge
Differential D25233

Fix PHP 8.1 Fatal error in Figlet: Use square brackets instead of curly braces
ClosedPublic
Actions

Authored by aklapper on May 20 2023, 12:50.
Tags
None
Referenced Files
F3251526: D25233.1742357698.diff
Tue, Mar 18, 04:14
F3224965: D25233.1742089925.diff
Sat, Mar 15, 01:52
F3223659: D25233.1741999034.diff
Fri, Mar 14, 00:37
F3222141: D25233.1741904544.diff
Wed, Mar 12, 22:22
F3221072: D25233.1741813600.diff
Tue, Mar 11, 21:06
F3221071: D25233.1741813596.diff
Tue, Mar 11, 21:06
F3212667: D25233.1741459323.diff
Fri, Mar 7, 18:42
F3006016: D25233.1740552390.diff
Tue, Feb 25, 06:46
View All 28 Files
Subscribers
Cigaryno
Matthew
speck
tobiaswiese
valerio.bozzolan

Details

Reviewers
valerio.bozzolan
Group Reviewers
O1: Blessed Committers
Maniphest Tasks
T15406: PHP Fatal error in Figlet due to use of curly braces
Commits
rPf97c9d68d031: Fix PHP 8.1 Fatal error in Figlet: Use square brackets instead of curly braces
Summary

Replace curly with square brackets when accessing an array value. See
https://www.php.net/manual/en/migration74.deprecated.php#migration74.deprecated.core.array-string-access-curly-brace
https://wiki.php.net/rfc/deprecate_curly_braces_array_access#wasn_t_the_curly_brace_syntax_deprecated_once_before

Closes T15406

Test Plan

Fixed these two lines; afterwards Figlet rendered in task comment preview. Example figlet:

figlet {{{ stuff }}}

Diff Detail

Repository
rP Phorge
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Revision Contents
Changeset List

PathSizePackages
externals/
pear-figlet/
Text/
4 linesO1: Blessed Committers

Diff 763

View Options

externals/pear-figlet/Text/Figlet.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