fix url for redirect

[quix0rs-gnu-social.git] / actions / register.php

diff --git a/actions/register.php b/actions/register.php
index fbff26af03efb93bdff7f630cc206081bb4e318f..3ed892e7ff16798c991f4a77a20fc42f7b04087e 100644 (file)
--- a/actions/register.php
+++ b/actions/register.php
@@ -54,7 +54,7 @@ class RegisterAction extends Action {
                } else if (!Validate::string($nickname, array('min_length' => 1,
                                                                                                          'max_length' => 64,
                                                                                                          'format' => VALIDATE_NUM . VALIDATE_ALPHA_LOWER))) {
-                       $this->show_form(_t('Nickname must have only letters and numbers and no spaces.'));
+                       $this->show_form(_t('Nickname must have only lowercase letters and numbers and no spaces.'));
                } else if ($this->nickname_exists($nickname)) {
                        $this->show_form(_t('Nickname already exists.'));
                } else if ($this->email_exists($email)) {
@@ -89,8 +89,11 @@ class RegisterAction extends Action {
        }
 
        function register_user($nickname, $password, $email) {
-               # TODO: wrap this in a transaction!
+               
                $profile = new Profile();
+               
+               $profile->query('BEGIN');
+               
                $profile->nickname = $nickname;
                $profile->profileurl = common_profile_url($nickname);
                $profile->created = DB_DataObject_Cast::dateTime(); # current time
@@ -103,40 +106,70 @@ class RegisterAction extends Action {
                $user->id = $id;
                $user->nickname = $nickname;
                $user->password = common_munge_password($password, $id);
-               $user->email = $email;
                $user->created =  DB_DataObject_Cast::dateTime(); # current time
-               $user->uri = common_mint_tag('user:'.$id);
+               $user->uri = common_user_uri($user);
                
                $result = $user->insert();
                if (!$result) {
-                       # Try to clean up...
-                       $profile->delete();
+                       return FALSE;
+               }
+
+               if ($email) {
+                       $confirm = new Confirm_email();
+                       $confirm->code = common_good_rand(16);
+                       $confirm->user_id = $user->id;
+                       $confirm->email = $email;
+                       
+                       $result = $confirm->insert();
+                       if (!$result) {
+                               return FALSE;
+                       }
+               }
+               
+               $profile->query('COMMIT');
+
+               if ($email) {
+                       mail_confirm_address($confirm->code,
+                                                                $profile->nickname,
+                                                                $email);
                }
+               
                return $result;
        }
 
+       function show_top($error=NULL) {
+               if ($error) {
+                       common_element('p', 'error', $error);
+               } else {
+                       common_element('p', 'instructions', 
+                                                  _t('You can create a new account to start posting notices.'));
+               }
+       }
+       
        function show_form($error=NULL) {
                global $config;
-               
-               common_show_header(_t('Register'));
+
+               common_show_header(_t('Register'), NULL, $error, array($this, 'show_top'));
                common_element_start('form', array('method' => 'POST',
                                                                                   'id' => 'login',
                                                                                   'action' => common_local_url('register')));
-               common_input('nickname', _t('Nickname'));
-               common_password('password', _t('Password'));
-               common_password('confirm', _t('Confirm'));
-               common_input('email', _t('Email'));
+               common_input('nickname', _t('Nickname'), NULL,
+                                        _t('1-64 lowercase letters or numbers, no punctuation or spaces'));
+               common_password('password', _t('Password'),                                             
+                                               _t('6 or more characters'));
+               common_password('confirm', _t('Confirm'),
+                                               _t('Same as password above'));
+               common_input('email', _t('Email'), NULL,
+                                        _t('Used only for updates, announcements, and password recovery'));
                common_element_start('p');
-               common_element_start('label', array('for' => 'license'));
-               common_text(_t('My text and files are available under '));
-               common_element('a', array(href => $config['license']['url']),
-                                          $config['license']['title']);
-               common_text(_t(' except this private data: password, email address, IM address, phone number.'));
-               common_element_end('label');
                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 => $config['license']['url']),
+                                          $config['license']['title']);
+               common_text(_t(' except this private data: password, email address, IM address, phone number.'));
                common_element_end('p');
                common_submit('submit', _t('Register'));
                common_element_end('form');