diff --git a/mod/regmod.php b/mod/regmod.php deleted file mode 100644 index 6a31a356a7..0000000000 --- a/mod/regmod.php +++ /dev/null @@ -1,59 +0,0 @@ -. - * - */ - -use Friendica\App; -use Friendica\DI; -use Friendica\Model\User; -use Friendica\Module\Security\Login; - -function regmod_content(App $a) -{ - if (!local_user()) { - info(DI::l10n()->t('Please login.')); - return Login::form(DI::args()->getQueryString(), intval(DI::config()->get('config', 'register_policy')) === \Friendica\Module\Register::CLOSED ? 0 : 1); - } - - if (!is_site_admin() || !empty($_SESSION['submanage'])) { - notice(DI::l10n()->t('Permission denied.')); - return ''; - } - - if ($a->argc != 3) { - exit(); - } - - $cmd = $a->argv[1]; - $hash = $a->argv[2]; - - if ($cmd === 'deny') { - if (User::deny($hash)) { - notice(DI::l10n()->t('Registration revoked')); - } - DI::baseUrl()->redirect('admin/users/'); - } - - if ($cmd === 'allow') { - if (User::allow($hash)) { - info(DI::l10n()->t('Account approved.')); - } - DI::baseUrl()->redirect('admin/users/'); - } -} diff --git a/src/Module/Admin/Users.php b/src/Module/Admin/Users.php index d8594695de..07363b732c 100644 --- a/src/Module/Admin/Users.php +++ b/src/Module/Admin/Users.php @@ -79,7 +79,6 @@ class Users extends BaseAdmin } if (!empty($_POST['page_users_approve'])) { - require_once 'mod/regmod.php'; foreach ($pending as $hash) { if (User::allow($hash)) { info(DI::l10n()->t('Account approved.')); @@ -88,7 +87,6 @@ class Users extends BaseAdmin } if (!empty($_POST['page_users_deny'])) { - require_once 'mod/regmod.php'; foreach ($pending as $hash) { if (User::deny($hash)) { notice(DI::l10n()->t('Registration revoked')); @@ -138,6 +136,16 @@ class Users extends BaseAdmin User::block($uid, false); notice(DI::l10n()->t('User "%s" unblocked', $user['username'])); break; + case 'allow': + parent::checkFormSecurityTokenRedirectOnError('/admin/users', 'admin_users', 't'); + User::allow(Register::getPendingForUser($uid)['hash'] ?? ''); + notice(DI::l10n()->t('Account approved.')); + break; + case 'deny': + parent::checkFormSecurityTokenRedirectOnError('/admin/users', 'admin_users', 't'); + User::deny(Register::getPendingForUser($uid)['hash'] ?? ''); + notice(DI::l10n()->t('Registration revoked')); + break; } DI::baseUrl()->redirect('admin/users'); diff --git a/view/templates/admin/users.tpl b/view/templates/admin/users.tpl index 6ca85fe7aa..8bbffbb7de 100644 --- a/view/templates/admin/users.tpl +++ b/view/templates/admin/users.tpl @@ -35,8 +35,8 @@ {{$u.email}} - - + + diff --git a/view/theme/frio/templates/admin/users.tpl b/view/theme/frio/templates/admin/users.tpl index 256d5cdaec..a9c5f003ee 100644 --- a/view/theme/frio/templates/admin/users.tpl +++ b/view/theme/frio/templates/admin/users.tpl @@ -49,8 +49,8 @@ {{$u.name}} {{$u.email}} - - + + {{if $u.note}} diff --git a/view/theme/quattro/templates/admin/users.tpl b/view/theme/quattro/templates/admin/users.tpl index 97875a0771..bd4e63ec75 100644 --- a/view/theme/quattro/templates/admin/users.tpl +++ b/view/theme/quattro/templates/admin/users.tpl @@ -35,8 +35,8 @@ {{$u.email}} - - + +