]> git.proxmox.com Git - proxmox-widget-toolkit.git/blobdiff - src/window/Edit.js
edit window: small code style/nits
[proxmox-widget-toolkit.git] / src / window / Edit.js
index c0e305b9af1ba3c7bfa556358f2ab2929fee7c70..573c24c3e629fac4fd52d97b206728348161c0da 100644 (file)
@@ -7,27 +7,40 @@ Ext.define('Proxmox.window.Edit', {
     // set extra options like params for the load request
     autoLoadOptions: undefined,
 
+    // to submit extra params on load and submit, useful, e.g., if not all ID
+    // parameters are included in the URL
+    extraRequestParams: {},
+
     resizable: false,
 
-    // use this tio atimatically generate a title like
-    // Create: <subject>
+    // use this to automatically generate a title like `Create: <subject>`
     subject: undefined,
 
-    // set isCreate to true if you want a Create button (instead
-    // OK and RESET)
+    // set isCreate to true if you want a Create button (instead OK and RESET)
     isCreate: false,
 
     // set to true if you want an Add button (instead of Create)
     isAdd: false,
 
-    // set to true if you want an Remove button (instead of Create)
+    // set to true if you want a Remove button (instead of Create)
     isRemove: false,
 
+    // set to false, if you don't want the reset button present
+    showReset: true,
+
     // custom submitText
     submitText: undefined,
 
     backgroundDelay: 0,
 
+    // string or function, called as (url, values) - useful if the ID of the
+    // new object is part of the URL, or that URL differs from GET/PUT URL
+    submitUrl: Ext.identityFn,
+
+    // string or function, called as (url, initialConfig) - mostly for
+    // consistency with submitUrl existing. If both are set `url` gets optional
+    loadUrl: Ext.identityFn,
+
     // needed for finding the reference to submitbutton
     // because we do not have a controller
     referenceHolder: true,
@@ -63,15 +76,16 @@ Ext.define('Proxmox.window.Edit', {
     getValues: function(dirtyOnly) {
        let me = this;
 
-        let values = {};
+       let values = {};
+       Ext.apply(values, me.extraRequestParams);
 
        let form = me.formPanel.getForm();
 
-        form.getFields().each(function(field) {
-            if (!field.up('inputpanel') && (!dirtyOnly || field.isDirty())) {
-                Proxmox.Utils.assemble_field_data(values, field.getSubmitData());
-            }
-        });
+       form.getFields().each(function(field) {
+           if (!field.up('inputpanel') && (!dirtyOnly || field.isDirty())) {
+               Proxmox.Utils.assemble_field_data(values, field.getSubmitData());
+           }
+       });
 
        Ext.Array.each(me.query('inputpanel'), function(panel) {
            Proxmox.Utils.assemble_field_data(values, panel.getValues(dirtyOnly));
@@ -129,7 +143,9 @@ Ext.define('Proxmox.window.Edit', {
            values.background_delay = me.backgroundDelay;
        }
 
-       let url = me.url;
+       let url = Ext.isFunction(me.submitUrl)
+           ? me.submitUrl(me.url, values)
+           : me.submitUrl || me.url;
        if (me.method === 'DELETE') {
            url = url + "?" + Ext.Object.toQueryString(values);
            values = undefined;
@@ -190,22 +206,30 @@ Ext.define('Proxmox.window.Edit', {
            waitMsgTarget: me,
        }, options);
 
+       if (Object.keys(me.extraRequestParams).length > 0) {
+           let params = newopts.params || {};
+           Ext.applyIf(params, me.extraRequestParams);
+           newopts.params = params;
+       }
+
+       let url = Ext.isFunction(me.loadUrl)
+           ? me.loadUrl(me.url, me.initialConfig)
+           : me.loadUrl || me.url;
+
        let createWrapper = function(successFn) {
            Ext.apply(newopts, {
-               url: me.url,
+               url: url,
                method: 'GET',
                success: function(response, opts) {
                    form.clearInvalid();
-                   me.digest = response.result.digest || response.result.data.digest;
+                   me.digest = response.result?.digest || response.result?.data?.digest;
                    if (successFn) {
                        successFn(response, opts);
                    } else {
                        me.setValues(response.result.data);
                    }
                    // hack: fix ExtJS bug
-                   Ext.Array.each(me.query('radiofield'), function(f) {
-                       f.resetOriginalValue();
-                   });
+                   Ext.Array.each(me.query('radiofield'), f => f.resetOriginalValue());
                },
                failure: function(response, opts) {
                    Ext.Msg.alert(gettext('Error'), response.htmlStatus, function() {
@@ -223,18 +247,23 @@ Ext.define('Proxmox.window.Edit', {
     initComponent: function() {
        let me = this;
 
-       if (!me.url) {
-           throw "no url specified";
+       if (!me.url && (
+               !me.submitUrl || !me.loadUrl || me.submitUrl === Ext.identityFn ||
+               me.loadUrl === Ext.identityFn
+           )
+       ) {
+           throw "neither 'url' nor both, submitUrl and loadUrl specified";
+       }
+       if (me.create) {
+           throw "deprecated parameter, use isCreate";
        }
-
-       if (me.create) {throw "deprecated parameter, use isCreate";}
 
        let items = Ext.isArray(me.items) ? me.items : [me.items];
 
        me.items = undefined;
 
        me.formPanel = Ext.create('Ext.form.Panel', {
-           url: me.url,
+           url: me.url, // FIXME: not in 'form' class, safe to remove??
            method: me.method || 'PUT',
            trackResetOnLoad: true,
            bodyPadding: me.bodyPadding !== undefined ? me.bodyPadding : 10,
@@ -292,8 +321,7 @@ Ext.define('Proxmox.window.Edit', {
            resetBtn.setDisabled(!dirty);
 
            if (inputPanel && inputPanel.hasAdvanced) {
-               // we want to show the advanced options
-               // as soon as some of it is not valid
+               // we want to show the advanced options as soon as some of it is not valid
                let advancedItems = me.down('#advancedContainer').query('field');
                let allAdvancedValid = true;
                advancedItems.forEach(function(field) {
@@ -304,7 +332,6 @@ Ext.define('Proxmox.window.Edit', {
 
                if (!allAdvancedValid) {
                    inputPanel.setAdvancedVisible(true);
-                   me.down('#advancedcb').setValue(true);
                }
            }
        };
@@ -323,7 +350,7 @@ Ext.define('Proxmox.window.Edit', {
            me.title = Proxmox.Utils.dialog_title(me.subject, me.isCreate, me.isAdd);
        }
 
-       if (me.isCreate) {
+       if (me.isCreate || !me.showReset) {
                me.buttons = [submitBtn];
        } else {
                me.buttons = [submitBtn, resetBtn];
@@ -333,22 +360,20 @@ Ext.define('Proxmox.window.Edit', {
            let sp = Ext.state.Manager.getProvider();
            let advchecked = sp.get('proxmox-advanced-cb');
            inputPanel.setAdvancedVisible(advchecked);
-           me.buttons.unshift(
-              {
-                  xtype: 'proxmoxcheckbox',
-                  itemId: 'advancedcb',
-                  boxLabelAlign: 'before',
-                  boxLabel: gettext('Advanced'),
-                  stateId: 'proxmox-advanced-cb',
-                  value: advchecked,
-                  listeners: {
-                      change: function(cb, val) {
-                          inputPanel.setAdvancedVisible(val);
-                          sp.set('proxmox-advanced-cb', val);
-                      },
-                  },
-              },
-           );
+           me.buttons.unshift({
+               xtype: 'proxmoxcheckbox',
+               itemId: 'advancedcb',
+               boxLabelAlign: 'before',
+               boxLabel: gettext('Advanced'),
+               stateId: 'proxmox-advanced-cb',
+               value: advchecked,
+               listeners: {
+                   change: function(cb, val) {
+                       inputPanel.setAdvancedVisible(val);
+                       sp.set('proxmox-advanced-cb', val);
+                   },
+               },
+           });
        }
 
        let onlineHelp = me.onlineHelp;