Page MenuHomePhorge

Fix /dashboard/adjust/remove/ PHP 8.1 strlen(null) errors
ClosedPublic

Authored by Sten on Jul 13 2023, 12:03.
Tags
None
Referenced Files
F3328333: D25345.1743454572.diff
Sun, Mar 30, 20:56
F3325165: D25345.1743410101.diff
Sun, Mar 30, 08:35
F3319165: D25345.1743300097.diff
Sat, Mar 29, 02:01
F3317994: D25345.1743270885.diff
Fri, Mar 28, 17:54
F3317823: D25345.1743269067.diff
Fri, Mar 28, 17:24
F3313206: D25345.1743215305.diff
Fri, Mar 28, 02:28
F3300476: D25345.1743040556.diff
Wed, Mar 26, 01:55
F3299063: D25345.1743014758.diff
Tue, Mar 25, 18:45

Details

Summary

/dashboard/adjust/remove/ POST requests fail under PHP 8.1 with strlen(null) errors.

This change fixes it.

Fixes T15549

Test Plan

View a dashboard which has some panels. Eg https://my.phorge.site/dashboard/view/1/
Click on a 'Manage Panel' dropdown and click on 'Remove Panel'

Confirm the panel is removed and no strlen(null) errors are generated.

Diff Detail

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