Page MenuHomePhabricator

Addressing PHP8 incompatibilities - Diffusion, Drydock, Harbormastor
Needs ReviewPublic

Authored by cspeckmim on Jun 5 2023, 1:11 PM.
Tags
None
Referenced Files
F18692903: D21873.id52178.diff
Fri, Sep 26, 11:34 PM
F18650341: D21873.id52178.diff
Sun, Sep 21, 12:21 AM
F18630622: D21873.id.diff
Sep 16 2025, 12:12 PM
F18622529: D21873.diff
Sep 15 2025, 12:12 PM
F18621833: D21873.id.diff
Sep 15 2025, 9:35 AM
F18618209: D21873.id52179.diff
Sep 14 2025, 11:26 PM
F18603486: D21873.diff
Sep 13 2025, 5:21 PM
F18573700: D21873.diff
Sep 10 2025, 9:08 AM
Subscribers
Restricted Owners Package

Details

Reviewers
None
Group Reviewers
Blessed Reviewers
Summary

Fixing a few more null-string issues.

Test Plan
  • I used the search icon next to a Repository field to list and locate diffusion repositories
  • I modified a repository's default branch name to something, something else, empty, something else - and verified the history page for the repository displayed properly
  • I modified a repository's staging area and view the history of the repository verified the history page displayed properly
  • I created and edited several Almanac resources (all the available different types), and several Drydock Blueprints, one for Almanac Host and one for Working Copy
  • Visited all the UIExample views

Diff Detail

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

Event Timeline

cspeckmim held this revision as a draft.
Owners added a subscriber: Restricted Owners Package.Jun 5 2023, 1:11 PM
cspeckmim retitled this revision from Addressing PHP8 incompatibilities - Miscellaneous to Addressing PHP8 incompatibilities - Diffusion, Drydock, Harbormastor.Jun 6 2023, 1:45 AM
cspeckmim edited the test plan for this revision. (Show Details)

Found a few things while running through tests. Revert change to DifferentialAuditorsField as it doesn't appear to be necessary (anymore?).

Found two more quick issues

  • Adding search results to dashboard hit an error
  • Viewing the prototype calendar event failed after creating an event