Fix issue #149

CHANGES.txt

@@ -1,3 +1,10 @@
+Next Release
+------------
+
+- Raise a ValueError exception when the prototype
+  for a field in a sequence has no name. See
+  https://github.com/Pylons/deform/issues/149
+
 0.9.7 (2013-03-06)
 ------------------
 

deform/tests/test_widget.py

@@ -1206,6 +1206,17 @@ def test_prototype_unicode(self):
         self.assertEqual(url_unquote(result), 'abc')
         self.assertEqual(protofield.cloned, True)
 
+    def test_prototype_field_has_no_name(self):
+        from deform.compat import url_unquote
+        renderer = DummyRenderer(text_type('abc'))
+        schema = DummySchema()
+        field = DummyField(schema, renderer)
+        widget = self._makeOne()
+        protofield = DummyField(None, renderer)
+        protofield.name = ''
+        field.children=[protofield]
+        self.assertRaises(ValueError, widget.prototype, field)
+
     def test_prototype_str(self):
         from deform.compat import url_unquote
         renderer = DummyRenderer('abc')

deform/widget.py

@@ -1271,6 +1271,9 @@ def prototype(self, field):
         # we clone the item field to bump the oid (for easier
         # automated testing; finding last node)
         item_field = field.children[0].clone()
+        if not item_field.name:
+            info = 'Prototype for %r has no name' % field
+            raise ValueError(info)
         # NB: item_field default should already be set up
         proto = item_field.render_template(self.item_template, parent=field)
         if isinstance(proto, string_types):