Fix config container reader for:

[section]
group.name = name

Author: c9s

src/Onion/Command/BuildCommand.php

@@ -66,6 +66,7 @@ function execute($arguments = array())
         else
             $logger->warn( '* doc/ directory not found.',1 );
 
+
         $logger->info( 'Configuring package.ini' );
         $config = new PackageConfigReader($logger);
         $config->readAsPackageXml();

src/Onion/ConfigContainer.php

@@ -11,6 +11,22 @@
 namespace Onion;
 use Exception;
 
+
+/**
+ * ini config container provides an interface for retrive strucutre config value
+ *
+ *  [section]
+ *  name = value
+ *
+ *  $config->get('section.name');
+ *
+ *
+ *  [section]
+ *  name.subname = value
+ *
+ *  $config->get('section.name.subname')
+ *
+ */
 class ConfigContainer 
 {
     public $array;
@@ -37,7 +53,7 @@ function __isset($k)
 
     function has( $refstr ) 
     {
-        $paths = explode('.',$refstr);
+        $paths = explode('.',$refstr,2);
         $ref = & $this->array;
         foreach( $paths as $p ) {
             if( ! isset( $ref[ $p ] ) )
@@ -49,7 +65,7 @@ function has( $refstr )
 
     function set( $refstr , $v ) 
     {
-        $paths = explode('.',$refstr);
+        $paths = explode('.',$refstr,2);
         $ref = & $this->array;
         foreach( $paths as $p ) {
             if( ! isset( $ref[ $p ] ) ) {
@@ -65,7 +81,7 @@ function set( $refstr , $v )
 
     function get( $refstr )
     {
-        $paths = explode('.',$refstr);
+        $paths = explode('.',$refstr,2);
         $ref = $this->array;
         foreach( $paths as $path ) {
             if( isset($ref[$path]) ) {

src/Onion/PackageConfigReader.php

@@ -17,6 +17,21 @@
 use Onion\ConfigContainer;
 use Onion\SpecUtils;
 
+/**
+ *
+ * package ini file reader:
+ *
+ *      $reader = new PackageIniReader;
+ *      $pkginfo = $reader->read( 'file.ini' );
+ *
+ *      $pkginfo->get( 'config' );
+ *      $pkginfo->get( 'config.name' );
+ *      $pkginfo->get( 'section.name' );
+ *
+ *      $pkgxml = new PackageXmlGenerator( $pkginfo );
+ *      $pkgxml->generate();
+ *
+ */
 class PackageConfigReader
 {
     public $file;
@@ -430,6 +445,7 @@ function generatePackageXml()
             die( $e->getMessage() );
         } 
     }
-
 }
 
+
+

tests/Onion/ConfigContainerTest.php

@@ -5,7 +5,7 @@ function test()
     {
         $config = new \Onion\ConfigContainer(array( 
             'key' => array( 
-                'subkey' => array( 'hash' => 1 ),
+                'subkey.hash' => 1
             )
         ));
         ok( $config );
@@ -23,7 +23,6 @@ function test()
 
         ok( ! $config->has('non') );
         ok( $config->has('key') );
-        ok( $config->has('key.subkey') );
         ok( $config->has('key.subkey2') );
         ok( $config->has('key.subkey.hash3') );
     }