Fix a bunch of plugins to cope with moved load_prefs/validate files.
[squirrelmail.git] / plugins / administrator / options.php
index 93370f64b1f96d463b006cb1a633930657a4fa5e..9842e2459fc43f42513121ac8edd76da4572e7cd 100644 (file)
@@ -16,70 +16,137 @@ function parseConfig( $cfg_file ) {
     global $newcfg;
 
     $cfg = file( $cfg_file );
-    $cm = FALSE;
-    $j = count( $cfg );
-
-    for ( $i=0; $i < $j; $i++ ) {
-        $l = '';
-        $first_char = $cfg[$i]{0};
-        do {
-            // Remove comments
-            $c = trim( $cfg[$i] );
-            $c = preg_replace( '/\/\*.*\*\//', '', $c );
-            $c = preg_replace( '/#.*$/', '', $c );
-            $c = preg_replace( '/\/\/.*$/', '', $c );
-            $l .= $c;
-            $i++;
-        } while( $first_char == '$' && substr( $c, -1 ) <> ';' && $i < $j );
-        $i--;
-        if ( $l <> '' ) {
-            if ( $cm ) {
-                if( substr( $l, -2 ) == '*/' ) {
-                    $l = '';
-                    $cm = FALSE;
-                } else if( $k = strpos( $l, '*/' ) ) {
-                    $l = substr( $l, $k );
-                    $cm = FALSE;
+    $mode = '';
+    $l = count( $cfg );
+    $modifier = FALSE;
+
+    for ($i=0;$i<$l;$i++) {
+        $line = trim( $cfg[$i] );
+        $s = strlen( $line );
+        for ($j=0;$j<$s;$j++) {
+            switch ( $mode ) {
+            case '=':
+                if ( $line{$j} == '=' ) {
+                    // Ok, we've got a right value, lets detect what type
+                    $mode = 'D';
+                } else if ( $line{$j} == ';' ) {
+                    // hu! end of command
+                    $key = $mode = '';
+                }
+                break;
+            case 'K':
+                // Key detect
+                if( $line{$j} == ' ' ) {
+                    $mode = '=';
                 } else {
-                    $l = '';
+                    $key .= $line{$j};
                 }
-            } else {
-                if( $l{0}.$l{1} == '/*' ) {
-                    $l = '';
-                    $cm = TRUE;
-                } else if ( $k = strpos( $l, '/*' ) ) {
-                    $l = substr( $l, 0, $k );
-                    $cm = TRUE;
+                break;
+            case ';':
+                // Skip until next ;
+                if ( $line{$j} == ';' ) {
+                    $mode = '';
+                }
+                break;
+            case 'S':
+                if ( $line{$j} == '\\' ) {
+                    $value .= $line{$j};
+                    $modifier = TRUE;
+                } else if ( $line{$j} == $delimiter && $modifier === FALSE ) {
+                    // End of string;
+                    $newcfg[$key] = $value . $delimiter;
+                    $key = $value = '';
+                    $mode = ';';
+                } else {
+                    $value .= $line{$j};
+                    $modifier = FALSE;
+                }
+                break;
+            case 'N':
+                if ( $line{$j} == ';' ) {
+                    $newcfg{$key} = $value;
+                    $key = $mode = '';
+                } else {
+                    $value .= $line{$j};
+                }
+                break;
+            case 'C':
+                // Comments
+                if ( $s > $j + 1  &&
+                     $line{$j}.$line{$j+1} == '*/' ) {
+                    $mode = '';
+                    $j++;
+                }
+                break;
+            case 'D':
+                // Delimiter detect
+                switch ( $line{$j} ) {
+                case '"':
+                case "'":
+                    // Double quote string
+                    $delimiter = $value = $line{$j};
+                    $mode = 'S';
+                    break;
+                case ' ':
+                    // Nothing yet
+                    break;
+                default:
+                    if ( strtoupper( substr( $line, $j, 4 ) ) == 'TRUE'  ) {
+                        // Boolean TRUE
+                        $newcfg{$key} = 'TRUE';
+                        $key = '';
+                        $mode = ';';
+                    } else if ( strtoupper( substr( $line, $j, 5 ) ) == 'FALSE'  ) {
+                        $newcfg{$key} = 'FALSE';
+                        $key = '';
+                        $mode = ';';
+                    } else {
+                        // Number or function call
+                        $mode = 'N';
+                        $value = $line{$j};
+                    }
+                }
+                break;
+            default:
+                if ( $line{$j} == '$' ) {
+                    // We must detect $key name
+                    $mode = 'K';
+                    $key = '$';
+                } else if ( $s < $j + 2 ) {
+                } else if ( strtoupper( substr( $line, $j, 7 ) ) == 'GLOBAL ' ) {
+                    // Skip untill next ;
+                    $mode = ';';
+                    $j += 6;
+                } else if ( $line{$j}.$line{$j+1} == '/*' ) {
+                    $mode = 'C';
+                    $j++;
+                } else if ( $line{$j} == '#' || $line{$j}.$line{$j+1} == '//' ) {
+                    // Delete till the end of the line
+                    $j = $s;
                 }
-            }
-    
-            if ( $k = strpos( $l, '=' ) ) {
-                $key = trim( substr( $l, 0, $k - 1 ) );
-                $val = str_replace( ';', '', trim( substr( $l, $k + 1 ) ) );
-                $newcfg[$key] = $val;
             }
         }
-
     }
 
 }
+
 /* ---------------------- main -------------------------- */
+
 chdir('..');
-require_once('../src/validate.php');
-require_once('../functions/page_header.php');
-require_once('../functions/imap.php');
-require_once('../src/load_prefs.php');
-require_once('../plugins/administrator/defines.php');
-
-$auth = FALSE;
-if ( $adm_id = fileowner('../config/config.php') ) {
-    $adm = posix_getpwuid( $adm_id );
-    if ( $username == $adm['name'] ) {
-        $auth = TRUE;
-    }
-}
+chdir('..');
+define('SM_PATH','../');
 
-if ( !auth ) {
+/* SquirrelMail required files. */
+require_once(SM_PATH . 'include/validate.php');
+require_once(SM_PATH . 'functions/page_header.php');
+require_once(SM_PATH . 'functions/imap.php');
+require_once(SM_PATH . 'include/load_prefs.php');
+require_once(SM_PATH . 'plugins/administrator/defines.php');
+require_once(SM_PATH . 'plugins/administrator/auth.php');
+
+GLOBAL $data_dir, $username;
+
+if ( !adm_check_user() ) {
     header("Location: ../../src/options.php") ;
     exit;
 }
@@ -96,10 +163,34 @@ $cfgfile = '../config/config.php';
 parseConfig( '../config/config_default.php' );
 parseConfig( $cfgfile );
 
-echo "<form action=$PHP_SELF method=post>" .
-    "<br><center><table width=95% bgcolor=\"$color[5]\"><tr><td>".
+$colapse = array( 'Titles' => 'off',
+                  'Group1' => getPref($data_dir, $username, 'adm_Group1', 'off' ),
+                  'Group2' => getPref($data_dir, $username, 'adm_Group2', 'on' ),
+                  'Group3' => getPref($data_dir, $username, 'adm_Group3', 'on' ),
+                  'Group4' => getPref($data_dir, $username, 'adm_Group4', 'on' ),
+                  'Group5' => getPref($data_dir, $username, 'adm_Group5', 'on' ),
+                  'Group6' => getPref($data_dir, $username, 'adm_Group6', 'on' ),
+                  'Group7' => getPref($data_dir, $username, 'adm_Group7', 'on' ),
+                  'Group8' => getPref($data_dir, $username, 'adm_Group8', 'on' ) );
+
+if ( isset( $switch ) ) {
+
+    if ( $colapse[$switch] == 'on' ) {
+       $colapse[$switch] = 'off';
+    } else {
+       $colapse[$switch] = 'on';
+    }
+    setPref($data_dir, $username, "adm_$switch", $colapse[$switch] );
+
+}
+
+echo "<form action=options.php method=post name=options>" .
+    "<center><table width=95% bgcolor=\"$color[5]\"><tr><td>".
     "<table width=100% cellspacing=0 bgcolor=\"$color[4]\">" ,
     "<tr bgcolor=\"$color[5]\"><th colspan=2>" . _("Configuration Administrator") . "</th></tr>";
+
+$act_grp = 'Titles';  /* Active group */
+
 foreach ( $newcfg as $k => $v ) {
     $l = strtolower( $v );
     $type = SMOPT_TYPE_UNDEFINED;
@@ -112,7 +203,11 @@ foreach ( $newcfg as $k => $v ) {
     if ( isset( $defcfg[$k] ) ) {
         $name = $defcfg[$k]['name'];
         $type = $defcfg[$k]['type'];
-        $size = $defcfg[$k]['size'];
+        if ( isset( $defcfg[$k]['size'] ) ) {
+            $size = $defcfg[$k]['size'];
+        } else {
+            $size = 40;
+        }
     } else if ( $l == 'true' ) {
         $v = 'TRUE';
         $type = SMOPT_TYPE_BOOLEAN;
@@ -125,86 +220,267 @@ foreach ( $newcfg as $k => $v ) {
         $type = SMOPT_TYPE_STRING;
     }
 
-    switch ( $type ) {
-    case SMOPT_TYPE_TITLE:
-        echo "<tr bgcolor=\"$color[0]\"><th colspan=2>$name</th></tr>";
-        break;
-    case SMOPT_TYPE_COMMENT:
-        $v = substr( $v, 1, strlen( $v ) - 2 );
-        echo "<tr><td>$name</td><td>";
-        echo "<b>$v</b>";
-        $newcfg[$k] = "'$v'";
-        break;
-    case SMOPT_TYPE_INTEGER:
-        if ( isset( $HTTP_POST_VARS[$e] ) ) {
-            $v = intval( $HTTP_POST_VARS[$e] );
-            $newcfg[$k] = $v;
-        }
-        echo "<tr><td>$name</td><td>";
-        echo "<input size=10 name=\"adm_$n\" value=\"$v\">";
-        break;
-    case SMOPT_TYPE_NUMLIST:
-        if ( isset( $HTTP_POST_VARS[$e] ) ) {
-            $v = $HTTP_POST_VARS[$e];
-            $newcfg[$k] = $v;
+    if ( substr( $k, 0, 7 ) == '$theme[' ) {
+        $type = SMOPT_TYPE_THEME;
+    } else if ( substr( $k, 0, 9 ) == '$plugins[' ) {
+        $type = SMOPT_TYPE_PLUGINS;
+    } else if ( substr( $k, 0, 13 ) == '$ldap_server[' ) {
+        $type = SMOPT_TYPE_LDAP;
+    }
+
+    if( $type == SMOPT_TYPE_TITLE || $colapse[$act_grp] == 'off' ) {
+
+        switch ( $type ) {
+        case SMOPT_TYPE_LDAP:
+        case SMOPT_TYPE_PLUGINS:
+        case SMOPT_TYPE_THEME:
+        case SMOPT_TYPE_HIDDEN:
+            break;
+        case SMOPT_TYPE_EXTERNAL:
+            echo "<tr><td>$name</td><td><b>" .
+                 $defcfg[$k]['value'] .
+                 "</b></td></tr>";
+            break;
+        case SMOPT_TYPE_TITLE:
+            if ( $colapse[$k] == 'on' ) {
+                $sw = '(+)';
+            } else {
+                $sw = '(-)';
+            }
+            echo "<tr bgcolor=\"$color[0]\"><th colspan=2>" .
+                 "<a href=options.php?switch=$k STYLE=\"text-decoration:none\"><b>$sw</b> </a>" .
+                 "$name</th></tr>";
+            $act_grp = $k;
+            break;
+        case SMOPT_TYPE_COMMENT:
+            $v = substr( $v, 1, strlen( $v ) - 2 );
+            echo "<tr><td>$name</td><td>".
+                 "<b>$v</b>";
+            $newcfg[$k] = "'$v'";
+            if ( isset( $defcfg[$k]['comment'] ) ) {
+                echo ' &nbsp; ' . $defcfg[$k]['comment'];
+            }
+            echo "</td></tr>\n";
+            break;
+        case SMOPT_TYPE_INTEGER:
+            if ( isset( $HTTP_POST_VARS[$e] ) ) {
+                $v = intval( $HTTP_POST_VARS[$e] );
+                $newcfg[$k] = $v;
+            }
+            echo "<tr><td>$name</td><td>".
+                 "<input size=10 name=\"adm_$n\" value=\"$v\">";
+            if ( isset( $defcfg[$k]['comment'] ) ) {
+                echo ' &nbsp; ' . $defcfg[$k]['comment'];
+            }
+            echo "</td></tr>\n";
+            break;
+        case SMOPT_TYPE_NUMLIST:
+            if ( isset( $HTTP_POST_VARS[$e] ) ) {
+                $v = $HTTP_POST_VARS[$e];
+                $newcfg[$k] = $v;
+            }
+            echo "<tr><td>$name</td><td>";
+            echo "<select name=\"adm_$n\">";
+            foreach ( $defcfg[$k]['posvals'] as $kp => $vp ) {
+                echo "<option value=\"$kp\"";
+                if ( $kp == $v ) {
+                    echo ' selected';
+                }
+                echo ">$vp</option>";
+            }
+            echo '</select>';
+            if ( isset( $defcfg[$k]['comment'] ) ) {
+                echo ' &nbsp; ' . $defcfg[$k]['comment'];
+            }
+            echo "</td></tr>\n";
+            break;
+        case SMOPT_TYPE_STRLIST:
+            if ( isset( $HTTP_POST_VARS[$e] ) ) {
+                $v = '"' . $HTTP_POST_VARS[$e] . '"';
+                $newcfg[$k] = $v;
+            }
+            echo "<tr><td>$name</td><td>".
+                 "<select name=\"adm_$n\">";
+            foreach ( $defcfg[$k]['posvals'] as $kp => $vp ) {
+                echo "<option value=\"$kp\"";
+                if ( $kp == substr( $v, 1, strlen( $v ) - 2 ) ) {
+                    echo ' selected';
+                }
+                echo ">$vp</option>";
+            }
+            echo '</select>';
+            if ( isset( $defcfg[$k]['comment'] ) ) {
+                echo ' &nbsp; ' . $defcfg[$k]['comment'];
+            }
+            echo "</td></tr>\n";
+            break;
+
+        case SMOPT_TYPE_TEXTAREA:
+            if ( isset( $HTTP_POST_VARS[$e] ) ) {
+                $v = '"' . $HTTP_POST_VARS[$e] . '"';
+                $newcfg[$k] = str_replace( "\n", '', $v );
+            }
+            echo "<tr><td valign=top>$name</td><td>".
+                 "<textarea cols=\"$size\" name=\"adm_$n\">" . substr( $v, 1, strlen( $v ) - 2 ) . "</textarea>";
+            if ( isset( $defcfg[$k]['comment'] ) ) {
+                echo ' &nbsp; ' . $defcfg[$k]['comment'];
+            }
+            echo "</td></tr>\n";
+            break;
+        case SMOPT_TYPE_STRING:
+            if ( isset( $HTTP_POST_VARS[$e] ) ) {
+                $v = '"' . $HTTP_POST_VARS[$e] . '"';
+                $newcfg[$k] = $v;
+            }
+            if ( $v == '""' && isset( $defcfg[$k]['default'] ) ) {
+                $v = "'" . $defcfg[$k]['default'] . "'";
+                $newcfg[$k] = $v;
+            }
+            echo "<tr><td>$name</td><td>".
+                 "<input size=\"$size\" name=\"adm_$n\" value=\"" . substr( $v, 1, strlen( $v ) - 2 ) . "\">";
+            if ( isset( $defcfg[$k]['comment'] ) ) {
+                echo ' &nbsp; ' . $defcfg[$k]['comment'];
+            }
+            echo "</td></tr>\n";
+            break;
+        case SMOPT_TYPE_BOOLEAN:
+            if ( isset( $HTTP_POST_VARS[$e] ) ) {
+                $v = $HTTP_POST_VARS[$e];
+                $newcfg[$k] = $v;
+            } else {
+                $v = strtoupper( $v );
+            }
+            if ( $v == 'TRUE' ) {
+                $ct = ' checked';
+                $cf = '';
+            } else {
+                $ct = '';
+                $cf = ' checked';
+            }
+            echo "<tr><td>$name</td><td>" .
+                 "<INPUT$ct type=radio NAME=\"adm_$n\" value=\"TRUE\">" . _("Yes") .
+                 "<INPUT$cf type=radio NAME=\"adm_$n\" value=\"FALSE\">" . _("No");
+            if ( isset( $defcfg[$k]['comment'] ) ) {
+                echo ' &nbsp; ' . $defcfg[$k]['comment'];
+            }
+            echo "</td></tr>\n";
+            break;
+        default:
+            echo "<tr><td>$name</td><td>" .
+                 "<b><i>$v</i></b>";
+            if ( isset( $defcfg[$k]['comment'] ) ) {
+                echo ' &nbsp; ' . $defcfg[$k]['comment'];
+            }
+            echo "</td></tr>\n";
         }
-        echo "<tr><td>$name</td><td>";
-        echo "<select name=\"adm_$n\">";
-        foreach ( $defcfg[$k]['posvals'] as $kp => $vp ) {
-            echo "<option value=\"$kp\"";
-            if ( $kp == $v ) {
-                echo ' selected';
-            }
-            echo ">$vp</option>";
+    }
+
+}
+
+/* Special Themes Block */
+if ( $colapse['Group7'] == 'off' ) {
+    $i = 0;
+    echo '<tr><th>' . _("Theme Name") .
+         '</th><th>' . _("Theme Path") .
+         '</th></tr>';
+    while ( isset( $newcfg["\$theme[$i]['NAME']"] ) ) {
+        $k1 = "\$theme[$i]['NAME']";
+        $e1 = "theme_name_$i";
+        if ( isset( $HTTP_POST_VARS[$e1] ) ) {
+            $v1 = '"' . str_replace( '\"', '"', $HTTP_POST_VARS[$e1] ) . '"';
+            $v1 = '"' . str_replace( '"', '\"', $v1 ) . '"';
+            $newcfg[$k1] = $v1;
+        } else {
+            $v1 = $newcfg[$k1];
         }
-        echo '</select>';
-        break;
-    case SMOPT_TYPE_STRLIST:
-        if ( isset( $HTTP_POST_VARS[$e] ) ) {
-            $v = '"' . $HTTP_POST_VARS[$e] . '"';
-            $newcfg[$k] = $v;
+        $k2 = "\$theme[$i]['PATH']";
+        $e2 = "theme_path_$i";
+        if ( isset( $HTTP_POST_VARS[$e2] ) ) {
+            $v2 = '"' . str_replace( '\"', '"', $HTTP_POST_VARS[$e2] ) . '"';
+            $v2 = '"' . str_replace( '"', '\"', $v2 ) . '"';
+            $newcfg[$k2] = $v2;
+        } else {
+            $v2 = $newcfg[$k2];
         }
-        echo "<tr><td>$name</td><td>";
-        echo "<select name=\"adm_$n\">";
-        foreach ( $defcfg[$k]['posvals'] as $kp => $vp ) {
-            echo "<option value=\"$kp\"";
-            if ( $kp == substr( $v, 1, strlen( $v ) - 2 ) ) {
-                echo ' selected';
-            }
-            echo ">$vp</option>";
+        $name = substr( $v1, 1, strlen( $v1 ) - 2 );
+        $path = substr( $v2, 1, strlen( $v2 ) - 2 );
+        echo '<tr>'.
+             "<td align=right>$i. <input name=\"$e1\" value=\"$name\" size=30></td>".
+             "<td><input name=\"$e2\" value=\"$path\" size=40></td>".
+             "</tr>\n";
+        $i++;
+
+    }
+}
+
+/* Special Plugins Block */
+if ( $colapse['Group8'] == 'on' ) {
+    $sw = '(+)';
+} else {
+    $sw = '(-)';
+}
+echo "<tr bgcolor=\"$color[0]\"><th colspan=2>" .
+     "<a href=options.php?switch=Group8 STYLE=\"text-decoration:none\"><b>$sw</b> </a>" .
+     _("Plugins") . '</th></tr>';
+
+if( $colapse['Group8'] == 'off' ) {
+
+    $fd = opendir( '../plugins/' );
+    $op_plugin = array();
+    $p_count = 0;
+    while (false!==($file = readdir($fd))) {
+        if ($file != '.' && $file != '..' && $file != 'CVS' ) {
+            if ( filetype( $file ) == 'dir' ) {
+                $op_plugin[] = $file;
+                $p_count++;
+            }
         }
-        echo '</select>';
-        break;
+    }
+    closedir($fd);
+    asort( $op_plugin );
 
-    case SMOPT_TYPE_STRING:
-        if ( isset( $HTTP_POST_VARS[$e] ) ) {
-            $v = '"' . $HTTP_POST_VARS[$e] . '"';
-            $newcfg[$k] = $v;
+    /* Lets get the plugins that are active */
+    $plugins = array();
+    if ( isset( $HTTP_POST_VARS['plg'] ) ) {
+        foreach ( $op_plugin as $plg ) {
+            if ( isset( $HTTP_POST_VARS["plgs_$plg"] ) &&
+                 $HTTP_POST_VARS["plgs_$plg"] == 'on' ) {
+                $plugins[] = $plg;
+            }
         }
-        echo "<tr><td>$name</td><td>";
-        echo "<input size=\"$size\" name=\"adm_$n\" value=\"" . substr( $v, 1, strlen( $v ) - 2 ) . "\">";
-        break;
-    case SMOPT_TYPE_BOOLEAN:
-        if ( isset( $HTTP_POST_VARS[$e] ) ) {
-            $v = $HTTP_POST_VARS[$e];
-            $newcfg[$k] = $v;
+        $i = 0;
+        foreach ( $plugins as $plg ) {
+            $k = "\$plugins[$i]";
+            $newcfg[$k] = "'$plg'";
+            $i++;
+        }
+        while ( isset( $newcfg["\$plugins[$i]"] ) ) {
+            $k = "\$plugins[$i]";
+            $newcfg[$k] = '';
+            $i++;
         }
-        if ( $v == 'TRUE' ) {
-            $ct = ' checked';
-            $cf = '';
+    } else {
+        $i = 0;
+        while ( isset( $newcfg["\$plugins[$i]"] ) ) {
+            $k = "\$plugins[$i]";
+            $v = $newcfg[$k];
+            $plugins[] = substr( $v, 1, strlen( $v ) - 2 );
+            $i++;
+        }
+    }
+    echo "<tr><td colspan=2><input type=hidden name=plg value=on><center><table><tr><td>";
+    foreach ( $op_plugin as $plg ) {
+        if ( in_array( $plg, $plugins ) ) {
+            $sw = ' checked';
         } else {
-            $ct = '';
-            $cf = ' checked';
+            $sw = '';
         }
-        echo "<tr><td>$name</td><td>";
-        echo "<INPUT$ct type=radio NAME=\"adm_$n\" value=\"TRUE\">" . _("Yes") .
-            "<INPUT$cf type=radio NAME=\"adm_$n\" value=\"FALSE\">" . _("No");
-        break;
-    default:
-        echo "<tr><td>$name</td><td>";
-        echo "<b><i>$v</i></b>";
+        echo '<tr>' .
+             "<td>$plg</td><td><input$sw type=checkbox name=plgs_$plg></td>".
+             "</tr>\n";
     }
-    echo "</td></tr>\n";
+    echo '</td></tr></table>';
+
 }
 echo "<tr bgcolor=\"$color[5]\"><th colspan=2><input value=\"" .
      _("Change Settings") . "\" type=submit></th></tr>" ,
@@ -213,40 +489,30 @@ echo "<tr bgcolor=\"$color[5]\"><th colspan=2><input value=\"" .
 /*
     Write the options to the file.
 */
-$fp = fopen( $cfgfile, 'w' );
-fwrite( $fp, "<?PHP\n".
-            "/**\n".
-            " * SquirrelMail Configuration File\n".
-            " * Created using the Administrator Plugin\n".
-            " */\n\n" );
-
-fwrite( $fp, 'GLOBAL ' );
-$not_first = FALSE;
-foreach ( $newcfg as $k => $v ) {
-    if ( $k{0} == '$' ) {
-        if( $i = strpos( $k, '[' ) ) {
-            if( strpos( $k, '[0]' ) ) {
-                if( $not_first ) {
-                    fwrite( $fp, ', ' );
-                }
-                fwrite( $fp, substr( $k, 0, $i) );
-                $not_first = TRUE;
-            }
-        } else {
-            if( $not_first ) {
-                fwrite( $fp, ', ' );
+
+if( $fp = @fopen( $cfgfile, 'w' ) ) {
+    fwrite( $fp, "<?PHP\n".
+    "/**\n".
+    " * SquirrelMail Configuration File\n".
+    " * Created using the Administrator Plugin\n".
+    " */\n" );
+
+    foreach ( $newcfg as $k => $v ) {
+        if ( $k{0} == '$' && $v <> '' ) {
+            if ( substr( $k, 1, 11 ) == 'ldap_server' ) {
+                $v = substr( $v, 0, strlen( $v ) - 1 ) . "\n)";
+                $v = str_replace( 'array(', "array(\n\t", $v );
+                $v = str_replace( "',", "',\n\t", $v );
             }
-            fwrite( $fp, $k );
-            $not_first = TRUE;
+            fwrite( $fp, "$k = $v;\n" );
         }
     }
+    fwrite( $fp, '?>' );
+    fclose( $fp );
+} else {
+    echo '<font size=+1><br>'.
+         _("Config file can't be opened. Please check config.php.").
+         '</font>';
 }
-fwrite( $fp, ";\n" );
-foreach ( $newcfg as $k => $v ) {
-    if ( $k{0} == '$' ) {
-        fwrite( $fp, "$k = $v;\n" );
-    }
-}
-fwrite( $fp, '?>' );
-fclose( $fp );
-?>
\ No newline at end of file
+
+?>