From 36cd4c78dc26c3fa323bf5bbead4db7c316b1ed7 Mon Sep 17 00:00:00 2001
From: "Pedro M. Baeza" <pedro.baeza@gmail.com>
Date: Mon, 16 Mar 2015 03:59:56 +0100
Subject: [PATCH] quality_control * Fix wizard set test

* Expand AUTHORS

* Fix permissions
---
 quality_control/__openerp__.py               | 4 +++-
 quality_control/models/qc_inspection.py      | 2 +-
 quality_control/security/ir.model.access.csv | 1 +
 quality_control/wizard/qc_test_wizard.py     | 7 +++++--
 4 files changed, 10 insertions(+), 4 deletions(-)

diff --git a/quality_control/__openerp__.py b/quality_control/__openerp__.py
index 622f493c4..934b5c52c 100644
--- a/quality_control/__openerp__.py
+++ b/quality_control/__openerp__.py
@@ -24,7 +24,9 @@
 {
     "name": "Quality control",
     "version": "1.0",
-    "author": "OdooMRP team",
+    "author": "OdooMRP team,"
+              "AvanzOSC,"
+              "Serv. Tecnol. Avanzados - Pedro M. Baeza",
     "website": "http://www.odoomrp.com",
     "contributors": [
         "Pedro M. Baeza <pedro.baeza@serviciosbaeza.com",
diff --git a/quality_control/models/qc_inspection.py b/quality_control/models/qc_inspection.py
index 149b6237a..c34f04907 100644
--- a/quality_control/models/qc_inspection.py
+++ b/quality_control/models/qc_inspection.py
@@ -153,7 +153,7 @@ class QcInspection(models.Model):
             del header['auto_generated']  # don't change auto_generated flag
             del header['user']  # don't change current user
             inspection.write(header)
-            self.inspection_lines.unlink()
+            inspection.inspection_lines.unlink()
             inspection.inspection_lines = inspection._prepare_inspection_lines(
                 trigger_line.test, force_fill=force_fill)
 
diff --git a/quality_control/security/ir.model.access.csv b/quality_control/security/ir.model.access.csv
index 42ef2d392..a4b01334b 100644
--- a/quality_control/security/ir.model.access.csv
+++ b/quality_control/security/ir.model.access.csv
@@ -3,6 +3,7 @@ access_user_qc_inspection,qc_inspection,quality_control.model_qc_inspection,qual
 access_user_qc_inspection_line,qc_inspection_line,quality_control.model_qc_inspection_line,quality_control.group_quality_control_user,1,1,1,1
 access_user_qc_test,qc_test user,quality_control.model_qc_test,quality_control.group_quality_control_user,1,0,0,0
 access_user_qc_test_question,qc_test_question user,quality_control.model_qc_test_question,quality_control.group_quality_control_user,1,0,0,0
+access_user_qc_test_question_value,qc_test_question_value_user,quality_control.model_qc_test_question_value,quality_control.group_quality_control_user,1,0,0,0
 access_manager_qc_test,qc_test manager,quality_control.model_qc_test,quality_control.group_quality_control_manager,1,1,1,1
 access_manager_qc_test_question,qc_test_question manager,quality_control.model_qc_test_question,quality_control.group_quality_control_manager,1,1,1,1
 access_manager_qc_test_question_value,qc_test_question_value,quality_control.model_qc_test_question_value,quality_control.group_quality_control_manager,1,1,1,1
diff --git a/quality_control/wizard/qc_test_wizard.py b/quality_control/wizard/qc_test_wizard.py
index 0545f1429..149d4ccfd 100644
--- a/quality_control/wizard/qc_test_wizard.py
+++ b/quality_control/wizard/qc_test_wizard.py
@@ -17,7 +17,10 @@ class QcInspectionSetTest(models.TransientModel):
 
     @api.multi
     def action_create_test(self):
-        inspection_obj = self.env['qc.inspection']
-        inspection_obj.browse(self.env.context['active_id']).set_test(
+        inspection = self.env['qc.inspection'].browse(
+            self.env.context['active_id'])
+        inspection.test = self.test
+        inspection.inspection_lines.unlink()
+        inspection.inspection_lines = inspection._prepare_inspection_lines(
             self.test)
         return True