Replace System::baseUrl() by App->getBaseURL() in mod/register

- Remove redundant System::baseUrl() in goaway() calls
- Add more generic Friendica\Model use statement
- Allow empty path in goaway()
This commit is contained in:
Hypolite Petovan 2018-10-14 08:58:27 -04:00
parent 6606c9cbb2
commit 0563a28438
2 changed files with 27 additions and 15 deletions

View File

@ -647,7 +647,7 @@ function killme()
/** /**
* @brief Redirect to another URL and terminate this process. * @brief Redirect to another URL and terminate this process.
*/ */
function goaway($path) function goaway($path = '')
{ {
if (strstr(normalise_link($path), 'http://')) { if (strstr(normalise_link($path), 'http://')) {
$url = $path; $url = $path;

View File

@ -12,7 +12,7 @@ use Friendica\Core\PConfig;
use Friendica\Core\System; use Friendica\Core\System;
use Friendica\Core\Worker; use Friendica\Core\Worker;
use Friendica\Database\DBA; use Friendica\Database\DBA;
use Friendica\Model\User; use Friendica\Model;
use Friendica\Module\Tos; use Friendica\Module\Tos;
use Friendica\Util\DateTimeFormat; use Friendica\Util\DateTimeFormat;
@ -67,7 +67,7 @@ function register_post(App $a)
$arr['language'] = L10n::getBrowserLanguage(); $arr['language'] = L10n::getBrowserLanguage();
try { try {
$result = User::create($arr); $result = Model\User::create($arr);
} catch (Exception $e) { } catch (Exception $e) {
notice($e->getMessage()); notice($e->getMessage());
return; return;
@ -76,7 +76,7 @@ function register_post(App $a)
$user = $result['user']; $user = $result['user'];
if ($netpublish && intval(Config::get('config', 'register_policy')) !== REGISTER_APPROVE) { if ($netpublish && intval(Config::get('config', 'register_policy')) !== REGISTER_APPROVE) {
$url = System::baseUrl() . '/profile/' . $user['nickname']; $url = $a->getBaseUrl() . '/profile/' . $user['nickname'];
Worker::add(PRIORITY_LOW, "Directory", $url); Worker::add(PRIORITY_LOW, "Directory", $url);
} }
@ -92,12 +92,18 @@ function register_post(App $a)
// Only send a password mail when the password wasn't manually provided // Only send a password mail when the password wasn't manually provided
if (!x($_POST, 'password1') || !x($_POST, 'confirm')) { if (!x($_POST, 'password1') || !x($_POST, 'confirm')) {
$res = User::sendRegisterOpenEmail( $res = Model\User::sendRegisterOpenEmail(
$user['email'], Config::get('config', 'sitename'), System::baseUrl(), $user['username'], $result['password'], $user); $user['email'],
Config::get('config', 'sitename'),
$a->getBaseUrl(),
$user['username'],
$result['password'],
$user
);
if ($res) { if ($res) {
info(L10n::t('Registration successful. Please check your email for further instructions.') . EOL); info(L10n::t('Registration successful. Please check your email for further instructions.') . EOL);
goaway(System::baseUrl()); goaway();
} else { } else {
notice( notice(
L10n::t('Failed to send email message. Here your accout details:<br> login: %s<br> password: %s<br><br>You can change your password after login.', L10n::t('Failed to send email message. Here your accout details:<br> login: %s<br> password: %s<br><br>You can change your password after login.',
@ -108,12 +114,12 @@ function register_post(App $a)
} }
} else { } else {
info(L10n::t('Registration successful.') . EOL); info(L10n::t('Registration successful.') . EOL);
goaway(System::baseUrl()); goaway();
} }
} elseif (intval(Config::get('config', 'register_policy')) === REGISTER_APPROVE) { } elseif (intval(Config::get('config', 'register_policy')) === REGISTER_APPROVE) {
if (!strlen(Config::get('config', 'admin_email'))) { if (!strlen(Config::get('config', 'admin_email'))) {
notice(L10n::t('Your registration can not be processed.') . EOL); notice(L10n::t('Your registration can not be processed.') . EOL);
goaway(System::baseUrl()); goaway();
} }
$hash = random_string(); $hash = random_string();
@ -146,9 +152,9 @@ function register_post(App $a)
'source_name' => $user['username'], 'source_name' => $user['username'],
'source_mail' => $user['email'], 'source_mail' => $user['email'],
'source_nick' => $user['nickname'], 'source_nick' => $user['nickname'],
'source_link' => System::baseUrl() . "/admin/users/", 'source_link' => $a->getBaseUrl() . "/admin/users/",
'link' => System::baseUrl() . "/admin/users/", 'link' => $a->getBaseUrl() . "/admin/users/",
'source_photo' => System::baseUrl() . "/photo/avatar/" . $user['uid'] . ".jpg", 'source_photo' => $a->getBaseUrl() . "/photo/avatar/" . $user['uid'] . ".jpg",
'to_email' => $admin['email'], 'to_email' => $admin['email'],
'uid' => $admin['uid'], 'uid' => $admin['uid'],
'language' => $admin['language'] ? $admin['language'] : 'en', 'language' => $admin['language'] ? $admin['language'] : 'en',
@ -156,11 +162,17 @@ function register_post(App $a)
]); ]);
} }
// send notification to the user, that the registration is pending // send notification to the user, that the registration is pending
User::sendRegisterPendingEmail( Model\User::sendRegisterPendingEmail(
$user['email'], Config::get('config', 'sitename'), $user['username']); $user['email'],
Config::get('config', 'sitename'),
$user['username'],
$a->getBaseURL(),
$user['nickname'],
$result['password']
);
info(L10n::t('Your registration is pending approval by the site owner.') . EOL); info(L10n::t('Your registration is pending approval by the site owner.') . EOL);
goaway(System::baseUrl()); goaway();
} }
return; return;