Severity: MEDIUM
Location: lib/Controller/SettingsController.php lines 1020–1023 (manualImport)
Description:
The success branch of manualImport is an empty if block:
if ($result['success'] === true) {
// empty
}
return new JSONResponse($result, 400);Every response — including successful imports — returns HTTP 400. The UI therefore always displays the import as failed, regardless of the actual outcome.
Suggested fix:
return new JSONResponse($result, $result['success'] === true ? 200 : 400);
Source: deep team-reviewer pass 2026-05-27
Severity: MEDIUM
Location:
lib/Controller/SettingsController.phplines 1020–1023 (manualImport)Description:
The success branch of
manualImportis an emptyifblock:Every response — including successful imports — returns HTTP 400. The UI therefore always displays the import as failed, regardless of the actual outcome.
Suggested fix:
Source: deep team-reviewer pass 2026-05-27