From 900ce3b0eeb9bec8797bea83372952e043e8d2fc Mon Sep 17 00:00:00 2001 From: jim-p Date: Wed, 16 Sep 2015 15:47:28 -0400 Subject: [PATCH] Work around a chicken-and-egg problem in user syncing. See #5152 See also: commit:5372d26d9d25d751d16865ed9d46869d3b0ec5e1 --- src/usr/local/www/system_usermanager.php | 3 +++ 1 file changed, 3 insertions(+) diff --git a/src/usr/local/www/system_usermanager.php b/src/usr/local/www/system_usermanager.php index 2cd524b6a6..d00d420a64 100644 --- a/src/usr/local/www/system_usermanager.php +++ b/src/usr/local/www/system_usermanager.php @@ -354,7 +354,10 @@ if ($_POST['save']) { $a_user[] = $userent; } + /* Add user to groups so PHP can see the memberships properly or else the user's shell account does not get proper permissions (if applicable) See #5152. */ + local_user_set_groups($userent,$_POST['groups']); local_user_set($userent); + /* Add user to groups again to ensure they are set everywhere, otherwise the user may not appear to be a member of the group. See commit:5372d26d9d25d751d16865ed9d46869d3b0ec5e1. */ local_user_set_groups($userent, $_POST['groups']); write_config();