Add a copy button that copies the configuration of one disk to a new
one.

Signed-off-by: Max Carrara <m.carr...@proxmox.com>
---
 www/manager6/panel/MultiDiskEdit.js | 78 +++++++++++++++++++++++++----
 1 file changed, 68 insertions(+), 10 deletions(-)

diff --git a/www/manager6/panel/MultiDiskEdit.js 
b/www/manager6/panel/MultiDiskEdit.js
index ea1f974d..dffbb4c5 100644
--- a/www/manager6/panel/MultiDiskEdit.js
+++ b/www/manager6/panel/MultiDiskEdit.js
@@ -13,14 +13,37 @@ Ext.define('PVE.panel.MultiDiskPanel', {
     controller: {
        xclass: 'Ext.app.ViewController',
 
+       disableableButtons: ['addButton', 'copyButton'],
+
        vmconfig: {},
 
        onAdd: function() {
            let me = this;
-           me.lookup('addButton').setDisabled(true);
+           me.disableButtons();
            me.addDisk();
-           let count = me.lookup('grid').getStore().getCount() + 1; // +1 is 
from ide2
-           me.lookup('addButton').setDisabled(count >= me.maxCount);
+           me.enableButtons();
+       },
+
+       onCopy: function(button, event) {
+           let me = this;
+           me.disableButtons();
+           me.addDisk(true);
+           me.enableButtons();
+       },
+
+       disableButtons: function() {
+           let me = this;
+           for (let buttonName of me.disableableButtons) {
+               me.lookup(buttonName).setDisabled(true);
+           }
+       },
+
+       enableButtons: function() {
+           let me = this;
+           let count = me.lookup('grid').getStore().getCount() + 1;
+           for (let buttonName of me.disableableButtons) {
+               me.lookup(buttonName).setDisabled(count >= me.maxCount);
+           }
        },
 
        getNextFreeDisk: function(vmconfig) {
@@ -34,7 +57,7 @@ Ext.define('PVE.panel.MultiDiskPanel', {
        // define in subclass
        diskSorter: undefined,
 
-       addDisk: function() {
+       addDisk: function(copy = false) {
            let me = this;
            let grid = me.lookup('grid');
            let store = grid.getStore();
@@ -54,6 +77,21 @@ Ext.define('PVE.panel.MultiDiskPanel', {
            })[0];
 
            let panel = me.addPanel(itemId, vmconfig, nextFreeDisk);
+
+           if (copy) {
+               let selection = grid.getSelection()[0];
+               let selectedItemId = selection.get('itemId');
+               let panelFrom = me.getView().getComponent(selectedItemId);
+
+               if (!panelFrom) {
+                   throw "unexpected missing panel";
+               }
+
+               let values = panelFrom.getValues(false, true);
+               values.deviceid = nextFreeDisk.id;
+               panel.setValues(values);
+           }
+
            panel.updateVMConfig(vmconfig);
 
            // we need to setup a validitychange handler, so that we can show
@@ -170,7 +208,7 @@ Ext.define('PVE.panel.MultiDiskPanel', {
 
            store.remove(record);
            me.getView().remove(record.get('itemId'));
-           me.lookup('addButton').setDisabled(false);
+           me.enableButtons();
            me.updateVMConfig();
            me.checkValidity();
        },
@@ -251,11 +289,31 @@ Ext.define('PVE.panel.MultiDiskPanel', {
                    ],
                },
                {
-                   xtype: 'button',
-                   reference: 'addButton',
-                   text: gettext('Add'),
-                   iconCls: 'fa fa-plus-circle',
-                   handler: 'onAdd',
+                   layout: 'hbox',
+                   border: false,
+                   defaults: {
+                       border: false,
+                       layout: 'anchor',
+                       flex: 1,
+                   },
+                   items: [
+                       {
+                           xtype: 'button',
+                           reference: 'addButton',
+                           text: gettext('Add'),
+                           iconCls: 'fa fa-plus-circle',
+                           handler: 'onAdd',
+                           margin: '0 2.5 0 0',
+                       },
+                       {
+                           xtype: 'button',
+                           reference: 'copyButton',
+                           text: gettext('Copy'),
+                           iconCls: 'fa fa-files-o',
+                           handler: 'onCopy',
+                           margin: '0 0 0 2.5',
+                       },
+                   ],
                },
                {
                    // dummy field to control wizard validation
-- 
2.39.2



_______________________________________________
pve-devel mailing list
pve-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

Reply via email to