X-Git-Url: https://git.mxchange.org/?a=blobdiff_plain;f=actions%2Ffinishopenidlogin.php;h=09102e8442806b7c18b1ac1d443cc5d48e9c53d8;hb=31b5ae1523c4f64159f00b32683912da1a9fb474;hp=da2a05c26bf914a882e23ec3b4e283fd97969a67;hpb=26c7d5f04ea7808c8a3addf1d4ed56d36a36b6c9;p=quix0rs-gnu-social.git diff --git a/actions/finishopenidlogin.php b/actions/finishopenidlogin.php index da2a05c26b..09102e8442 100644 --- a/actions/finishopenidlogin.php +++ b/actions/finishopenidlogin.php @@ -29,6 +29,11 @@ class FinishopenidloginAction extends Action { common_user_error(_t('Already logged in.')); } else if ($_SERVER['REQUEST_METHOD'] == 'POST') { if ($this->arg('create')) { + if (!$this->boolean('license')) { + $this->show_form(_t('You can\'t register if you don\'t agree to the license.'), + $this->trimmed('newname')); + return; + } $this->create_new_user(); } else if ($this->arg('connect')) { $this->connect_user(); @@ -42,8 +47,7 @@ class FinishopenidloginAction extends Action { } } - function show_form($error=NULL, $username=NULL) { - common_show_header(_t('OpenID Account Setup')); + function show_top($error=NULL) { if ($error) { common_element('div', array('class' => 'error'), $error); } else { @@ -54,7 +58,13 @@ class FinishopenidloginAction extends Action { _t(' so we must connect your OpenID to a local account. ' . ' You can either create a new account, or connect with ' . ' your existing account, if you have one.')); - } + } + } + + function show_form($error=NULL, $username=NULL) { + common_show_header(_t('OpenID Account Setup'), NULL, $error, + array($this, 'show_top')); + common_element_start('form', array('method' => 'POST', 'id' => 'account_connect', 'action' => common_local_url('finishopenidlogin'))); @@ -65,6 +75,16 @@ class FinishopenidloginAction extends Action { common_input('newname', _t('New nickname'), ($username) ? $username : '', _t('1-64 lowercase letters or numbers, no punctuation or spaces')); + common_element_start('p'); + common_element('input', array('type' => 'checkbox', + 'id' => 'license', + 'name' => 'license', + 'value' => 'true')); + common_text(_t('My text and files are available under ')); + common_element('a', array(href => common_config('license', 'url')), + common_config('license', 'title')); + common_text(_t(' except this private data: password, email address, IM address, phone number.')); + common_element_end('p'); common_submit('create', _t('Create')); common_element('h2', NULL, 'Connect existing account'); @@ -110,6 +130,7 @@ class FinishopenidloginAction extends Action { oid_set_last($display); oid_update_user($user, $sreg); common_set_user($user->nickname); + common_real_login(true); $this->go_home($user->nickname); } else { $this->save_values($display, $canonical, $sreg); @@ -147,6 +168,11 @@ class FinishopenidloginAction extends Action { $this->show_form(_t('Nickname must have only letters and numbers and no spaces.')); return; } + + if (!User::allowed_nickname($nickname)) { + $this->show_form(_t('Nickname not allowed.')); + return; + } if (User::staticGet('nickname', $nickname)) { $this->show_form(_t('Nickname already in use. Try another one.')); @@ -203,7 +229,7 @@ class FinishopenidloginAction extends Action { $user = new User(); $user->id = $id; $user->nickname = $nickname; - $user->uri = common_mint_tag('user:'.$id); + $user->uri = common_user_uri($user); if ($sreg['email'] && Validate::email($sreg['email'], true)) { $user->email = $sreg['email']; @@ -228,6 +254,7 @@ class FinishopenidloginAction extends Action { oid_set_last($display); common_set_user($user->nickname); + common_real_login(true); common_redirect(common_local_url('showstream', array('nickname' => $user->nickname))); } @@ -262,6 +289,7 @@ class FinishopenidloginAction extends Action { oid_update_user($user, $sreg); oid_set_last($display); common_set_user($user->nickname); + common_real_login(true); $this->go_home($user->nickname); } @@ -317,6 +345,9 @@ class FinishopenidloginAction extends Action { 'max_length' => 64, 'format' => VALIDATE_NUM . VALIDATE_ALPHA_LOWER))) { return false; + } + if (!User::allowed_nickname($str)) { + return false; } if (User::staticGet('nickname', $str)) { return false;