# HG changeset patch
# User Thierry Florac <thierry.florac@onf.fr>
# Date 1466437560 -7200
# Node ID de12e7f11dade465787aa1b67f56de3924a824df
# Parent  54e9500c3a0078d33147133382a2f9edde765cf6
Corrected owner change handler

diff -r 54e9500c3a00 -r de12e7f11dad src/pyams_content/shared/common/zmi/owner.py
--- a/src/pyams_content/shared/common/zmi/owner.py	Mon Jun 20 15:54:32 2016 +0200
+++ b/src/pyams_content/shared/common/zmi/owner.py	Mon Jun 20 17:46:00 2016 +0200
@@ -104,9 +104,14 @@
             previous_owner = next(iter(roles.owner))
             roles.owner = {new_owner}
             contributors = roles.contributors
-            if (previous_owner in contributors) and not data.get('keep_owner_as_contributor'):
-                contributors.remove(previous_owner)
+            if data.get('keep_owner_as_contributor'):
+                if previous_owner not in contributors:
+                    contributors.add(previous_owner)
+            else:
+                if previous_owner in contributors:
+                    contributors.remove(previous_owner)
             contributors.add(new_owner)
+            roles.contributors = contributors
             self.request.registry.notify(ObjectModifiedEvent(version))