More misc fixes and rewrites (sorry, lame description)
[mailer.git] / inc / libs / user_functions.php
index 0737b412fd450bad2987e4d02073a19d157e4b14..ec3958a6455f70468f6ed8815f4b6fe713056254 100644 (file)
@@ -93,18 +93,18 @@ function SortLinks ($letter, $sortby, $colspan, $return=false) {
        $add = "&page=".REQUEST_GET('page')."&offset=".REQUEST_GET('offset');
 
        // Add status or mode
-       if (REQUEST_ISSET_GET(('status'))) $add .= "&mode=".REQUEST_GET(('status'));
-       elseif (REQUEST_ISSET_GET('mode')) $add .= "&mode=".REQUEST_GET('mode');
+       if (REQUEST_ISSET_GET('status'))   $add .= '&mode=' . REQUEST_GET('status');
+       elseif (REQUEST_ISSET_GET('mode')) $add .= '&mode=' . REQUEST_GET('mode');
 
        // Makes order by links..
        if ($letter == "front") $letter = getMessage('_ALL2');
 
        // Prepare array with all possible sorters
        $list = array(
-               'userid'                => getMessage('_UID'),
-               'family'                => getMessage('FAMILY_NAME'),
-               'email'                 => getMessage('ADDY'),
-               'REMOTE_ADDR'   => getMessage('REMOTE_IP')
+               'userid'      => getMessage('_UID'),
+               'family'      => getMessage('FAMILY_NAME'),
+               'email'       => getMessage('ADDY'),
+               'REMOTE_ADDR' => getMessage('REMOTE_IP')
        );
 
        // Add nickname if extension is installed
@@ -114,16 +114,16 @@ function SortLinks ($letter, $sortby, $colspan, $return=false) {
 
        foreach ($list as $sort => $title) {
                if ($sortby == $sort) {
-                       $OUT .= "<strong>".$title."</strong>&nbsp;|&nbsp;";
+                       $OUT .= "<strong>" . $title . "</strong>&nbsp;|&nbsp;";
                } else {
-                       $OUT .= "<a href=\"{!URL!}/modules.php?module=admin&amp;what=list_user&amp;letter=".$letter."&amp;sortby=".$sort.$add."\">".$title."</a>&nbsp;|&nbsp;";
+                       $OUT .= "<a href=\"{!URL!}/modules.php?module=admin&amp;what=list_user&amp;letter=" . $letter . "&amp;sortby=" . $sort.$add . "\">" . $title . "</a>&nbsp;|&nbsp;";
                }
        } // END - foreach
 
        define('__SORT_LIST', substr($OUT, 0, -13));
 
        // Load template
-       $OUT = LOAD_TEMPLATE("admin_list_user_sort", true);
+       $OUT = LOAD_TEMPLATE('admin_list_user_sort', true);
        if ($return === true) {
                // Return code
                return $OUT;
@@ -135,56 +135,50 @@ function SortLinks ($letter, $sortby, $colspan, $return=false) {
 
 // Add page navigation
 function ADD_PAGENAV ($PAGES, $offset, $show_form, $colspan,$return=false) {
-       if (!$show_form) {
-               // Empty row
-               define('__FORM_HEADER', "<tr><td colspan=\"".$colspan."\" class=\"seperator\">&nbsp;</td></tr>");
-       } else {
+       if ($show_form) {
                // Load form for changing number of lines
-               define('__FORM_HEADER', LOAD_TEMPLATE("admin_list_user_sort_form", true));
-       }
-
-       if (!$show_form) {
-               // Add line with bottom border
-               define('__FORM_FOOTER', "<tr><td colspan=\"{!__COLSPAN2!}\" class=\"seperator bottom2\">&nbsp;</td></tr>");
+               define('__FORM_HEADER', LOAD_TEMPLATE('admin_list_user_sort_form', true));
+               define('__FORM_FOOTER', '<tr><td colspan="{!__COLSPAN2!}" class="seperator bottom2">&nbsp;</td></tr>');
        } else {
-               // Add line without bottom border
-               define('__FORM_FOOTER', "<tr><td colspan=\"{!__COLSPAN2!}\" class=\"seperator bottom2\">&nbsp;</td></tr>");
+               // Empty row
+               define('__FORM_HEADER', '<tr><td colspan="' . $colspan . '" class="seperator">&nbsp;</td></tr>');
+               define('__FORM_FOOTER', '<tr><td colspan="{!__COLSPAN2!}" class="seperator bottom2">&nbsp;</td></tr>');
        }
 
        $OUT = '';
        for ($page = 1; $page <= $PAGES; $page++) {
                if (($page == REQUEST_GET('page')) || ((!REQUEST_ISSET_GET('page')) && ($page == '1'))) {
-                       $OUT .= "<strong>-";
+                       $OUT .= '<strong>-';
                } else {
                        if (!REQUEST_ISSET_GET('letter')) REQUEST_SET_GET('letter', getMessage('_ALL2'));
                        if (!REQUEST_ISSET_GET('sortby')) REQUEST_SET_GET('sortby', 'userid');
 
                        // Base link
-                       $OUT .= "<a href=\"{!URL!}/modules.php?module=admin&amp;what=".getWhat();
+                       $OUT .= '<a href="{!URL!}/modules.php?module=admin&amp;what=' . getWhat();
 
                        // Add status or mode
-                       if (REQUEST_ISSET_GET(('status'))) $OUT .= "&amp;mode=".REQUEST_GET(('status'));
-                       elseif (REQUEST_ISSET_GET('mode')) $OUT .= "&amp;mode=".REQUEST_GET('mode');
+                       if (REQUEST_ISSET_GET('status'))    $OUT .= '&amp;mode=' . REQUEST_GET('status');
+                        elseif (REQUEST_ISSET_GET('mode')) $OUT .= '&amp;mode=' . REQUEST_GET('mode');
 
                        // Letter and so on
-                       $OUT .= "&amp;letter=".REQUEST_GET('letter')."&amp;sortby=".REQUEST_GET('sortby')."&amp;page=".$page."&amp;offset=".$offset."\">";
+                       $OUT .= '&amp;letter=' . REQUEST_GET('letter') . '&amp;sortby=' . REQUEST_GET('sortby') . '&amp;page=' . $page . '&amp;offset=' . $offset . '">';
                }
 
                $OUT .= $page;
 
                if (($page == REQUEST_GET('page')) || ((!REQUEST_ISSET_GET('page')) && ($page == '1'))) {
-                       $OUT .= "-</strong>";
+                       $OUT .= '-</strong>';
                } else  {
-                       $OUT .= "</a>";
+                       $OUT .= '</a>';
                }
 
-               if ($page < $PAGES) $OUT .= "&nbsp;|&nbsp;";
+               if ($page < $PAGES) $OUT .= '&nbsp;|&nbsp;';
        } // END - for
 
        define('__PAGENAV_LIST', $OUT);
 
        // Load template
-       $OUT = LOAD_TEMPLATE("admin_list_user_pagenav", true);
+       $OUT = LOAD_TEMPLATE('admin_list_user_pagenav', true);
        if ($return === true) {
                // Return code
                return $OUT;
@@ -195,17 +189,21 @@ function ADD_PAGENAV ($PAGES, $offset, $show_form, $colspan,$return=false) {
 }
 
 // Create email link to user's account
-function USER_generateMemberEmailLink($email, $mod='admin') {
+function generateUserEmailLink($email, $mod = 'admin') {
        // Show contact link only if user is confirmed by default
        $locked = " AND `status`='CONFIRMED'";
 
        // But admins shall always see it
        if (IS_ADMIN()) $locked = '';
 
-       $result = SQL_QUERY_ESC("SELECT userid
-FROM `{!_MYSQL_PREFIX!}_user_data`
-WHERE `email`='%s'".$locked." LIMIT 1",
-       array($email), __FUNCTION__, __LINE__);
+       $result = SQL_QUERY_ESC("SELECT
+       `userid`
+FROM
+       `{!_MYSQL_PREFIX!}_user_data`
+WHERE
+       `email`='%s'" . $locked."
+LIMIT 1",
+               array($email), __FUNCTION__, __LINE__);
        if (SQL_NUMROWS($result) == 1) {
                // Load userid
                list($uid) = SQL_FETCHROW($result);
@@ -238,7 +236,7 @@ function determineRandomReferalId () {
 
                        // Look for random user
                        $result = SQL_QUERY_ESC("SELECT `userid` FROM `{!_MYSQL_PREFIX!}_user_data` WHERE `status`='CONFIRMED' AND `rand_confirmed` >= %s ORDER BY `rand_confirmed` DESC LIMIT %s, 1",
-                       array(getConfig('user_min_confirmed'), $randNum), __FUNCTION__, __LINE__);
+                               array(getConfig('user_min_confirmed'), $randNum), __FUNCTION__, __LINE__);
 
                        // Do we have one entry there?
                        if (SQL_NUMROWS($result) == 1) {
@@ -246,8 +244,8 @@ function determineRandomReferalId () {
                                list($refid) = SQL_FETCHROW($result);
 
                                // Reset this user's counter
-                               SQL_QUERY_ESC("UPDATE `{!_MYSQL_PREFIX!}_user_data` SET `rand_confirmed`=0 WHERE userid=%s LIMIT 1",
-                               array($refid), __FUNCTION__, __LINE__);
+                               SQL_QUERY_ESC("UPDATE `{!_MYSQL_PREFIX!}_user_data` SET `rand_confirmed`=0 WHERE `userid`=%s LIMIT 1",
+                                       array($refid), __FUNCTION__, __LINE__);
                        } // END - if
 
                        // Free result
@@ -280,13 +278,13 @@ function USER_DO_LOGIN ($uid, $passwd) {
                );
 
                // Check login data
-               if ((EXT_IS_ACTIVE('nickname')) && (NICKNAME_PROBE_ON_USERID($uid))) {
+               if ((EXT_IS_ACTIVE('nickname')) && (NICKNAME_IS_ACTIVE($uid))) {
                        // Nickname entered
-                       $result = SQL_QUERY_ESC("SELECT userid, password, last_online".$lastOnline." FROM `{!_MYSQL_PREFIX!}_user_data` WHERE nickname='%s' AND `status`='CONFIRMED' LIMIT 1",
+                       $result = SQL_QUERY_ESC("SELECT `userid`, `password`, `last_online`" . $lastOnline . " FROM `{!_MYSQL_PREFIX!}_user_data` WHERE `nickname`='%s' AND `status`='CONFIRMED' LIMIT 1",
                                array($uid), __FUNCTION__, __LINE__);
                } else {
                        // Direct userid entered
-                       $result = SQL_QUERY_ESC("SELECT userid, password, last_online".$lastOnline." FROM `{!_MYSQL_PREFIX!}_user_data` WHERE userid=%s AND `status`='CONFIRMED' LIMIT 1",
+                       $result = SQL_QUERY_ESC("SELECT `userid`, `password`, `last_online`" . $lastOnline . " FROM `{!_MYSQL_PREFIX!}_user_data` WHERE `userid`=%s AND `status`='CONFIRMED' LIMIT 1",
                                array($uid, $content['hash']), __FUNCTION__, __LINE__);
                }
 
@@ -315,8 +313,8 @@ function USER_DO_LOGIN ($uid, $passwd) {
 
                                // ... and update database
                                // @TODO Make this filter working: $ADDON = runFilterChain('post_login_update', $content);
-                               SQL_QUERY_ESC("UPDATE `{!_MYSQL_PREFIX!}_user_data` SET password='%s' WHERE userid=%s AND `status`='CONFIRMED' LIMIT 1",
-                               array($content['hash'], $uid), __FUNCTION__, __LINE__);
+                               SQL_QUERY_ESC("UPDATE `{!_MYSQL_PREFIX!}_user_data` SET `password`='%s' WHERE `userid`=%s AND `status`='CONFIRMED' LIMIT 1",
+                                       array($content['hash'], $uid), __FUNCTION__, __LINE__);
 
                                // No login bonus by default
                                $GLOBALS['bonus_payed'] = false;
@@ -326,13 +324,13 @@ function USER_DO_LOGIN ($uid, $passwd) {
                                if (!empty($content['last_login'])) $probe = time() - $content['last_login'];
                                if ((GET_EXT_VERSION('bonus') >= '0.2.2') && ($probe >= getConfig('login_timeout'))) {
                                        // Add login bonus to user's account
-                                       $add = sprintf(", login_bonus=login_bonus+%s",
+                                       $add = sprintf(", `login_bonus`=`login_bonus`+%s",
                                        (float)getConfig('login_bonus')
                                        );
                                        $GLOBALS['bonus_payed'] = true;
 
                                        // Subtract login bonus from userid's account or jackpot
-                                       if ((GET_EXT_VERSION('bonus') >= '0.3.5') && (getConfig('bonus_mode') != "ADD")) BONUS_POINTS_HANDLER('login_bonus');
+                                       if ((GET_EXT_VERSION('bonus') >= '0.3.5') && (getConfig('bonus_mode') != 'ADD')) BONUS_POINTS_HANDLER('login_bonus');
                                } // END - if
 
                                // Calculate new hash with the secret key and master salt together
@@ -345,8 +343,8 @@ function USER_DO_LOGIN ($uid, $passwd) {
                                // Try to set session data (which shall normally always work!)
                                if ((setSession('userid', $uid )) && (setSession('u_hash', $content['hash']))) {
                                        // Update database records
-                                       SQL_QUERY_ESC("UPDATE `{!_MYSQL_PREFIX!}_user_data` SET total_logins=total_logins+1".$add." WHERE userid=%s LIMIT 1",
-                                       array($uid), __FUNCTION__, __LINE__);
+                                       SQL_QUERY_ESC("UPDATE `{!_MYSQL_PREFIX!}_user_data` SET `total_logins`=`total_logins +1" . $add . " WHERE `userid`=%s LIMIT 1",
+                                               array($uid), __FUNCTION__, __LINE__);
                                        if (SQL_AFFECTEDROWS() == 1) {
                                                // Procedure to checking for login data
                                                if (($GLOBALS['bonus_payed']) && (EXT_IS_ACTIVE('bonus'))) {
@@ -366,8 +364,8 @@ function USER_DO_LOGIN ($uid, $passwd) {
                                }
                        } elseif (GET_EXT_VERSION('sql_patches') >= '0.6.1') {
                                // Update failture counter
-                               SQL_QUERY_ESC("UPDATE `{!_MYSQL_PREFIX!}_user_data` SET login_failures=login_failures+1,last_failure=NOW() WHERE userid=%s LIMIT 1",
-                               array($uid), __FUNCTION__, __LINE__);
+                               SQL_QUERY_ESC("UPDATE `{!_MYSQL_PREFIX!}_user_data` SET `login_failures`=`login_failures`+1,`last_failure`=NOW() WHERE `userid`=%s LIMIT 1",
+                                       array($uid), __FUNCTION__, __LINE__);
 
                                // Wrong password!
                                $URL = 'modules.php?module=index&amp;what=login&amp;login='.getCode('WRONG_PASS');
@@ -375,8 +373,8 @@ function USER_DO_LOGIN ($uid, $passwd) {
                } elseif (((isNicknameUsed($content['userid'])) && (!empty($content['userid']))) || ($content['userid'] == $uid)) {
                        // Other account status?
                        // @TODO Can this query be merged with above query?
-                       $result = SQL_QUERY_ESC("SELECT status FROM `{!_MYSQL_PREFIX!}_user_data` WHERE userid=%s LIMIT 1",
-                       array($uid), __FUNCTION__, __LINE__);
+                       $result = SQL_QUERY_ESC("SELECT status FROM `{!_MYSQL_PREFIX!}_user_data` WHERE `userid`=%s LIMIT 1",
+                               array($uid), __FUNCTION__, __LINE__);
 
                        // Entry found?
                        if (SQL_NUMROWS($result) == 1) {
@@ -411,18 +409,18 @@ function USER_DO_NEW_PASSWORD ($email, $uid) {
        $result = false;
 
        // Probe userid/nickname
-       if ((EXT_IS_ACTIVE('nickname')) && (NICKNAME_PROBE_ON_USERID($uid))) {
+       if ((EXT_IS_ACTIVE('nickname')) && (NICKNAME_IS_ACTIVE($uid))) {
                // Nickname entered
-               $result = SQL_QUERY_ESC("SELECT userid, status FROM `{!_MYSQL_PREFIX!}_user_data` WHERE nickname='%s' OR email='%s' LIMIT 1",
-               array($uid, $email), __FUNCTION__, __LINE__);
+               $result = SQL_QUERY_ESC("SELECT `userid`, `status` FROM `{!_MYSQL_PREFIX!}_user_data` WHERE `nickname`='%s' OR `email`='%s' LIMIT 1",
+                       array($uid, $email), __FUNCTION__, __LINE__);
        } elseif (($uid > 0) && (empty($email))) {
                // Direct userid entered
-               $result = SQL_QUERY_ESC("SELECT userid, status FROM `{!_MYSQL_PREFIX!}_user_data` WHERE userid=%s LIMIT 1",
-               array(bigintval($uid)), __FUNCTION__, __LINE__);
+               $result = SQL_QUERY_ESC("SELECT `userid`, `status` FROM `{!_MYSQL_PREFIX!}_user_data` WHERE `userid`=%s LIMIT 1",
+                       array(bigintval($uid)), __FUNCTION__, __LINE__);
        } elseif (!empty($email)) {
                // Email entered
-               $result = SQL_QUERY_ESC("SELECT userid, status FROM `{!_MYSQL_PREFIX!}_user_data` WHERE email='%s' LIMIT 1",
-               array($email), __FUNCTION__, __LINE__);
+               $result = SQL_QUERY_ESC("SELECT `userid`, `status` FROM `{!_MYSQL_PREFIX!}_user_data` WHERE `email`='%s' LIMIT 1",
+                       array($email), __FUNCTION__, __LINE__);
        } else {
                // Userid not set!
                DEBUG_LOG(__FUNCTION__, __LINE__, 'Userid is not set! BUG!');
@@ -437,14 +435,14 @@ function USER_DO_NEW_PASSWORD ($email, $uid) {
                if ($status == 'CONFIRMED') {
                        // Ooppps, this was missing! ;-) We should update the database...
                        $NEW_PASS = generatePassword();
-                       SQL_QUERY_ESC("UPDATE `{!_MYSQL_PREFIX!}_user_data` SET password='%s' WHERE userid=%s LIMIT 1",
-                       array(generateHash($NEW_PASS), $uid), __FUNCTION__, __LINE__);
+                       SQL_QUERY_ESC("UPDATE `{!_MYSQL_PREFIX!}_user_data` SET `password`='%s' WHERE `userid`=%s LIMIT 1",
+                               array(generateHash($NEW_PASS), $uid), __FUNCTION__, __LINE__);
 
                        // Prepare data and message for email
-                       $msg = LOAD_EMAIL_TEMPLATE('new-pass', array('new_pass' => $NEW_PASS), $uid);
+                       $message = LOAD_EMAIL_TEMPLATE('new-pass', array('new_pass' => $NEW_PASS), $uid);
 
                        // ... and send it away
-                       sendEmail($uid, getMessage('GUEST_NEW_PASSWORD'), $msg);
+                       sendEmail($uid, getMessage('GUEST_NEW_PASSWORD'), $message);
 
                        // Output note to user
                        LOAD_TEMPLATE('admin_settings_saved', false, getMessage('GUEST_NEW_PASSWORD_SEND'));