quick (un)lock for users

This commit is contained in:
Roland Gruber 2012-04-09 13:20:24 +00:00
parent 0d1e3e89dc
commit 770458b82f
7 changed files with 336 additions and 36 deletions

View File

@ -158,6 +158,16 @@ class baseType {
usort($ret, 'compareDN');
return $ret;
}
/**
* This function is called after the edit page is processed and before the page content is generated.
* This can be used to run custom handlers after each page processing.
*
* @param accountContainer $container account container
*/
public function runEditPagePostAction($container) {
}
}

View File

@ -751,6 +751,7 @@ class accountContainer {
$result = array();
$stopProcessing = false; // when set to true, no module options are displayed
$errorsOccured = false;
$typeObject = new $this->type();
$profileLoaded = $this->loadProfileIfRequested();
if ($this->subpage=='') $this->subpage='attributes';
if (isset($_POST['accountContainerReset'])) {
@ -805,8 +806,10 @@ class accountContainer {
}
}
}
$this->sortModules();
$this->sortModules();
}
// run type post actions
$typeObject->runEditPagePostAction($this);
// save account
if (!$errorsOccured && isset($_POST['accountContainerSaveAccount'])) {
// check if all modules are complete
@ -877,7 +880,6 @@ class accountContainer {
}
}
// update titles
$typeObject = new $this->type();
$this->titleBarTitle = $typeObject->getTitleBarTitle($this);
$this->titleBarSubtitle = $typeObject->getTitleBarSubtitle($this);
// prints a module content page

View File

@ -267,7 +267,7 @@ class posixAccount extends baseModule implements passwordService {
'example' => _('Steve Miller,Room 2.14,123-123-1234,123-123-1234')
)
);
if ($this->areGroupOfNamesActive()) {
if (self::areGroupOfNamesActive()) {
$return['upload_columns'][] = array(
'name' => 'posixAccount_gon',
'description' => _('Group of names'),
@ -321,7 +321,7 @@ class posixAccount extends baseModule implements passwordService {
'cn' => _('Common name'),
'userPassword' => _('Password')
);
if ($this->areGroupOfNamesActive()) {
if (self::areGroupOfNamesActive()) {
$return['PDF_fields']['gon'] = _('Group of names');
}
// help Entries
@ -479,7 +479,7 @@ class posixAccount extends baseModule implements passwordService {
}
$this->groups_orig = $this->groups;
// get additional group of names memberships
if ($this->areGroupOfNamesActive()) {
if (self::areGroupOfNamesActive()) {
$gonList1 = searchLDAPByAttribute('member', $this->getAccountContainer()->dn_orig, 'groupOfNames', array('dn'), array('gon', 'group'));
$gonList2 = searchLDAPByAttribute('uniqueMember', $this->getAccountContainer()->dn_orig, 'groupOfUniqueNames', array('dn'), array('gon', 'group'));
$gonList = array_merge($gonList1, $gonList2);
@ -618,7 +618,7 @@ class posixAccount extends baseModule implements passwordService {
}
}
// set group of names
if ($this->areGroupOfNamesActive()) {
if (self::areGroupOfNamesActive()) {
$gons = $this->findGroupOfNames();
$toAdd = array_values(array_diff($this->gonList, $this->gonList_orig));
$toRem = array_values(array_diff($this->gonList_orig, $this->gonList));
@ -766,10 +766,10 @@ class posixAccount extends baseModule implements passwordService {
}
// Load attributes
if (isset($_POST['lockPassword'])) {
$this->attributes['userPassword'][0] = pwd_disable($this->attributes['userPassword'][0]);
$this->lock();
}
if (isset($_POST['unlockPassword'])) {
$this->attributes['userPassword'][0] = pwd_enable($this->attributes['userPassword'][0]);
$this->unlock();
}
if (isset($_POST['removePassword'])) {
unset($this->attributes['userPassword']);
@ -955,7 +955,7 @@ class posixAccount extends baseModule implements passwordService {
$this->groups = array_delete($_POST['removegroups'], $this->groups);
}
// group of names
if ($this->areGroupOfNamesActive()) {
if (self::areGroupOfNamesActive()) {
if (isset($_POST['addgons']) && isset($_POST['addgons_button'])) { // Add groups to list
// add new group
$this->gonList = @array_merge($this->gonList, $_POST['addgons']);
@ -1224,7 +1224,7 @@ class posixAccount extends baseModule implements passwordService {
$return->addElement($unixContainer);
if ($this->areGroupOfNamesActive()) {
if (self::areGroupOfNamesActive()) {
$return->addElement(new htmlSpacer('100px', null));
$gons = $this->findGroupOfNames();
@ -1371,7 +1371,7 @@ class posixAccount extends baseModule implements passwordService {
$addGroupSelect->setTransformSingleSelect(false);
$return->addElement($addGroupSelect, true);
// group of names
if ($this->areGroupOfNamesActive()) {
if (self::areGroupOfNamesActive()) {
$gons = $this->findGroupOfNames();
$gonList = array();
foreach ($gons as $dn => $attr) {
@ -1475,7 +1475,7 @@ class posixAccount extends baseModule implements passwordService {
if (isset($this->attributes['gecos'][0])) {
$return['posixAccount_gecos'] = array('<block><key>' . _('Gecos') . '</key><value>' . $this->attributes['gecos'][0] . '</value></block>');
}
if ($this->areGroupOfNamesActive()) {
if (self::areGroupOfNamesActive()) {
$allGons = $this->findGroupOfNames();
$gons = array();
for ($i = 0; $i < sizeof($this->gonList); $i++) {
@ -1577,7 +1577,7 @@ class posixAccount extends baseModule implements passwordService {
}
$existingGroups = array_keys($groupMap);
// get list of existing group of names
if ($this->areGroupOfNamesActive()) {
if (self::areGroupOfNamesActive()) {
$gons = $this->findGroupOfNames();
$gonList = array();
foreach ($gons as $dn => $attr) {
@ -1679,7 +1679,7 @@ class posixAccount extends baseModule implements passwordService {
}
}
// group of names
if ($this->areGroupOfNamesActive() && ($rawAccounts[$i][$ids['posixAccount_gon']] != "")) {
if (self::areGroupOfNamesActive() && ($rawAccounts[$i][$ids['posixAccount_gon']] != "")) {
$groups = explode(",", $rawAccounts[$i][$ids['posixAccount_gon']]);
for ($g = 0; $g < sizeof($groups); $g++) {
if (!in_array($groups[$g], $gonList)) {
@ -1850,7 +1850,7 @@ class posixAccount extends baseModule implements passwordService {
$groupMap[$groupList[$i][0]] = $groupList[$i][1];
}
// get list of existing group of names
if ($this->areGroupOfNamesActive()) {
if (self::areGroupOfNamesActive()) {
$gonList = $this->findGroupOfNames();
$gonMap = array();
foreach ($gonList as $dn => $attr) {
@ -2334,7 +2334,7 @@ class posixAccount extends baseModule implements passwordService {
*
* @return boolean group of names are active
*/
private function areGroupOfNamesActive() {
public static function areGroupOfNamesActive() {
if (!isset($_SESSION['config'])) {
return false;
}
@ -2377,6 +2377,43 @@ class posixAccount extends baseModule implements passwordService {
return null;
}
/**
* Returns if the Unix part of the current account is locked.
*
* @return boolean password is locked
*/
public function isLocked() {
return isset($this->attributes['userPassword'][0]) && !pwd_is_enabled($this->attributes['userPassword'][0]);
}
/**
* Locks the user password of this account.
*/
public function lock() {
$this->attributes['userPassword'][0] = pwd_disable($this->attributes['userPassword'][0]);
}
/**
* Unlocks the user password of this account.
*/
public function unlock() {
$this->attributes['userPassword'][0] = pwd_enable($this->attributes['userPassword'][0]);
}
/**
* Removes all Unix group memberships from this user.
*/
public function removeFromUnixGroups() {
$this->groups = array();
}
/**
* Removes all group of names memberships from this user.
*/
public function removeFromGONGroups() {
$this->gonList = array();
}
}
?>

View File

@ -619,7 +619,7 @@ class sambaSamAccount extends baseModule implements passwordService {
* @return boolean true, if settings are complete
*/
public function module_complete() {
if (!in_array('sambaSamAccount', $this->attributes['objectClass'])) {
if (!$this->isExtensionEnabled()) {
return true;
}
if ($this->get_scope() == "host") {
@ -672,7 +672,7 @@ class sambaSamAccount extends baseModule implements passwordService {
// skip saving if the extension was not added/modified
return array();
}
if (in_array('sambaSamAccount', $this->attributes['objectClass'])) {
if ($this->isExtensionEnabled()) {
if ($this->expirePassword === true) {
$this->attributes['sambaPwdLastSet'][0] = '0';
}
@ -710,7 +710,7 @@ class sambaSamAccount extends baseModule implements passwordService {
return array();
}
// skip processing if extension is not active
if (!in_array('sambaSamAccount', $this->attributes['objectClass'])) {
if (!$this->isExtensionEnabled()) {
return array();
}
// delete LM hash if needed
@ -1043,7 +1043,7 @@ class sambaSamAccount extends baseModule implements passwordService {
*/
function display_html_attributes() {
$return = new htmlTable();
if (in_array('sambaSamAccount', $this->attributes['objectClass'])) {
if ($this->isExtensionEnabled()) {
if ($this->get_scope() == "host") {
$attrs = $this->getAccountContainer()->getAccountModule('posixAccount')->getAttributes();
if (substr($attrs['uid'][0], -1, 1) != '$') {
@ -2326,7 +2326,39 @@ class sambaSamAccount extends baseModule implements passwordService {
$this->attributes['sambaKickoffTime'][0] = gmmktime(0, 0, 0, intval($month), intval($day),
intval($year));
}
/**
* Returns if the Samba extension is enabled.
*
* @return boolean Samba extension is active
*/
public function isExtensionEnabled() {
return in_array('sambaSamAccount', $this->attributes['objectClass']);
}
/**
* Returns if the Samba part of the current account is deactivated.
*
* @return boolean account is locked
*/
public function isDeactivated() {
return $this->deactivated;
}
/**
* Deactivates this account.
*/
public function deactivate() {
$this->deactivated = true;
}
/**
* Activates this account.
*/
public function activate() {
$this->deactivated = false;
}
}
?>

View File

@ -173,9 +173,9 @@ class user extends baseType {
$personalAttributes = $container->getAccountModule('inetOrgPerson')->getAttributes();
}
if ($personalAttributes == null) {
return null;
return $this->buildAccountStatusIcon($container);
}
$subtitle = '';
$subtitle = $this->buildAccountStatusIcon($container);
$spacer = '&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;';
// check if an email address can be shown
if (isset($personalAttributes['mail'][0])) {
@ -198,17 +198,226 @@ class user extends baseType {
/**
* Builds the HTML code for the icon that shows the account status (locked/unlocked).
*
* @param array $attributes LDAP attributes
* @param accountContainer $container account container
* @return String HTML code for icon
*/
private function buildAccountStatusIcon($attributes) {
$lowerAttributes = array_change_key_case($attributes, CASE_LOWER);
if (!lamUserList::isUnixAvailable($lowerAttributes) && !lamUserList::isUnixAvailable($lowerAttributes) && !lamUserList::isPPolicyAvailable($lowerAttributes)) {
private function buildAccountStatusIcon($container) {
// check if there are account parts that can be locked
$unixAvailable = ($container->getAccountModule('posixAccount') != null);
$sambaAvailable = (($container->getAccountModule('sambaSamAccount') != null) && $container->getAccountModule('sambaSamAccount')->isExtensionEnabled());
$ppolicyAvailable = ($container->getAccountModule('ppolicyUser') != null);
if (!$unixAvailable && !$sambaAvailable && !$ppolicyAvailable) {
return '';
}
return '<img src="../../graphics/lock.png">&nbsp;&nbsp;&nbsp;';
// get locking status
$unixLocked = false;
if ($unixAvailable && $container->getAccountModule('posixAccount')->isLocked()) {
$unixLocked = true;
}
$sambaLocked = false;
if ($sambaAvailable && $container->getAccountModule('sambaSamAccount')->isDeactivated()) {
$sambaLocked = true;
}
$ppolicyLocked = false;
if ($ppolicyAvailable && $container->getAccountModule('ppolicyUser')->isLocked()) {
$ppolicyLocked = true;
}
$partiallyLocked = $unixLocked || $sambaLocked || $ppolicyLocked;
$fullyLocked = ($unixAvailable || $sambaAvailable || $ppolicyAvailable)
&& (!$unixAvailable || $unixLocked)
&& (!$sambaAvailable || $sambaLocked)
&& (!$ppolicyAvailable || $ppolicyLocked);
// build tooltip
$icon = 'unlocked.png';
if ($fullyLocked) {
$icon = 'lock.png';
}
elseif ($partiallyLocked) {
$icon = 'partiallyLocked.png';
}
$statusTable = '<table border=0>';
// Unix
if ($unixAvailable) {
$unixIcon = 'unlocked.png';
if ($unixLocked) {
$unixIcon = 'lock.png';
}
$statusTable .= '<tr><td>' . _('Unix') . '&nbsp;&nbsp;</td><td><img height=16 width=16 src=&quot;../../graphics/' . $unixIcon . '&quot;></td></tr>';
}
// Samba
if ($sambaAvailable) {
$sambaIcon = 'unlocked.png';
if ($sambaLocked) {
$sambaIcon = 'lock.png';
}
$statusTable .= '<tr><td>' . _('Samba') . '&nbsp;&nbsp;</td><td><img height=16 width=16 src=&quot;../../graphics/' . $sambaIcon . '&quot;></td></tr>';
}
// PPolicy
if ($ppolicyAvailable) {
$ppolicyIcon = 'unlocked.png';
if ($ppolicyLocked) {
$ppolicyIcon = 'lock.png';
}
$statusTable .= '<tr><td>' . _('Password policy') . '&nbsp;&nbsp;</td><td><img height=16 width=16 src=&quot;../../graphics/' . $ppolicyIcon . '&quot;></td></tr>';
}
$statusTable .= '</table>';
$tipContent = $statusTable;
$tipContent .= '<br><img alt=&quot;hint&quot; src=&quot;../../graphics/light.png&quot;> ';
$tipContent .= _('Please click to lock/unlock this account.');
$tooltip = "'" . $tipContent . "', TITLE, '" . _('Account status') . "'";
$dialogDiv = $this->buildAccountStatusDialogDiv($unixAvailable, $unixLocked, $sambaAvailable, $sambaLocked, $ppolicyAvailable, $ppolicyLocked);
$onClick = 'onclick="showConfirmationDialog(\'' . _('Change account status') . '\', \'' . _('Ok') . '\', \'' . _('Cancel') . '\', \'lam_accountStatusDialog\', \'inputForm\', \'lam_accountStatusResult\');"';
return $dialogDiv . '<a href="#"><img id="lam_accountStatus" alt="status" ' . $onClick . ' onmouseout="UnTip()" onmouseover="Tip(' . $tooltip . ')" height=16 width=16 src="../../graphics/' . $icon . '"></a>&nbsp;&nbsp;&nbsp;';
}
/**
* Builds the dialog to (un)lock parts of an account.
*
* @param boolean $unixAvailable Unix part is active
* @param boolean $unixLocked Unix part is locked
* @param boolean $sambaAvailable Samba part is active
* @param boolean $sambaLocked Samba part is locked
* @param boolean $ppolicyAvailable PPolicy part is active
* @param boolean $ppolicyLocked PPolicy part is locked
*/
private function buildAccountStatusDialogDiv($unixAvailable, $unixLocked, $sambaAvailable, $sambaLocked, $ppolicyAvailable, $ppolicyLocked) {
$partiallyLocked = $unixLocked || $sambaLocked || $ppolicyLocked;
$fullyLocked = ($unixAvailable || $sambaAvailable || $ppolicyAvailable)
&& (!$unixAvailable || $unixLocked)
&& (!$sambaAvailable || $sambaLocked)
&& (!$ppolicyAvailable || $ppolicyLocked);
$dialog = '<div id="lam_accountStatusDialog" class="hidden">';
// show radio buttons for lock/unlock
$radioDisabled = ' readonly="readonly"';
$lockChecked = ' checked="checked"';
$unlockChecked = '';
$onchange = '';
if ($partiallyLocked && !$fullyLocked) {
$radioDisabled = '';
$onchange = ' onchange="if (jQuery(\'#lam_accountStatusActionRadioLock:checked\').val()) {' .
'jQuery(\'#lam_accountStatusDialogLockDiv\').removeClass(\'hidden\');' .
'jQuery(\'#lam_accountStatusDialogUnlockDiv\').addClass(\'hidden\');' .
'}' .
'else {' .
'jQuery(\'#lam_accountStatusDialogLockDiv\').addClass(\'hidden\');' .
'jQuery(\'#lam_accountStatusDialogUnlockDiv\').removeClass(\'hidden\');' .
'};"';
}
if ($fullyLocked) {
$lockChecked = '';
$unlockChecked = ' checked="checked"';
}
$dialog .= '<input id="lam_accountStatusActionRadioLock" type="radio" name="lam_accountStatusAction" value="lock"' . $onchange . $lockChecked . $radioDisabled . '> ' . _('Lock') . '<br>';
$dialog .= '<input id="lam_accountStatusActionRadioUnlock" type="radio" name="lam_accountStatusAction" value="unlock"' . $onchange . $unlockChecked . $radioDisabled . '> ' . _('Unlock') . '<br><br>';
$dialog .= '<input type="hidden" name="lam_accountStatusResult" id="lam_accountStatusResult" value="cancel">';
// locking part
if (!$fullyLocked) {
$dialog .= '<div id="lam_accountStatusDialogLockDiv"><table border=0>';
if ($unixAvailable && !$unixLocked) {
$dialog .= '<tr><td><input type="checkbox" name="lam_accountStatusLockUnix" checked="checked"></td>';
$dialog .= '<td><img alt="" src="../../graphics/tux.png"></td>';
$dialog .= '<td>' . _('Unix') . '</td>';
}
if ($sambaAvailable && !$sambaLocked) {
$dialog .= '<tr><td><input type="checkbox" name="lam_accountStatusLockSamba" checked="checked"></td>';
$dialog .= '<td><img alt="" src="../../graphics/samba.png"></td>';
$dialog .= '<td>' . _('Samba') . '</td>';
}
if ($ppolicyAvailable && !$ppolicyLocked) {
$dialog .= '<tr><td><input type="checkbox" name="lam_accountStatusLockPPolicy" checked="checked"></td>';
$dialog .= '<td><img alt="" src="../../graphics/security.png"></td>';
$dialog .= '<td>' . _('PPolicy') . '</td>';
}
if ($unixAvailable) {
$dialog .= '<tr><td><input type="checkbox" name="lam_accountStatusRemoveUnixGroups" checked="checked"></td>';
$dialog .= '<td><img alt="" src="../../graphics/groupBig.png"></td>';
$dialog .= '<td>' . _('Remove from all Unix groups') . '</td>';
}
if (posixAccount::areGroupOfNamesActive()) {
$dialog .= '<tr><td><input type="checkbox" name="lam_accountStatusRemoveGONGroups" checked="checked"></td>';
$dialog .= '<td><img alt="" src="../../graphics/groupBig.png"></td>';
$dialog .= '<td>' . _('Remove from all group of (unique) names') . '</td>';
}
$dialog .= '</table></div>';
}
// unlocking part
if ($partiallyLocked) {
$unlockClass = 'hidden';
if ($fullyLocked) {
$unlockClass = '';
}
$dialog .= '<div id="lam_accountStatusDialogUnlockDiv" class="' . $unlockClass . '"><table border=0>';
if ($unixAvailable && $unixLocked) {
$dialog .= '<tr><td><input type="checkbox" name="lam_accountStatusUnlockUnix" checked="checked"></td>';
$dialog .= '<td><img alt="" src="../../graphics/tux.png"></td>';
$dialog .= '<td>' . _('Unix') . '</td>';
}
if ($sambaAvailable && $sambaLocked) {
$dialog .= '<tr><td><input type="checkbox" name="lam_accountStatusUnlockSamba" checked="checked"></td>';
$dialog .= '<td><img alt="" src="../../graphics/samba.png"></td>';
$dialog .= '<td>' . _('Samba') . '</td>';
}
if ($ppolicyAvailable && $ppolicyLocked) {
$dialog .= '<tr><td><input type="checkbox" name="lam_accountStatusUnlockPPolicy" checked="checked"></td>';
$dialog .= '<td><img alt="" src="../../graphics/security.png"></td>';
$dialog .= '<td>' . _('PPolicy') . '</td>';
}
$dialog .= '</table></div>';
}
$dialog .= '</div>';
return $dialog;
}
/**
* This function is called after the edit page is processed and before the page content is generated.
* This can be used to run custom handlers after each page processing.
*
* @param accountContainer $container account container
*/
public function runEditPagePostAction($container) {
// check if account status should be changed
if (isset($_POST['lam_accountStatusResult']) && ($_POST['lam_accountStatusResult'] == 'ok')) {
// lock account
if ($_POST['lam_accountStatusAction'] == 'lock') {
// Unix
if (isset($_POST['lam_accountStatusLockUnix']) && ($_POST['lam_accountStatusLockUnix'] == 'on')) {
$container->getAccountModule('posixAccount')->lock();
}
// Samba
if (isset($_POST['lam_accountStatusLockSamba']) && ($_POST['lam_accountStatusLockSamba'] == 'on')) {
$container->getAccountModule('sambaSamAccount')->deactivate();
}
// PPolicy
if (isset($_POST['lam_accountStatusLockPPolicy']) && ($_POST['lam_accountStatusLockPPolicy'] == 'on')) {
$container->getAccountModule('ppolicyUser')->lock();
}
// remove Unix groups
if (isset($_POST['lam_accountStatusRemoveUnixGroups']) && ($_POST['lam_accountStatusRemoveUnixGroups'] == 'on')) {
$container->getAccountModule('posixAccount')->removeFromUnixGroups();
}
// remove group of names memberships
if (isset($_POST['lam_accountStatusRemoveGONGroups']) && ($_POST['lam_accountStatusRemoveGONGroups'] == 'on')) {
$container->getAccountModule('posixAccount')->removeFromGONGroups();
}
}
// unlock account
elseif ($_POST['lam_accountStatusAction'] == 'unlock') {
// Unix
if (isset($_POST['lam_accountStatusUnlockUnix']) && ($_POST['lam_accountStatusUnlockUnix'] == 'on')) {
$container->getAccountModule('posixAccount')->unlock();
}
// Samba
if (isset($_POST['lam_accountStatusUnlockSamba']) && ($_POST['lam_accountStatusUnlockSamba'] == 'on')) {
$container->getAccountModule('sambaSamAccount')->activate();
}
// PPolicy
if (isset($_POST['lam_accountStatusUnlockPPolicy']) && ($_POST['lam_accountStatusUnlockPPolicy'] == 'on')) {
$container->getAccountModule('ppolicyUser')->unlock();
}
}
}
}
}
/**

View File

@ -238,7 +238,7 @@ $profileNewPwd2->setFieldSize(15);
$container->addElement($profileNewPwd2, true);
$newProfileButton = new htmlButton('btnAddProfile', _('Add'));
$newProfileButton->setOnClick("jQuery('#action').val('add');showConfirmationDialog('" . _("Add profile") . "', '" .
_('Ok') . "', '" . _('Cancel') . "', 'passwordDialogDiv', 'profileForm');");
_('Ok') . "', '" . _('Cancel') . "', 'passwordDialogDiv', 'profileForm', null);");
$container->addElement($newProfileButton, true);
$container->addElement($topicSpacer, true);
@ -250,7 +250,7 @@ $oldProfileInput->setFieldSize(15);
$container->addElement($oldProfileInput, true);
$renameProfileButton = new htmlButton('btnRenameProfile', _('Rename'));
$renameProfileButton->setOnClick("jQuery('#action').val('rename');showConfirmationDialog('" . _("Rename profile") . "', '" .
_('Ok') . "', '" . _('Cancel') . "', 'passwordDialogDiv', 'profileForm');");
_('Ok') . "', '" . _('Cancel') . "', 'passwordDialogDiv', 'profileForm', null);");
$container->addElement($renameProfileButton, true);
$container->addElement($topicSpacer, true);
@ -259,7 +259,7 @@ $container->addElement(new htmlSubTitle(_("Delete profile")), true);
$container->addElement(new htmlTableExtendedSelect('delfilename', $files, array(), _('Profile name'), '232'), true);
$deleteProfileButton = new htmlButton('btnDeleteProfile', _('Delete'));
$deleteProfileButton->setOnClick("jQuery('#action').val('delete');showConfirmationDialog('" . _("Delete profile") . "', '" .
_('Ok') . "', '" . _('Cancel') . "', 'passwordDialogDiv', 'profileForm');");
_('Ok') . "', '" . _('Cancel') . "', 'passwordDialogDiv', 'profileForm', null);");
$container->addElement($deleteProfileButton, true);
$container->addElement($topicSpacer, true);
@ -276,7 +276,7 @@ $profileSetPwd2->setFieldSize(15);
$container->addElement($profileSetPwd2, true);
$setPasswordProfileButton = new htmlButton('btnSetPasswordProfile', _('Set profile password'));
$setPasswordProfileButton->setOnClick("jQuery('#action').val('setpass');showConfirmationDialog('" . _("Set profile password") . "', '" .
_('Ok') . "', '" . _('Cancel') . "', 'passwordDialogDiv', 'profileForm');");
_('Ok') . "', '" . _('Cancel') . "', 'passwordDialogDiv', 'profileForm', null);");
$container->addElement($setPasswordProfileButton, true);
$container->addElement($topicSpacer, true);
@ -287,7 +287,7 @@ $container->addElement(new htmlSubTitle(_("Change default profile")), true);
$container->addElement(new htmlTableExtendedSelect('defaultfilename', $files, array($defaultprofile), _('Profile name'), '234'), true);
$defaultProfileButton = new htmlButton('btnDefaultProfile', _('Ok'));
$defaultProfileButton->setOnClick("jQuery('#action').val('setdefault');showConfirmationDialog('" . _("Change default profile") . "', '" .
_('Ok') . "', '" . _('Cancel') . "', 'passwordDialogDiv', 'profileForm');");
_('Ok') . "', '" . _('Cancel') . "', 'passwordDialogDiv', 'profileForm', null);");
$container->addElement($defaultProfileButton, true);
$container->addElement($topicSpacer, true);

View File

@ -309,11 +309,22 @@ function passwordHandleReply(data) {
* @param cancelText text for Cancel button
* @param dialogDiv div that contains dialog content
* @param formName form to submit
* @param resultField (hidden) input field whose value is set to ok/cancel when button is pressed
*/
function showConfirmationDialog(title, okText, cancelText, dialogDiv, formName) {
function showConfirmationDialog(title, okText, cancelText, dialogDiv, formName, resultField) {
var buttonList = {};
buttonList[cancelText] = function() { jQuery(this).dialog("close"); };
buttonList[okText] = function() { document.forms[formName].submit(); };
buttonList[cancelText] = function() {
if (resultField) {
jQuery('#' + resultField).val('cancel');
};
jQuery(this).dialog("close");
};
buttonList[okText] = function() {
if (resultField) {
jQuery('#' + resultField).val('ok');
};
document.forms[formName].submit();
};
jQuery('#' + dialogDiv).dialog({
modal: true,
title: title,
@ -325,4 +336,3 @@ function showConfirmationDialog(title, okText, cancelText, dialogDiv, formName)
jQuery('#' + dialogDiv).parent().appendTo(document.forms[formName]);
}