Page MenuHomePhabricator

Addressing some PHP8 incompatibilities - Misc. Applications
ClosedPublic

Authored by cspeckmim on May 29 2023, 3:50 AM.
Tags
None
Referenced Files
F13326598: D21868.diff
Sat, Jun 15, 7:59 AM
F13321118: D21868.diff
Fri, Jun 14, 6:25 AM
F13310216: D21868.diff
Mon, Jun 10, 11:15 AM
F13304858: D21868.diff
Sat, Jun 8, 1:47 PM
F13276351: D21868.diff
Fri, May 31, 5:52 AM
F13270584: D21868.id52164.diff
Wed, May 29, 9:58 AM
F13270583: D21868.id52163.diff
Wed, May 29, 9:58 AM
F13270582: D21868.id52161.diff
Wed, May 29, 9:57 AM
Subscribers

Details

Summary

Browsing through all other non-prototype applications and using them a bit to track down PHP8 incompatibilities.

Refs T13588

Test Plan

I tried basic usage of the following applications which found & fixed:

  • Almanac, creating resources, networks, services, etc.
  • Drydock, creating blue prints
  • Pholio mocks
  • Setting up TOTP MFA
  • Phame blog and post
  • Creating default Space
  • Creating and signing a legalpad document
  • Creating a conpherence room

Diff Detail

Repository
rP Phabricator
Branch
cspeck-php8-apps
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 25821
Build 35649: arc lint + arc unit

Event Timeline

cspeckmim held this revision as a draft.

Found a few incompatibilities with Conpherence

Ran all unit tests, found a few more PHP8 incompatibilities with mail.

cspeckmim edited the summary of this revision. (Show Details)
epriestley added inline comments.
src/applications/metamta/exception/PhabricatorMetaMTAReceivedMailProcessingException.php
17 ↗(On Diff #52163)

(This requires PHP 5.5, but so does Phabricator after T13492, so this should be fine.)

This revision is now accepted and ready to land.May 29 2023, 4:28 AM
src/applications/metamta/exception/PhabricatorMetaMTAReceivedMailProcessingException.php
17 ↗(On Diff #52163)

buh - the error I got from this led me to https://php.watch/versions/8.2/partially-supported-callable-deprecation. The backwards-compat notice at the bottom mentions ::class notation is available in 5.3+. I tried messing around with this in https://3v4l.org/E9eTR#v5.4.0 but nothing immediately stood out to me as functional in both 5.4 and 8.2.