openerp-dev-web team mailing list archive

["gpa\(Open ERP\)" <gpa@xxxxxxxxxxx>]

gpa(Open ERP) has proposed merging lp:~openerp-dev/openobject-addons/evaluation_duplicate_question 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/evaluation_duplicate_question/+merge/60030

Fixed for the duplicate question in the evaluation when click on the load question button
-- 
https://code.launchpad.net/~openerp-dev/openobject-addons/evaluation_duplicate_question/+merge/60030
Your team OpenERP R&D Team is subscribed to branch lp:~openerp-dev/openobject-addons/evaluation_duplicate_question.

=== modified file 'training_evaluation/training_evaluation.py'
--- training_evaluation/training_evaluation.py	2011-04-11 08:57:50 +0000
+++ training_evaluation/training_evaluation.py	2011-05-05 09:04:28 +0000
@@ -394,12 +394,15 @@
 
         ev_line = self.pool.get('training.evaluation.line')
         for obj in self.browse(cr, uid, ids, context=context):
+            cr.execute('select question_id from training_evaluation_line where evaluation_id=%s',(obj.id,))
+            question_ids = map(lambda x: x[0], cr.fetchall())
             for question_rel in obj.questionnaire_id.question_ids:
-                values = {
-                    'evaluation_id' : obj.id,
-                    'question_id' : question_rel.question_id.id,
-                }
-                ev_line.create(cr, uid, values, context=context)
+                if question_rel.question_id.id not in question_ids:
+                    values = {
+                        'evaluation_id' : obj.id,
+                        'question_id' : question_rel.question_id.id,
+                    }
+                    ev_line.create(cr, uid, values, context=context)
 
         return True