Chirag Patel (OpenERP) has proposed merging 
lp:~openerp-dev/openobject-client-web/6.0-opw-572736-cpa into 
lp:openobject-client-web.

Requested reviews:
  OpenERP Core Team (openerp)

For more details, see:
https://code.launchpad.net/~openerp-dev/openobject-client-web/6.0-opw-572736-cpa/+merge/97626

Hello,

Fixed message not display when print report without select any record from 
listgrid.

Demo: Sales > Sales > Sales Order
1. Click on report "Quotation / Order" in sidebar.

Observed: Message ("You must select one or several records") not display and 
print first record from listgrid.
Expected: Message should be display.

Thanks.
-- 
https://code.launchpad.net/~openerp-dev/openobject-client-web/6.0-opw-572736-cpa/+merge/97626
Your team OpenERP R&D Team is subscribed to branch 
lp:~openerp-dev/openobject-client-web/6.0-opw-572736-cpa.
=== modified file 'addons/openerp/static/javascript/form.js'
--- addons/openerp/static/javascript/form.js	2011-12-19 08:20:25 +0000
+++ addons/openerp/static/javascript/form.js	2012-03-15 12:11:26 +0000
@@ -961,12 +961,8 @@
             new ListView('_terp_list').getSelectedRecords().join(',') +
             ']';
         if (eval(params['_terp_selection']).length == 0) {
-            var ids = eval(jQuery('#_terp_ids').val());
-            if (ids && ids.length > 0){
-                params['_terp_selection'] = '[' + ids[0] + ']';
-            } else {
-                error_display(_('You must select one or several records !'));
-            }
+            error_display(_('You must select one or several records !'));
+            return;
         }
         var id = eval(params['_terp_selection'])[0]
     } else {

_______________________________________________
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