Merge pull request #8 from irabbi360/dev

vm storage detach remove

Author: irabbi360

src/ProxmoxNodeVm.php

@@ -819,7 +819,7 @@ public function attachDisk(string $node, int $vmid, array $params)
             $result = $this->makeRequest('POST', "nodes/{$node}/qemu/{$vmid}/config", $diskParams);
 
             if (!isset($result['data'])) {
-                throw new Exception('Failed to attach disk: No response data received');
+                return ResponseHelper::generate(false,"Failed to attach disk: No response data received");
             }
 
             $successResponse = [
@@ -835,14 +835,33 @@ public function attachDisk(string $node, int $vmid, array $params)
         }
     }
 
-    public function detachDisk($node, $vmid, $disk)
+    /**
+     * @throws Exception
+     */
+    public function detachDisk(string $node, int $vmid, string $disk, bool $type = false)
     {
-        // API endpoint to remove a disk
-        $result = $this->makeRequest('DELETE', "nodes/{$node}/qemu/{$vmid}/config", [
-            'delete' => $disk, // Specify the disk to delete (e.g., 'scsi0')
-        ]);
+        try {
+            // API endpoint to remove a disk
+            $result = $this->makeRequest('POST', "nodes/{$node}/qemu/{$vmid}/config", [
+                'delete' => $disk, // Specify the disk to delete (e.g., 'scsi0')
+                'force' => $type
+            ]);
+
+            if (!isset($result['data'])) {
+                return ResponseHelper::generate(false,"Failed to detach disk");
+            }
 
-        return $result;
+            $successResponse = [
+                'node' => $node,
+                'vmid' => $vmid,
+                'disk' => $disk,
+                'data' => $result['data']
+            ];
+
+            return ResponseHelper::generate(true, 'Disk attach success', $successResponse);
+        } catch (Exception $e) {
+            return ResponseHelper::generate(false,"Failed to detach disk: " . $e->getMessage());
+        }
     }
 
     /**