From 4df6cb915187f9067133f014e673edeedc329796 Mon Sep 17 00:00:00 2001
From: nuxsmin <nuxsmin@syspass.org>
Date: Thu, 6 Dec 2018 11:46:25 +0100
Subject: [PATCH] * [FIX] Wrong behaviour when copying account with permissions
 set. Thanks to @Edll for the feedback. Closes #1145

Signed-off-by: nuxsmin <nuxsmin@syspass.org>
---
 app/modules/web/Forms/AccountForm.php                      | 2 --
 .../material-blue/views/account/account-permissions.inc    | 7 +++++++
 2 files changed, 7 insertions(+), 2 deletions(-)

diff --git a/app/modules/web/Forms/AccountForm.php b/app/modules/web/Forms/AccountForm.php
index dc7a5c41..c4d4bc11 100644
--- a/app/modules/web/Forms/AccountForm.php
+++ b/app/modules/web/Forms/AccountForm.php
@@ -103,8 +103,6 @@ final class AccountForm extends FormBase implements FormInterface
         $this->accountRequest->url = $this->request->analyzeString('url');
         $this->accountRequest->notes = $this->request->analyzeString('notes');
         $this->accountRequest->userEditId = $this->context->getUserData()->getId();
-        $this->accountRequest->otherUserEdit = (int)$this->request->analyzeBool('other_user_edit_enabled', false);
-        $this->accountRequest->otherUserGroupEdit = (int)$this->request->analyzeBool('other_usergroup_edit_enabled', false);
         $this->accountRequest->pass = $this->request->analyzeEncrypted('password');
         $this->accountRequest->isPrivate = (int)$this->request->analyzeBool('private_enabled', false);
         $this->accountRequest->isPrivateGroup = (int)$this->request->analyzeBool('private_group_enabled', false);
diff --git a/app/modules/web/themes/material-blue/views/account/account-permissions.inc b/app/modules/web/themes/material-blue/views/account/account-permissions.inc
index 6f7f8000..35e6d7e6 100644
--- a/app/modules/web/themes/material-blue/views/account/account-permissions.inc
+++ b/app/modules/web/themes/material-blue/views/account/account-permissions.inc
@@ -148,6 +148,13 @@
                     </div>
                 </td>
             </tr>
+
+            <?php if ($_getvar('copyAction')): // Forces an update of permissions when copying from another account ?>
+                <input type="hidden" name="other_users_view_update" value="1">
+                <input type="hidden" name="other_users_edit_update" value="1">
+                <input type="hidden" name="other_usergroups_view_update" value="1">
+                <input type="hidden" name="other_usergroups_edit_update" value="1">
+            <?php endif; ?>
         <?php endif; ?>
 
         <?php if ($_getvar('editAction')