From 098a5f68475fd518bfbef2eadc8800fdc180c09b Mon Sep 17 00:00:00 2001 From: Raphael Collet Date: Tue, 7 Feb 2012 12:20:15 +0100 Subject: [PATCH] [IMP] project_planning: improve badly built query bzr revid: rco@openerp.com-20120207112015-am6s4jux0nj2xybi --- addons/project_planning/project_planning.py | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/addons/project_planning/project_planning.py b/addons/project_planning/project_planning.py index 5bfba1c427b..02b214a2ccd 100644 --- a/addons/project_planning/project_planning.py +++ b/addons/project_planning/project_planning.py @@ -60,11 +60,12 @@ class report_account_analytic_planning(osv.osv): result = {} for user_id in ids: child_ids = [] - cr.execute('SELECT dept.id FROM hr_department AS dept \ - LEFT JOIN hr_employee AS emp ON dept.manager_id = emp.id \ - WHERE emp.id IN \ - (SELECT emp.id FROM hr_employee \ - JOIN resource_resource r ON r.id = emp.resource_id WHERE r.user_id=' + str(user_id) + ') ') + cr.execute("""SELECT dept.id FROM hr_department AS dept + LEFT JOIN hr_employee AS emp ON dept.manager_id = emp.id + WHERE emp.id IN + (SELECT emp.id FROM hr_employee + JOIN resource_resource r ON r.id = emp.resource_id WHERE r.user_id = %s) + """, (user_id,)) mgnt_dept_ids = [x[0] for x in cr.fetchall()] ids_dept = obj_dept.search(cr, uid, [('id', 'child_of', mgnt_dept_ids)], context=context) if ids_dept: