]> git.mxchange.org Git - mailer.git/blobdiff - inc/libs/admins_functions.php
Fixes for #226, renamed function:
[mailer.git] / inc / libs / admins_functions.php
index 407308222aa1bd1ef4551b88dd9f13fe86f263a8..5f5075a574ff7e8f131933632b448db9ea4fd8cf 100644 (file)
@@ -16,7 +16,7 @@
  * $Author::                                                          $ *
  * -------------------------------------------------------------------- *
  * Copyright (c) 2003 - 2009 by Roland Haeder                           *
- * Copyright (c) 2009 - 2011 by Mailer Developer Team                   *
+ * Copyright (c) 2009 - 2012 by Mailer Developer Team                   *
  * For more information visit: http://mxchange.org                      *
  *                                                                      *
  * This program is free software; you can redistribute it and/or modify *
@@ -164,7 +164,7 @@ LIMIT 1",
                $email = '{%url=modules.php?module=' . $mod . '&what=admins_contct&id=' . bigintval($email) . '%}';
        } else {
                // This is strange and needs fixing
-               debug_report_bug(__FUNCTION__, __LINE__, 'email[' . gettype($email) . ']=' . $email . ',mod=' . $mod . ' - This should not happen.');
+               reportBug(__FUNCTION__, __LINE__, 'email[' . gettype($email) . ']=' . $email . ',mod=' . $mod . ' - This should not happen.');
        }
 
        // Return rewritten (?) email address
@@ -201,7 +201,7 @@ function adminsChangeAdminAccount ($postData, $element = '', $displayMessage = t
 
                                // Save password when set
                                if (!empty($postData['pass1'][$id])) {
-                                       $add = sprintf(", `password`='%s'", SQL_ESCAPE($hash));
+                                       $add = sprintf(",`password`='%s'", SQL_ESCAPE($hash));
                                } // END - if
 
                                // Get admin's id
@@ -246,7 +246,7 @@ LIMIT 1",
                                        array(
                                                $login,
                                                $postData['email'][$id],
-                                               $postData['mode'][$id],
+                                               $postData['access_mode'][$id],
                                                $postData['la_mode'][$id],
                                                $id
                                        ), __FUNCTION__, __LINE__);
@@ -277,28 +277,7 @@ LIMIT 1",
                        }
                } else {
                        // Update whole array
-                       $SQL = 'UPDATE `{?_MYSQL_PREFIX?}_admins` SET ';
-                       foreach ($postData as $entry => $value) {
-                               // Skip login/id entry
-                               if (in_array($entry, array('login', 'id'))) {
-                                       continue;
-                               } // END - if
-
-                               // Do we have a non-string (e.g. number, NULL, NOW() or back-tick at the beginning?
-                               if (is_null($value[$id])) {
-                                       // NULL detected
-                                       $SQL .= '`' . $entry . '`=NULL, ';
-                               } elseif ((bigintval($value[$id], true, false) === $value[$id]) || ($value[$id] == 'NOW()') || (substr($value[$id], 0, 1) == '`'))  {
-                                       // No need for ticks (')
-                                       $SQL .= '`' . $entry . '`=' . $value[$id] . ', ';
-                               } else {
-                                       // Strings need ticks (') around them
-                                       $SQL .= '`' . $entry . "`='" . SQL_ESCAPE($value[$id]) . "', ";
-                               }
-                       } // END - foreach
-
-                       // Remove last 2 chars and finish query
-                       $SQL = substr($SQL, 0, -2) . ' WHERE `id`=%s LIMIT 1';
+                       $SQL = getUpdateSqlFromArray($postData, 'admins', 'id', '%s', array('login', 'id'), $id);
 
                        // Run it
                        SQL_QUERY_ESC($SQL, array(bigintval($id)), __FUNCTION__, __LINE__);
@@ -314,11 +293,10 @@ LIMIT 1",
                }
        } // END - foreach
 
-       // Display message
-       if (!empty($message)) {
-               if ($displayMessage === true) {
-                       displayMessage($message);
-               } // END - if
+       // Display message if not empty and allowed
+       if ((!empty($message)) && ($displayMessage === true)) {
+               // Display it
+               displayMessage($message);
        } // END - if
 
        // Remove cache file
@@ -352,10 +330,10 @@ function adminsEditAdminAccount ($postData) {
                        // Shall we allow changing default ACL?
                        if ($currMode == 'allow') {
                                // Allow changing it
-                               $content['mode'] = '{%pipe,generateAdminAccessModeSelectionBox=' . $id . '%}';
+                               $content['access_mode'] = '{%pipe,generateAdminAccessModeSelectionBox=' . $id . '%}';
                        } else {
                                // Don't allow it
-                               $content['mode'] = ' ';
+                               $content['access_mode'] = ' ';
                        }
 
                        // Load row template and switch color
@@ -373,10 +351,20 @@ function adminsEditAdminAccount ($postData) {
 // Generate access mode selection box for given admin id
 function generateAdminAccessModeSelectionBox ($adminId = NULL) {
        // Start the selection box
-       $OUT = '<select name="mode[' . $adminId . ']" size="1" class="form_select">';
+       $OUT = '<select name="access_mode[' . $adminId . ']" size="1" class="form_select">';
 
        // Add option list
-       $OUT .= generateOptionList('/ARRAY/', array('allow', 'deny'), array('{--ADMIN_ADMINS_ACCESS_MODE_ALLOW--}', '{--ADMIN_ADMINS_ACCESS_MODE_DENY--}'), getAdminDefaultAcl($adminId));
+       $OUT .= generateOptions(
+               '/ARRAY/',
+               array(
+                       'allow',
+                       'deny'
+               ), array(
+                       '{--ADMIN_ADMINS_ACCESS_MODE_ALLOW--}',
+                       '{--ADMIN_ADMINS_ACCESS_MODE_DENY--}'
+               ),
+               getAdminDefaultAcl($adminId)
+       );
 
        // Finish it
        $OUT .= '</select>';
@@ -388,10 +376,22 @@ function generateAdminAccessModeSelectionBox ($adminId = NULL) {
 // Generate menu mode selection box for given admin it
 function generateAdminMenuModeSelectionBox ($adminId = NULL) {
        // Start the selection box
-       $OUT = '<select name="la_mode[{%pipe,makeNullToZero=' . makeZeroToNull($adminId) . '%}]" size="1" class="form_select">';
+       $OUT = '<select name="la_mode[{%pipe,convertNullToZero=' . convertZeroToNull($adminId) . '%}]" size="1" class="form_select">';
 
        // Add option list
-       $OUT .= generateOptionList('/ARRAY/', array('global', 'OLD', 'NEW'), array('{--ADMIN_ADMINS_LA_MODE_GLOBAL--}', '{--ADMIN_ADMINS_LA_MODE_OLD--}', '{--ADMIN_ADMINS_LA_MODE_NEW--}'), getAdminMenuMode($adminId));
+       $OUT .= generateOptions(
+               '/ARRAY/',
+               array(
+                       'global',
+                       'OLD',
+                       'NEW'
+               ), array(
+                       '{--ADMIN_ADMINS_LA_MODE_GLOBAL--}',
+                       '{--ADMIN_ADMINS_LA_MODE_OLD--}',
+                       '{--ADMIN_ADMINS_LA_MODE_NEW--}'
+               ),
+               getAdminMenuMode($adminId)
+       );
 
        // Finish it
        $OUT .= '</select>';
@@ -411,15 +411,15 @@ function adminsDeleteAdminAccount ($postData) {
                        $id = bigintval($id);
 
                        // Get the admin's data
-                       $result = SQL_QUERY_ESC("SELECT `login`,`email`,`default_acl` AS `mode`,`la_mode` FROM `{?_MYSQL_PREFIX?}_admins` WHERE `id`=%s LIMIT 1",
+                       $result = SQL_QUERY_ESC("SELECT `login`,`email`,`default_acl` AS `access_mode`,`la_mode` FROM `{?_MYSQL_PREFIX?}_admins` WHERE `id`=%s LIMIT 1",
                                array($id), __FUNCTION__, __LINE__);
 
                        // Do we have an entry?
                        if (SQL_NUMROWS($result) == 1) {
                                // Entry found, so load data
                                $content = SQL_FETCHARRAY($result);
-                               $content['mode']    = '{--ADMIN_ADMINS_ACCESS_MODE_' . strtoupper($content['mode'])    . '--}';
-                               $content['la_mode'] = '{--ADMIN_ADMINS_LA_MODE_' . strtoupper($content['la_mode']) . '--}';
+                               $content['access_mode'] = '{--ADMIN_ADMINS_ACCESS_MODE_' . strtoupper($content['access_mode'])    . '--}';
+                               $content['la_mode']     = '{--ADMIN_ADMINS_LA_MODE_' . strtoupper($content['la_mode']) . '--}';
 
                                // Prepare some more data
                                $content['id'] = $id;
@@ -467,12 +467,21 @@ function adminsRemoveAdminAccount ($postData) {
 // List all admin accounts
 function adminsListAdminAccounts() {
        // Select all admin accounts
-       $result = SQL_QUERY('SELECT `id`,`login`,`email`,`default_acl` AS mode, `la_mode` FROM `{?_MYSQL_PREFIX?}_admins` ORDER BY `login` ASC', __FUNCTION__, __LINE__);
+       $result = SQL_QUERY('SELECT
+       `id`,
+       `login`,
+       `email`,
+       `default_acl` AS `access_mode`,
+       `la_mode`
+FROM
+       `{?_MYSQL_PREFIX?}_admins`
+ORDER BY
+       `login` ASC', __FUNCTION__, __LINE__);
        $OUT = '';
        while ($content = SQL_FETCHARRAY($result)) {
                // Compile some variables
-               $content['mode']    = '{--ADMIN_ADMINS_ACCESS_MODE_' . strtoupper($content['mode'])    . '--}';
-               $content['la_mode'] = '{--ADMIN_ADMINS_LA_MODE_' . strtoupper($content['la_mode']) . '--}';
+               $content['access_mode'] = '{--ADMIN_ADMINS_ACCESS_MODE_' . strtoupper($content['access_mode'])    . '--}';
+               $content['la_mode']     = '{--ADMIN_ADMINS_LA_MODE_' . strtoupper($content['la_mode']) . '--}';
 
                // Load row template and switch color
                $OUT .= loadTemplate('admin_list_admins_row', true, $content);
@@ -487,7 +496,7 @@ function adminsListAdminAccounts() {
 
 // Sends out mail to all administrators
 // IMPORTANT: Please use sendAdminNotification() instead of calling this function directly
-function sendAdminsEmails ($subj, $template, $content, $userid) {
+function sendAdminsEmails ($subject, $template, $content, $userid) {
        // Trim template name
        $template = trim($template);
 
@@ -503,12 +512,16 @@ function sendAdminsEmails ($subj, $template, $content, $userid) {
                // Create new entry (to all admins)
                SQL_QUERY_ESC("INSERT INTO `{?_MYSQL_PREFIX?}_admins_mails` (`admin_id`,`mail_template`) VALUES (0, '%s')",
                        array($template), __FUNCTION__, __LINE__);
+
+               // Select all email adresses (default)
+               $result = SQL_QUERY('SELECT `email` FROM `{?_MYSQL_PREFIX?}_admins` ORDER BY `id` ASC',
+                       __FUNCTION__, __LINE__);
        } else {
                // Load admin ids...
                // @TODO This can be, somehow, rewritten
                $adminIds = array();
                while ($content = SQL_FETCHARRAY($result)) {
-                       $adminIds[] = $content['admin_id'];
+                       array_push($adminIds, $content['admin_id']);
                } // END - while
 
                // Free memory
@@ -519,15 +532,18 @@ function sendAdminsEmails ($subj, $template, $content, $userid) {
 
                // "implode" ids and query string
                $adminId = implode(',', $adminIds);
+
+               // To which admin shall we sent it?
                if ($adminId == '-1') {
+                       // Is an "event"
                        if (isExtensionActive('events')) {
                                // Add line to user events
-                               EVENTS_ADD_LINE($subj, $message, $userid);
+                               EVENTS_ADD_LINE($subject, $message, $userid);
                        } else {
                                // Log error for debug
-                               logDebugMessage(__FUNCTION__, __LINE__, sprintf("Extension 'events' missing: tpl=%s,subj=%s,userid=%s",
+                               logDebugMessage(__FUNCTION__, __LINE__, sprintf("Extension 'ext-events' missing: template=%s,subj=%s,userid=%s",
                                        $template,
-                                       $subj,
+                                       $subject,
                                        $userid
                                ));
                        }
@@ -542,9 +558,18 @@ function sendAdminsEmails ($subj, $template, $content, $userid) {
                }
        }
 
+       // Default is no special mail header
+       $mailHeader = '';
+
+       // Is the template a bug report?
+       if ($template == 'admin_report_bug') {
+               // Then set 'Reply-To:' again
+               $mailHeader = 'Reply-To: webmaster@mxchange.org' . chr(10);
+       } // END - if
+
        // Load email addresses and send away
        while ($content = SQL_FETCHARRAY($result)) {
-               sendEmail($content['email'], $subj, $message);
+               sendEmail($content['email'], $subject, $message, 'N', $mailHeader);
        } // END - while
 
        // Free memory
@@ -629,7 +654,7 @@ function getAminsExpertWarning () {
 function getAdminLoginFailures ($adminId) {
        // Admin login should not be empty
        if (empty($adminId)) {
-               debug_report_bug(__FUNCTION__, __LINE__, 'adminId is empty.');
+               reportBug(__FUNCTION__, __LINE__, 'adminId is empty.');
        } // END - if
 
        // By default no admin is found
@@ -665,7 +690,7 @@ function getAdminLoginFailures ($adminId) {
 function getAdminLastFailure ($adminId) {
        // Admin login should not be empty
        if (empty($adminId)) {
-               debug_report_bug(__FUNCTION__, __LINE__, 'adminId is empty.');
+               reportBug(__FUNCTION__, __LINE__, 'adminId is empty.');
        } // END - if
 
        // By default no admin is found
@@ -701,7 +726,7 @@ function getAdminLastFailure ($adminId) {
 //                             Wrapper functions
 //-----------------------------------------------------------------------------
 
-// Wrapper function to check wether expert setting warning is enabled
+// Wrapper function to check whether expert setting warning is enabled
 function isAdminsExpertWarningEnabled () {
        return (getAminsExpertWarning() == 'Y');
 }