Merge lp:~openerp-dev/openobject-client-web/6.0-opw-5796-sma into lp:openobject-client-web

[Sananaz (Open ERP)]

Hello,

To generate problem:
- open a sale order line in a sale order
- you can't open the m2o fields (product_id or product_uom)

If can't generate problem please clear your browser cache.

Thank you.

[Xavier (Open ERP) (xmo-deactivatedaccount)]

The '/' prefix was added at the same time the conditional above was modified to add a 'split' in it.

I'm pretty sure if we go through conditional, things are going to break because there will not be a final '/' and so we'll get `somefield_terp_model` instead of `somefield/_terp_model`.

I think we need to leave it before `_terp_model` but remove adding of trailing '/' in original creation of `prefix`. What do you think?

review: Needs Fixing

[lp:~openerp-dev/openobject-client-web/6.0-opw-5796-sma]

4619. By Xavier (Open ERP) on 2011-05-26

[FIX] make substring search on m2o fields (without popping up the m2o search) significantly more reliable, thanks Stefan

4620. By Launchpad Translations on behalf of openerp on 2011-05-27

Launchpad automatic translations update.

4621. By Launchpad Translations on behalf of openerp on 2011-05-29

Launchpad automatic translations update.

4622. By Launchpad Translations on behalf of openerp on 2011-05-30

Launchpad automatic translations update.

4623. By Launchpad Translations on behalf of openerp on 2011-05-31

Launchpad automatic translations update.

4624. By Navrang Oza on 2011-05-31

[FIX] Syntax errors were not handled properly.

4625. By Navrang Oza on 2011-06-01

[FIX] Removed unnecessary comma (,) to avoide IE issue. (OPW-5969).

[Navrang Oza (noz-tiny)]

@xmo, You are right, we should not add '/' in prefix if we are going to remove after words.

As sma is not here so I made changes in her branch and resubmitting here.

review: Needs Resubmitting

[lp:~openerp-dev/openobject-client-web/6.0-opw-5796-sma]

4626. By Navrang Oza on 2011-06-02

[FIX] Import : Web client adds an extra seperated line [] and sends to server, which turns into crash at server failing to read data within the line.

4627. By Launchpad Translations on behalf of openerp on 2011-06-03

Launchpad automatic translations update.

4628. By Launchpad Translations on behalf of openerp on 2011-06-05

Launchpad automatic translations update.

4629. By Launchpad Translations on behalf of openerp on 2011-06-06

Launchpad automatic translations update.

4630. By Navrang Oza on 2011-06-06

[FIX] m2m contextmenu doesn't appear in o2m popup.

=== modified file 'addons/openerp/static/javascript/form.js'
--- addons/openerp/static/javascript/form.js	2011-06-01 04:54:40 +0000
+++ addons/openerp/static/javascript/form.js	2011-06-01 07:32:37 +0000
@@ -802,8 +802,8 @@
 function makeContextMenu(id, kind, relation, val){
     var act = get_form_action('get_context_menu');
 
-    var prefix = id.indexOf('/') > -1 ? id.slice(0, id.lastIndexOf('/')) + '/' : '';
-    if ((prefix.split('/')[0])== '_terp_listfields') {
+    var prefix = id.indexOf('/') > -1 ? id.slice(0, id.lastIndexOf('/')) : '';
+    if ((prefix.split('/')[0]) == '_terp_listfields') {
         prefix = (prefix.split('/')[1]);
     }