From fd519b3c39d032f128961d88b3f52c05ad99bfd1 Mon Sep 17 00:00:00 2001 From: Ralf Becker Date: Wed, 30 Jan 2019 17:22:12 +0100 Subject: [PATCH] disable omitting of unchanged fields for non SQL account backends as existing backend code then tries to remove the value, which fails with an error from LDAP server, as eg. last name is required --- admin/inc/class.admin_account.inc.php | 2 ++ 1 file changed, 2 insertions(+) diff --git a/admin/inc/class.admin_account.inc.php b/admin/inc/class.admin_account.inc.php index 447fb4404e..2260102b86 100644 --- a/admin/inc/class.admin_account.inc.php +++ b/admin/inc/class.admin_account.inc.php @@ -183,6 +183,8 @@ class admin_account // only record real changes if (isset($content['old_account']) && + // currently LDAP (and probably also AD and UCS) can not skip unchanged fields! + get_class($GLOBALS['egw']->accounts->backend) === 'EGroupware\\Api\\Accounts\\Sql' && (!isset($content[$c_name]) && $c_name !== 'account_expires' || // account_expires is not set when empty! $content['old_account'][$a_name] == $content[$c_name])) {