More SQL rewrites, TODO: Put all table and column names in backticks (`)

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

diff --git a/inc/modules/admin/what-send_newsletter.php b/inc/modules/admin/what-send_newsletter.php
index 8e12bd96a15a3711247f6ce8b2e5e0923fb6828a..68091e0907705bddfae84740f3e9a429a7b4a58d 100644 (file)
--- a/inc/modules/admin/what-send_newsletter.php
+++ b/inc/modules/admin/what-send_newsletter.php
@@ -40,25 +40,22 @@ if ((!defined('__SECURITY')) || (!IS_ADMIN())) {
 // Add description as navigation point
 ADD_DESCR("admin", __FILE__);
 
-if (isset($_POST['ok']))
-{
-       $result = SQL_QUERY("SELECT userid, email FROM "._MYSQL_PREFIX."_user_data WHERE status='CONFIRMED' AND nl_receive='Y' ORDER BY userid", __FILE__, __LINE__);
-       if (SQL_NUMROWS($result) > 0)
-       {
+if (isset($_POST['ok'])) {
+       $result = SQL_QUERY("SELECT userid, email
+FROM `"._MYSQL_PREFIX."_user_data`
+WHERE status='CONFIRMED' AND nl_receive='Y'
+ORDER BY userid ASC", __FILE__, __LINE__);
+       if (SQL_NUMROWS($result) > 0) {
                // Members are available so we can send out the newsletter!
-               while (list($id, $email) = SQL_FETCHROW($result))
-               {
+               while (list($id, $email) = SQL_FETCHROW($result)) {
                        // Construct mail...
                        $template = "newsletter";
 
                        // Check for extension and sending-mode
-                       if (!EXT_IS_ACTIVE("html_mail", true) && ($_POST['mode'] == "html"))
-                       {
+                       if (!EXT_IS_ACTIVE("html_mail", true) && ($_POST['mode'] == "html")) {
                                // Set mode to text mode
                                $_POST['mode'] == "text";
-                       }
-                        elseif ($_POST['mode'] == "html")
-                       {
+                       } elseif ($_POST['mode'] == "html") {
                                // Set HTML templates
                                $template = "newsletter_html";
                        }
@@ -79,18 +76,13 @@ if (isset($_POST['ok']))
                // Output message
                LOAD_TEMPLATE("admin_settings_saved", false, ADMIN_NL_SEND_DONE);
        }
-}
- else
-{
+} else {
        // Copy data into constants for the template and load it
        define('_DATESTAMP', MAKE_DATETIME(time(), "3"));
-       if (EXT_IS_ACTIVE("html_mail"))
-       {
+       if (EXT_IS_ACTIVE("html_mail")) {
                // Load template with HTML mode
                LOAD_TEMPLATE("admin_newsletter");
-       }
-        else
-       {
+       } else {
                // Load template with only text mode
                LOAD_TEMPLATE("admin_newsletter_nohtml");
        }