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
=== modified file 'addons/web/static/src/js/view_form.js'
--- addons/web/static/src/js/view_form.js 2013-04-17 13:34:38 +0000
+++ addons/web/static/src/js/view_form.js 2013-05-28 06:15:31 +0000
@@ -2405,6 +2405,9 @@
2405 }2405 }
2406 this._super.apply(this, [value_]);2406 this._super.apply(this, [value_]);
2407 },2407 },
2408 get_value: function() {
2409 return this.get('value') || 0;
2410 },
2408 focus: function () {2411 focus: function () {
2409 var $input = this.$('input:first');2412 var $input = this.$('input:first');
2410 return $input.length ? $input.select() : false;2413 return $input.length ? $input.select() : false;