Make zope.app.publication dependency optional.

CHANGES.txt

@@ -5,7 +5,7 @@ CHANGES
 1.3.5 (unreleased)
 ------------------
 
-- Nothing changed yet.
+- Make zope.app.publication dependency optional.
 
 
 1.3.4 (2012-01-20)

setup.py

@@ -58,7 +58,6 @@ def read(*rnames):
                 'zope.testing',
                 ]),
       install_requires=['setuptools',
-                        'zope.app.publication',
                         'zope.browserpage',
                         'zope.browserresource',
                         'zope.component',

src/zc/resourcelibrary/configure.zcml

@@ -19,6 +19,7 @@
       />
 
   <utility
+      zcml:condition="installed zope.app.publication"
       component=".publication.Request"
       provides="zope.app.publication.interfaces.IBrowserRequestFactory"
       />

src/zc/resourcelibrary/publication.py

@@ -15,7 +15,6 @@
 $Id: publication.py 4528 2005-12-23 02:45:25Z gary $
 """
 from zope import interface
-from zope.app.publication.interfaces import IBrowserRequestFactory
 from zope.component import queryMultiAdapter, getMultiAdapter
 from zope.publisher.browser import BrowserRequest, BrowserResponse
 from zope.publisher.browser import isHTML
@@ -25,6 +24,11 @@
 import zc.resourcelibrary
 import zope.component.hooks
 
+try:
+    from zope.app.publication.interfaces import IBrowserRequestFactory
+except:
+    class IBrowserRequestFactory(interface.Interface):
+        pass
 
 class Request(BrowserRequest):
     interface.classProvides(IBrowserRequestFactory)
@@ -97,7 +101,7 @@ def retry(self):
     def _implicitResult(self, body):
         #figure out the content type
         content_type = self.getHeader('content-type')
-        if content_type is None:
+        if content_type is None and self._status != 304:
             if isHTML(body):
                 content_type = 'text/html'
             else: