Page MenuHomePhorge

Fix strlen(null) in DifferentialChangesetViewController loadCoverage()
ClosedPublic

Authored by Sten on Jul 4 2023, 14:19.

Details

Summary

DifferentialChangesetViewController loadCoverage() fails under PHP 8.1 when test coverage is provided for some but not all modified files.

Update the code to replace strlen() with phutil_nonempty_string().

Fixes T15521

Test Plan

View a diff with test coverage for some but not all files.

Diff Detail

Repository
rP Phorge
Branch
DifferentialChangesetViewController (branched from master)
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 632
Build 632: arc lint + arc unit