Page MenuHomePhabricator

Addressing PHP8 incompatibilities - Miscellaneous
ClosedPublic

Authored by cspeckmim on May 29 2023, 11:38 PM.
Tags
None
Referenced Files
F19064695: D21869.id52165.diff
Sat, Nov 29, 10:26 PM
F19049898: D21869.diff
Thu, Nov 27, 6:33 PM
F19023890: D21869.id52165.diff
Nov 24 2025, 2:03 AM
F19001170: D21869.id52171.diff
Nov 21 2025, 2:04 AM
F19000760: D21869.diff
Nov 21 2025, 12:46 AM
F18993708: D21869.id.diff
Nov 19 2025, 8:20 AM
F18955349: D21869.id.diff
Nov 12 2025, 9:10 PM
F18944086: D21869.diff
Nov 11 2025, 3:15 PM
Subscribers

Details

Summary

More perusing through Phabricator to find and address incompatibilities.

Refs T13588

Test Plan
  • Creating macros, changing their text.
  • Upload to use existing SSH key instead of generate one.
  • Modify alt text of an uploaded file.
  • Create phame post
  • Delete phriction document
  • Move tasks around on workboard, assign story points
  • Generate Diviner docs
  • Bulk modify tasks

Diff Detail

Repository
rP Phabricator
Branch
cspeck-php8-misc
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 25830
Build 35658: arc lint + arc unit

Event Timeline

cspeckmim held this revision as a draft.

One actual logical issue inline.

scripts/repository/commit_hook.php
122

This incorrectly prevents use of the common and desirable username "0".

(A user with this wholly reasonable username has a registered account on this server and has contributed to Phabricator in the past.)

This revision is now accepted and ready to land.May 30 2023, 2:34 PM
scripts/repository/commit_hook.php
122

👀

Correct the check for an empty username

This revision is now accepted and ready to land.May 30 2023, 3:21 PM