diff options
| author | Nikunj Ladava <nla@odoo.com> | 2018-01-31 16:51:13 +0530 |
|---|---|---|
| committer | Aaron Bohy <aab@odoo.com> | 2018-02-15 17:25:00 +0100 |
| commit | 4c186f30c3a77e661f0dfdd39fe2c1216ef1ac7b (patch) | |
| tree | a3e0455cbf90f257f6ea7a74b5309bfc0402f583 /addons/web/static/src | |
| parent | 8b9e854be4bfe3eda9a15368a864f7af606dee31 (diff) | |
[FIX] web: ActionManager: handle false actions
Server actions return an action to execute, or false. When they
return false, an 'ir.actions.act_window_close' must be executed,
such that the current dialog closes (if any) and mostly, the
'on_close' handler is executed, which ensures for example that the
current view is reloaded.
The case where server actions return false has been broken by rev.
32b8cec53. It produced tracebacks in several models, when clicking
on some actions in the 'Action' dropdown. For instance, in lunch,
go to Configuration > Products, open one in a form view, and click
on 'Lunch: Archive/Restore products' in the Action dropdown.
Diffstat (limited to 'addons/web/static/src')
| -rw-r--r-- | addons/web/static/src/js/chrome/action_manager.js | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/addons/web/static/src/js/chrome/action_manager.js b/addons/web/static/src/js/chrome/action_manager.js index 237ce1f1c96..1651e5d72d9 100644 --- a/addons/web/static/src/js/chrome/action_manager.js +++ b/addons/web/static/src/js/chrome/action_manager.js @@ -510,6 +510,7 @@ var ActionManager = Widget.extend({ }, }); return this.dp.add(runDef).then(function (action) { + action = action || { type: 'ir.actions.act_window_close' }; return self.doAction(action, options); }); }, |
