[FEATURE] Install script: Simplify config generation

Simplify the generation of the local configuration file
by replacing various loops with shorter PHP standard methods.

This will cause the local configuration to have a slightly
different style, but it is still an array therefore is
readable and changeable by PHP developers.

diff --git a/install.php b/install.php
index 51cf7b8268a07aa85b6d1014bfb6c3846199c3f9..df4f106490992fa72623da13a484f7399e40b9c0 100644 (file)
--- a/install.php
+++ b/install.php
@@ -20,7 +20,7 @@
 define ('JIRAFEAU_ROOT', dirname (__FILE__) . '/');
 define ('NL', "\n");
 define ('QUOTE', "'");
 define ('JIRAFEAU_ROOT', dirname (__FILE__) . '/');
 define ('NL', "\n");
 define ('QUOTE', "'");

-define ('JIRAFEAU_CFG', JIRAFEAU_ROOT.'lib/config.local.php');
+define ('JIRAFEAU_CFG', JIRAFEAU_ROOT . 'lib/config.local.php');

 define ('JIRAFEAU_VAR_RAND_LENGTH', 15);
 
 require (JIRAFEAU_ROOT . 'lib/settings.php');
 define ('JIRAFEAU_VAR_RAND_LENGTH', 15);
 
 require (JIRAFEAU_ROOT . 'lib/settings.php');


@@ -40,32 +40,17 @@ jirafeau_quoted ($str)
 function
 jirafeau_export_cfg ($cfg)
 {
 function
 jirafeau_export_cfg ($cfg)
 {

-    $handle = fopen (JIRAFEAU_CFG, 'w');
-    fwrite ($handle, '<?php' . NL);
-    fwrite ($handle,
-            '/* ' .
-            t ('This file was generated by the install process. ' .
+    $content = '<?php' . NL;
+    $content .= '/* ' . t ('This file was generated by the install process. ' .

                'You can edit it. Please see config.original.php to understand the ' .
                'You can edit it. Please see config.original.php to understand the ' .

-               'configuration items.') . ' */' . NL);
-    foreach ($cfg as $key => $item)
-    {
-        fwrite ($handle, '$cfg[' . jirafeau_quoted ($key) . '] = ');
-        if (is_bool ($item))
-            fwrite ($handle, ($item ? 'true' : 'false'));
-        else if (is_string ($item))
-            fwrite ($handle, jirafeau_quoted ($item));
-        else if (is_int ($item))
-            fwrite ($handle, $item);
-        else if (is_array ($item))
-            fwrite ($handle, str_replace(array("\n", "\r"), "",
-                                         var_export ($item, true)));
-        else
-            fwrite ($handle, 'null');
-        fwrite ($handle, ';'.NL);
+               'configuration items.') . ' */' . NL;
+    $content .= '$cfg = ' . var_export($cfg, true) . ';';
+
+    $fileWrite = file_put_contents(JIRAFEAU_CFG, $content);
+
+    if (false === $fileWrite) {
+        jirafeau_fatal_error(t('Can not write local configuration file'));

     }
     }

-    /* No newline at the end of the file to be able to send headers. */
-    fwrite ($handle, '?>');
-    fclose ($handle);

 }
 
 function
 }
 
 function

diff --git a/lib/settings.php b/lib/settings.php
index 37951221130c3bcb5171011d54b3a658267e7b88..316741431ca3cc1e3ba12cd94308f0c948d41243 100644 (file)
--- a/lib/settings.php
+++ b/lib/settings.php
@@ -21,10 +21,13 @@ global $cfg;
 
 // Read config files
 require (JIRAFEAU_ROOT . 'lib/config.original.php');
 
 // Read config files
 require (JIRAFEAU_ROOT . 'lib/config.original.php');

-

 if (file_exists(JIRAFEAU_ROOT . 'lib/config.local.php'))
 {
 if (file_exists(JIRAFEAU_ROOT . 'lib/config.local.php'))
 {

+       // read local copy and merge with original values
+       $cfgOriginal = $cfg;

        require (JIRAFEAU_ROOT . 'lib/config.local.php');
        require (JIRAFEAU_ROOT . 'lib/config.local.php');

+       $cfg = array_merge($cfgOriginal, $cfg);
+       unset($cfgOriginal);

 }
 
 /* Jirafeau constants */
 }
 
 /* Jirafeau constants */