Begin some re-orginization of UI controls. Remove support for $theme_css, since...
[squirrelmail.git] / config / conf.pl
index 7ad551e64a43511bb637b8051939d288273a11bb..d87b702e8e334753792111253d52f3a017a59d80 100755 (executable)
@@ -188,14 +188,26 @@ while ( $line = <FILE> ) {
             $sub =~ s/\]\[['"]NAME['"]\]//;
             $sub =~ s/.*\[//;
             $theme_name[$sub] = $options[1];
-        } elsif ( $options[0] =~ /^aTemplateSet\[[0-9]+\]\[['"]PATH['"]\]/ ) {
+        } elsif ( $options[0] =~ /^aTemplateSet\[[0-9]+\]\[['"]ID['"]\]/ ) {
             $sub = $options[0];
-            $sub =~ s/\]\[['"]PATH['"]\]//;
+            $sub =~ s/\]\[['"]ID['"]\]//;
             $sub =~ s/.*\[//;
             if ( -e "../templates" ) {
                 $options[1] =~ s/^\.\.\/config/\.\.\/templates/;
             }
-            $templateset_path[$sub] = &change_to_rel_path($options[1]);
+            $templateset_id[$sub] = $options[1];
+##### FIXME: This section BELOW here so old prefs files don't blow up when running conf.pl
+#####        Remove after a month or two 
+} elsif ( $options[0] =~ /^aTemplateSet\[[0-9]+\]\[['"]PATH['"]\]/ ) {
+    $sub = $options[0];
+    $sub =~ s/\]\[['"]PATH['"]\]//;
+    $sub =~ s/.*\[//;
+    if ( -e "../templates" ) {
+        $options[1] =~ s/^\.\.\/config/\.\.\/templates/;
+    }
+    $templateset_id[$sub] = $options[1];
+##### FIXME: This section ABOVE here so old prefs files don't blow up when running conf.pl
+#####        Remove after a month or two 
         } elsif ( $options[0] =~ /^aTemplateSet\[[0-9]+\]\[['"]NAME['"]\]/ ) {
             $sub = $options[0];
             $sub =~ s/\]\[['"]NAME['"]\]//;
@@ -318,7 +330,7 @@ while ( $line = <FILE> ) {
             $ldap_writeable[$sub] = $writeable;
             $ldap_search_tree[$sub] = $search_tree;
             $ldap_starttls[$sub] = $starttls;
-        } elsif ( $options[0] =~ /^(data_dir|attachment_dir|theme_css|org_logo|signout_page)$/ ) {
+        } elsif ( $options[0] =~ /^(data_dir|attachment_dir|theme_css|org_logo|signout_page|icon_theme_def)$/ ) {
             ${ $options[0] } = &change_to_rel_path($options[1]);
         } else {
             ${ $options[0] } = $options[1];
@@ -345,8 +357,6 @@ $noselect_fix_enable = 'false'          if ( !$noselect_fix_enable );
 $frame_top = "_top"                     if ( !$frame_top );
 $provider_uri = ''                      if ( !$provider_uri );
 $provider_name = ''                     if ( !$provider_name );
-$edit_identity = 'true'                 if ( !$edit_identity );
-$edit_name = 'true'                     if ( !$edit_name );
 $no_list_for_subscribe = 'false'        if ( !$no_list_for_subscribe );
 $allow_charset_search = 'true'          if ( !$allow_charset_search );
 $allow_advanced_search = 0              if ( !$allow_advanced_search) ;
@@ -359,6 +369,9 @@ $default_use_javascript_addr_book = 'false' if (! $default_use_javascript_addr_b
 
 # since 1.2.0
 $hide_sm_attributions = 'false'         if ( !$hide_sm_attributions );
+# since 1.2.5
+$edit_identity = 'true'                 if ( !$edit_identity );
+$edit_name = 'true'                     if ( !$edit_name );
 
 # since 1.4.0
 $use_smtp_tls= 'false'                  if ( !$use_smtp_tls);
@@ -395,6 +408,9 @@ $prefs_user_size = 128                  if ( !$prefs_user_size );
 $prefs_key_size = 64                    if ( !$prefs_key_size );
 $prefs_val_size = 65536                 if ( !$prefs_val_size );
 
+# since 1.5.2
+$icon_theme_def = ''                                   if ( !$icon_theme_def );
+
 # add qmail-inject test here for backwards compatibility
 if ( !$sendmail_args && $sendmail_path =~ /qmail-inject/ ) {
     $sendmail_args = '';
@@ -423,6 +439,9 @@ $disable_server_sort = 'false'         if ( !$disable_server_sort );
 
 # since 1.5.2
 $abook_file_line_length = 2048         if ( !$abook_file_line_length );
+$config_location_base = ''             if ( !$config_location_base );
+$smtp_sitewide_user = ''               if ( !$smtp_sitewide_user );
+$smtp_sitewide_pass = ''               if ( !$smtp_sitewide_pass );
 
 if ( $ARGV[0] eq '--install-plugin' ) {
     print "Activating plugin " . $ARGV[1] . "\n";
@@ -465,7 +484,7 @@ while ( ( $command ne "q" ) && ( $command ne "Q" ) && ( $command ne ":q" ) ) {
         print "2.  Server Settings\n";
         print "3.  Folder Defaults\n";
         print "4.  General Options\n";
-        print "5.  Templates\n";
+        print "5.  User Interface\n";
         print "6.  Address Books\n";
         print "7.  Message of the Day (MOTD)\n";
         print "8.  Plugins\n";
@@ -524,7 +543,7 @@ while ( ( $command ne "q" ) && ( $command ne "Q" ) && ( $command ne ":q" ) ) {
             print "4.   SMTP Server           : $WHT$smtpServerAddress$NRM\n";
             print "5.   SMTP Port             : $WHT$smtpPort$NRM\n";
             print "6.   POP before SMTP       : $WHT$pop_before_smtp$NRM\n";
-            print "7.   SMTP Authentication   : $WHT$smtp_auth_mech$NRM\n";
+            print "7.   SMTP Authentication   : $WHT$smtp_auth_mech" . display_smtp_sitewide_userpass() ."$NRM\n";
             print "8.   Secure SMTP (TLS)     : $WHT" . display_use_tls($use_smtp_tls) . "$NRM\n";
             print "9.   Header encryption key : $WHT$encode_header_key$NRM\n";
             print "\n";
@@ -596,21 +615,17 @@ while ( ( $command ne "q" ) && ( $command ne "Q" ) && ( $command ne ":q" ) ) {
         print "13. Allow advanced search       : $WHT$allow_advanced_search$NRM\n";
         print "14. PHP session name            : $WHT$session_name$NRM\n";
         print "15. Time zone configuration     : $WHT$time_zone_type$NRM\n";
+        print "16. Location base               : $WHT$config_location_base$NRM\n";
         print "\n";
         print "R   Return to Main Menu\n";
     } elsif ( $menu == 5 ) {
-        print $WHT. "Themes\n" . $NRM;
-        print "1.  Change Template set\n";
-#        for ( $count = 0 ; $count <= $#theme_name/2 ; $count++ ) {
-#            $temp_name = $theme_name[$count*2];
-#            printf "     %s%*s    %s\n", $temp_name,
-#                   40 - length($temp_name), " ",
-#                   $theme_name[($count*2)+1];
-#        }
-        print "2.  CSS File : $WHT$theme_css$NRM\n";
-        print "3.  Default font size: $WHT$default_fontsize$NRM\n";
-        print "4.  Change available font sets\n";
-        print "5.  Change Themes\n";
+        print $WHT. "User Interface\n" . $NRM;
+        print "1.  Modify Template sets\n";
+        print "2.  Use Icons                    : $WHT$use_icons$NRM\n";
+        print "3.  Default Icon Set             : $WHT$icon_theme_def$NRM\n";
+        print "4.  Default font size            : $WHT$default_fontsize$NRM\n";
+        print "5.  Modify available font sets\n";
+#        print "4.  Change Themes\n";
 
         print "\n";
         print "R   Return to Main Menu\n";
@@ -699,14 +714,13 @@ while ( ( $command ne "q" ) && ( $command ne "Q" ) && ( $command ne ":q" ) ) {
     } elsif ( $menu == 11 ) {
     print $WHT. "Interface tweaks\n" . $NRM;
     print "1.  Display html mails in iframe : $WHT$use_iframe$NRM\n";
-    print "2.  Use Icons                    : $WHT$use_icons$NRM\n";
     print "\n";
     print $WHT. "PHP tweaks\n" . $NRM;
-    print "3.  Use php recode functions     : $WHT$use_php_recode$NRM\n";
-    print "4.  Use php iconv functions      : $WHT$use_php_iconv$NRM\n";
+    print "4.  Use php recode functions     : $WHT$use_php_recode$NRM\n";
+    print "5.  Use php iconv functions      : $WHT$use_php_iconv$NRM\n";
     print "\n";
     print $WHT. "Configuration tweaks\n" . $NRM;
-    print "5.  Allow remote configtest     : $WHT$allow_remote_configtest$NRM\n";
+    print "6.  Allow remote configtest     : $WHT$allow_remote_configtest$NRM\n";
     print "\n";
         print "R   Return to Main Menu\n";
     }
@@ -831,12 +845,13 @@ while ( ( $command ne "q" ) && ( $command ne "Q" ) && ( $command ne ":q" ) ) {
             elsif ( $command == 13 ) { $allow_advanced_search    = command316(); }
             elsif ( $command == 14 ) { $session_name             = command317(); }
             elsif ( $command == 15 ) { $time_zone_type           = command318(); }
+            elsif ( $command == 16 ) { $config_location_base     = command_config_location_base(); }
         } elsif ( $menu == 5 ) {
             if ( $command == 1 ) { $templateset_default = command_templates(); }
-            elsif ( $command == 2 ) { $theme_css = command42(); }
-            elsif ( $command == 3 ) { $default_fontsize = command_default_fontsize(); }
-            elsif ( $command == 4 ) { command_fontsets(); }
-            elsif ( $command == 5 ) { command41(); }
+            elsif ( $command == 2 ) { $use_icons      = commandB3(); }
+            elsif ( $command == 3 ) { $icon_theme_def = commandB7(); }
+            elsif ( $command == 4 ) { $default_fontsize = command_default_fontsize(); }
+            elsif ( $command == 5 ) { command_fontsets(); }
         } elsif ( $menu == 6 ) {
             if    ( $command == 1 ) { command61(); }
             elsif ( $command == 2 ) { command62(); }
@@ -868,10 +883,9 @@ while ( ( $command ne "q" ) && ( $command ne "Q" ) && ( $command ne ":q" ) ) {
             elsif ( $command == 5 ) { $lossy_encoding                = commandA5(); }
         } elsif ( $menu == 11 ) {
             if    ( $command == 1 ) { $use_iframe     = commandB2(); }
-            elsif ( $command == 2 ) { $use_icons      = commandB3(); }
-            elsif ( $command == 3 ) { $use_php_recode = commandB4(); }
-            elsif ( $command == 4 ) { $use_php_iconv  = commandB5(); }
-            elsif ( $command == 5 ) { $allow_remote_configtest = commandB6(); }
+            elsif ( $command == 4 ) { $use_php_recode = commandB4(); }
+            elsif ( $command == 5 ) { $use_php_iconv  = commandB5(); }
+            elsif ( $command == 6 ) { $allow_remote_configtest = commandB6(); }
         }
     }
 }
@@ -962,8 +976,6 @@ sub command3 {
     print "If your Organization Title includes a '\$', please precede it with a \\. \n";
     print "Other '\$' will be considered the beginning of a variable that\n";
     print "must be defined before the \$org_title is printed.\n";
-    print "\$version, for example can be used, and will print the\n";
-    print "string representing the current SquirrelMail version.\n";
     print "\n";
     print "[$WHT$org_title$NRM]: $WHT";
     $new_org_title = <STDIN>;
@@ -1171,26 +1183,6 @@ sub command17 {
     return $new_smtpPort;
 }
 
-# authenticated server
-sub command18 {
-    return;
-    # This sub disabled by tassium - it has been replaced with smtp_auth_mech
-    print "Do you wish to use an authenticated SMTP server?  Your server must\n";
-    print "support this in order for SquirrelMail to work with it.  We implemented\n";
-    print "it according to RFC 2554.\n";
-
-    $YesNo = 'n';
-    $YesNo = 'y' if ( lc($use_authenticated_smtp) eq 'true' );
-
-    print "Use authenticated SMTP server (y/n) [$WHT$YesNo$NRM]: $WHT";
-
-    $new_use_authenticated_smtp = <STDIN>;
-    $new_use_authenticated_smtp =~ tr/yn//cd;
-    return 'true'  if ( $new_use_authenticated_smtp eq "y" );
-    return 'false' if ( $new_use_authenticated_smtp eq "n" );
-    return $use_authenticated_smtp;
-}
-
 # pop before SMTP
 sub command18a {
     print "Do you wish to use POP3 before SMTP?  Your server must\n";
@@ -1284,8 +1276,11 @@ sub command111 {
 # Now offers to detect supported mechs, assuming server & port are set correctly
 
 sub command112a {
-    if ($use_imap_tls =~ /^true\b/i) {
-        print "Auto-detection of login methods is unavailable when using TLS.\n";
+    if ($use_imap_tls ne "0") {
+        # 1. Script does not handle TLS.
+        # 2. Server does not have to declare all supported authentication mechs when 
+        #    STARTTLS is used. Supported mechs are declared only after STARTTLS.
+        print "Auto-detection of login methods is unavailable when using TLS or STARTTLS.\n";
     } else {
         print "If you have already set the hostname and port number, I can try to\n";
         print "detect the mechanisms your IMAP server supports.\n";
@@ -1347,9 +1342,10 @@ sub command112a {
 # SMTP authentication type
 # Possible choices: none, plain, cram-md5, digest-md5
 sub command112b {
-    if ($use_smtp_tls =~ /^true\b/i) {
-        print "Auto-detection of login methods is unavailable when using TLS.\n";
-    } else {
+    if ($use_smtp_tls ne "0") {
+        print "Auto-detection of login methods is unavailable when using TLS or STARTTLS.\n";
+    } elsif (eval ("use IO::Socket; 1")) {
+        # try loading IO::Socket module
         print "If you have already set the hostname and port number, I can try to\n";
         print "automatically detect some of the mechanisms your SMTP server supports.\n";
         print "Auto-detection is *optional* - you can safely say \"n\" here.\n";
@@ -1363,7 +1359,6 @@ sub command112b {
             # Special case!
             # Check none by trying to relay to junk@microsoft.com
             $host = $smtpServerAddress . ':' . $smtpPort;
-            use IO::Socket;
             my $sock = IO::Socket::INET->new($host);
             print "Testing none:\t\t$WHT";
             if (!defined($sock)) {
@@ -1437,18 +1432,96 @@ sub command112b {
     $inval=<STDIN>;
     chomp($inval);
     if ($inval =~ /^none\b/i) {
-      # SMTP doesn't necessarily require logins
-      return "none";
+        # remove sitewide smtp authentication information
+        $smtp_sitewide_user = '';
+        $smtp_sitewide_pass = '';
+        # SMTP doesn't necessarily require logins
+        return "none";
+    } elsif ( ($inval =~ /^cram-md5\b/i) || ($inval =~ /^digest-md5\b/i) ||
+              ($inval =~ /^login\b/i) || ($inval =~/^plain\b/i)) {
+        command_smtp_sitewide_userpass($inval);
+        return lc($inval);
+    } elsif (trim($inval) eq '') {
+        # user selected default value
+        command_smtp_sitewide_userpass($smtp_auth_mech);
+        return $smtp_auth_mech;
+    } else {
+        # user entered garbage 
+        return $smtp_auth_mech;
+    }
+}
+
+sub command_smtp_sitewide_userpass($) {
+    # get first function argument
+    my $auth_mech = shift(@_);
+    my $default, $tmp;
+    $auth_mech = lc(trim($auth_mech));
+    if ($auth_mech eq 'none') {
+        return;
     }
-    if ( ($inval =~ /^cram-md5\b/i) || ($inval =~ /^digest-md5\b/i) ||
-    ($inval =~ /^login\b/i) || ($inval =~/^plain\b/i)) {
-      return lc($inval);
+    print "SMTP authentication uses IMAP username and password by default.\n";
+    print "\n";
+    print "Would you like to use other login and password for all SquirrelMail \n";
+    print "SMTP connections?";
+    if ($smtp_sitewide_user ne '') {
+        $default = 'y';
+        print " [Yn]:";
+    } else {
+        $default = 'n';
+        print " [yN]:";
+    }
+    $tmp=<STDIN>;
+    $tmp = trim($tmp);
+    
+    if ($tmp eq '') {
+        $tmp = $default;
     } else {
-      # user entered garbage, or default value so nothing needs to be set
-      return $smtp_auth_mech;
+        $tmp = lc($tmp);
+    }
+
+    if ($tmp eq 'n') {
+        $smtp_sitewide_user = '';
+        $smtp_sitewide_pass = '';
+    } elsif ($tmp eq 'y') {
+        print "Enter username [$smtp_sitewide_user]:";
+        my $new_user = <STDIN>;
+        $new_user = trim($new_user);
+        if ($new_user ne '') {
+            $smtp_sitewide_user = $new_user;
+        }
+        if ($smtp_sitewide_user ne '') {
+            print "If you don't enter any password, current sitewide password will be used.\n";
+            print "If you enter space, password will be set to empty string.\n";
+            print "Enter password:";
+            my $new_pass = <STDIN>;
+            if ($new_pass ne "\n") {
+                $smtp_sitewide_pass = trim($new_pass);
+            }
+        } else {
+            print "Invalid input. You must set username used for SMTP authentication.\n";
+            print "Click any key to continue\n";
+            $tmp = <STDIN>;
+        }
+    } else {
+        print "Invalid input\n";
+        print "Click any key to continue\n";
+        $tmp = <STDIN>;
     }
 }
 
+# Sub adds information about SMTP authentication type to menu
+sub display_smtp_sitewide_userpass() {
+    my $ret = '';
+    if ($smtp_auth_mech ne 'none') {
+        if ($smtp_sitewide_user ne '') {
+            $ret = ' (with custom username and password)';
+        } else {
+            $ret = ' (with IMAP username and password)';
+        }
+    }
+    return $ret;
+}
+
 # TLS
 # This sub is reused for IMAP and SMTP
 # Args: service name, default value
@@ -2441,6 +2514,25 @@ sub command318 {
     return $time_zone_type;
 }
 
+# set the location base for redirects (since 1.5.2)
+sub command_config_location_base {
+    print "Here you can set the base part of the SquirrelMail URL.\n";
+    print "It is normally autodetected but if that fails, use this\n";
+    print "option to override.\n";
+    print "It should contain only the protocol and hostname/port parts\n";
+    print "of the URL; the full path will be appended automatically.\n\n";
+    print "Examples:\nhttp://webmail.example.org\nhttp://webmail.example.com:8080\nhttps://webmail.example.com:6691\n\n";
+    print "Do not add any path elements.\n";
+
+    print "URL base? [" .$WHT."autodetect$NRM]: $WHT";
+    $new_config_location_base = <STDIN>;
+    chomp($new_config_location_base);
+    $config_location_base = $new_config_location_base;
+    
+    return $config_location_base;
+}
+
+
 
 sub command41 {
     print "\nDefine the themes that you wish to use.  If you have added ";
@@ -2610,11 +2702,12 @@ sub command_templates {
         }
 
         print " $count.  $name";
-        print "($templateset_path[$count])\n";
+        print "($templateset_id[$count])\n";
 
         $count++;
    }
 
+# FIXME: what is this stuff?  it is commented out, can we just blast it?
 #    opendir( DIR, "../templates" );
 #    @files          = readdir(DIR);
 #    $pos            = 0;
@@ -2623,7 +2716,7 @@ sub command_templates {
 #        if ( -d "../templates/" . $files[$i] && $files[$i] !~ /^\./ && $files[$i] ne "CVS" ) {
 #            $filename = "../templates/" . $files[$cnt];
 #            $found = 0;
-#            for ( $x = 0 ; $x <= $#templateset_path ; $x++ ) {
+#            for ( $x = 0 ; $x <= $#templateset_id ; $x++ ) {
 #                if ( $theme_path[$x] eq $filename ) {
 #                    $found = 1;
 #                }
@@ -2652,20 +2745,26 @@ sub command_templates {
 #    }
 #    closedir DIR;
 
+    $menu_text = ".-------------------------------------.\n"
+               . "| t            (detect templates set) |\n"
+               . "| +                (add template set) |\n"
+               . "| - N           (remove template set) |\n"
+               . "| m N     (mark default template set) |\n"
+               . "| l              (list template sets) |\n"
+               . "| d                            (done) |\n"
+               . "|-------------------------------------|\n"
+               . "| where N is a template set number    |\n"
+               . "`-------------------------------------'\n";
     print "\n";
-    print ".--------------------------------.\n";
-    print "| t       (detect templates set) |\n";
-    print "| +           (add template set) |\n";
-    print "| - N      (remove template set) |\n";
-    print "| m N        (mark template set) |\n";
-    print "| l          (list template set) |\n";
-    print "| d                       (done) |\n";
-    print "`--------------------------------'\n";
+    print $menu_text;
     print "\n[template set] command (?=help) > ";
 
     $input = <STDIN>;
     $input =~ s/[\r\n]//g;
     while ( $input ne "d" ) {
+
+        # list template sets
+        #
         if ( $input =~ /^\s*l\s*/i ) {
             $count = 0;
             while ( $count <= $#templateset_name ) {
@@ -2684,10 +2783,13 @@ sub command_templates {
                 }
 
                 print " $count.  $name";
-                print "($templateset_path[$count])\n";
+                print "($templateset_id[$count])\n";
 
                 $count++;
             }
+
+        # mark default template set
+        #
         } elsif ( $input =~ /^\s*m\s*[0-9]+/i ) {
             $old_def       = $templateset_default;
             $templateset_default = $input;
@@ -2696,79 +2798,89 @@ sub command_templates {
                 print "Cannot set default template set to $templateset_default.  That template set does not exist.\n";
                 $templateset_default = $old_def;
             }
+
+        # add template set
+        #
         } elsif ( $input =~ /^\s*\+/ ) {
-            print "What is the name of this template: ";
+            print "\nWhat is the name of this template (as shown to your users): ";
             $name = <STDIN>;
             $name =~ s/[\r\n]//g;
             $templateset_name[ $#templateset_name + 1 ] = $name;
-            print "Be sure to put ../templates/ before the dirname.\n";
-            print "What file is this stored in (ex: ../templates/default/): ";
+            print "\n\nThe directory name should not contain any path information\n"
+                . "or slashes, and should be the name of the directory that the\n"
+                . "template set is found in within the SquirrelMail templates\n"
+                . "directory.\n\n";
+            print "What directory is this stored in (ex: default_advanced): ";
             $name = <STDIN>;
             $name =~ s/[\r\n]//g;
-            $templateset_path[ $#templateset_path + 1 ] = $name;
+            $templateset_id[ $#templateset_id + 1 ] = $name;
+
+        # detect template sets
+        #
         } elsif ( $input =~ /^\s*t\s*/i ) {
             print "\nStarting detection...\n\n";
             opendir( DIR, "../templates" );
             @files = readdir(DIR);
             $cnt = 0;
-            $detected = 0;
             while ( $cnt <= $#files ) {
                 if ( -d "../templates/" . $files[$cnt] && $files[$cnt] !~ /^\./ && $files[$cnt] ne "CVS" ) {
-                    $filename = "../templates/" . $files[$cnt]. "/";
+                    $filename = $files[$cnt];
                     $found = 0;
-                    for ( $x = 0 ; $x <= $#templateset_path ; $x++ ) {
-                        if ( $templateset_path[$x] eq $filename ) {
+                    for ( $x = 0 ; $x <= $#templateset_id ; $x++ ) {
+                        if ( $templateset_id[$x] eq $filename ) {
                             $found = 1;
+                            last;
                         }
                     }
-                    if ( $found != 1 && $detected == 0) {
-                        $templateset_path[ $#templateset_path + 1 ] = $filename;
-                        $templateset_name[ $#templateset_name + 1 ] = "Default template";
-                        $aTemplateSet[ $#templateset_name + 1 ] = "Default template";
-                        $aTemplateSet[ $#templateset_path + 1 ] = $filename;
-                    } elsif ( $found != 1) {
+                    if ( $found != 1) {
                         print "** Found template set: $filename\n";
-                        print "   What is it's name? ";
+                        print "   What is it's name (as shown to your users)? ";
                         $nm = <STDIN>;
                         $nm =~ s/[\n\r]//g;
-                        $templateset_path[ $#templateset_path + 1 ] = $filename;
+                        $templateset_id[ $#templateset_id + 1 ] = $filename;
                         $templateset_name[ $#templateset_name + 1 ] = $nm;
-
-                        $aTemplateSet[ $#templateset_name + 1 ] = $nm;
-                        $aTemplateSet[ $#templateset_path + 1 ] = $filename;
+# FIXME: why are these two lines here?
+#                        $aTemplateSet[ $#templateset_name + 1 ] = $nm;
+#                        $aTemplateSet[ $#templateset_id + 1 ] = $filename;
                     }
-                    $detected++;
                 }
                 $cnt++;
             }
             print "\n";
-            for ( $cnt = 0 ; $cnt <= $#templateset_path ; $cnt++ ) {
-                $filename = $templateset_path[$cnt];
-                if ( !(-d  $filename) ) {
-                    print "  Removing $filename (file not found)\n";
+            for ( $cnt= 0 ; $cnt <= $#templateset_id ; ) {
+                $filename = $templateset_id[$cnt];
+                if ( !(-d  change_to_rel_path('SM_PATH . \'templates/' . $filename)) ) {
+                    print "  Removing \"$filename\" (template set directory not found)\n";
+                    if ( $templateset_default gt $cnt ) { $templateset_default--; }
+                    elsif ( $templateset_default eq $cnt ) { $templateset_default = 0; }
                     $offset         = 0;
                     @new_templateset_name = ();
-                    @new_templateset_path = ();
-                    for ( $x = 0 ; $x < $#templateset_path ; $x++ ) {
-                        if ( $templateset_path[$x] eq $filename ) {
+                    @new_templateset_id = ();
+                    for ( $x = 0 ; $x < $#templateset_id ; $x++ ) {
+                        if ( $templateset_id[$x] eq $filename ) {
                             $offset = 1;
                         }
                         if ( $offset == 1 ) {
                             $new_templateset_name[$x] = $templateset_name[ $x + 1 ];
-                            $new_theme_path[$x] = $templateset_path[ $x + 1 ];
+                            $new_templateset_id[$x] = $templateset_id[ $x + 1 ];
                         } else {
                             $new_templateset_name[$x] = $templateset_name[$x];
-                            $new_templateset_path[$x] = $templateset_path[$x];
+                            $new_templateset_id[$x] = $templateset_id[$x];
                         }
                     }
                     @templateset_name = @new_templateset_name;
-                    @templateset_path = @new_templateset_path;
-                }
+                    @templateset_id = @new_templateset_id;
+                } else { $cnt++; }
             }
             print "\nDetection complete!\n\n";
 
             closedir DIR;
-        } elsif ( $input =~ /^\s*-\s*[0-9]?/ ) {
+
+        # remove template set
+        #
+        # undocumented functionality of removing last template set isn't that great
+        #} elsif ( $input =~ /^\s*-\s*[0-9]?/ ) {
+        } elsif ( $input =~ /^\s*-\s*[0-9]+/ ) {
             if ( $input =~ /[0-9]+\s*$/ ) {
                 $rem_num = $input;
                 $rem_num =~ s/^\s*-\s*//g;
@@ -2781,29 +2893,25 @@ sub command_templates {
             } else {
                 $count          = 0;
                 @new_templateset_name = ();
-                @new_templateset_path = ();
+                @new_templateset_id = ();
                 while ( $count <= $#templateset_name ) {
                     if ( $count != $rem_num ) {
                         @new_templateset_name = ( @new_templateset_name, $templateset_name[$count] );
-                        @new_templateset_path = ( @new_templateset_path, $templateset_path[$count] );
+                        @new_templateset_id = ( @new_templateset_id, $templateset_id[$count] );
                     }
                     $count++;
                 }
                 @templateset_name = @new_templateset_name;
-                @templateset_path = @new_templateset_path;
+                @templateset_id = @new_templateset_id;
                 if ( $templateset_default > $rem_num ) {
                     $templateset_default--;
                 }
             }
+
+        # help
+        #
         } elsif ( $input =~ /^\s*\?\s*/ ) {
-            print ".--------------------------------.\n";
-            print "| t       (detect templates set) |\n";
-            print "| +           (add template set) |\n";
-            print "| - N      (remove template set) |\n";
-            print "| m N        (mark template set) |\n";
-            print "| l          (list template set) |\n";
-            print "| d                       (done) |\n";
-            print "`--------------------------------'\n";
+            print $menu_text;
         }
         print "[template set] command (?=help) > ";
         $input = <STDIN>;
@@ -2813,34 +2921,6 @@ sub command_templates {
 }
 
 
-# Theme - CSS file
-sub command42 {
-    print "You may specify a cascading style-sheet (CSS) file to be included\n";
-    print "on each html page generated by SquirrelMail. The CSS file is useful\n";
-    print "for specifying a site-wide font. If you're not familiar with CSS\n";
-    print "files, leave this blank.\n";
-    print "\n";
-    print "To clear out an existing value, just type a space for the input.\n";
-    print "\n";
-    print "Please be aware of the following: \n";
-    print "  - Relative URLs are relative to the config dir\n";
-    print "    to use the themes directory, use ../themes/css/newdefault.css\n";
-    print "  - To specify a css file defined outside the SquirrelMail source tree\n";
-    print "    use the absolute URL the webserver would use to include the file\n";
-    print "    e.g. http://some.host.com/css/mystyle.css or /css/mystyle.css\n";
-    print "\n";
-    print "[$WHT$theme_css$NRM]: $WHT";
-    $new_theme_css = <STDIN>;
-
-    if ( $new_theme_css eq "\n" ) {
-        $new_theme_css = $theme_css;
-    } else {
-        $new_theme_css =~ s/[\r\n]//g;
-    }
-    $new_theme_css =~ s/^\s*//;
-    return $new_theme_css;
-}
-
 # sets default font size option
 sub command_default_fontsize {
     print "Enter default font size [$WHT$$default_fontsize$NRM]: $WHT";
@@ -2984,7 +3064,11 @@ sub command61 {
             $sub = $#ldap_host + 1;
 
             print "First, we need to have the hostname or the IP address where\n";
-            print "this LDAP server resides.  Example: ldap.bigfoot.com\n";
+            print "this LDAP server resides. Example: ldap.bigfoot.com\n";
+           print "\n";
+           print "You can use any URI compatible with your LDAP library. Please\n";
+            print "note that StartTLS option is not compatible with ldaps and\n";
+           print "ldapi URIs.\n";
             print "hostname: ";
             $name = <STDIN>;
             $name =~ s/[\r\n]//g;
@@ -3817,7 +3901,32 @@ sub commandB6 {
     return $allow_remote_configtest;
 }
 
-
+# Default Icon theme
+sub commandB7 {
+       print "You may change the path to the default icon theme to be used, if icons\n";
+       print "have been enabled.  This theme will be used when an icon cannot be\n";
+       print "found in the current theme, or when no icon theme is specified.  If\n";
+       print "left blank, and icons are enabled, the default theme will be used\n";
+       print "from images/themes/default/.\n";
+    print "\n";
+    print "To clear out an existing value, just type a space for the input.\n";
+    print "\n";
+    print "Please be aware of the following: \n";
+    print "  - Relative URLs are relative to the config dir\n";
+    print "    to use the icon themes directory, use ../images/themes/newtheme/\n";
+    print "  - The icon theme may be outside the SquirrelMail directory, but\n";
+    print "    it must be web accessible.\n";
+    print "[$WHT$icon_theme_def$NRM]: $WHT";
+    $new_icon_theme_def = <STDIN>;
+
+    if ( $new_icon_theme_def eq "\n" ) {
+        $new_icon_theme_def = $icon_theme_def;
+    } else {
+        $new_icon_theme_def =~ s/[\r\n]//g;
+    }
+    $new_icon_theme_def =~ s/^\s*//;
+    return $new_icon_theme_def;
+}
 
 sub save_data {
     $tab = "    ";
@@ -3971,9 +4080,9 @@ sub save_data {
         # boolean
         print CF "\$hide_auth_header         = $hide_auth_header;\n";
         # boolean
-        print CF "\$disable_thread_sort     = $disable_thread_sort;\n";
+        print CF "\$disable_thread_sort      = $disable_thread_sort;\n";
         # boolean
-        print CF "\$disable_server_sort     = $disable_server_sort;\n";
+        print CF "\$disable_server_sort      = $disable_server_sort;\n";
         # boolean
         print CF "\$allow_charset_search     = $allow_charset_search;\n";
         # integer
@@ -3982,6 +4091,9 @@ sub save_data {
         # integer
         print CF "\$time_zone_type           = $time_zone_type;\n";
         print CF "\n";
+        # string
+        print CF "\$config_location_base     = '$config_location_base';\n";
+        print CF "\n";
 
         # all plugins are strings
         for ( $ct = 0 ; $ct <= $#plugins ; $ct++ ) {
@@ -4006,9 +4118,16 @@ sub save_data {
 
         if ( $templateset_default eq '' ) { $templateset_default = '0'; }
         print CF "\$templateset_default = $templateset_default;\n";
+# FIXME: need to make this a setting the user can change herein
+#        This REALLY needs to be done, since the index of the "default"
+#        set cannot always be predicted!
+#        Heck, why are default and fallback indexes?  If we make them
+#        into the ID strings, then it would not cause such issues
+$templateset_fallback = 0;
+        print CF "\$templateset_fallback = $templateset_fallback;\n";
 
         for ( $count = 0 ; $count <= $#templateset_name ; $count++ ) {
-            print CF "\$aTemplateSet[$count]['PATH'] = " . &change_to_SM_path($templateset_path[$count]) . ";\n";
+            print CF "\$aTemplateSet[$count]['ID'] = '" . $templateset_id[$count] . "';\n";
             # escape theme name so it can contain single quotes.
             $esc_name =  $templateset_name[$count];
             $esc_name =~ s/\\/\\\\/g;
@@ -4150,6 +4269,8 @@ sub save_data {
 
         # string
         print CF "\$smtp_auth_mech = '$smtp_auth_mech';\n";
+        print CF "\$smtp_sitewide_user = '". quote_single($smtp_sitewide_user) ."';\n";
+        print CF "\$smtp_sitewide_pass = '". quote_single($smtp_sitewide_pass) ."';\n";
         # string
         print CF "\$imap_auth_mech = '$imap_auth_mech';\n";
         # boolean
@@ -4167,6 +4288,8 @@ sub save_data {
         # boolean
         print CF "\$use_icons = $use_icons;\n";
         print CF "\n";
+        # string
+        print CF "\$icon_theme_def = " . &change_to_SM_path($icon_theme_def) . ";\n";
         # boolean
         print CF "\$use_php_recode = $use_php_recode;\n";
         print CF "\n";
@@ -4177,15 +4300,6 @@ sub save_data {
         print CF "\$allow_remote_configtest = $allow_remote_configtest;\n";
         print CF "\n";
 
-        print CF "\@include SM_PATH . 'config/config_local.php';\n";
-
-        print CF "\n/**\n";
-        print CF " * Make sure there are no characters after the PHP closing\n";
-        print CF " * tag below (including newline characters and whitespace).\n";
-        print CF " * Otherwise, that character will cause the headers to be\n";
-        print CF " * sent and regular output to begin, which will majorly screw\n";
-        print CF " * things up when we try to send more headers later.\n";
-        print CF " */\n";
         close CF;
 
         print "Data saved in config.php\n";
@@ -4476,6 +4590,7 @@ sub change_to_rel_path() {
     my $new_path = $old_path;
 
     if ( $old_path =~ /^SM_PATH/ ) {
+        # FIXME: the following replacement loses the opening quote mark!
         $new_path =~ s/^SM_PATH . \'/\.\.\//;
         $new_path =~ s/\.\.\/config\///;
     }
@@ -4483,13 +4598,16 @@ sub change_to_rel_path() {
     return $new_path;
 }
 
-sub detect_auth_support {
 # Attempts to auto-detect if a specific auth mechanism is supported.
 # Called by 'command112a' and 'command112b'
 # ARGS: service-name (IMAP or SMTP), host:port, mech-name (ie. CRAM-MD5)
-
+sub detect_auth_support {
+    # Try loading IO::Socket
+    unless (eval("use IO::Socket; 1")) {
+        print "Perl IO::Socket module is not available.";
+        return undef;
+    }
     # Misc setup
-    use IO::Socket;
     my $service = shift;
     my $host = shift;
     my $mech = shift;
@@ -4586,8 +4704,8 @@ sub clear_screen() {
 # returns 0 (folder name is not correct) or 1 (folder name is correct)
 sub check_imap_folder($) {
     my $folder_name = shift(@_);
-    if ($folder_name =~ /[\x{80}-\x{FFFF}]/) {
-        # check for 8bit. Using iso-10646 range, because x80-xFF range does not match unicode chars
+
+    if ($folder_name =~ /[\x80-\xFFFF]/) {
         print "Folder name contains 8bit characters. Configuration utility requires\n";
         print "UTF7-IMAP encoded folder names.\n";
         print "Press any key to continue...";
@@ -4608,3 +4726,10 @@ sub check_imap_folder($) {
         return 1;
     }
 }
+
+# quotes string written in single quotes
+sub quote_single($) {
+    my $string = shift(@_);
+    $string =~ s/\'/\\'/g;
+    return $string;
+}