[ticket/10280] Change the display of user activation settings in the ACP.

Use a select box to be consistent with other settings.

PHPBB3-10280
This commit is contained in:
RMcGirr83 2011-07-16 08:22:29 -04:00 committed by Oleg Pudeyev
parent 372698218a
commit 3ce5b1d386

View file

@ -234,7 +234,7 @@ class acp_board
'max_name_chars' => array('lang' => 'USERNAME_LENGTH', 'validate' => 'int:8:180', 'type' => false, 'method' => false, 'explain' => false,), 'max_name_chars' => array('lang' => 'USERNAME_LENGTH', 'validate' => 'int:8:180', 'type' => false, 'method' => false, 'explain' => false,),
'max_pass_chars' => array('lang' => 'PASSWORD_LENGTH', 'validate' => 'int:8:255', 'type' => false, 'method' => false, 'explain' => false,), 'max_pass_chars' => array('lang' => 'PASSWORD_LENGTH', 'validate' => 'int:8:255', 'type' => false, 'method' => false, 'explain' => false,),
'require_activation' => array('lang' => 'ACC_ACTIVATION', 'validate' => 'int', 'type' => 'custom', 'method' => 'select_acc_activation', 'explain' => true), 'require_activation' => array('lang' => 'ACC_ACTIVATION', 'validate' => 'int', 'type' => 'select', 'method' => 'select_acc_activation', 'explain' => true),
'new_member_post_limit' => array('lang' => 'NEW_MEMBER_POST_LIMIT', 'validate' => 'int:0:255', 'type' => 'text:4:4', 'explain' => true, 'append' => ' ' . $user->lang['POSTS']), 'new_member_post_limit' => array('lang' => 'NEW_MEMBER_POST_LIMIT', 'validate' => 'int:0:255', 'type' => 'text:4:4', 'explain' => true, 'append' => ' ' . $user->lang['POSTS']),
'new_member_group_default'=> array('lang' => 'NEW_MEMBER_GROUP_DEFAULT', 'validate' => 'bool', 'type' => 'radio:yes_no', 'explain' => true), 'new_member_group_default'=> array('lang' => 'NEW_MEMBER_GROUP_DEFAULT', 'validate' => 'bool', 'type' => 'radio:yes_no', 'explain' => true),
'min_name_chars' => array('lang' => 'USERNAME_LENGTH', 'validate' => 'int:1', 'type' => 'custom:5:180', 'method' => 'username_length', 'explain' => true), 'min_name_chars' => array('lang' => 'USERNAME_LENGTH', 'validate' => 'int:1', 'type' => 'custom:5:180', 'method' => 'username_length', 'explain' => true),
@ -768,24 +768,28 @@ class acp_board
/** /**
* Select account activation method * Select account activation method
*/ */
function select_acc_activation($value, $key = '') function select_acc_activation($selected_value, $value)
{ {
global $user, $config; global $user, $config;
$radio_ary = array( $act_ary = array(
USER_ACTIVATION_DISABLE => 'ACC_DISABLE', 'ACC_DISABLE' => USER_ACTIVATION_DISABLE,
USER_ACTIVATION_NONE => 'ACC_NONE', 'ACC_NONE' => USER_ACTIVATION_NONE,
); );
if ($config['email_enable']) if ($config['email_enable'])
{ {
$radio_ary[USER_ACTIVATION_SELF] = 'ACC_USER'; $act_ary['ACC_USER'] = USER_ACTIVATION_SELF;
$radio_ary[USER_ACTIVATION_ADMIN] = 'ACC_ADMIN'; $act_ary['ACC_ADMIN'] = USER_ACTIVATION_ADMIN;
}
$act_options = '';
foreach ($act_ary as $key => $value)
{
$selected = ($selected_value == $value) ? ' selected="selected"' : '';
$act_options .= '<option value="' . $value . '"' . $selected . '>' . $user->lang[$key] . '</option>';
} }
$radio_text = h_radio('config[require_activation]', $radio_ary, $value, 'require_activation', $key, '<br />'); return $act_options;
return $radio_text;
} }
/** /**