diff options
| author | Aaron Bohy <aab@odoo.com> | 2019-09-12 08:31:05 +0000 |
|---|---|---|
| committer | Julien Mougenot <jum@odoo.com> | 2019-09-18 08:24:13 +0000 |
| commit | af65475a4cd33ca8c6b25ba525d2f59cc28b3f43 (patch) | |
| tree | 4823248c368a9b98d4fd28518bd02ac10e044ef4 /addons/web/static/src/js | |
| parent | ac85176ce81427ba6f55b59e28c1ff6e7e080c32 (diff) | |
[FIX] web: list: multi edit input of date(time) fields
The Date(Time)Fields listen to 'input' and 'change' events (like
all InputFields) to keep track of changes. However, the datepicker
already listens to the 'change' event. As a consequence, in multi
edition in a list view, if the user changed the value of a
date(time) field by directly editing the input, the fieldChanged
event was triggered twice, and there were 2 dialogs asking the user
if he wanted to save the change.
This rev. makes the Date(Time)Fields stop listening themselves to
'input' and 'change' events, and instead completely rely on the
datepicker widget.
Task 2068280
Diffstat (limited to 'addons/web/static/src/js')
| -rw-r--r-- | addons/web/static/src/js/fields/basic_fields.js | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/addons/web/static/src/js/fields/basic_fields.js b/addons/web/static/src/js/fields/basic_fields.js index 38276373cc2..abdd3e57c37 100644 --- a/addons/web/static/src/js/fields/basic_fields.js +++ b/addons/web/static/src/js/fields/basic_fields.js @@ -488,6 +488,9 @@ var FieldDate = InputField.extend({ className: "o_field_date", tagName: "span", supportedFieldTypes: ['date', 'datetime'], + // we don't need to listen on 'input' nor 'change' events because the + // datepicker widget is already listening, and will correctly notify changes + events: AbstractField.prototype.events, /** * @override |