bug: BackupModel/ProfileModel loadFormData() returns array when object declared — TypeError #80

New Issue

2026-06-21T23:01:44Z

jmiller commented

2026-06-21 23:01:44 +00:00

Severity: MEDIUM

loadFormData() declares return type object but getUserState() can return an array. On PHP 8.x this throws a TypeError, crashing the edit views when Joomla restores form state after validation failure.

Fix

return is_array($data) ? (object) $data : $data;

Files

src/Model/BackupModel.php:32-39
src/Model/ProfileModel.php (same pattern)

## Severity: MEDIUM `loadFormData()` declares return type `object` but `getUserState()` can return an array. On PHP 8.x this throws a `TypeError`, crashing the edit views when Joomla restores form state after validation failure. ## Fix ```php return is_array($data) ? (object) $data : $data; ``` ## Files - `src/Model/BackupModel.php:32-39` - `src/Model/ProfileModel.php` (same pattern)

jmiller added the component: admin label 2026-06-21 23:01:44 +00:00

jmiller referenced this issue

2026-06-21 23:04:13 +00:00

test: Deep dive audit — verify and fix all findings #81

jmiller referenced this issue from a commit

2026-06-21 23:09:44 +00:00

fix: critical and high audit findings (#81)

jmiller referenced a pull request that will close this issue

2026-06-21 23:09:56 +00:00

fix: Critical and high audit findings (#81) #83

jmiller closed this issue

2026-06-21 23:10:24 +00:00