Skip to content

Commit 035af89

Browse files
committed
vm rename api update
1 parent 90471d6 commit 035af89

File tree

1 file changed

+5
-21
lines changed

1 file changed

+5
-21
lines changed

src/ProxmoxNodeVm.php

Lines changed: 5 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -422,38 +422,22 @@ public function renameVM(string $node, int $vmid, string $newName)
422422

423423
// Check if VM is running
424424
if ($vmStatus['data']['status'] === 'running') {
425-
return [
426-
'success' => false,
427-
'message' => "Cannot rename running VM. Please stop the VM first.",
428-
'data' => null
429-
];
425+
return ResponseHelper::generate(false, 'Cannot rename running VM. Please stop the VM first.');
430426
}
431427

432428
// Set new name
433429
$result = $this->setVMConfig($node, $vmid, [
434430
'name' => $newName
435431
]);
436432

437-
if (isset($result['success']) && $result['success']) {
438-
return [
439-
'success' => true,
440-
'message' => "VM renamed successfully.",
441-
'data' => $result['data'] ?? null
442-
];
433+
if (isset($result['data']) && $result['data']) {
434+
return ResponseHelper::generate(true, 'VM renamed successfully.');
443435
}
444436

445-
return [
446-
'success' => false,
447-
'message' => "Failed to rename VM.",
448-
'data' => $result['data'] ?? null
449-
];
437+
return ResponseHelper::generate(false, 'Failed to rename VM.', $result['data'] ?? null);
450438

451439
} catch (Exception $e) {
452-
return [
453-
'success' => false,
454-
'message' => $e->getMessage(),
455-
'data' => null
456-
];
440+
return ResponseHelper::generate(false, $e->getMessage());
457441
}
458442
}
459443

0 commit comments

Comments
 (0)