Merge lp:~openerp-commiter/openobject-server/partner_address_filter into lp:openobject-server

Proposed by gpa(OpenERP)
Status: Rejected
Rejected by: Fabien (Open ERP)
Proposed branch: lp:~openerp-commiter/openobject-server/partner_address_filter
Merge into: lp:openobject-server
Diff against target: 12 lines (+1/-1)
1 file modified
openerp/addons/base/res/partner/partner.py (+1/-1)
To merge this branch: bzr merge lp:~openerp-commiter/openobject-server/partner_address_filter
Reviewer Review Type Date Requested Status
Fabien (Open ERP) Disapprove
Review via email: mp+49208@code.launchpad.net

Description of the change

Improvement in the name search method for addresses Partner search filters on Reference Field

To post a comment you must log in.
Revision history for this message
Fabien (Open ERP) (fp-tinyerp) wrote :

If we find a reference exactly matching, it has precedence on the name.

review: Disapprove

Unmerged revisions

3355. By gpa(OpenERP)

[FIX] Search Addresses Partner search filters on Reference Field

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'openerp/addons/base/res/partner/partner.py'
--- openerp/addons/base/res/partner/partner.py 2011-01-17 15:17:57 +0000
+++ openerp/addons/base/res/partner/partner.py 2011-02-10 11:33:58 +0000
@@ -193,7 +193,7 @@
193 if not context:193 if not context:
194 context={}194 context={}
195 if name:195 if name:
196 ids = self.search(cr, uid, [('ref', '=', name)] + args, limit=limit, context=context)196 ids = self.search(cr, uid, ['|',('ref', '=', name),('name', '=', name)] + args, limit=limit, context=context)
197 if not ids:197 if not ids:
198 ids = self.search(cr, uid, [('name', operator, name)] + args, limit=limit, context=context)198 ids = self.search(cr, uid, [('name', operator, name)] + args, limit=limit, context=context)
199 else:199 else: