Merge lp:~openerp-dev/openobject-addons/6.0-opw-516452-rgo into lp:openobject-addons/6.0

Proposed by Ravi Gohil (OpenERP)
Status: Merged
Approved by: Naresh(OpenERP)
Approved revision: no longer in the source branch.
Merged at revision: 5207
Proposed branch: lp:~openerp-dev/openobject-addons/6.0-opw-516452-rgo
Merge into: lp:openobject-addons/6.0
Diff against target: 32 lines (+11/-8)
1 file modified
base_report_creator/base_report_creator.py (+11/-8)
To merge this branch: bzr merge lp:~openerp-dev/openobject-addons/6.0-opw-516452-rgo
Reviewer Review Type Date Requested Status
Naresh(OpenERP) (community) Approve
Review via email: mp+95555@code.launchpad.net

Description of the change

Hello,

Steps to reproduce the issue:

-> Installed base_report_creator.
-> Created a very simple report with partner name and phone.
-> Executed the report
-> Trying to filter data on tree by name or phone,

This will throw a traceback.

This fixes the issue, fix is back-ported from trunk.

Kindly review this,

Thanks.

To post a comment you must log in.
Revision history for this message
Naresh(OpenERP) (nch-openerp) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'base_report_creator/base_report_creator.py'
--- base_report_creator/base_report_creator.py 2011-01-17 08:00:48 +0000
+++ base_report_creator/base_report_creator.py 2012-04-27 10:05:21 +0000
@@ -201,18 +201,21 @@
201 else:201 else:
202 fields['column_count'] = (False, 'Count')202 fields['column_count'] = (False, 'Count')
203 newargs = []203 newargs = []
204 newargs2 = []204 query_params = []
205 for a in args:205 for a in args:
206 if fields[a[0]][0]:206 if fields[a[0]][0]:
207 res = self.pool.get(fields[a[0]][0])._where_calc(cr, user, [[fields[a[0]][1], a[1], a[2]]], active_test = False, context = context)207 model = self.pool.get(fields[a[0]][0])
208 newargs += res[0]208 if a[1] in ('like', 'ilike', 'not like', 'not ilike'):
209 newargs2 += res[1]209 right = '%%%s%%' % (a[2],)
210 else:210 else:
211 newargs += [("count(*) " + a[1] +" " + str(a[2]))]211 right = a[2]
212 newargs.append(str(model._table+"."+fields[a[0]][1] + " " +a[1] + " %s "))
213 query_params.append(right)
212 ctx = context or {}214 ctx = context or {}
213 ctx['getid'] = True215 ctx['getid'] = True
214 sql_query = report.sql_query216 report_pool = self.pool.get('base_report_creator.report')
215 cr.execute(sql_query) # TODO: FILTER217 reports = report_pool._sql_query_get(cr, user, [context_id], 'sql_query', None, ctx, where_plus=newargs, limit=limit, offset=offset)
218 cr.execute(reports[context_id], query_params)
216 result = cr.fetchall()219 result = cr.fetchall()
217 return map(lambda x: x[0], result)220 return map(lambda x: x[0], result)
218221