Merge pull request #1 from zopefoundation/fix-parse

- Fixed: When parsing configuration text, sections were input and

src/zc/metarecipe/README.txt

@@ -227,6 +227,11 @@ That's pretty much it.
 Changes
 =======
 
+- Fixed: When parsing configuration text, sections were input and
+         evaluated at the same time in section sorted order. This
+         caused problems if a section that sorted early referred to a
+         section that sorted late.
+
 0.2.0 (2012-09-24)
 ------------------
 

src/zc/metarecipe/__init__.py

@@ -20,9 +20,14 @@ def __setitem__(self, name, data):
     def parse(self, data):
         parser = ConfigParser.RawConfigParser()
         parser.readfp(cStringIO.StringIO(textwrap.dedent(data)))
-
-        for section in sorted(parser.sections()):
-            self[section] = dict(parser.items(section))
+        buildout = self.buildout
+        raw = buildout._raw
+        sections = sorted(parser.sections())
+
+        for section in sections:
+            raw[section] = dict(parser.items(section))
+        for section in sections:
+            buildout[section]
 
 validtypes = unicode, int
 

src/zc/metarecipe/tests.py

@@ -40,6 +40,27 @@ def test_funky_option_types():
 
     """
 
+class Dict(dict):
+
+    def __setitem__(self, name, value):
+        print 'raw setitem', name, value
+        dict.__setitem__(self, name, value)
+
+def parse_does_raw_before_calling_getitem():
+    r"""
+    >>> import zc.metarecipe.testing
+    >>> buildout = zc.metarecipe.testing.Buildout()
+    >>> buildout._raw = Dict()
+    >>> recipe = zc.metarecipe.Recipe(buildout, '', {})
+    >>> recipe.parse("[a]\nx=1\n[b]\ny=2\n")
+    raw setitem a {'x': '1'}
+    raw setitem b {'y': '2'}
+    [a]
+    x = 1
+    [b]
+    y = 2
+    """
+
 def test_suite():
     return unittest.TestSuite((
         manuel.testing.TestSuite(