Skip to content

Commit

Permalink
Don't send notif mail when updating Protected account's Access Level
Browse files Browse the repository at this point in the history
Prior to this, Mantis would effectively not update the protected user
account's 'Access Level' field, but would still send a notification to
the user informing them that the field was modified, causing confusion.

The confirmation message for protected account update was reworded.

Fixes #15247
  • Loading branch information
dregad committed Nov 29, 2012
1 parent 1c847ef commit efdb61e
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 1 deletion.
2 changes: 1 addition & 1 deletion lang/strings_english.txt
Expand Up @@ -997,7 +997,7 @@ $s_account_reset_msg2 = 'Account password has been set to blank...';
$s_account_unlock_msg = 'The account has been unlocked.';

# manage_user_update.php
$s_manage_user_protected_msg = 'Account protected. Access level and enabled protected. Otherwise, account has been updated...';
$s_manage_user_protected_msg = 'Account protected; the \'Access Level\' and \'Enabled\' fields cannot be modified. Other fields were successfully updated.';
$s_manage_user_updated_msg = 'Account successfully updated...';
$s_email_user_updated_subject = 'Account updated';
$s_email_user_updated_msg = 'Your account has been updated by an administrator. A list of these changes is provided below. You can update your account details and preferences at any time by visiting the following URL:';
Expand Down
2 changes: 2 additions & 0 deletions manage_user_update.php
Expand Up @@ -132,6 +132,8 @@
protected=" . db_param() . ", realname=" . db_param() . "
WHERE id=" . db_param();
$query_params = Array( $c_username, $c_email, $c_protected, $c_realname, $c_user_id );
# Prevent e-mail notification for a change that did not happen
$f_access_level = $t_old_access_level;
} else {
$query = "UPDATE $t_user_table
SET username=" . db_param() . ", email=" . db_param() . ",
Expand Down

0 comments on commit efdb61e

Please sign in to comment.