do not special case a folder named 'None'. There's no reason
authorkink <kink@7612ce4b-ef26-0410-bec9-ea0150e637f0>
Tue, 6 Mar 2007 14:57:12 +0000 (14:57 +0000)
committerkink <kink@7612ce4b-ef26-0410-bec9-ea0150e637f0>
Tue, 6 Mar 2007 14:57:12 +0000 (14:57 +0000)
to not use the empty string and this prevents people who want
to have a "None" folder from using it properly.

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@12315 7612ce4b-ef26-0410-bec9-ea0150e637f0

38 files changed:
ChangeLog
functions/imap_mailbox.php
functions/page_header.php
help/en_US/options.hlp
plugins/abook_take/take.php
plugins/administrator/options.php
plugins/bug_report/bug_report.php
plugins/calendar/calendar.php
plugins/calendar/day.php
plugins/calendar/event_create.php
plugins/calendar/event_delete.php
plugins/calendar/event_edit.php
plugins/change_password/options.php
plugins/filters/options.php
plugins/filters/spamoptions.php
plugins/info/options.php
plugins/listcommands/options.php
plugins/mail_fetch/fetch.php
plugins/mail_fetch/options.php
plugins/newmail/newmail_opt.php
plugins/spamcop/options.php
plugins/squirrelspell/sqspell_functions.php
plugins/translate/options.php
src/about.php
src/addressbook.php
src/compose.php
src/folders.php
src/help.php
src/image.php
src/left_main.php
src/options.php
src/options_highlight.php
src/options_identities.php
src/options_order.php
src/search.php
src/vcard.php
src/view_text.php
templates/default/page_header.tpl

index e7ee71f..fb4ce22 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -186,6 +186,7 @@ Version 1.5.2 - SVN
     better-than-nothing alternative.
   - Fix Priority and Receipt compose options being reset after return from
     HTML addressbook, and allow returning from an empty address book (#1673056).
+  - Do not special case the 'None' folder.
 
 Version 1.5.1 (branched on 2006-02-12)
 --------------------------------------
index 5d0e5ab..e7c4130 100755 (executable)
@@ -360,8 +360,7 @@ function sqimap_mailbox_exists ($imap_stream, $mailbox, $mailboxlist=null) {
  * @since 1.0 or older
  */
 function sqimap_mailbox_select ($imap_stream, $mailbox) {
-    // FIX ME: WHAAAA DO NOT USE "None" for something that does not exist. Use false or NULL instead
-    if ($mailbox == 'None') {
+    if (empty($mailbox)) {
         return;
     }
 
index 530774a..c181479 100644 (file)
@@ -196,7 +196,7 @@ function displayInternalLink($path, $text, $target='') {
  * @return void
  */
 
-function displayPageHeader($color, $mailbox, $sHeaderJs='', $sBodyTagJs = '') {
+function displayPageHeader($color, $mailbox='', $sHeaderJs='', $sBodyTagJs = '') {
 
     global $reply_focus, $hide_sm_attributions, $frame_top,
         $provider_name, $provider_uri, $startMessage,
index 9cf3346..31d6508 100644 (file)
       messages, the INBOX will become bold and a number will appear to the right of it
       to say how many new messages are in it.  If you set it to All Folders, this 
       behavior will happen on all folders.  If you notice that loading the folder list
-      is really slow, you can set this to INBOX or None and that should speed it up.
+      is really slow, you can set this to INBOX and that should speed it up.
       <br /><br />
       
       <b>Unseen message notification type</b><br />
index 1f9c8d5..3ac2c50 100644 (file)
@@ -22,7 +22,7 @@ require('../../include/init.php');
 require(SM_PATH . 'functions/forms.php');
 require(SM_PATH . 'functions/addressbook.php');
 
-displayPageHeader($color, 'None');
+displayPageHeader($color);
 
 /* input form data */
 sqgetGlobalVar('email', $email, SQ_POST);
@@ -45,4 +45,4 @@ $formdata=array('email'=>$addrs);
 abook_create_form(SM_PATH . 'src/addressbook.php','addaddr',_("Add to address book"),_("Add address"),$formdata);
 echo '</form>';
 ?>
-</body></html>
\ No newline at end of file
+</body></html>
index 15a1c4b..57e631b 100644 (file)
@@ -233,7 +233,7 @@ if ( !adm_check_user() ) {
     exit;
 }
 
-displayPageHeader($color, 'None');
+displayPageHeader($color);
 
 $newcfg = array( );
 
@@ -646,4 +646,4 @@ if ( $fp = @fopen( $cfgfile, 'w' ) ) {
 }
 
 ?>
-</body></html>
\ No newline at end of file
+</body></html>
index c62498c..2e84758 100644 (file)
@@ -24,7 +24,7 @@ require_once(SM_PATH . 'functions/forms.php');
 /** load plugin functions */
 require_once(SM_PATH . 'plugins/bug_report/functions.php');
 
-displayPageHeader($color, 'None');
+displayPageHeader($color);
 
 /** is bug_report plugin disabled or called by wrong user */
 if (! is_plugin_enabled('bug_report') || ! bug_report_check_user()) {
@@ -131,4 +131,4 @@ echo addSubmit(_("Start Bug Report Form"));
     </form>
     <br />
   </body>
-</html>
\ No newline at end of file
+</html>
index dae2a87..f76e2d0 100644 (file)
@@ -169,7 +169,7 @@ if( !isset($day) || $day <= 0){
 $todayis = date( 'mdY' );
 $calself=basename($PHP_SELF);
 
-displayPageHeader($color, 'None');
+displayPageHeader($color);
 calendar_header();
 readcalendardata();
 startcalendar();
@@ -177,4 +177,4 @@ drawmonthview();
 endcalendar();
 
 ?>
-</body></html>
\ No newline at end of file
+</body></html>
index a2a2049..40e4a4d 100644 (file)
@@ -169,7 +169,7 @@ $calself=basename($PHP_SELF);
 
 $daily_events = array();
 
-displayPageHeader($color, 'None');
+displayPageHeader($color);
 calendar_header();
 readcalendardata();
 day_header();
@@ -177,4 +177,4 @@ initialize_events();
 display_events();
 ?>
 </table></td></tr></table>
-</body></html>
\ No newline at end of file
+</body></html>
index 86fe2e8..db2347c 100644 (file)
@@ -122,7 +122,7 @@ if (!isset($hour) || $hour <= 0){
 $calself=basename($PHP_SELF);
 
 
-displayPageHeader($color, 'None');
+displayPageHeader($color);
 //load calendar menu
 calendar_header();
 
@@ -175,4 +175,4 @@ if(!isset($event_text)){
 
 ?>
 </table></td></tr></table>
-</body></html>
\ No newline at end of file
+</body></html>
index dc90191..79867ae 100644 (file)
@@ -119,7 +119,7 @@ if ($day <= 0){
 
 $calself=basename($PHP_SELF);
 
-displayPageHeader($color, 'None');
+displayPageHeader($color);
 //load calendar menu
 calendar_header();
 
@@ -145,4 +145,4 @@ if (isset($dyear) && isset($dmonth) && isset($dday) && isset($dhour) && isset($d
 
 ?>
 </table></td></tr></table>
-</body></html>
\ No newline at end of file
+</body></html>
index 7d38c64..5b94a1b 100644 (file)
@@ -255,7 +255,7 @@ if ($hour <= 0){
 
 $calself=basename($PHP_SELF);
 
-displayPageHeader($color, 'None');
+displayPageHeader($color);
 //load calendar menu
 calendar_header();
 
@@ -311,4 +311,4 @@ if (!isset($updated)){
 
 ?>
 </table></td></tr></table>
-</body></html>
\ No newline at end of file
+</body></html>
index 15b674e..9b72d6c 100644 (file)
@@ -62,7 +62,7 @@ if(sqgetGlobalVar('cpw_go', $cpw_go, SQ_POST)) {
     }
 }
 
-displayPageHeader($color, 'None');
+displayPageHeader($color);
 
 do_hook('change_password_init', $null);
 ?>
index 8056a83..9cc1e54 100644 (file)
@@ -18,7 +18,7 @@ include_once(SM_PATH . 'functions/imap_general.php');
 include_once(SM_PATH . 'functions/forms.php');
 include_once(SM_PATH . 'plugins/filters/filters.php');
 
-displayPageHeader($color, 'None');
+displayPageHeader($color);
 
 /* get globals */
 sqgetGlobalVar('delimiter', $delimiter, SQ_SESSION);
index bcef8e5..16b2583 100644 (file)
@@ -25,7 +25,7 @@ sqgetGlobalVar('delimiter', $delimiter, SQ_SESSION);
 sqgetGlobalVar('action', $action, SQ_GET);
 /* end globals */
 
-displayPageHeader($color, 'None');
+displayPageHeader($color);
 
 if (sqgetGlobalVar('spam_submit',$spam_submit,SQ_POST)) {
     $spam_filters = load_spam_filters();
index 79fa0d1..d09ee6c 100644 (file)
@@ -26,7 +26,7 @@ require_once(SM_PATH . 'plugins/info/functions.php');
 
 global $username, $color, $folder_prefix, $default_charset;
 $default_charset = strtoupper($default_charset);
-displayPageHeader($color, 'None');
+displayPageHeader($color);
 $mailbox = 'INBOX';
 
 /**
index 26e4ec2..67dfc01 100644 (file)
@@ -70,7 +70,7 @@ if (sqGetGlobalVar('addlist', $ignore, SQ_FORM)
 
 
 
-displayPageHeader($color, 'None');
+displayPageHeader($color);
 
 $oTemplate->assign('lists', $lists);
 $oTemplate->display('plugins/listcommands/non_rfc_lists.tpl');
index 918fe29..17711ec 100644 (file)
@@ -119,7 +119,7 @@ function Mail_Fetch_Select_Server($mailfetch) {
 }
 
 $mailfetch = Mail_Fetch_Servers();
-displayPageHeader($color, 'None');
+displayPageHeader($color);
 
 echo '<br />';
 
index 81780f1..7931bb6 100644 (file)
@@ -55,7 +55,7 @@ sqgetGlobalVar('submit_mailfetch', $submit_mailfetch, SQ_POST);
 
 /* end globals */
 
-displayPageHeader( $color, 'None' );
+displayPageHeader( $color );
 
 switch( $mf_action ) {
  case 'add':
index 5b194f3..24678ee 100644 (file)
@@ -23,7 +23,7 @@ include_once(SM_PATH . 'plugins/newmail/functions.php');
 
 include_once(SM_PATH . 'functions/forms.php');
 
-displayPageHeader($color, 'None');
+displayPageHeader($color);
 
 echo html_tag( 'table', '', 'center', $color[0], 'width="95%" cellpadding="1" cellspacing="0" border="0"' ) . "\n" .
         html_tag( 'tr' ) . "\n" .
index f30987a..8e18618 100755 (executable)
@@ -18,7 +18,7 @@ require('../../include/init.php');
 /* plugin functions */
 include_once(SM_PATH . 'plugins/spamcop/functions.php');
 
-displayPageHeader($color, 'None');
+displayPageHeader($color);
 
 /** is spamcop plugin disabled */
 // option changes do nothing, if read_body_header_right hook is not active.
@@ -194,4 +194,4 @@ echo '<p><b>' . _("More information") . '</b><br />';
 printf(_("For more information about SpamCop, it's services, spam in general, and many related topics, try reading through SpamCop's %sHelp and Feedback%s section."), '<a href="http://spamcop.net/help.shtml">', '</a>');
 echo "</p>\n";
 ?>
-</body></html>
\ No newline at end of file
+</body></html>
index 4044f23..7f8d771 100644 (file)
@@ -125,7 +125,7 @@ function sqspell_makePage($title, $scriptsrc, $body){
       $MOD = 'options_main';
   }
 
-  displayPageHeader($color, 'None');
+  displayPageHeader($color);
   echo "&nbsp;<br />\n";
   /**
    * Check if we need to link in a script.
index 7974d6f..7e18ef2 100644 (file)
@@ -20,7 +20,7 @@ require('../../include/init.php');
 /** Plugin functions */
 include_once(SM_PATH . 'plugins/translate/functions.php');
 
-displayPageHeader($color, 'None');
+displayPageHeader($color);
 
 ?>
    <table width="95%" align="center" border="0" cellpadding="1" cellspacing="0"><tr><td bgcolor="<?php echo $color[0]; ?>">
@@ -78,4 +78,4 @@ if (!$disable_compose_translate) {
 </td></tr>
 </table>
 </form>
-</body></html>
\ No newline at end of file
+</body></html>
index 98ed98d..5a12188 100644 (file)
@@ -21,7 +21,7 @@
  */
 require('../include/init.php');
 
-displayPageHeader($color, 'None' );
+displayPageHeader($color);
 
 ?>
 <p align="center">
index bfb0a25..69e45f7 100644 (file)
@@ -40,7 +40,7 @@ sqgetGlobalVar('doedit',        $doedit,        SQ_POST);
 $abook_sort_order = get_abook_sort();
 
 /* Create page header before addressbook_init in order to  display error messages correctly. */
-displayPageHeader($color, 'None');
+displayPageHeader($color);
 
 /* Open addressbook with error messages on.
  remote backends (LDAP) are enabled because they can be used. (list_addr function)
index 9e72e91..2da8690 100644 (file)
@@ -383,7 +383,7 @@ if (!isset($compose_messages[$session]) || ($compose_messages[$session] == NULL)
     $composeMessage=$compose_messages[$session];
 }
 
-if (!isset($mailbox) || $mailbox == '' || ($mailbox == 'None')) {
+if (empty($mailbox)) {
     $mailbox = 'INBOX';
 }
 
index 91e41d7..ae3c896 100644 (file)
@@ -22,7 +22,7 @@ require_once(SM_PATH . 'functions/imap_general.php');
 require_once(SM_PATH . 'functions/folder_manip.php');
 require_once(SM_PATH . 'functions/forms.php');
 
-displayPageHeader($color, 'None');
+displayPageHeader($color);
 
 /* get globals we may need */
 sqgetGlobalVar('delimiter', $delimiter, SQ_SESSION);
index fd0525e..ebfa81b 100644 (file)
@@ -16,7 +16,7 @@
  */
 require('../include/init.php');
 
-displayPageHeader($color, 'None' );
+displayPageHeader($color);
 
 $helpdir[0] = 'basic.hlp';
 $helpdir[1] = 'main_folder.hlp';
index e1a2680..2bf417c 100644 (file)
@@ -16,7 +16,7 @@
  */
 require('../include/init.php');
 
-displayPageHeader($color, 'None');
+displayPageHeader($color);
 
 /* globals */
 if ( sqgetGlobalVar('passed_id', $temp, SQ_GET) ) {
index 3805ec1..5b1cf17 100644 (file)
@@ -69,7 +69,7 @@ if ($auto_create_special && !isset($auto_create_done)) {
     $autocreate = array($sent_folder, $trash_folder, $draft_folder);
     $folders_created = false;
     foreach( $autocreate as $folder ) {
-        if (($folder != '') && ($folder != 'none')) {
+        if ($folder != '') {
             /**
              * If $show_only_subscribed_folders is true, don't use 
              * $mailboxes array for checking if mailbox exists.
index 499a435..4350c59 100644 (file)
@@ -237,7 +237,7 @@ if ($optmode == SMOPT_MODE_SUBMIT) {
 /* Finally, display whatever page we are supposed to show now. */
 /***************************************************************/
 
-displayPageHeader($color, 'None', (isset($optpage_data['xtra']) ? $optpage_data['xtra'] : ''));
+displayPageHeader($color, null, (isset($optpage_data['xtra']) ? $optpage_data['xtra'] : ''));
 
 /*
  * The main option page has a different layout then the rest of the option
index c7678ca..1b9768b 100644 (file)
@@ -104,7 +104,7 @@ if (isset($theid) && ($action == 'delete') ||
 
     setPref($data_dir, $username, 'hililist', serialize($message_highlight_list));
 }
-displayPageHeader($color, 'None');
+displayPageHeader($color);
 
 /**
  * Display the current rule list
index 7484948..7a49817 100644 (file)
@@ -61,7 +61,7 @@ if (!empty($return)) {
     exit;
 }
 
-displayPageHeader($color, 'None');
+displayPageHeader($color);
 
 /* since 1.1.3 */
 do_hook('options_identities_top', $null);
index b3559bd..8f081ac 100644 (file)
@@ -73,7 +73,7 @@ if (!sqgetGlobalVar('account', $account,  SQ_GET)) {
 /* Finally, display whatever page we are supposed to show now. */
 /***************************************************************/
 
-displayPageHeader($color, 'None', (isset($optpage_data['xtra']) ? $optpage_data['xtra'] : ''));
+displayPageHeader($color, null, (isset($optpage_data['xtra']) ? $optpage_data['xtra'] : ''));
 
 
 /**
index c2a1054..1606105 100644 (file)
@@ -1312,7 +1312,7 @@ $imapConnection = sqimap_login($username, false, $imapServerAddress, $imapPort,
 $boxes = sqimap_mailbox_list($imapConnection);
 /* ensure we have a valid default mailbox name */
 $mailbox = asearch_nz($mailbox_array[0]);
-if (($mailbox == '') || ($mailbox == 'None')) //Workaround for sm quirk IMHO (what if I really have a mailbox called None?)
+if ($mailbox == '')
     $mailbox = $boxes[0]['unformatted']; //Usually INBOX ;)
 
 
index 7589bfe..b9766fb 100644 (file)
@@ -41,7 +41,7 @@ sqgetGlobalVar('startMessage', $startMessage, SQ_GET);
 $imapConnection = sqimap_login($username, false, $imapServerAddress, $imapPort, 0);
 sqimap_mailbox_select($imapConnection, $mailbox);
 
-displayPageHeader($color, 'None');
+displayPageHeader($color);
 
 $msg_url = 'read_body.php?mailbox='.urlencode($mailbox).
     '&amp;startMessage='.urlencode($startMessage).
@@ -185,4 +185,4 @@ $oTemplate->assign('info', $opts);
 $oTemplate->display('vcard.tpl');
 
 $oTemplate->display('footer.tpl');
-?>
\ No newline at end of file
+?>
index 8f43447..ce84feb 100644 (file)
@@ -75,7 +75,7 @@ if ($type1 == 'html' || (isset($override_type1) &&  $override_type1 == 'html'))
     translateText($body, $wrap_at, $charset);
 }
 
-displayPageHeader($color, 'None');
+displayPageHeader($color);
 
 $oTemplate->assign('view_message_href', $msg_url);
 $oTemplate->assign('download_href', $dwnld_url);
index 3860ac1..99a4de4 100644 (file)
@@ -17,7 +17,7 @@ extract($t);
 
 
 $current_folder_str = '';
-if ( $shortBoxName <> '' && strtolower( $shortBoxName ) <> 'none' ) {
+if ( $shortBoxName != '' ) {
     $current_folder_str .= _("Current Folder") . ": <em>$shortBoxName&nbsp;</em>\n";
 } else {
     $current_folder_str .= '&nbsp;';