Huge rewrite of default parameters, ext-network continued:

[mailer.git] / inc / modules / admin / what-list_user.php

diff --git a/inc/modules/admin/what-list_user.php b/inc/modules/admin/what-list_user.php
index 480eea6662ea5e57fc419fbbf86e3daa8fb83c64..0722b871e4affee3a8b048e2f5332912eeb5cd99 100644 (file)
--- a/inc/modules/admin/what-list_user.php
+++ b/inc/modules/admin/what-list_user.php
@@ -52,7 +52,7 @@ if ((!isExtensionInstalled('user')) || (!isExtensionActive('user'))) {
 
 // Init variables
 $MORE = '';
-$colspan = '4';
+$colspan = 4;
 
 // Add nickname if extension is found&active
 // @TODO Rewrite these if-blocks in a filter
@@ -87,13 +87,13 @@ if (isExtensionActive('country')) {
 // Init unset data
 if (!isGetRequestElementSet('letter')) { setRequestGetElement('letter', getMessage('_ALL2')); }
 if (!isGetRequestElementSet('sortby')) { setRequestGetElement('sortby', 'userid');            }
-if (!isGetRequestElementSet('page'))   { setRequestGetElement('page'  , '1');                 }
+if (!isGetRequestElementSet('page'))   { setRequestGetElement('page'  , 1);                 }
 
 // Set base URL
 $base = '[<a href="{?URL?}/modules.php?module=admin';
 
 if (isGetRequestElementSet('userid')) {
-       // Secure the user ID
+       // Secure the user id
        $userid = bigintval(getRequestElement('userid'));
 
        // Does the account exists?
@@ -131,10 +131,10 @@ LIMIT 1",
                // Is this above zero?
                if ($stamp > 0) {
                        // Then use it
-                       $content['birthday'] = generateDateTime($stamp, '3');
+                       $content['birthday'] = generateDateTime($stamp, 3);
                } else {
                        // Zero or below so set zero!
-                       $content['birthday'] = generateDateTime(0, '3');
+                       $content['birthday'] = generateDateTime(0, 3);
                }
 
                // Prepare data for template
@@ -142,15 +142,15 @@ LIMIT 1",
                $content['gender']            = translateGender($content['gender']);
                $content['email_link']        = generateEmailLink($content['email'], 'user_data');
                $content['status']            = translateUserStatus($content['status']);
-               $content['last_online']       = generateDateTime($content['last_online'], '0');
+               $content['last_online']       = generateDateTime($content['last_online'], 0);
                $content['used_points']       = translateComma($content['used_points']);
                if ($content['emails_sent'] > 0) $content['emails_sent'] = $base . '&amp;what=email_details&amp;userid=' . $userid . '">' . translateComma($content['emails_sent']) . '</a>]';
-               $content['joined']            = generateDateTime($content['joined'], '0');
-               $content['last_update']       = generateDateTime($content['last_update'], '0');
-               $content['last_profile_sent'] = generateDateTime($content['last_profile_sent'], '0');
+               $content['joined']            = generateDateTime($content['joined'], 0);
+               $content['last_update']       = generateDateTime($content['last_update'], 0);
+               $content['last_profile_sent'] = generateDateTime($content['last_profile_sent'], 0);
                $content['total']             = translateComma(countSumTotalData($userid, 'user_points', 'points'));
                $content['locked']            = translateComma(countSumTotalData($userid, 'user_points', 'locked_points'));
-               $content['lock_timestamp']    = generateDateTime($content['lock_timestamp'], '2');
+               $content['lock_timestamp']    = generateDateTime($content['lock_timestamp'], 2);
 
                // Is the lock reason not set?
                if (!isset($content['lock_reason'])) $content['lock_reason'] = '---';
@@ -172,7 +172,7 @@ LIMIT 1",
                } // END - if
 
                // Clickrate
-               $content['click_rate'] = 0;
+               $content['click_rate'] = '0';
                if ($content['emails_received'] > 0) {
                        $content['click_rate'] = translateComma($content['mails_confirmed'] / $content['emails_received'] * 100);
                } // END - if
@@ -185,8 +185,8 @@ LIMIT 1",
                if (isExtensionActive('country')) {
                        // Then overwrite country information
                        $content['country'] = generateCountryInfo($content['country_code']);
-               } elseif ($content['country'] == 0) {
-                       // Zero ID???
+               } elseif ($content['country'] == '0') {
+                       // Zero id???
                        $content['country'] = '???';
                }
 
@@ -247,7 +247,7 @@ LIMIT 1",
        $result_master = SQL_QUERY($sql, __FILE__, __LINE__);
 
        // Calculate page count (0.5 fixes a bug with page count)
-       if (getConfig('user_limit') == 0) {
+       if (getConfig('user_limit') == '0') {
                setConfigEntry('user_limit', 100);
                loadTemplate('admin_settings_saved', false, getMessage('EXTENSION_WARNING_USER_LIMIT'));
        } // END - if
@@ -255,7 +255,7 @@ LIMIT 1",
        // Activate the extension please!
        $PAGES = round(SQL_NUMROWS($result_master) / getConfig('user_limit') + 0.5);
 
-       if (!isGetRequestElementSet('page'))   setRequestGetElement('page'  , '1');
+       if (!isGetRequestElementSet('page'))   setRequestGetElement('page'  , 1);
        if (!isGetRequestElementSet('offset')) setRequestGetElement('offset', getConfig('user_limit'));
 
        // Add limitation to SQL string and run him again
@@ -312,7 +312,7 @@ LIMIT 1",
                        $pointsTotal = countSumTotalData($content['userid'], 'user_points', 'points')  - countSumTotalData($content['userid'], 'user_data', 'used_points');
 
                        // Clickrate
-                       $clickRate = 0;
+                       $clickRate = '0';
                        if ($content['emails_received'] > 0) {
                                $clickRate = $content['mails_confirmed'] / $content['emails_received'] * 100;
                        } // END - if
@@ -326,9 +326,9 @@ LIMIT 1",
                        $content['points']         = translateComma($pointsTotal);
                        $content['rate']           = translateComma($clickRate);
                        $content['locked']         = translateComma(countSumTotalData($content['userid'], 'user_points', 'locked_points'));
-                       $content['lock_timestamp'] = generateDateTime($content['lock_timestamp'], '2');
+                       $content['lock_timestamp'] = generateDateTime($content['lock_timestamp'], 2);
                        $content['status']         = translateUserStatus($content['status']);
-                       $content['refs']           = countSumTotalData($content['userid'], 'refsystem', 'counter');
+                       $content['refs']           = countSumTotalData($content['userid'], 'user_refs', 'id', 'userid', true);
 
                        // If we have at least one referal, make it clickable to referal list
                        if ($content['refs'] > 0) $content['refs'] = $base . '&amp;what=list_refs&amp;userid=' . $content['userid'] . '">' . $content['refs'] . '</a>]';
@@ -342,7 +342,7 @@ LIMIT 1",
                                // Then overwrite country information
                                $content['country'] = generateCountryInfo($content['country_code']);
                        } elseif ($content['country'] == '') {
-                               // Zero ID???
+                               // Zero id???
                                $content['country'] = '???';
                        }