Merge lp:~openerp-dev/openerp-web/7.0-opw-592647-pna into lp:openerp-web/7.0

Proposed by Pinakin Nayi (OpenERP)
Status: Needs review
Proposed branch: lp:~openerp-dev/openerp-web/7.0-opw-592647-pna
Merge into: lp:openerp-web/7.0
Diff against target: 13 lines (+3/-0)
1 file modified
addons/web/static/src/js/view_form.js (+3/-0)
To merge this branch: bzr merge lp:~openerp-dev/openerp-web/7.0-opw-592647-pna
Reviewer Review Type Date Requested Status
Naresh(OpenERP) Pending
Review via email: mp+165961@code.launchpad.net

Description of the change

Hello Sir,

I fixed issue in view_form.

Steps to reproduce :
When creating bank statement and make amount field empty and click on red
button gives client error like ..

Uncaught Error: TypeError: bad operand type for abs(): 'bool'

Thanks,
pna

To post a comment you must log in.

Unmerged revisions

3936. By Pinakin Nayi (OpenERP)

[FIX]view_form : problem with FieldFloat widget ex. abs(amount) and if there is amount is blank than it will pass boolean and gives client error

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'addons/web/static/src/js/view_form.js'
2--- addons/web/static/src/js/view_form.js 2013-04-17 13:34:38 +0000
3+++ addons/web/static/src/js/view_form.js 2013-05-28 06:15:31 +0000
4@@ -2405,6 +2405,9 @@
5 }
6 this._super.apply(this, [value_]);
7 },
8+ get_value: function() {
9+ return this.get('value') || 0;
10+ },
11 focus: function () {
12 var $input = this.$('input:first');
13 return $input.length ? $input.select() : false;