A lot texts rewritten and exclamation signs removed
[mailer.git] / inc / modules / guest / what-register.php
index ee33a91fde18c9784eaad0d6c6369896ad9a204c..144b1e1833271e49c992af614daca1a62d0cb340 100644 (file)
@@ -53,7 +53,7 @@ if (!isExtensionActive('register')) {
 }
 
 // Initialize variables
-$isFailed   = false;
+$isOkay     = true;
 $shortPass  = false;
 $cats       = 0;
 $ipTimedOut = false;
@@ -83,31 +83,31 @@ if (isFormSent()) {
        // Did he agree to our Terms Of Usage?
        if (postRequestElement('agree') != 'Y') {
                setRequestPostElement('agree', '!');
-               $isFailed = true;
+               $isOkay = false;
        } // END - if
 
        // Did he enter a valid email address? (we really don't care about
        // that, he has to click on a confirmation link :P )
        if ((!isPostRequestElementSet('email')) || (!isEmailValid(postRequestElement('email')))) {
                setRequestPostElement('email', '!');
-               $isFailed = true;
+               $isOkay = false;
        } // END - if
 
        // And what about surname and family's name?
        if (!isPostRequestElementSet('surname')) {
                setRequestPostElement('surname', '!');
-               $isFailed = true;
+               $isOkay = false;
        } // END - if
        if (!isPostRequestElementSet('family')) {
                setRequestPostElement('family', '!');
-               $isFailed = true;
+               $isOkay = false;
        } // END - if
 
        // Get temporary array for modification
        $postArray = postRequestArray();
 
        // Check for required fields
-       if ($isFailed === false) $isFailed = ifRequiredRegisterFieldsAreSet($postArray);
+       if ($isOkay === true) $isOkay = ifRequiredRegisterFieldsAreSet($postArray);
 
        // Set it back in request
        setPostRequestArray($postArray);
@@ -121,13 +121,13 @@ if (isFormSent()) {
                        if (!isPostRequestElementSet('pass1')) { setRequestPostElement('pass1', '!'); } else { setRequestPostElement('pass1', ''); }
                        if (!isPostRequestElementSet('pass2')) { setRequestPostElement('pass2', '!'); } else { setRequestPostElement('pass2', ''); }
                }
-               $isFailed = true;
+               $isOkay = false;
        } // END - if
 
        // Is the password long enouth?
-       if ((strlen(postRequestElement('pass1')) < getConfig('pass_len')) && ($isFailed === false)) {
+       if ((strlen(postRequestElement('pass1')) < getConfig('pass_len')) && ($isOkay === true)) {
                $shortPass = true;
-               $isFailed = true;
+               $isOkay = false;
        } // END - if
 
        // No admin? Admins can always register!
@@ -139,7 +139,7 @@ if (isFormSent()) {
 
                if ($cats < getConfig('least_cats')) {
                        // ... nope!
-                       $isFailed = true;
+                       $isOkay = false;
                } // END - if
        } // END - if
 
@@ -148,7 +148,7 @@ if (isFormSent()) {
                $CHK = isEmailTaken(postRequestElement('email'));
                if ($CHK === true) {
                        setRequestPostElement('email', '?');
-                       $isFailed = true;
+                       $isOkay = false;
                } // END - if
        } // END - if
 
@@ -168,18 +168,20 @@ LIMIT 1",
                        // Same IP in timeout range and different email address entered... Eat this, faker! ;-)
                        // But admins are allowed to fake their own exchange service.
                        $ipTimedOut = true;
-                       $isFailed = true;
+                       $isOkay = false;
                } // END - if
-       } // END - if
 
-       // Free memory
-       SQL_FREERESULT($result);
+               // Free memory
+               SQL_FREERESULT($result);
+       } // END - if
 } // END - if
 
-if ((isFormSent()) && (($isFailed === false) || (isAdmin()))) {
+// Is the form sent and all went fine or admin logged in?
+//* DEBUG: */ print intval(isFormSent()).'/'.intval($isOkay).'/'.intval(isAdmin()).'<br />';
+if ((isFormSent()) && (($isOkay === true) || (isAdmin()))) {
        // Prepapre month and day of birth
-       if (strlen(postRequestElement('day'))   == 1) setRequestPostElement('day'  , '0'.postRequestElement('day'));
-       if (strlen(postRequestElement('month')) == 1) setRequestPostElement('month', '0'.postRequestElement('month'));
+       if (strlen(postRequestElement('day'))   == 1) setRequestPostElement('day'  , 0 . postRequestElement('day'));
+       if (strlen(postRequestElement('month')) == 1) setRequestPostElement('month', 0 . postRequestElement('month'));
 
        // Get total ...
        // ... confirmed, ...
@@ -196,7 +198,7 @@ if ((isFormSent()) && (($isFailed === false) || (isAdmin()))) {
        // @TODO Rewrite these all to a single filter
        $ADD1 = '';
        $ADD2 = '';
-       if (getExtensionVersion('theme') >= '0.0.8') {
+       if (isExtensionInstalledAndNewer('theme', '0.0.8')) {
                // Okay, add design here
                $ADD1 = ', `curr_theme`';
                $ADD2 = ", '".getCurrentTheme()."'";
@@ -205,7 +207,7 @@ if ((isFormSent()) && (($isFailed === false) || (isAdmin()))) {
        // Check if I shall disable sending mail to newly registered members out about active/begging rallye
        //
        // First comes first: begging rallye
-       if (getExtensionVersion('beg') >= '0.1.7') {
+       if (isExtensionInstalledAndNewer('beg', '0.1.7')) {
                // Okay, shall I disable now?
                if (getConfig('beg_new_mem_notify') != 'Y') {
                        $ADD1 .= ', `beg_ral_notify`, `beg_ral_en_notify`';
@@ -214,7 +216,7 @@ if ((isFormSent()) && (($isFailed === false) || (isAdmin()))) {
        } // END - if
 
        // Second: active rallye
-       if (getExtensionVersion('bonus') >= '0.7.7') {
+       if (isExtensionInstalledAndNewer('bonus', '0.7.7')) {
                // Okay, shall I disable now?
                if (getConfig('bonus_new_mem_notify') != 'Y') {
                        $ADD1 .= ', `bonus_ral_notify`, `bonus_ral_en_notify`';
@@ -288,11 +290,11 @@ VALUES ('%s','%s','%s','%s','%s',%s,'%s','%s',%s, %s,%s,'%s',%s, %s,'%s','UNCONF
                if (getConfig('ref_payout') > 0) $locked = 'locked_points';
 
                // @TODO Rewrite this to addPointsDirectly()
-               SQL_QUERY_ESC("INSERT INTO `{?_MYSQL_PREFIX?}_user_points` (`userid`, `ref_depth`, `".$locked."`) VALUES (%s,0,'%s')",
-                       array(bigintval($userid), getConfig('points_register')), __FILE__, __LINE__);
+               SQL_QUERY_ESC("INSERT INTO `{?_MYSQL_PREFIX?}_user_points` (`userid`, `ref_depth`, `".$locked."`) VALUES (%s,0,'{?points_register?}')",
+                       array(bigintval($userid)), __FILE__, __LINE__);
 
                // Update mediadata as well
-               if ((getExtensionVersion('mediadata') >= '0.0.4') && ($locked == 'points')) {
+               if ((isExtensionInstalledAndNewer('mediadata', '0.0.4')) && ($locked == 'points')) {
                        // Update database
                        updateMediadataEntry(array('total_points'), 'add', getConfig('points_register'));
                } // END - if
@@ -312,8 +314,8 @@ VALUES ('%s','%s','%s','%s','%s',%s,'%s','%s',%s, %s,%s,'%s',%s, %s,'%s','UNCONF
        // Rewrite gender
        $gender = translateGender(postRequestElement('gender'));
 
-       // ... rewrite a zero referal ID to the main title
-       if (postRequestElement('refid') == '0') setRequestPostElement('refid', getConfig('MAIN_TITLE'));
+       // ... rewrite a zero referal id to the main title
+       if (postRequestElement('refid') == 0) setRequestPostElement('refid', getConfig('MAIN_TITLE'));
 
        // Is ZIP code set?
        if (isPostRequestElementSet('zip')) {
@@ -321,7 +323,7 @@ VALUES ('%s','%s','%s','%s','%s',%s,'%s','%s',%s, %s,%s,'%s',%s, %s,'%s','UNCONF
                // Start with the gender...
                $content = array(
                        'hash'    => $hash,
-                       'userid'     => $userid,
+                       'userid'  => $userid,
                        'gender'  => $gender,
                        'surname' => SQL_ESCAPE(postRequestElement('surname')),
                        'family'  => SQL_ESCAPE(postRequestElement('family')),
@@ -337,7 +339,7 @@ VALUES ('%s','%s','%s','%s','%s',%s,'%s','%s',%s, %s,%s,'%s',%s, %s,'%s','UNCONF
                // No ZIP code entered
                $content = array(
                        'hash'    => $hash,
-                       'userid'     => $userid,
+                       'userid'  => $userid,
                        'gender'  => $gender,
                        'surname' => SQL_ESCAPE(postRequestElement('surname')),
                        'family'  => SQL_ESCAPE(postRequestElement('family')),