lp://qastaging/~openerp-dev/openobject-server/7.0-partner-name-search-limit-mat

Branch merges

Propose for merging

No branches dependent on this one.

Merged into lp://qastaging/openobject-server/7.0 at revision 5216

OpenERP Core Team: Pending requested 2014-01-29

Diff: 97 lines (+40/-15)

2 files modified

openerp/addons/base/res/res_partner.py (+26/-15)
openerp/addons/base/tests/test_base.py (+14/-0)

api

Related source package recipes

No recipes using this branch. (?)

Create packaging recipe

Related snap packages

Related bugs

Bug #1203727: 7.0: res.partner - name_search applies limit too early

Low

Fix Released

Link a bug report

Related blueprints

5220. By Martin Trigaux (OpenERP) on 2014-01-29

[IMP] make sure the records will be in right order (and trigger chirurgiens-dentistes.fr for the second name_search)

5219. By Martin Trigaux (OpenERP) on 2014-01-29

[FIX] better test, add missing parentheses in query, return empty list in case of no idea (instead of triggering super)

5218. By Martin Trigaux (OpenERP) on 2014-01-29

[ref] move up!

5217. By Martin Trigaux (OpenERP) on 2014-01-29

[FIX] revert previous rev for better where and add test

5216. By Martin Trigaux (OpenERP) on 2014-01-29

[IMP] remove useless variable, where_clause at least '' as created from a str.join()

5215. By Martin Trigaux (OpenERP) on 2014-01-29

[FIX] res_partner: name_search search in one pass

The original query did two passes (SQL query and orm search). This was a problem due to the limit parameter where the second query could reduce the number of results to a smaller number than the asked limit.

This code reproduces the orm _search behaviour to execute only one SQL query mixing both the ACL clauses and the complexe name_search without degrading the performances (removing the limit in the first query would have).

5214. By Martin Trigaux (OpenERP) on 2014-01-27

[MERGE] [FIX] expression: do not look at translation in parse when searching with empty string or false
This fixes traceback when applying filter 'is not set' on a translatable field

5213. By Launchpad Translations on behalf of openerp on 2014-01-26

Launchpad automatic translations update.

5212. By Launchpad Translations on behalf of openerp on 2014-01-24

Launchpad automatic translations update.

5211. By Launchpad Translations on behalf of openerp on 2014-01-23

Launchpad automatic translations update.