From f99b59672da9e7b8e9cbce1a29cd331d186b7701 Mon Sep 17 00:00:00 2001
From: Juliana <juliana@le-filament.com>
Date: Tue, 14 Jan 2020 18:17:58 +0100
Subject: [PATCH] =?UTF-8?q?cgscop=20#67=20/=20#49=20:=20Mise=20=C3=A0=20jo?=
 =?UTF-8?q?ur=20champs=20d=C3=A9c=C3=A9s=20+=20modif=20droit=20sur=20res?=
 =?UTF-8?q?=20partner?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 models/scop_period.py       | 17 +++++++++--------
 security/security_rules.xml |  2 +-
 2 files changed, 10 insertions(+), 9 deletions(-)

diff --git a/models/scop_period.py b/models/scop_period.py
index 97d7827..01bb3cd 100644
--- a/models/scop_period.py
+++ b/models/scop_period.py
@@ -68,13 +68,14 @@ class ScopPeriod(models.Model):
     @api.multi
     def write(self, vals):
         for period in self:
-            vals.pop('partner_id', False)
-            vals.pop('id_riga', False)
-            vals.pop('start', False)
-            vals.pop('end', False)
-            vals.pop('end_reason', False)
-            vals.pop('comments', False)
-            vals.pop('dissolution_reason_id', False)
+            partner_vals = dict(vals)
+            partner_vals.pop('partner_id', False)
+            partner_vals.pop('id_riga', False)
+            partner_vals.pop('start', False)
+            partner_vals.pop('end', False)
+            partner_vals.pop('end_reason', False)
+            partner_vals.pop('comments', False)
+            partner_vals.pop('dissolution_reason_id', False)
             # Update partner
-            period.partner_id.write(vals)
+            period.partner_id.write(partner_vals)
         return super(ScopPeriod, self).write(vals)
diff --git a/security/security_rules.xml b/security/security_rules.xml
index f8ba7e9..e9a7e92 100644
--- a/security/security_rules.xml
+++ b/security/security_rules.xml
@@ -52,7 +52,7 @@
             <field name="name">partner rule administrator</field>
             <field name="model_id" ref="base.model_res_partner"/>
             <field name="domain_force">[(1,'=',1)]</field>
-            <field name="groups" eval="[(6, 0, [ref('cgscop_partner.group_cg_administrator')]), (6, 0, [ref('cgscop_partner.group_add_period')]), (6, 0, [ref('cgscop_partner.group_edit_period')])]"/>
+            <field name="groups" eval="[(6, 0, [ref('cgscop_partner.group_cg_administrator'), ref('cgscop_partner.group_add_period'), ref('cgscop_partner.group_edit_period')])]"/>
             <field name="perm_read" eval="False"/>
             <field name="perm_write" eval="True"/>
             <field name="perm_create" eval="True"/>
-- 
GitLab