store invite code in session so openidfinish can find it
This commit is contained in:
parent
34d904b180
commit
231c61a7eb
|
@ -191,11 +191,28 @@ class FinishopenidloginAction extends Action
|
||||||
{
|
{
|
||||||
# FIXME: save invite code before redirect, and check here
|
# FIXME: save invite code before redirect, and check here
|
||||||
|
|
||||||
if (common_config('site', 'closed') || common_config('site', 'inviteonly')) {
|
if (common_config('site', 'closed')) {
|
||||||
$this->clientError(_('Registration not allowed.'));
|
$this->clientError(_('Registration not allowed.'));
|
||||||
return;
|
return;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
$invite = null;
|
||||||
|
|
||||||
|
if (common_config('site', 'inviteonly')) {
|
||||||
|
$code = $_SESSION['invitecode'];
|
||||||
|
if (empty($code)) {
|
||||||
|
$this->clientError(_('Registration not allowed.'));
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
|
$invite = Invitation::staticGet($code);
|
||||||
|
|
||||||
|
if (empty($invite)) {
|
||||||
|
$this->clientError(_('Not a valid invitation code.'));
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
$nickname = $this->trimmed('newname');
|
$nickname = $this->trimmed('newname');
|
||||||
|
|
||||||
if (!Validate::string($nickname, array('min_length' => 1,
|
if (!Validate::string($nickname, array('min_length' => 1,
|
||||||
|
@ -257,10 +274,16 @@ class FinishopenidloginAction extends Action
|
||||||
# XXX: add language
|
# XXX: add language
|
||||||
# XXX: add timezone
|
# XXX: add timezone
|
||||||
|
|
||||||
$user = User::register(array('nickname' => $nickname,
|
$args = array('nickname' => $nickname,
|
||||||
'email' => $email,
|
'email' => $email,
|
||||||
'fullname' => $fullname,
|
'fullname' => $fullname,
|
||||||
'location' => $location));
|
'location' => $location);
|
||||||
|
|
||||||
|
if (!empty($invite)) {
|
||||||
|
$args['code'] = $invite->code;
|
||||||
|
}
|
||||||
|
|
||||||
|
$user = User::register($args);
|
||||||
|
|
||||||
$result = oid_link_user($user->id, $canonical, $display);
|
$result = oid_link_user($user->id, $canonical, $display);
|
||||||
|
|
||||||
|
|
|
@ -55,6 +55,44 @@ class RegisterAction extends Action
|
||||||
|
|
||||||
var $registered = false;
|
var $registered = false;
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Prepare page to run
|
||||||
|
*
|
||||||
|
*
|
||||||
|
* @param $args
|
||||||
|
* @return string title
|
||||||
|
*/
|
||||||
|
|
||||||
|
function prepare()
|
||||||
|
{
|
||||||
|
$this->code = $this->trimmed('code');
|
||||||
|
|
||||||
|
if (empty($this->code)) {
|
||||||
|
common_ensure_session();
|
||||||
|
if (!empty($_SESSION['invitecode'])) {
|
||||||
|
$this->code = $_SESSION['invitecode'];
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
if (common_config('site', 'inviteonly') && empty($this->code)) {
|
||||||
|
$this->clientError(_('Sorry, only invited people can register.'));
|
||||||
|
return false;
|
||||||
|
}
|
||||||
|
|
||||||
|
if (!empty($this->code)) {
|
||||||
|
$this->invite = Invitation::staticGet($code);
|
||||||
|
if (empty($this->invite)) {
|
||||||
|
$this->clientError(_('Sorry, invalid invitation code.'));
|
||||||
|
return false;
|
||||||
|
}
|
||||||
|
// Store this in case we need it
|
||||||
|
common_ensure_session();
|
||||||
|
$_SESSION['invitecode'] = $this->code;
|
||||||
|
}
|
||||||
|
|
||||||
|
return true;
|
||||||
|
}
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* Title of the page
|
* Title of the page
|
||||||
*
|
*
|
||||||
|
@ -112,7 +150,7 @@ class RegisterAction extends Action
|
||||||
$token = $this->trimmed('token');
|
$token = $this->trimmed('token');
|
||||||
if (!$token || $token != common_session_token()) {
|
if (!$token || $token != common_session_token()) {
|
||||||
$this->showForm(_('There was a problem with your session token. '.
|
$this->showForm(_('There was a problem with your session token. '.
|
||||||
'Try again, please.'));
|
'Try again, please.'));
|
||||||
return;
|
return;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -145,14 +183,14 @@ class RegisterAction extends Action
|
||||||
|
|
||||||
if (!$this->boolean('license')) {
|
if (!$this->boolean('license')) {
|
||||||
$this->showForm(_('You can\'t register if you don\'t '.
|
$this->showForm(_('You can\'t register if you don\'t '.
|
||||||
'agree to the license.'));
|
'agree to the license.'));
|
||||||
} else if ($email && !Validate::email($email, true)) {
|
} else if ($email && !Validate::email($email, true)) {
|
||||||
$this->showForm(_('Not a valid email address.'));
|
$this->showForm(_('Not a valid email address.'));
|
||||||
} else if (!Validate::string($nickname, array('min_length' => 1,
|
} else if (!Validate::string($nickname, array('min_length' => 1,
|
||||||
'max_length' => 64,
|
'max_length' => 64,
|
||||||
'format' => NICKNAME_FMT))) {
|
'format' => NICKNAME_FMT))) {
|
||||||
$this->showForm(_('Nickname must have only lowercase letters '.
|
$this->showForm(_('Nickname must have only lowercase letters '.
|
||||||
'and numbers and no spaces.'));
|
'and numbers and no spaces.'));
|
||||||
} else if ($this->nicknameExists($nickname)) {
|
} else if ($this->nicknameExists($nickname)) {
|
||||||
$this->showForm(_('Nickname already in use. Try another one.'));
|
$this->showForm(_('Nickname already in use. Try another one.'));
|
||||||
} else if (!User::allowed_nickname($nickname)) {
|
} else if (!User::allowed_nickname($nickname)) {
|
||||||
|
@ -160,9 +198,9 @@ class RegisterAction extends Action
|
||||||
} else if ($this->emailExists($email)) {
|
} else if ($this->emailExists($email)) {
|
||||||
$this->showForm(_('Email address already exists.'));
|
$this->showForm(_('Email address already exists.'));
|
||||||
} else if (!is_null($homepage) && (strlen($homepage) > 0) &&
|
} else if (!is_null($homepage) && (strlen($homepage) > 0) &&
|
||||||
!Validate::uri($homepage,
|
!Validate::uri($homepage,
|
||||||
array('allowed_schemes' =>
|
array('allowed_schemes' =>
|
||||||
array('http', 'https')))) {
|
array('http', 'https')))) {
|
||||||
$this->showForm(_('Homepage is not a valid URL.'));
|
$this->showForm(_('Homepage is not a valid URL.'));
|
||||||
return;
|
return;
|
||||||
} else if (!is_null($fullname) && mb_strlen($fullname) > 255) {
|
} else if (!is_null($fullname) && mb_strlen($fullname) > 255) {
|
||||||
|
@ -180,13 +218,13 @@ class RegisterAction extends Action
|
||||||
} else if ($password != $confirm) {
|
} else if ($password != $confirm) {
|
||||||
$this->showForm(_('Passwords don\'t match.'));
|
$this->showForm(_('Passwords don\'t match.'));
|
||||||
} else if ($user = User::register(array('nickname' => $nickname,
|
} else if ($user = User::register(array('nickname' => $nickname,
|
||||||
'password' => $password,
|
'password' => $password,
|
||||||
'email' => $email,
|
'email' => $email,
|
||||||
'fullname' => $fullname,
|
'fullname' => $fullname,
|
||||||
'homepage' => $homepage,
|
'homepage' => $homepage,
|
||||||
'bio' => $bio,
|
'bio' => $bio,
|
||||||
'location' => $location,
|
'location' => $location,
|
||||||
'code' => $code))) {
|
'code' => $code))) {
|
||||||
if (!$user) {
|
if (!$user) {
|
||||||
$this->showForm(_('Invalid username or password.'));
|
$this->showForm(_('Invalid username or password.'));
|
||||||
return;
|
return;
|
||||||
|
@ -259,17 +297,17 @@ class RegisterAction extends Action
|
||||||
|
|
||||||
// overrided to add hentry, and content-inner class
|
// overrided to add hentry, and content-inner class
|
||||||
function showContentBlock()
|
function showContentBlock()
|
||||||
{
|
{
|
||||||
$this->elementStart('div', array('id' => 'content', 'class' => 'hentry'));
|
$this->elementStart('div', array('id' => 'content', 'class' => 'hentry'));
|
||||||
$this->showPageTitle();
|
$this->showPageTitle();
|
||||||
$this->showPageNoticeBlock();
|
$this->showPageNoticeBlock();
|
||||||
$this->elementStart('div', array('id' => 'content_inner',
|
$this->elementStart('div', array('id' => 'content_inner',
|
||||||
'class' => 'entry-content'));
|
'class' => 'entry-content'));
|
||||||
// show the actual content (forms, lists, whatever)
|
// show the actual content (forms, lists, whatever)
|
||||||
$this->showContent();
|
$this->showContent();
|
||||||
$this->elementEnd('div');
|
$this->elementEnd('div');
|
||||||
$this->elementEnd('div');
|
$this->elementEnd('div');
|
||||||
}
|
}
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* Instructions or a notice for the page
|
* Instructions or a notice for the page
|
||||||
|
@ -343,90 +381,79 @@ class RegisterAction extends Action
|
||||||
|
|
||||||
function showFormContent()
|
function showFormContent()
|
||||||
{
|
{
|
||||||
$code = $this->trimmed('code');
|
|
||||||
|
|
||||||
if ($code) {
|
|
||||||
$invite = Invitation::staticGet($code);
|
|
||||||
}
|
|
||||||
|
|
||||||
if (common_config('site', 'inviteonly') && !($code && $invite)) {
|
|
||||||
$this->clientError(_('Sorry, only invited people can register.'));
|
|
||||||
return;
|
|
||||||
}
|
|
||||||
|
|
||||||
$this->elementStart('form', array('method' => 'post',
|
$this->elementStart('form', array('method' => 'post',
|
||||||
'id' => 'form_register',
|
'id' => 'form_register',
|
||||||
'class' => 'form_settings',
|
'class' => 'form_settings',
|
||||||
'action' => common_local_url('register')));
|
'action' => common_local_url('register')));
|
||||||
$this->elementStart('fieldset');
|
$this->elementStart('fieldset');
|
||||||
$this->element('legend', null, 'Account settings');
|
$this->element('legend', null, 'Account settings');
|
||||||
$this->hidden('token', common_session_token());
|
$this->hidden('token', common_session_token());
|
||||||
|
|
||||||
if ($code) {
|
if ($this->code) {
|
||||||
$this->hidden('code', $code);
|
$this->hidden('code', $this->code);
|
||||||
}
|
}
|
||||||
|
|
||||||
$this->elementStart('ul', 'form_data');
|
$this->elementStart('ul', 'form_data');
|
||||||
if (Event::handle('StartRegistrationFormData', array($this))) {
|
if (Event::handle('StartRegistrationFormData', array($this))) {
|
||||||
$this->elementStart('li');
|
$this->elementStart('li');
|
||||||
$this->input('nickname', _('Nickname'), $this->trimmed('nickname'),
|
$this->input('nickname', _('Nickname'), $this->trimmed('nickname'),
|
||||||
_('1-64 lowercase letters or numbers, '.
|
_('1-64 lowercase letters or numbers, '.
|
||||||
'no punctuation or spaces. Required.'));
|
'no punctuation or spaces. Required.'));
|
||||||
$this->elementEnd('li');
|
$this->elementEnd('li');
|
||||||
$this->elementStart('li');
|
$this->elementStart('li');
|
||||||
$this->password('password', _('Password'),
|
$this->password('password', _('Password'),
|
||||||
_('6 or more characters. Required.'));
|
_('6 or more characters. Required.'));
|
||||||
$this->elementEnd('li');
|
$this->elementEnd('li');
|
||||||
$this->elementStart('li');
|
$this->elementStart('li');
|
||||||
$this->password('confirm', _('Confirm'),
|
$this->password('confirm', _('Confirm'),
|
||||||
_('Same as password above. Required.'));
|
_('Same as password above. Required.'));
|
||||||
$this->elementEnd('li');
|
$this->elementEnd('li');
|
||||||
$this->elementStart('li');
|
$this->elementStart('li');
|
||||||
if ($invite && $invite->address_type == 'email') {
|
if ($this->invite && $this->invite->address_type == 'email') {
|
||||||
$this->input('email', _('Email'), $invite->address,
|
$this->input('email', _('Email'), $this->invite->address,
|
||||||
_('Used only for updates, announcements, '.
|
_('Used only for updates, announcements, '.
|
||||||
'and password recovery'));
|
'and password recovery'));
|
||||||
} else {
|
} else {
|
||||||
$this->input('email', _('Email'), $this->trimmed('email'),
|
$this->input('email', _('Email'), $this->trimmed('email'),
|
||||||
_('Used only for updates, announcements, '.
|
_('Used only for updates, announcements, '.
|
||||||
'and password recovery'));
|
'and password recovery'));
|
||||||
}
|
}
|
||||||
$this->elementEnd('li');
|
$this->elementEnd('li');
|
||||||
$this->elementStart('li');
|
$this->elementStart('li');
|
||||||
$this->input('fullname', _('Full name'),
|
$this->input('fullname', _('Full name'),
|
||||||
$this->trimmed('fullname'),
|
$this->trimmed('fullname'),
|
||||||
_('Longer name, preferably your "real" name'));
|
_('Longer name, preferably your "real" name'));
|
||||||
$this->elementEnd('li');
|
$this->elementEnd('li');
|
||||||
$this->elementStart('li');
|
$this->elementStart('li');
|
||||||
$this->input('homepage', _('Homepage'),
|
$this->input('homepage', _('Homepage'),
|
||||||
$this->trimmed('homepage'),
|
$this->trimmed('homepage'),
|
||||||
_('URL of your homepage, blog, '.
|
_('URL of your homepage, blog, '.
|
||||||
'or profile on another site'));
|
'or profile on another site'));
|
||||||
$this->elementEnd('li');
|
$this->elementEnd('li');
|
||||||
$this->elementStart('li');
|
$this->elementStart('li');
|
||||||
$this->textarea('bio', _('Bio'),
|
$this->textarea('bio', _('Bio'),
|
||||||
$this->trimmed('bio'),
|
$this->trimmed('bio'),
|
||||||
_('Describe yourself and your '.
|
_('Describe yourself and your '.
|
||||||
'interests in 140 chars'));
|
'interests in 140 chars'));
|
||||||
$this->elementEnd('li');
|
$this->elementEnd('li');
|
||||||
$this->elementStart('li');
|
$this->elementStart('li');
|
||||||
$this->input('location', _('Location'),
|
$this->input('location', _('Location'),
|
||||||
$this->trimmed('location'),
|
$this->trimmed('location'),
|
||||||
_('Where you are, like "City, '.
|
_('Where you are, like "City, '.
|
||||||
'State (or Region), Country"'));
|
'State (or Region), Country"'));
|
||||||
$this->elementEnd('li');
|
$this->elementEnd('li');
|
||||||
Event::handle('EndRegistrationFormData', array($this));
|
Event::handle('EndRegistrationFormData', array($this));
|
||||||
$this->elementStart('li', array('id' => 'settings_rememberme'));
|
$this->elementStart('li', array('id' => 'settings_rememberme'));
|
||||||
$this->checkbox('rememberme', _('Remember me'),
|
$this->checkbox('rememberme', _('Remember me'),
|
||||||
$this->boolean('rememberme'),
|
$this->boolean('rememberme'),
|
||||||
_('Automatically login in the future; '.
|
_('Automatically login in the future; '.
|
||||||
'not for shared computers!'));
|
'not for shared computers!'));
|
||||||
$this->elementEnd('li');
|
$this->elementEnd('li');
|
||||||
$attrs = array('type' => 'checkbox',
|
$attrs = array('type' => 'checkbox',
|
||||||
'id' => 'license',
|
'id' => 'license',
|
||||||
'class' => 'checkbox',
|
'class' => 'checkbox',
|
||||||
'name' => 'license',
|
'name' => 'license',
|
||||||
'value' => 'true');
|
'value' => 'true');
|
||||||
if ($this->boolean('license')) {
|
if ($this->boolean('license')) {
|
||||||
$attrs['checked'] = 'checked';
|
$attrs['checked'] = 'checked';
|
||||||
}
|
}
|
||||||
|
@ -435,9 +462,9 @@ class RegisterAction extends Action
|
||||||
$this->elementStart('label', array('class' => 'checkbox', 'for' => 'license'));
|
$this->elementStart('label', array('class' => 'checkbox', 'for' => 'license'));
|
||||||
$this->text(_('My text and files are available under '));
|
$this->text(_('My text and files are available under '));
|
||||||
$this->element('a', array('href' => common_config('license', 'url')),
|
$this->element('a', array('href' => common_config('license', 'url')),
|
||||||
common_config('license', 'title'), _("Creative Commons Attribution 3.0"));
|
common_config('license', 'title'), _("Creative Commons Attribution 3.0"));
|
||||||
$this->text(_(' except this private data: password, '.
|
$this->text(_(' except this private data: password, '.
|
||||||
'email address, IM address, and phone number.'));
|
'email address, IM address, and phone number.'));
|
||||||
$this->elementEnd('label');
|
$this->elementEnd('label');
|
||||||
$this->elementEnd('li');
|
$this->elementEnd('li');
|
||||||
}
|
}
|
||||||
|
|
Loading…
Reference in New Issue
Block a user