summaryrefslogtreecommitdiff
path: root/addons/web
diff options
context:
space:
mode:
authorAaron Bohy <aab@odoo.com>2017-08-23 16:10:44 +0200
committerAaron Bohy <aab@odoo.com>2017-08-24 07:20:22 +0200
commit6e4b59cc9cf2ede2ff54579d0cb672210f2b80ab (patch)
treebabb7757b74ddaadb476dfa6a599f0231da34b02 /addons/web
parent42f22a21614ee7967b900736001b8e68b8ad756e (diff)
[FIX] web: Form: display button box in create mode
Before this rev., the button box wasn't displayed on new (not saved yet) records. This wasn't the case before the new views. This is a problem in lunch, as the lunch.order form view is opened in create mode, and the balance is displayed in the button box.
Diffstat (limited to 'addons/web')
-rw-r--r--addons/web/static/src/js/views/form/form_renderer.js4
-rw-r--r--addons/web/static/tests/views/form_tests.js6
2 files changed, 3 insertions, 7 deletions
diff --git a/addons/web/static/src/js/views/form/form_renderer.js b/addons/web/static/src/js/views/form/form_renderer.js
index 2ef0f077e09..999006e54c2 100644
--- a/addons/web/static/src/js/views/form/form_renderer.js
+++ b/addons/web/static/src/js/views/form/form_renderer.js
@@ -246,10 +246,6 @@ var FormRenderer = BasicRenderer.extend({
_renderButtonBox: function (node) {
var self = this;
var $result = $('<' + node.tag + '>', { 'class': 'o_not_full' });
- // Avoid to show buttons if we are in create mode (edit mode without res_id)
- if (this.mode === 'edit' && !this.state.res_id) {
- return $result;
- }
var buttons = _.map(node.children, function (child) {
if (child.tag === 'button') {
return self._renderStatButton(child);
diff --git a/addons/web/static/tests/views/form_tests.js b/addons/web/static/tests/views/form_tests.js
index ada05e5ed28..3ceb2046756 100644
--- a/addons/web/static/tests/views/form_tests.js
+++ b/addons/web/static/tests/views/form_tests.js
@@ -2642,7 +2642,7 @@ QUnit.module('Views', {
form.destroy();
});
- QUnit.test('button box is not rendered in create mode', function (assert) {
+ QUnit.test('button box is rendered in create mode', function (assert) {
assert.expect(3);
var form = createView({
@@ -2672,8 +2672,8 @@ QUnit.module('Views', {
// create mode (leave edition first!)
form.$buttons.find('.o_form_button_cancel').click();
form.$buttons.find('.o_form_button_create').click();
- assert.strictEqual(form.$('.oe_stat_button').length, 0,
- "button box should not be displayed when creating a new record");
+ assert.strictEqual(form.$('.oe_stat_button').length, 1,
+ "button box should be displayed when creating a new record as well");
form.destroy();
});