From a55813d6d7bebb08ccdfc12211170805d4f5665d Mon Sep 17 00:00:00 2001 From: Dietmar Maurer Date: Thu, 30 Mar 2017 06:30:14 +0200 Subject: [PATCH] do not override getSubmitValue, add note about strange semantics --- Toolkit.js | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/Toolkit.js b/Toolkit.js index 811bbca..b1709bf 100644 --- a/Toolkit.js +++ b/Toolkit.js @@ -201,17 +201,15 @@ Ext.define('Proxmox.Datepicker', { hideMode: 'visibility' }); -// ExtJS 6.0.1 has no get/setSubmitValue() (although you find it in the docs). +// ExtJS 6.0.1 has no setSubmitValue() (although you find it in the docs). +// Note: this.submitValue is a boolean flag, whereas getSubmitValue() returns +// data to be submitted. Ext.define('Proxmox.form.field.Text', { override: 'Ext.form.field.Text', setSubmitValue: function(v) { this.submitValue = v; }, - - getSubmitValue: function(v) { - return this.submitValue; - } }); // force alert boxes to be rendered with an Error Icon