Merge lp:~openerp-dev/openobject-server/trunk-M2O_with_selection-rga into lp:openobject-server

[Ravi Gadhia (OpenERP)]

Hello,

Improve selection field.

Now M2O field with widget="selection" no need to pre-load selection value (at time of fields_view_get) it's(combobox) get value on popup by name_search and we can apply domain as like M2O field

related client branch:
lp:~openerp-dev/openobject-client/trunk-m2o_with_selection-rga

[OpenERP buildbot (openerp-buildbot)]

review: Approve

[Naresh(OpenERP) (nch-openerp)]

Hello,

The diff shows that you partially removed the code for widget="selection". you need to completely remove the implementation of widget="selection" from the server i.e the domains, context handling part. approx line no. in orm.py is 1227 to 1246

Thanks,

review: Needs Resubmitting

[lp:~openerp-dev/openobject-server/trunk-M2O_with_selection-rga]

3370. By Ravi Gadhia (OpenERP) on 2011-04-19

[IMP] client call name_search when combobox popup so no need to eval domain/context here

3371. By Ravi Gadhia (OpenERP) on 2011-04-19

Merge with trunk server

Unmerged revisions

3371. By Ravi Gadhia (OpenERP) on 2011-04-19

Merge with trunk server

3370. By Ravi Gadhia (OpenERP) on 2011-04-19

[IMP] client call name_search when combobox popup so no need to eval domain/context here

3369. By Ravi Gadhia (OpenERP) on 2011-03-15

[IMP] Improve selection field at client side so it's no need to send selection value at view load time, client get it's by name_search when combobox popup

=== modified file 'openerp/osv/orm.py'
--- openerp/osv/orm.py	2011-02-21 17:27:57 +0000
+++ openerp/osv/orm.py	2011-03-15 10:04:28 +0000
@@ -1241,9 +1241,6 @@
                         search_context = dict(context)
                         if column._context and not isinstance(column._context, basestring):
                             search_context.update(column._context)
-                        attrs['selection'] = relation._name_search(cr, user, '', dom, context=search_context, limit=None, name_get_uid=1)
-                        if (node.get('required') and not int(node.get('required'))) or not column.required:
-                            attrs['selection'].append((False, ''))
                 fields[node.get('name')] = attrs
 
         elif node.tag in ('form', 'tree'):