Mohammed Shekha(Open ERP) has proposed merging 
lp:~openerp-dev/openerp-web/6.1-opw-578475-msh into lp:openerp-web/6.1.

Requested reviews:
  OpenERP Core Team (openerp)

For more details, see:
https://code.launchpad.net/~openerp-dev/openerp-web/6.1-opw-578475-msh/+merge/121832

Hello,

Fixed the issue of Advance filter, if you select float field from the advance 
filter and if you don't enter anything to the input field and try to search it 
will throw error.

Demo :- Accounting -> Journal Entries -> Journal Items -> Advance Filter -> 
Select Balance from the field list -> and don't enter anything in input field, 
it will throw traceback.

Because by default web-client sends False as a value and balance field is 
function field type='float', and the function has some operation on the value 
and value False raising traceback.

Thanks.
-- 
https://code.launchpad.net/~openerp-dev/openerp-web/6.1-opw-578475-msh/+merge/121832
Your team OpenERP R&D Team is subscribed to branch 
lp:~openerp-dev/openerp-web/6.1-opw-578475-msh.
=== modified file 'addons/web/static/src/js/search.js'
--- addons/web/static/src/js/search.js	2012-06-29 12:18:05 +0000
+++ addons/web/static/src/js/search.js	2012-08-29 11:18:31 +0000
@@ -1432,7 +1432,7 @@
     ],
     get_value: function() {
         try {
-            return openerp.web.parse_value(this.$element.val(), {'widget': 'float'});
+            return openerp.web.parse_value(this.$element.val(), {'widget': 'float'}) || 0.0;
         } catch (e) {
             return "";
         }

_______________________________________________
Mailing list: https://launchpad.net/~openerp-dev-gtk
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~openerp-dev-gtk
More help   : https://help.launchpad.net/ListHelp

Reply via email to