expire_options and formatter_options should not be filled up with

default values, resolves #52

cfg/.gitignore

@@ -0,0 +1 @@
+/conf.ini

lib/configuration.php

@@ -95,11 +95,13 @@ class configuration
                 }
             }
         }
+        $opts = '_options';
         foreach ($this->_defaults as $section => $values)
         {
-            if (!array_key_exists($section, $config))
+            // fill missing sections with default values
+            if (!array_key_exists($section, $config) || count($config[$section]) == 0)
             {
-                $this->_configuration[$section] = $this->_defaults[$section];
+                $this->_configuration[$section] = $values;
                 if (array_key_exists('dir', $this->_configuration[$section]))
                 {
                     $this->_configuration[$section]['dir'] = PATH . $this->_configuration[$section]['dir'];
@@ -117,47 +119,71 @@ class configuration
                     'opt' => array(PDO::ATTR_PERSISTENT => true),
                 );
             }
-            foreach ($values as $key => $val)
+
+            // "*_options" sections don't require all defaults to be set
+            if (
+                $section !== 'model_options' &&
+                ($from = strlen($section) - strlen($opts)) >= 0 &&
+                strpos($section, $opts, $from) !== false
+            )
             {
-                if ($key == 'dir')
+                if (is_int(current($values)))
                 {
-                    $val = PATH . $val;
+                    $config[$section] = array_map('intval', $config[$section]);
                 }
-                $result = $val;
-                if (array_key_exists($key, $config[$section]))
+                $this->_configuration[$section] = $config[$section];
+            }
+            // check for missing keys and set defaults if necessary
+            else
+            {
+                foreach ($values as $key => $val)
                 {
-                    if ($val === null)
+                    if ($key == 'dir')
                     {
-                        $result = $config[$section][$key];
+                        $val = PATH . $val;
                     }
-                    elseif (is_bool($val))
+                    $result = $val;
+                    if (array_key_exists($key, $config[$section]))
                     {
-                        $val = strtolower($config[$section][$key]);
-                        if (in_array($val, array('true', 'yes', 'on')))
+                        if ($val === null)
                         {
-                            $result = true;
+                            $result = $config[$section][$key];
                         }
-                        elseif (in_array($val, array('false', 'no', 'off')))
+                        elseif (is_bool($val))
                         {
-                            $result = false;
+                            $val = strtolower($config[$section][$key]);
+                            if (in_array($val, array('true', 'yes', 'on')))
+                            {
+                                $result = true;
+                            }
+                            elseif (in_array($val, array('false', 'no', 'off')))
+                            {
+                                $result = false;
+                            }
+                            else
+                            {
+                                $result = (bool) $config[$section][$key];
+                            }
                         }
-                        else
+                        elseif (is_int($val))
                         {
-                            $result = (bool) $config[$section][$key];
+                            $result = (int) $config[$section][$key];
+                        }
+                        elseif (is_string($val) && !empty($config[$section][$key]))
+                        {
+                            $result = (string) $config[$section][$key];
                         }
                     }
-                    elseif (is_int($val))
-                    {
-                        $result = (int) $config[$section][$key];
-                    }
-                    elseif (is_string($val) && !empty($config[$section][$key]))
-                    {
-                        $result = (string) $config[$section][$key];
-                    }
+                    $this->_configuration[$section][$key] = $result;
                 }
-                $this->_configuration[$section][$key] = $result;
             }
         }
+
+        // ensure a valid expire default key is set
+        if (!array_key_exists($this->_configuration['expire']['default'], $this->_configuration['expire_options']))
+        {
+            $this->_configuration['expire']['default'] = key($this->_configuration['expire_options']);
+        }
     }
 
     /**

tst/configuration.php

@@ -130,8 +130,9 @@ class configurationTest extends PHPUnit_Framework_TestCase
 
     public function testHandleWrongTypes()
     {
-        $this->_options['main']['syntaxhighlightingtheme'] = 'foo';
-        $options = $this->_options;
+        $original_options = $this->_options;
+        $original_options['main']['syntaxhighlightingtheme'] = 'foo';
+        $options = $original_options;
         $options['main']['discussion'] = 'true';
         $options['main']['opendiscussion'] = 0;
         $options['main']['password'] = -1; // evaluates to TRUE
@@ -140,6 +141,21 @@ class configurationTest extends PHPUnit_Framework_TestCase
         $options['formatter_options'][] = 'foo';
         helper::createIniFile(CONF, $options);
         $conf = new configuration;
-        $this->assertEquals($this->_options, $conf->get(), 'incorrect types are corrected');
+        $original_options['expire_options']['foo'] = intval('bar');
+        $original_options['formatter_options'][0] = 'foo';
+        $this->assertEquals($original_options, $conf->get(), 'incorrect types are corrected');
+    }
+
+    public function testHandleMissingSubKeys()
+    {
+        $options = $this->_options;
+        unset($options['expire_options']['1week']);
+        unset($options['expire_options']['1year']);
+        unset($options['expire_options']['never']);
+        helper::createIniFile(CONF, $options);
+        $conf = new configuration;
+        $options['expire']['default'] = '5min';
+        $this->assertEquals($options, $conf->get(), 'not overriding "missing" subkeys');
     }
+
 }