Complete rewrite and debug message surpressed:

[mailer.git] / inc / modules / member / what-newsletter.php

diff --git a/inc/modules/member/what-newsletter.php b/inc/modules/member/what-newsletter.php
index 5cac3962b529e4b53346fc740078936b61a6b5c9..1e815de44523942417d3a495bdd3029dc5dda0c0 100644 (file)
--- a/inc/modules/member/what-newsletter.php
+++ b/inc/modules/member/what-newsletter.php
@@ -43,7 +43,7 @@ if (!defined('__SECURITY')) {
 } elseif (!IS_MEMBER()) {
        redirectToUrl('modules.php?module=index');
 } elseif ((!EXT_IS_ACTIVE('newsletter')) && (!IS_ADMIN())) {
-       addFatalMessage(__FILE__, __LINE__, sprintf(getMessage('EXTENSION_PROBLEM_EXT_INACTIVE'), 'newsletter'));
+       addFatalMessage(__FILE__, __LINE__, generateExtensionInactiveNotInstalledMessage('newsletter'));
        return;
 }
 
@@ -52,17 +52,17 @@ ADD_DESCR('member', __FILE__);
 
 // Load status
 $result = SQL_QUERY_ESC("SELECT nl_receive, nl_until, nl_timespan FROM `{!_MYSQL_PREFIX!}_user_data` WHERE userid=%s LIMIT 1",
-       array(getUserId()), __FILE__, __LINE__);
+array(getUserId()), __FILE__, __LINE__);
 list($status, $until, $span) = SQL_FETCHROW($result);
 SQL_FREERESULT($result);
 
 // Remember charge value
 define('__CHARGE_VALUE', translateComma(getConfig('nl_charge')));
 
-if ((IS_FORM_SENT()) && ($status == 'Y') && ($span == '0')) {
+if ((isFormSent()) && ($status == 'Y') && ($span == '0')) {
        // Save request
        SQL_QUERY_ESC("UPDATE `{!_MYSQL_PREFIX!}_user_data` SET nl_timespan='".(getConfig('one_day') * 30)."' WHERE userid=%s LIMIT 1",
-               array(getUserId()), __FILE__, __LINE__);
+       array(getUserId()), __FILE__, __LINE__);
 
        // Load admin message
        $admin_msg = LOAD_EMAIL_TEMPLATE("admin_newsletter_request", array(), getUserId());
@@ -90,17 +90,17 @@ VALUES ('0','NEW','NL_UNSUBSCRIBE','{--ADMIN_NL_SUBJECT--}','".$admin_msg."', UN
        // Set status message and submit button
        switch ($status)
        {
-       case 'Y': // Receives the newsletter
-               define('__STATUS_VALUE', getMessage('NL_MEMBER_ON'));
-               define('__UNTIL_VALUE' , '');
-               define('NL_SUBMIT'     , getMessage('NL_MEMBER_SUBMIT_OFF'));
-               break;
+               case 'Y': // Receives the newsletter
+                       define('__STATUS_VALUE', getMessage('NL_MEMBER_ON'));
+                       define('__UNTIL_VALUE' , '');
+                       define('NL_SUBMIT'     , getMessage('NL_MEMBER_SUBMIT_OFF'));
+                       break;
 
-       case 'N': // Does not receive the newsletter
-               define('__STATUS_VALUE', getMessage('NL_MEMBER_OFF'));
-               define('__UNTIL_VALUE' , generateDateTime($until, "2"));
-               define('NL_SUBMIT'     , getMessage('NL_MEMBER_SUBMIT_ON'));
-               break;
+               case 'N': // Does not receive the newsletter
+                       define('__STATUS_VALUE', getMessage('NL_MEMBER_OFF'));
+                       define('__UNTIL_VALUE' , generateDateTime($until, '2'));
+                       define('NL_SUBMIT'     , getMessage('NL_MEMBER_SUBMIT_ON'));
+                       break;
        }
 
        // Load template and member settings