Yogesh(Open ERP) has proposed merging 
lp:~openerp-dev/openobject-addons/training_exam_report_task_2336_ysa into 
lp:~openobject-training/openobject-addons/training.

Requested reviews:
  Bhumika (OpenERP) (sbh-openerp)

For more details, see:
https://code.launchpad.net/~openerp-dev/openobject-addons/training_exam_report_task_2336_ysa/+merge/64013

[FIX] training report:- fix problem in exam sheet report.
-- 
https://code.launchpad.net/~openerp-dev/openobject-addons/training_exam_report_task_2336_ysa/+merge/64013
Your team OpenERP R&D Team is subscribed to branch 
lp:~openerp-dev/openobject-addons/training_exam_report_task_2336_ysa.
=== modified file 'training_extra_report/report/exam_sheet.py'
--- training_extra_report/report/exam_sheet.py	2011-05-10 12:34:43 +0000
+++ training_extra_report/report/exam_sheet.py	2011-06-09 13:24:46 +0000
@@ -181,15 +181,13 @@
         res = []
         if self.localcontext.get('show_response_note',False) and not participation:
             return []
-        part_line = False
         if self.localcontext.get('show_response_note',False):
             # we are displaying was the user checked
             part_line_pool = self.pool.get('training.participation.line')
             part_line_ids = part_line_pool.search(self.cr, self.uid, [('participation_id','=',participation.id),('question_id','=',question.id)])
-            if part_line_ids:
-                part_line = part_line_pool.browse(self.cr, self.uid, part_line_ids[0])
+            part_line = part_line_pool.browse(self.cr, self.uid, part_line_ids[0])
 
-        if part_line and question_type not in['plain','yesno']:
+        if question_type not in['plain','yesno']:
             self.cr.execute('select name,id,is_solution from training_exam_question_answer where question_id = %s',(question.id,))
             res = self.cr.dictfetchall()
             if self.localcontext.get('show_response_note',False):
@@ -197,7 +195,7 @@
                 for row in res:
                     row['is_solution'] = (row['id'] in part_mcq_answers) and 'yes' or 'no'
             return res
-        elif part_line and question_type =='yesno':
+        elif question_type =='yesno':
             resp_yesno = question.response_yesno
             if self.localcontext.get('show_response_note',False):
                 resp_yesno = part_line.response_yesno
@@ -299,13 +297,13 @@
         quizz_id = ids[0]
 
         if data.get('model', '') == 'training.participation':
-            participation = pool.get('training.participation').browse(cr, uid, data['id'], context=context)
+            participation = pool.get('training.participation').browse(cr, uid, data['id'])
             quizz_id = participation.questionnaire_id.id
             if isinstance(quizz_id, bool) and quizz_id == False:
                 raise Exception(_('Could not generate exam sheet report, no questionnaire is defined on the participation'))
 
         quizz_proxy = pool.get('training.exam.questionnaire')
-        quizz = quizz_proxy.browse(cr, uid, quizz_id, context=context)
+        quizz = quizz_proxy.browse(cr, uid, quizz_id)
 
         def write_orig_document():
             r = PdfFileReader(StringIO.StringIO(original_doc[0]))
@@ -379,13 +377,8 @@
 
 class report_preview(report_sxw.report_sxw, report_attachment):
     def create(self, cr, uid, ids, data, context=None):
-        pool = pooler.get_pool(cr.dbname)
-        quizz_id = ids[0]
-        if data.get('model', '') == 'training.participation':
-            participation = pool.get('training.participation').browse(cr, uid, data['id'], context=context)
-            quizz_id = participation.questionnaire_id.id
-            if isinstance(quizz_id, bool) and quizz_id == False:
-                raise Exception(_('Could not generate exam sheet report, no questionnaire is defined on the participation'))
+        if context is None:
+            context = {}
         z = super(report_preview, self).create(cr, uid, ids, data, context=context)
         return self.create_with_attachment(cr, uid, ids, data, z, context=context)
 #
@@ -449,7 +442,7 @@
 
 # Exam Sheet Report
 class training_participation_exam_sheet_parser(training_participation_exam_sheet):
-    REPORT_CONTEXT_FILTERS = ['show_page_header', 'show_participation_firstpage']
+    REPORT_CONTEXT_FILTERS = ['show_page_head', 'show_participation_firstpage']
     REPORT_CONTEXT_FORCE = {
         'show_page_header': True,
         'show_participation_firstpage': True,

_______________________________________________
Mailing list: https://launchpad.net/~openerp-dev-gtk
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~openerp-dev-gtk
More help   : https://help.launchpad.net/ListHelp

Reply via email to