← Back to team overview

openerp-dev-web team mailing list archive

[Merge] lp:~openerp-dev/openobject-client-web/6.0-opw-4311-sma into lp:openobject-client-web/6.0

 

Sananaz (Open ERP) has proposed merging lp:~openerp-dev/openobject-client-web/6.0-opw-4311-sma into lp:openobject-client-web/6.0.

Requested reviews:
  OpenERP Core Team (openerp)
Related bugs:
  Bug #726491 in OpenERP Web Client: "Wizards do not respect the existing OSV.OSV record and gets called disregard to the record exists or not."
  https://bugs.launchpad.net/openobject-client-web/+bug/726491

For more details, see:
https://code.launchpad.net/~openerp-dev/openobject-client-web/6.0-opw-4311-sma/+merge/53953

Hello,

Please check with the sidebar actions:

Go to Product list > Click on `New` (Don't Save) > Click Procurement Request (Actions at sidebar) 
It gives traceback.
So just give the message (`You must save this record to perform the action !`) same as GTK client if form is not saved don't perform any action.

Thank you.

-- 
https://code.launchpad.net/~openerp-dev/openobject-client-web/6.0-opw-4311-sma/+merge/53953
Your team OpenERP R&D Team is subscribed to branch lp:~openerp-dev/openobject-client-web/6.0-opw-4311-sma.
=== modified file 'addons/openerp/static/javascript/form.js'
--- addons/openerp/static/javascript/form.js	2011-01-28 15:21:15 +0000
+++ addons/openerp/static/javascript/form.js	2011-03-18 07:09:31 +0000
@@ -963,6 +963,10 @@
         var id = eval(params['_terp_selection'])[0]
     } else {
         var id = jQuery('[id="'+field+'"]').val();
+        if (id == 'False') {
+        	error_display(_("You must save this record to perform the action !"));
+            return;
+        }
     }
 
     var action_id = $src.attr('action_id') || null;


Follow ups