MDL-16267 and MDL-10107

I did some further work tidying up the form.  I also just treat the popup like another plugin and moved all the config into the plugin.
This still needs help buttons everywhere now!

diff --git a/lang/en_utf8/messageprocessor_jabber.php b/lang/en_utf8/messageprocessor_jabber.php
index 22ebe0b3242f6b66aeec662c271c7968039682f8..97b38709a05f5cc6c6142656121b60e68714652b 100644 (file)
--- a/lang/en_utf8/messageprocessor_jabber.php
+++ b/lang/en_utf8/messageprocessor_jabber.php
@@ -1,3 +1,4 @@
 <?PHP // $Id$ 
 $string['jabber'] = 'Jabber message';
+$string['jabberid'] = 'Jabber ID';
 ?>

diff --git a/message/edit.php b/message/edit.php
index fc6a8b160e9befdd22ae5f3d789a6b99a684c7bc..7c50015d047e752dbb12a57ac9bd90a410a1db82 100644 (file)
--- a/message/edit.php
+++ b/message/edit.php
@@ -93,12 +93,6 @@ if ($user->id == $USER->id) {
 if ( ($form = data_submitted()) && confirm_sesskey()) {
     $preferences = array();
 
-    /// Set the overall preferences
-    $preferences['message_showmessagewindow'] = $form->showmessagewindow?1:0;
-    $preferences['message_blocknoncontacts']  = $form->blocknoncontacts?1:0;
-    $preferences['message_beepnewmessage']    = $form->beepnewmessage?1:0;
-    $preferences['message_noframesjs']        = $form->noframesjs?1:0;
-    
     /// Set all the preferences for all the message providers
     $providers = message_get_my_providers();
     foreach ( $providers as $providerid => $provider){
@@ -114,6 +108,7 @@ if ( ($form = data_submitted()) && confirm_sesskey()) {
             $preferences['message_provider_'.$provider->component.'_'.$provider->name.'_'.$state] = $linepref;
         }
     }
+
     /// Set all the processor options as well
     $processors = $DB->get_records('message_processors');
     foreach ( $processors as $processorid => $processor){
@@ -141,11 +136,6 @@ if ( ($form = data_submitted()) && confirm_sesskey()) {
 //load preferences so show
 $preferences = new object();
 
-//get the message general preferences
-$preferences->showmessagewindow =  get_user_preferences( 'message_showmessagewindow', 1, $user->id);
-$preferences->blocknoncontacts  =  get_user_preferences( 'message_blocknoncontacts', '', $user->id);
-$preferences->beepnewmessage    =  get_user_preferences( 'message_beepnewmessage', '', $user->id);
-$preferences->noframesjs        =  get_user_preferences( 'message_noframesjs', '', $user->id);
 //get providers preferences
 $providers = message_get_my_providers();
 foreach ( $providers as $providerid => $provider){
@@ -204,22 +194,12 @@ $showroles = 1;
 $currenttab = 'editmessage';
 require('../user/tabs.php');
 
-echo '<form method="post" action="'.$CFG->wwwroot.'/message/edit.php">';
-
-echo '<div class="generalbox">';
-echo '<table>';
-echo '<tr><td colspan="2"><h3>'.get_string('private_config', 'message').'</h3></td></tr>';
-echo '<tr><td>'.get_string('showmessagewindow', 'message').'</td><td><input type="checkbox" name="showmessagewindow" '.($preferences->showmessagewindow==1?" checked=\"checked\"":"").' /></td></tr>';
-echo '<tr><td>'.get_string('blocknoncontacts', 'message').'</td><td><input type="checkbox" name="blocknoncontacts" '.($preferences->blocknoncontacts==1?" checked=\"checked\"":"").' /></td></tr>';
-echo '<tr><td>'.get_string('beepnewmessage', 'message').'</td><td><input type="checkbox" name="beepnewmessage" '.($preferences->beepnewmessage==1?" checked=\"checked\"":"").' /></td></tr>';
-echo '<tr><td>'.get_string('noframesjs', 'message').'</td><td><input type="checkbox" name="noframesjs" '.($preferences->noframesjs==1?" checked=\"checked\"":"").' /></td></tr>';
-echo '</table>';
-echo '</div>';
+echo '<form class="mform" method="post" action="'.$CFG->wwwroot.'/message/edit.php">';
 
 //output settings table
-echo '<div class="generalbox">';
+echo '<fieldset id="providers" class="clearfix">';
+echo '<legend class="ftoggler">'.get_string('providers_config', 'message').'</legend>';
 echo '<table>';
-echo '<tr><td><h3>'.get_string('providers_config', 'message').'</h3></td></tr>'."\n";
 $providers = message_get_my_providers();
 $processors = $DB->get_records('message_processors');
 $number_procs = count($processors);
@@ -254,11 +234,8 @@ foreach ( $providers as $providerid => $provider){
 }
 echo '</table>';
 echo '</td></tr></table>';
-echo '</div>';
+echo '</fieldset>';
 
-echo '<div class="generalbox">';
-echo '<table>';
-echo '<tr><td colspan="2"><h3>'.get_string('processor_config', 'message').'</h3></td></tr>'."\n";
 //get a listing of all the message processors
 $processors = $DB->get_records('message_processors');
 
@@ -278,8 +255,6 @@ foreach ( $processors as $processorid => $processor){
         }
     }
 }
-echo '</table>';
-echo '</div>';
 
 echo '<p><input type="hidden" name="sesskey" value="'.sesskey().'" /> </p>';
 echo '<div style="text-align:center"><input name="submit" value="'. get_string('updatemyprofile') .'" type="submit" /></div>';

diff --git a/message/output/email/message_output_email.php b/message/output/email/message_output_email.php
index 36e8ef3f55ed9767a68ccc90da1fc2ea0ea802d1..ac96cfd31483504d544a91328dfde6a14ebd3893 100644 (file)
--- a/message/output/email/message_output_email.php
+++ b/message/output/email/message_output_email.php
@@ -83,10 +83,10 @@ class message_output_email extends message_output {
      * @param object $mform preferences form class
      */
     function config_form($preferences){               
-        $dest = get_string('email', 'messageprocessor_email');
-        echo '<tr><td colspan="2"><b>'.get_string('processortag', 'message').$dest.'</b></td></tr>'."\n";
-        echo '<tr><td align="right">'.get_string('email').'</td><td><input name="email_email" value="'.$preferences->email_email.'" /></td></tr>'."\n";
-        return true;
+        echo '<fieldset id="messageprocessor_email" class="clearfix">';
+        echo '<legend class="ftoggler">'.get_string('email', 'messageprocessor_email').'</legend>';
+        echo get_string('email').': <input name="email_email" value="'.$preferences->email_email.'" />';
+        echo '</fieldset>';
     }
 
     /**

diff --git a/message/output/jabber/message_output_jabber.php b/message/output/jabber/message_output_jabber.php
index 167e4a2544ac94a7ccf98db549c6f69d63650057..766d504a22785b1d81799d2fe9108f4bf3a9127e 100644 (file)
--- a/message/output/jabber/message_output_jabber.php
+++ b/message/output/jabber/message_output_jabber.php
@@ -83,10 +83,10 @@ class message_output_jabber extends message_output {
      * @param object $mform preferences form class
      */
     function config_form($preferences){
-        $dest = get_string('jabber', 'messageprocessor_jabber');
-        echo '<tr><td colspan="2"><b>'.get_string('processortag', 'message').$dest.'</b></td></tr>'."\n";
-        echo '<tr><td align="right">Jabber ID</td><td><input name="jabber_jabberid" value="'.$preferences->jabber_jabberid.'" /></td></tr>'."\n";
-        return true;
+        echo '<fieldset id="messageprocessor_email" class="clearfix">';
+        echo '<legend class="ftoggler">'.get_string('jabber', 'messageprocessor_jabber').'</legend>';
+        echo get_string('jabberid', 'messageprocessor_jabber').': <input name="jabber_jabberid" value="'.$preferences->jabber_jabberid.'" />';
+        echo '</fieldset>';
     }
 
     /**

diff --git a/message/output/lib.php b/message/output/lib.php
index 7e390a6b72c32ede586553954178ccf31ab60a79..be8c56c879b151a073ee111fb90199e99a0495b6 100644 (file)
--- a/message/output/lib.php
+++ b/message/output/lib.php
@@ -39,6 +39,7 @@ abstract class message_output {
     public abstract function send_message($message);
     public abstract function process_form($form, &$preferences);
     public abstract function load_data(&$preferences, $userid);
+    public abstract function config_form($preferences);
 }
 
 

diff --git a/message/output/popup/message_output_popup.php b/message/output/popup/message_output_popup.php
index 799da5b5e93812c197daed780cc49724ed58572c..0a57a8ec6bfed7a659d6c6f326e42d812b77f242 100644 (file)
--- a/message/output/popup/message_output_popup.php
+++ b/message/output/popup/message_output_popup.php
@@ -45,7 +45,7 @@ class message_output_popup extends message_output{
      * @param object $message the message to be sent
      * @return true if ok, false if error
      */
-    public function send_message($message){
+    public function send_message($message) {
         global $DB;
         
         //put the process record into db
@@ -62,14 +62,30 @@ class message_output_popup extends message_output{
         return true;
     }
     
-    function config_form($preferences){
-        return true;
+    function config_form($preferences) {
+        echo '<fieldset id="messageprocessor_popup" class="clearfix">';
+        echo '<legend class="ftoggler">'.get_string('popup', 'messageprocessor_popup').'</legend>';
+        echo '<table>';
+        echo '<tr><td>'.get_string('showmessagewindow', 'message').'</td><td><input type="checkbox" name="showmessagewindow" '.($preferences->showmessagewindow==1?" checked=\"checked\"":"").' /></td></tr>';
+        echo '<tr><td>'.get_string('blocknoncontacts', 'message').'</td><td><input type="checkbox" name="blocknoncontacts" '.($preferences->blocknoncontacts==1?" checked=\"checked\"":"").' /></td></tr>';
+        echo '<tr><td>'.get_string('beepnewmessage', 'message').'</td><td><input type="checkbox" name="beepnewmessage" '.($preferences->beepnewmessage==1?" checked=\"checked\"":"").' /></td></tr>';
+        echo '<tr><td>'.get_string('noframesjs', 'message').'</td><td><input type="checkbox" name="noframesjs" '.($preferences->noframesjs==1?" checked=\"checked\"":"").' /></td></tr>';
+        echo '</table>';
+        echo '</fieldset>';
     }
     
-    public function process_form($form, &$preferences){
+    public function process_form($form, &$preferences) {
+        $preferences['message_showmessagewindow'] = $form->showmessagewindow?1:0;
+        $preferences['message_blocknoncontacts']  = $form->blocknoncontacts?1:0;
+        $preferences['message_beepnewmessage']    = $form->beepnewmessage?1:0;
+        $preferences['message_noframesjs']        = $form->noframesjs?1:0;
         return true;
     }
-    public function load_data(&$preferences, $userid){
+    public function load_data(&$preferences, $userid) {
+        $preferences->showmessagewindow =  get_user_preferences( 'message_showmessagewindow', 1, $user->id);
+        $preferences->blocknoncontacts  =  get_user_preferences( 'message_blocknoncontacts', '', $user->id);
+        $preferences->beepnewmessage    =  get_user_preferences( 'message_beepnewmessage', '', $user->id);
+        $preferences->noframesjs        =  get_user_preferences( 'message_noframesjs', '', $user->id);
         return true;
     }
 }

diff --git a/theme/standard/styles_layout.css b/theme/standard/styles_layout.css
index fbafdbc7d38aa4dbd7b9742032664b35a49148a5..e20572d4eab128d97129ce0f0de6513fd243c083 100644 (file)
--- a/theme/standard/styles_layout.css
+++ b/theme/standard/styles_layout.css
@@ -2569,6 +2569,10 @@ body#message-send td.fixeditor {
   text-align:center;
 }
 
+body#message-edit fieldset {
+  padding: 20px;
+}
+
 /***
  *** Notes
  ***/