Fix suspend/unsuspend/terminate — wrong body field name

JS sends account_id but PHP was reading id; both now accept either.
Same fix applied to terminate.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This commit is contained in:
2026-06-08 22:39:22 +00:00
parent fe2d3d457c
commit 77f88ca5bf
+3 -3
View File
@@ -129,7 +129,7 @@ match ($action) {
})(), })(),
'suspend' => (function() use ($db, $body, $ownerClause) { 'suspend' => (function() use ($db, $body, $ownerClause) {
$id = (int)($body['id'] ?? 0); $id = (int)($body['id'] ?? $body['account_id'] ?? 0);
$acct = $db->fetchOne( $acct = $db->fetchOne(
"SELECT a.id, a.username, a.domain, u.email FROM accounts a JOIN users u ON u.id = a.user_id WHERE a.id = ? $ownerClause", "SELECT a.id, a.username, a.domain, u.email FROM accounts a JOIN users u ON u.id = a.user_id WHERE a.id = ? $ownerClause",
[$id] [$id]
@@ -143,7 +143,7 @@ match ($action) {
})(), })(),
'unsuspend' => (function() use ($db, $body, $ownerClause) { 'unsuspend' => (function() use ($db, $body, $ownerClause) {
$id = (int)($body['id'] ?? 0); $id = (int)($body['id'] ?? $body['account_id'] ?? 0);
AccountManager::unsuspend($id); AccountManager::unsuspend($id);
audit('account.unsuspend', "account:$id"); audit('account.unsuspend', "account:$id");
Response::success(null, 'Account unsuspended'); Response::success(null, 'Account unsuspended');
@@ -151,7 +151,7 @@ match ($action) {
'terminate' => (function() use ($db, $body, $user) { 'terminate' => (function() use ($db, $body, $user) {
Auth::getInstance()->require('admin'); Auth::getInstance()->require('admin');
$id = (int)($body['id'] ?? 0); $id = (int)($body['id'] ?? $body['account_id'] ?? 0);
AccountManager::terminate($id); AccountManager::terminate($id);
audit('account.terminate', "account:$id"); audit('account.terminate', "account:$id");
Response::success(null, 'Account terminated'); Response::success(null, 'Account terminated');