Page MenuHomePhorge

Fix PHP 8.1 and 8.2 incompatibilities mostly
AbandonedPublic

Authored by MacFan4000 on Dec 7 2021, 14:09.
Referenced Files
F2197821: D25030.1716292996.diff
Mon, May 20, 12:03
Unknown Object (File)
Sat, May 18, 05:29
Unknown Object (File)
Tue, May 14, 17:32
Unknown Object (File)
Wed, May 8, 09:56
Unknown Object (File)
Fri, May 3, 13:08
Unknown Object (File)
Mon, Apr 29, 06:35
Unknown Object (File)
Sat, Apr 27, 19:58
Unknown Object (File)
Sat, Apr 27, 08:23

Details

Summary

This fixes most issues with php 8.1. Some issues still linger that I was unable to fix. Part of T15064.

Test Plan

Used php linting, tested in browser, checked error logs, etc

Diff Detail

Repository
rARC Arcanist
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

Ekubischta subscribed.

Just a warning that I found a few bugs just by spot checking...

I think that this revision should be abandoned

  1. It is impossible to test
  2. Should we be suppressing "null to string" errors with "@" ? (The issue here is that the built in strlen function does not accept null)

I think a better approach would be to

  • Write a replacement global function called phorge_strlen
  • Replace all strlen functions with our new function

Repeat for any other functions that we feel have this issue

src/console/grid/ArcanistGridView.php
158

Find and Replace bug

src/console/view/PhutilConsoleList.php
45

Find and Replace bug

src/console/view/PhutilConsoleTable.php
82

Find and Replace bug

228

Find and Replace bug

236

Find and Replace bug

This revision now requires changes to proceed.Nov 11 2022, 16:09
valerio.bozzolan retitled this revision from Fix PHP 8.1 incompatibilities mostly to Fix PHP 8.1 and 8.2 incompatibilities mostly.Mar 26 2023, 15:08