Merge lp:~openerp/openobject-server/qdp-res-currency-name-search-case-insensitive into lp:openobject-server

Proposed by qdp (OpenERP)
Status: Rejected
Rejected by: qdp (OpenERP)
Proposed branch: lp:~openerp/openobject-server/qdp-res-currency-name-search-case-insensitive
Merge into: lp:openobject-server
Diff against target: 16 lines (+2/-2)
1 file modified
openerp/addons/base/res/res_currency.py (+2/-2)
To merge this branch: bzr merge lp:~openerp/openobject-server/qdp-res-currency-name-search-case-insensitive
Reviewer Review Type Date Requested Status
OpenERP Core Team Pending
Review via email: mp+78933@code.launchpad.net

Description of the change

name search of res.currency improved to be case insensitive, such as it is for other search fields

To post a comment you must log in.
Revision history for this message
qdp (OpenERP) (qdp) wrote :

fixed in a more clean way by revision 3753 of server, by odo.

Unmerged revisions

3751. By Quentin (OpenERP) <email address hidden>

[IMP] base, res.currency: improved name_search to be case insensitive

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'openerp/addons/base/res/res_currency.py'
2--- openerp/addons/base/res/res_currency.py 2011-10-05 00:29:56 +0000
3+++ openerp/addons/base/res/res_currency.py 2011-10-11 09:46:28 +0000
4@@ -110,10 +110,10 @@
5 if not args:
6 args = []
7 if name:
8- ids = self.search(cr, user, ([('name','=',name)] + args), limit=limit, context=context)
9+ ids = self.search(cr, user, ([('name','ilike',name)] + args), limit=limit, context=context)
10 name_match = CURRENCY_DISPLAY_PATTERN.match(name)
11 if not ids and name_match:
12- ids = self.search(cr, user, [('name','=', name_match.group(1))] + args, limit=limit, context=context)
13+ ids = self.search(cr, user, [('name','ilike', name_match.group(1))] + args, limit=limit, context=context)
14 else:
15 ids = self.search(cr, user, args, limit=limit, context=context)
16 return self.name_get(cr, user, ids, context=context)