diff --git a/src/www/system_usermanager.php b/src/www/system_usermanager.php index 7758c1341..6683a7186 100644 --- a/src/www/system_usermanager.php +++ b/src/www/system_usermanager.php @@ -119,7 +119,7 @@ if ($_SERVER['REQUEST_METHOD'] === 'GET') { exit; } elseif ($act == 'new' || $act == 'edit') { // edit user, load or init data - $fieldnames = array('user_dn', 'descr', 'expires', 'scope', 'uid', 'priv', 'ipsecpsk', 'lifetime', 'otp_seed'); + $fieldnames = array('user_dn', 'descr', 'expires', 'scope', 'uid', 'priv', 'ipsecpsk', 'lifetime', 'otp_seed', 'email', 'comment'); if (isset($id)) { if (isset($a_user[$id]['authorizedkeys'])) { $pconfig['authorizedkeys'] = base64_decode($a_user[$id]['authorizedkeys']); @@ -195,7 +195,7 @@ if ($_SERVER['REQUEST_METHOD'] === 'GET') { echo json_encode($keyData); } exit; - } elseif ($act =='delApiKey' && isset($id)) { + } elseif ($act =='delApiKey' && isset($id)) { $username = $a_user[$id]['name']; if (!empty($pconfig['api_delete'])) { $authFactory = new \OPNsense\Auth\AuthenticationFactory(); @@ -331,6 +331,18 @@ if ($_SERVER['REQUEST_METHOD'] === 'GET') { unset($userent['disabled']); } + if (!empty($pconfig['email'])) { + $userent['email'] = $pconfig['email']; + } elseif (isset($userent['email'])) { + unset($userent['email']); + } + + if (!empty($pconfig['comment'])) { + $userent['comment'] = $pconfig['comment']; + } elseif (isset($userent['comment'])) { + unset($userent['comment']); + } + if (isset($id)) { $a_user[$id] = $userent; } else { @@ -344,15 +356,12 @@ if ($_SERVER['REQUEST_METHOD'] === 'GET') { if (!empty($pconfig['chkNewCert'])) { // redirect to cert manager when a new cert is requested for this user - header(url_safe('Location: /system_certmanager.php?act=new&userid=%s', array(count($a_user) - 1))); + header(url_safe('Location: /system_certmanager.php?act=new&userid=%s', array(isset($id) ? $id : count($a_user) - 1))); } else { - header(url_safe('Location: /system_usermanager.php')); + header(url_safe('Location: /system_usermanager.php?act=edit&userid=%s&savemsg=%s', array(isset($id) ? $id : count($a_user) - 1, get_std_save_message()))); exit; } } - } elseif (isset($id)) { - header(url_safe('Location: /system_usermanager.php?userid=%s', array($id))); - exit; } else { header(url_safe('Location: /system_usermanager.php')); exit; @@ -584,6 +593,24 @@ $( document ).ready(function() { +