Adding reminder

plugins/fortune/setup.php

@@ -13,24 +13,28 @@
  * @version $Id$
  * @package plugins
  * @subpackage fortune
+ *
+ * FIXME
+ * There should be no code in setup.php, just hook registrations. Create functions.php
+ * and move the code there.
  */
 
 /**
  * Init plugin
- * @access private 
+ * @access private
  */
 function squirrelmail_plugin_init_fortune() {
   global $squirrelmail_plugin_hooks;
-  
+
   $squirrelmail_plugin_hooks['mailbox_index_before']['fortune'] = 'fortune';
   $squirrelmail_plugin_hooks['options_display_inside']['fortune'] = 'fortune_options';
   $squirrelmail_plugin_hooks['options_display_save']['fortune'] = 'fortune_save';
-  $squirrelmail_plugin_hooks['loading_prefs']['fortune'] = 'fortune_load';  
+  $squirrelmail_plugin_hooks['loading_prefs']['fortune'] = 'fortune_load';
 }
 
 /**
  * Show fortune
- * @access private 
+ * @access private
  */
 function fortune() {
     global $fortune_visible, $color;
@@ -42,22 +46,22 @@ function fortune() {
     $fortune_location = '/usr/games/fortune';
     $exist = file_exists($fortune_location);
     echo "<center><table cellpadding=\"0\" cellspacing=\"0\" border=\"0\" bgcolor=\"$color[10]\">\n".
-         "<tr><td><table width=\"100%\" cellpadding=\"2\" cellspacing=\"1\" border=\"0\" bgcolor=\"$color[5]\">\n".
-	 "<tr><td align=\"center\">\n";
+        "<tr><td><table width=\"100%\" cellpadding=\"2\" cellspacing=\"1\" border=\"0\" bgcolor=\"$color[5]\">\n".
+        "<tr><td align=\"center\">\n";
     echo '<table><tr><td>';
     if (!$exist) {
         echo "$fortune_location" . _(" not found.");
     } else {
         echo "<center><em>" . _("Today's Fortune") . "</em><br /></font></center><pre>";
         htmlspecialchars(system($fortune_location));
-    } 
-  
+    }
+
     echo '</pre></td></tr></table></td></tr></table></td></tr></table></center>';
 }
 
 /**
  * Get fortune prefs
- * @access private 
+ * @access private
  */
 function fortune_load() {
     global $username, $data_dir, $fortune_visible;
@@ -67,21 +71,21 @@ function fortune_load() {
 
 /**
  * Add fortune options
- * @access private 
+ * @access private
  */
 function fortune_options() {
-  global $fortune_visible;
+    global $fortune_visible;
 
-  echo "<tr>" . html_tag('td',_("Fortunes:"),'right','','nowrap') . "\n";
-  echo '<td><input name="fortune_fortune_visible" type="checkbox"';
-  if ($fortune_visible)
-    echo ' checked="checked"';
-  echo " /> " . _("Show fortunes at top of mailbox") . "</td></tr>\n";
+    echo "<tr>" . html_tag('td',_("Fortunes:"),'right','','nowrap') . "\n";
+    echo '<td><input name="fortune_fortune_visible" type="checkbox"';
+    if ($fortune_visible)
+        echo ' checked="checked"';
+    echo " /> " . _("Show fortunes at top of mailbox") . "</td></tr>\n";
 }
 
 /**
  * Save fortune prefs
- * @access private 
+ * @access private
  */
 function fortune_save() {
     global $username,$data_dir;
@@ -93,4 +97,4 @@ function fortune_save() {
     }
 }
 
-?>
+?>