From a20c6e321660429d764815350235bb8445738d10 Mon Sep 17 00:00:00 2001 From: Michael Vogel Date: Thu, 3 Nov 2022 21:13:03 +0100 Subject: [PATCH] Apply suggestions from code review Co-authored-by: Hypolite Petovan --- src/Module/Admin/Site.php | 2 +- src/Module/Register.php | 1 - 2 files changed, 1 insertion(+), 2 deletions(-) diff --git a/src/Module/Admin/Site.php b/src/Module/Admin/Site.php index 0b5f3e9f3..e27719460 100644 --- a/src/Module/Admin/Site.php +++ b/src/Module/Admin/Site.php @@ -494,7 +494,7 @@ class Site extends BaseAdmin '$enable_multi_reg' => ['enable_multi_reg', DI::l10n()->t('Enable multiple registrations'), !DI::config()->get('system', 'block_extended_register'), DI::l10n()->t('Enable users to register additional accounts for use as pages.')], '$enable_openid' => ['enable_openid', DI::l10n()->t('Enable OpenID'), !DI::config()->get('system', 'no_openid'), DI::l10n()->t('Enable OpenID support for registration and logins.')], '$enable_regfullname' => ['enable_regfullname', DI::l10n()->t('Enable Fullname check'), !DI::config()->get('system', 'no_regfullname'), DI::l10n()->t('Enable check to only allow users to register with a space between the first name and the last name in their full name.')], - '$register_notification' => ['register_notification', DI::l10n()->t('Notify admin on new registration'), DI::config()->get('system', 'register_notification'), DI::l10n()->t('If enabled and the system is set to an open registration, a notification for each new registration is sent to the admin.')], + '$register_notification' => ['register_notification', DI::l10n()->t('Email administrators on new registration'), DI::config()->get('system', 'register_notification'), DI::l10n()->t('If enabled and the system is set to an open registration, an email for each new registration is sent to the administrators.')], '$community_page_style' => ['community_page_style', DI::l10n()->t('Community pages for visitors'), DI::config()->get('system', 'community_page_style'), DI::l10n()->t('Which community pages should be available for visitors. Local users always see both pages.'), $community_page_style_choices], '$max_author_posts_community_page' => ['max_author_posts_community_page', DI::l10n()->t('Posts per user on community page'), DI::config()->get('system', 'max_author_posts_community_page'), DI::l10n()->t('The maximum number of posts per user on the community page. (Not valid for "Global Community")')], '$mail_able' => function_exists('imap_open'), diff --git a/src/Module/Register.php b/src/Module/Register.php index b820f4ae1..c168c517e 100644 --- a/src/Module/Register.php +++ b/src/Module/Register.php @@ -333,7 +333,6 @@ class Register extends BaseModule if ($res) { DI::sysmsg()->addInfo(DI::l10n()->t('Registration successful. Please check your email for further instructions.')); - $this->sendNotification($user, 'SYSTEM_REGISTER_NEW'); if (DI::config()->get('system', 'register_notification')) { $this->sendNotification($user, 'SYSTEM_REGISTER_NEW'); }