diff --git a/models/scop_period.py b/models/scop_period.py
index 97d78279a7af0df502b240865fc6a902d4de49a6..01bb3cdd270fca634aa6a31b084e6c22adbe5ac7 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 f8ba7e9875f288f40a9c95c17cbe387220dfa311..e9a7e9212bd82afb8d5ad516d4aa5f9fb2fd6391 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"/>