]> git.proxmox.com Git - proxmox-widget-toolkit.git/blobdiff - src/node/APTRepositories.js
APTRepositories: change updating button text/state to viewcontroller
[proxmox-widget-toolkit.git] / src / node / APTRepositories.js
index fd4a35003d7fee0bb60b5fe46893fbdec1dfff1b..bb2bde0cdd8fca7f2b2c4aecfce547758dc56d81 100644 (file)
@@ -3,7 +3,7 @@ Ext.define('apt-repolist', {
     fields: [
        'Path',
        'Index',
-       'OfficialHost',
+       'Origin',
        'FileType',
        'Enabled',
        'Comment',
@@ -15,6 +15,92 @@ Ext.define('apt-repolist', {
     ],
 });
 
+Ext.define('Proxmox.window.APTRepositoryAdd', {
+    extend: 'Proxmox.window.Edit',
+    alias: 'widget.pmxAPTRepositoryAdd',
+
+    isCreate: true,
+    isAdd: true,
+
+    subject: gettext('Repository'),
+
+    initComponent: function() {
+       let me = this;
+
+       if (!me.repoInfo || me.repoInfo.length === 0) {
+           throw "repository information not initialized";
+       }
+
+       let description = Ext.create('Ext.form.field.Display', {
+           fieldLabel: gettext('Description'),
+           name: 'description',
+       });
+
+       let status = Ext.create('Ext.form.field.Display', {
+           fieldLabel: gettext('Status'),
+           name: 'status',
+           renderer: function(value) {
+               let statusText = gettext('Not yet configured');
+               if (value !== '') {
+                   statusText = Ext.String.format(
+                       '{0}: {1}',
+                       gettext('Configured'),
+                       value ? gettext('enabled') : gettext('disabled'),
+                   );
+               }
+
+               return statusText;
+           },
+       });
+
+       let repoSelector = Ext.create('Proxmox.form.KVComboBox', {
+           fieldLabel: gettext('Repository'),
+           xtype: 'proxmoxKVComboBox',
+           name: 'handle',
+           allowBlank: false,
+           comboItems: me.repoInfo.map(info => [info.handle, info.name]),
+           isValid: function() {
+               const handle = this.value;
+
+               if (!handle) {
+                   return false;
+               }
+
+               const info = me.repoInfo.find(elem => elem.handle === handle);
+
+               if (!info) {
+                   return false;
+               }
+
+               // not yet configured
+               return info.status === undefined || info.status === null;
+           },
+           listeners: {
+               change: function(f, value) {
+                   const info = me.repoInfo.find(elem => elem.handle === value);
+                   description.setValue(info.description);
+                   status.setValue(info.status);
+               },
+           },
+       });
+
+       repoSelector.setValue(me.repoInfo[0].handle);
+
+       let items = [
+           repoSelector,
+           description,
+           status,
+       ];
+
+       Ext.apply(me, {
+           items: items,
+           repoSelector: repoSelector,
+       });
+
+       me.callParent();
+    },
+});
+
 Ext.define('Proxmox.node.APTRepositoriesErrors', {
     extend: 'Ext.grid.GridPanel',
 
@@ -24,6 +110,8 @@ Ext.define('Proxmox.node.APTRepositoriesErrors', {
 
     store: {},
 
+    border: false,
+
     viewConfig: {
        stripeRows: false,
        getRowClass: () => 'proxmox-invalid-row',
@@ -52,6 +140,8 @@ Ext.define('Proxmox.node.APTRepositoriesGrid', {
 
     cls: 'proxmox-apt-repos', // to allow applying styling to general components with local effect
 
+    border: false,
+
     tbar: [
        {
            text: gettext('Reload'),
@@ -63,10 +153,31 @@ Ext.define('Proxmox.node.APTRepositoriesGrid', {
        },
        {
            text: gettext('Add'),
-           menu: {
-               plain: true,
-               itemId: "addMenu",
-               items: [],
+           id: 'addButton',
+           disabled: true,
+           repoInfo: undefined,
+           handler: function(button, event, record) {
+               Proxmox.Utils.checked_command(() => {
+                   let me = this;
+                   let panel = me.up('proxmoxNodeAPTRepositories');
+
+                   let extraParams = {};
+                   if (panel.digest !== undefined) {
+                      extraParams.digest = panel.digest;
+                   }
+
+                   Ext.create('Proxmox.window.APTRepositoryAdd', {
+                       repoInfo: me.repoInfo,
+                       url: `/api2/json/nodes/${panel.nodename}/apt/repositories`,
+                       method: 'PUT',
+                       extraRequestParams: extraParams,
+                       listeners: {
+                           destroy: function() {
+                               panel.reload();
+                           },
+                       },
+                   }).show();
+               });
            },
        },
        '-',
@@ -77,6 +188,9 @@ Ext.define('Proxmox.node.APTRepositoriesGrid', {
            altText: gettext('Disable'),
            id: 'repoEnableButton',
            disabled: true,
+           bind: {
+               text: '{enableButtonText}',
+           },
            handler: function(button, event, record) {
                let me = this;
                let panel = me.up('proxmoxNodeAPTRepositories');
@@ -123,27 +237,12 @@ Ext.define('Proxmox.node.APTRepositoriesGrid', {
 
     columns: [
        {
-           header: gettext('Official'),
-           dataIndex: 'OfficialHost',
-           renderer: function(value, cell, record) {
-               let icon = (cls) => `<i class="fa fa-fw ${cls}"></i>`;
-
-               const enabled = record.data.Enabled;
-
-               if (value === undefined || value === null) {
-                   return icon('fa-question-circle-o');
-               }
-               if (!value) {
-                   return icon('fa-times ' + (enabled ? 'critical' : 'faded'));
-               }
-               return icon('fa-check ' + (enabled ? 'good' : 'faded'));
-           },
-           width: 70,
-       },
-       {
+           xtype: 'checkcolumn',
            header: gettext('Enabled'),
            dataIndex: 'Enabled',
-           renderer: Proxmox.Utils.format_enabled_toggle,
+           listeners: {
+               beforecheckchange: () => false, // veto, we don't want to allow inline change - to subtle
+           },
            width: 90,
        },
        {
@@ -205,6 +304,20 @@ Ext.define('Proxmox.node.APTRepositoriesGrid', {
            },
            flex: 1,
        },
+       {
+           header: gettext('Origin'),
+           dataIndex: 'Origin',
+           width: 120,
+           renderer: (value, meta, rec) => {
+               let cls = 'fa fa-fw fa-question-circle-o';
+               if (value.match(/^\s*Proxmox\s*$/i)) {
+                   cls = 'pmx-itype-icon pmx-itype-icon-proxmox-x';
+               } else if (value.match(/^\s*Debian\s*$/i)) {
+                   cls = 'pmx-itype-icon pmx-itype-icon-debian-swirl';
+               }
+               return `<i class='${cls}'></i> ${value}`;
+           },
+       },
        {
            header: gettext('Comment'),
            dataIndex: 'Comment',
@@ -216,7 +329,7 @@ Ext.define('Proxmox.node.APTRepositoriesGrid', {
        let me = this;
 
        let warnings = {};
-       let officialHosts = {};
+       let origins = {};
 
        let addLine = function(obj, key, line) {
            if (obj[key]) {
@@ -233,14 +346,14 @@ Ext.define('Proxmox.node.APTRepositoriesGrid', {
                (info.kind === 'ignore-pre-upgrade-warning' && !me.majorUpgradeAllowed)
            ) {
                addLine(warnings, key, gettext('Warning') + ": " + info.message);
-           } else if (info.kind === 'badge' && info.message === 'official host name') {
-               officialHosts[key] = true;
+           } else if (info.kind === 'origin') {
+               origins[key] = info.message;
            }
        }
 
        gridData.forEach(function(record) {
            const key = `${record.Path}:${record.Index}`;
-           record.OfficialHost = !!officialHosts[key];
+           record.Origin = origins[key];
        });
 
        me.rowBodyFeature.getAdditionalData = function(innerData, rowIndex, record, orig) {
@@ -296,19 +409,6 @@ Ext.define('Proxmox.node.APTRepositoriesGrid', {
 
        me.callParent();
     },
-
-    listeners: {
-       selectionchange: function() {
-           let me = this;
-
-           if (me.onSelectionChange) {
-               let sm = me.getSelectionModel();
-               let rec = sm.getSelection()[0];
-
-               me.onSelectionChange(rec, sm);
-           }
-       },
-    },
 });
 
 Ext.define('Proxmox.node.APTRepositories', {
@@ -320,6 +420,20 @@ Ext.define('Proxmox.node.APTRepositories', {
 
     product: 'Proxmox VE', // default
 
+    controller: {
+       xclass: 'Ext.app.ViewController',
+
+       selectionChange: function(grid, selection) {
+           let me = this;
+           if (!selection || selection.length < 1) {
+               return;
+           }
+           let rec = selection[0];
+           let vm = me.getViewModel();
+           vm.set('selectionenabled', rec.get('Enabled'));
+       },
+    },
+
     viewModel: {
        data: {
            product: 'Proxmox VE', // default
@@ -327,9 +441,12 @@ Ext.define('Proxmox.node.APTRepositories', {
            subscriptionActive: '',
            noSubscriptionRepo: '',
            enterpriseRepo: '',
+           selectionenabled: false,
        },
        formulas: {
            noErrors: (get) => get('errorCount') === 0,
+           enableButtonText: (get) => get('selectionenabled')
+               ? gettext('Disable') : gettext('Enable'),
            mainWarning: function(get) {
                // Not yet initialized
                if (get('subscriptionActive') === '' ||
@@ -358,20 +475,33 @@ Ext.define('Proxmox.node.APTRepositories', {
        },
     },
 
+    scrollable: true,
+    layout: {
+       type: 'vbox',
+       align: 'stretch',
+    },
+
     items: [
        {
-           title: gettext('Warning'),
-           name: 'repositoriesMainWarning',
-           xtype: 'panel',
+           xtype: 'header',
+           baseCls: 'x-panel-header',
            bind: {
+               hidden: '{!mainWarning}',
                title: '{mainWarning}',
+           },
+       },
+       {
+           xtype: 'box',
+           bind: {
                hidden: '{!mainWarning}',
            },
+           height: 5,
        },
        {
            xtype: 'proxmoxNodeAPTRepositoriesErrors',
            name: 'repositoriesErrors',
            hidden: true,
+           padding: '0 0 5 0',
            bind: {
                hidden: '{noErrors}',
            },
@@ -383,12 +513,8 @@ Ext.define('Proxmox.node.APTRepositories', {
                nodename: '{nodename}',
            },
            majorUpgradeAllowed: false, // TODO get release information from an API call?
-           onSelectionChange: function(rec, sm) {
-               let me = this;
-               if (rec) {
-                   let btn = me.up('proxmoxNodeAPTRepositories').down('#repoEnableButton');
-                   btn.setText(rec.get('Enabled') ? gettext('Disable') : gettext('Enable'));
-               }
+           listeners: {
+               selectionchange: 'selectionChange',
            },
        },
     ],
@@ -413,8 +539,8 @@ Ext.define('Proxmox.node.APTRepositories', {
        let me = this;
        let vm = me.getViewModel();
 
-       let menu = me.down('#addMenu');
-       menu.removeAll();
+       let addButton = me.down('#addButton');
+       addButton.repoInfo = [];
 
        for (const standardRepo of standardRepos) {
            const handle = standardRepo.handle;
@@ -426,45 +552,11 @@ Ext.define('Proxmox.node.APTRepositories', {
                vm.set('noSubscriptionRepo', status);
            }
 
-           let status_text = '';
-           if (status !== undefined && status !== null) {
-               status_text = Ext.String.format(
-                   ' ({0}, {1})',
-                   gettext('configured'),
-                   status ? gettext('enabled') : gettext('disabled'),
-               );
-           }
-
-           menu.add({
-               text: standardRepo.name + status_text,
-               disabled: status !== undefined && status !== null,
-               repoHandle: handle,
-               handler: function(menuItem) {
-                   Proxmox.Utils.checked_command(() => {
-                       let params = {
-                           handle: menuItem.repoHandle,
-                       };
-
-                       if (me.digest !== undefined) {
-                           params.digest = me.digest;
-                       }
-
-                       Proxmox.Utils.API2Request({
-                           url: `/nodes/${me.nodename}/apt/repositories`,
-                           method: 'PUT',
-                           params: params,
-                           failure: function(response, opts) {
-                               Ext.Msg.alert(gettext('Error'), response.htmlStatus);
-                               me.reload();
-                           },
-                           success: function(response, opts) {
-                               me.reload();
-                           },
-                       });
-                   });
-               },
-           });
+           addButton.repoInfo.push(standardRepo);
+           addButton.digest = me.digest;
        }
+
+       addButton.setDisabled(false);
     },
 
     reload: function() {