]> git.proxmox.com Git - proxmox-widget-toolkit.git/commitdiff
form: displaye-edit: add one of the two missing returns
authorThomas Lamprecht <t.lamprecht@proxmox.com>
Fri, 17 Nov 2023 14:00:30 +0000 (15:00 +0100)
committerThomas Lamprecht <t.lamprecht@proxmox.com>
Fri, 17 Nov 2023 14:00:32 +0000 (15:00 +0100)
the other one _should_ not be problematic, as field-container itself
isn't picked up as a "real" field itself, but we might bind to that
somewhere, where enabling could break this.

The editable one seems to not be used yet, according to Dominik, so
fix that now already.

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
src/form/DisplayEdit.js

index 97b443faddc49a1ec165116d41e95a6ce2f1f495..fe1b83becb21aa8d89535fe31b41ee30f5f483f6 100644 (file)
@@ -24,7 +24,7 @@ Ext.define('Proxmox.form.field.DisplayEdit', {
     getEditable: function() {
        let me = this;
        let vm = me.getViewModel();
-       vm.get('editable');
+       return vm.get('editable');
     },
 
     setValue: function(value) {
@@ -37,6 +37,7 @@ Ext.define('Proxmox.form.field.DisplayEdit', {
     getValue: function() {
        let me = this;
        let vm = me.getViewModel();
+       // FIXME: add return, but check all use-sites for regressions then
        vm.get('value');
     },