Page MenuHomePhabricator

Addressing PHP8 incompatibilities - Miscellaneous
ClosedPublic

Authored by cspeckmim on May 29 2023, 11:38 PM.
Tags
None
Referenced Files
F15449708: D21869.id52176.diff
Fri, Mar 28, 11:47 AM
F15449698: D21869.id52171.diff
Fri, Mar 28, 11:42 AM
F15449691: D21869.id52167.diff
Fri, Mar 28, 11:41 AM
F15449676: D21869.id.diff
Fri, Mar 28, 11:37 AM
F15443934: D21869.id52167.diff
Thu, Mar 27, 6:57 AM
F15438957: D21869.diff
Wed, Mar 26, 4:56 AM
F15434942: D21869.id52165.diff
Tue, Mar 25, 6:09 AM
F15390829: D21869.id52176.diff
Sat, Mar 15, 6:54 AM
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 25826
Build 35654: 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