- The ``view`` attribute of the ``view`` ZCML directive is no longer

  required if the ZCML directive has a ``template`` attribute.

2 files changed, 35 insertions, 1 deletions

diff --git a/repoze/bfg/tests/test_zcml.py b/repoze/bfg/tests/test_zcml.py
index dcf6e17f8..208e8e754 100644
--- a/repoze/bfg/tests/test_zcml.py
+++ b/repoze/bfg/tests/test_zcml.py
@@ -355,6 +355,35 @@ class TestViewDirective(unittest.TestCase):
         result = wrapper(None, None)
         self.assertEqual(result.body, 'Hello.\n')
 
+    def test_with_template_no_view_callable(self):
+        from zope.interface import Interface
+        from zope.component import getSiteManager
+        from repoze.bfg.interfaces import IRequest
+        from repoze.bfg.interfaces import IView
+        from repoze.bfg.interfaces import IViewPermission
+
+        import repoze.bfg.tests
+
+        context = DummyContext(repoze.bfg.tests)
+        class IFoo(Interface):
+            pass
+        import os
+        fixture = 'fixtures/minimal.txt'
+        self._callFUT(context, 'repoze.view', IFoo, template=fixture)
+        actions = context.actions
+        self.assertEqual(len(actions), 1)
+
+        action = actions[0]
+        discrim = ('view', IFoo, '', IRequest, IView, None, None, None, None,
+                   None)
+        self.assertEqual(action['discriminator'], discrim)
+        register = action['callable']
+        register()
+        sm = getSiteManager()
+        wrapper = sm.adapters.lookup((IFoo, IRequest), IView, name='')
+        result = wrapper(None, None)
+        self.assertEqual(result.body, 'Hello.\n')
+
     def test_request_type_asinterface(self):
         from zope.component import getSiteManager
         from zope.interface import Interface
diff --git a/repoze/bfg/zcml.py b/repoze/bfg/zcml.py
index bca28d47e..ff2549447 100644
--- a/repoze/bfg/zcml.py
+++ b/repoze/bfg/zcml.py
@@ -89,7 +89,12 @@ def view(
     ):
 
     if not view:
-        raise ConfigurationError('"view" attribute was not specified')
+        if template:
+            def view(context, request):
+                return {}
+        else:
+            raise ConfigurationError('"view" attribute was not specified and '
+                                     'no template specified')
 
     sm = getSiteManager()