Merge lp:~openerp-dev/openobject-addons/saas-3-remove-survey-warning-ptr into lp:~openerp/openobject-addons/saas-3

Proposed by Pariket Trivedi(OpenERP)
Status: Needs review
Proposed branch: lp:~openerp-dev/openobject-addons/saas-3-remove-survey-warning-ptr
Merge into: lp:~openerp/openobject-addons/saas-3
Diff against target: 12 lines (+1/-1)
1 file modified
survey/survey.py (+1/-1)
To merge this branch: bzr merge lp:~openerp-dev/openobject-addons/saas-3-remove-survey-warning-ptr
Reviewer Review Type Date Requested Status
OpenERP Core Team Pending
Review via email: mp+213204@code.launchpad.net

Description of the change

hello,

I have to fixed warning: "The domain term '('survey_id', '=', [2])' should use the 'in' or 'not in' operator."
genrate in backend (terminal).

Steps to Reproduce:

1) Go to Tools module.
2) Go to Surveys tree view Or form View.
3) Click "Print Answer" button.
4) Check terminal you will get the Warning.

thanks,
ptr

To post a comment you must log in.

Unmerged revisions

9352. By Pariket Trivedi(OpenERP)

[FIX]: warning: The domain term '('survey_id', '=', [1])' should be use the 'in' operator.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'survey/survey.py'
2--- survey/survey.py 2013-10-27 12:31:04 +0000
3+++ survey/survey.py 2014-03-28 06:50:40 +0000
4@@ -114,7 +114,7 @@
5 response_id = context.get('response_id', 0)
6 datas['ids'] = [context.get('survey_id', 0)]
7 else:
8- response_id = self.pool.get('survey.response').search(cr, uid, [('survey_id','=', ids)], context=context)
9+ response_id = self.pool.get('survey.response').search(cr, uid, [('survey_id', 'in', ids)], context=context)
10 datas['ids'] = ids
11 page_setting = {'orientation': 'vertical', 'without_pagebreak': 0, 'paper_size': 'letter', 'page_number': 1, 'survey_title': 1}
12 report = {}

Subscribers

People subscribed via source and target branches