MDL-26046 prevent jabber messages wehn noemailever active
authorPetr Skoda <commits@skodak.org>
Sun, 30 Jan 2011 09:56:18 +0000 (10:56 +0100)
committerPetr Skoda <commits@skodak.org>
Sun, 30 Jan 2011 10:01:27 +0000 (11:01 +0100)
Standardise also the debug when emailing message.

config-dist.php
message/output/email/message_output_email.php
message/output/jabber/message_output_jabber.php

index f9909be..79e0d9d 100644 (file)
@@ -379,7 +379,8 @@ $CFG->admin = 'admin';
 // Prevent core_string_manager on-disk cache
 // $CFG->langstringcache = false; // NOT FOR PRODUCTION SERVERS!
 //
-// When working with production data on test servers, no emails should ever be send to real users
+// When working with production data on test servers, no emails or other messages
+// should ever be send to real users
 // $CFG->noemailever = true;    // NOT FOR PRODUCTION SERVERS!
 //
 // Divert all outgoing emails to this address to test and debug emailing features
index 3edfb1f..90bf792 100644 (file)
@@ -38,7 +38,13 @@ class message_output_email extends message_output {
      * @param object $eventdata the event data submitted by the message sender plus $eventdata->savedmessageid
      */
     function send_message($eventdata) {
-        global $SITE;
+        global $CFG;
+
+        if (!empty($CFG->noemailever)) {
+            // hidden setting for development sites, set in config.php if needed
+            debugging('$CFG->noemailever active, no email message sent.', DEBUG_MINIMAL);
+            return true;
+        }
 
         //hold onto email preference because /admin/cron.php sends a lot of messages at once
         static $useremailaddresses = array();
index 6f3c4a5..a6f9f65 100644 (file)
@@ -45,6 +45,12 @@ class message_output_jabber extends message_output {
         global $CFG;
 
         if (message_output_jabber::_jabber_configured()) {
+            if (!empty($CFG->noemailever)) {
+                // hidden setting for development sites, set in config.php if needed
+                debugging('$CFG->noemailever active, no jabber message sent.', DEBUG_MINIMAL);
+                return true;
+            }
+
             //hold onto jabber id preference because /admin/cron.php sends a lot of messages at once
             static $jabberaddresses = array();
 
@@ -86,7 +92,7 @@ class message_output_jabber extends message_output {
      */
     function config_form($preferences){
         global $CFG;
-        
+
         if (!message_output_jabber::_jabber_configured()) {
             return get_string('notconfigured','message_jabber');
         } else {