From 0de032132c60f51d34beb285e55e27c9f5a68012 Mon Sep 17 00:00:00 2001
From: helmutm <helmutm@fd906abe-77d9-0310-91a1-e0d9ade77398>
Date: Sun, 25 Mar 2007 12:36:21 +0000
Subject: [PATCH] provide 'options' field for loops root and menu objects; make
 versioning depend on 'useVersioning' option

git-svn-id: svn://svn.cy55.de/Zope3/src/loops/trunk@1664 fd906abe-77d9-0310-91a1-e0d9ade77398
---
 base.py                   |  4 ++++
 browser/common.py         | 11 +++++++++++
 browser/concept_macros.pt | 19 +++++++++++--------
 browser/node.py           | 11 +++++++++++
 configure.zcml            |  2 +-
 interfaces.py             | 14 ++++++++++++++
 search/search.pt          | 19 +++++++++++--------
 7 files changed, 63 insertions(+), 17 deletions(-)

diff --git a/base.py b/base.py
index 6256733..4e6aef2 100644
--- a/base.py
+++ b/base.py
@@ -51,6 +51,10 @@ class Loops(Folder):
     def setSkinName(self, skinName): self._skinName = skinName
     skinName = property(getSkinName, setSkinName)
 
+    def getOptions(self): return getattr(self, '_options', [])
+    def setOptions(self, value): self._options = value
+    options = property(getOptions, setOptions)
+
     def getLoopsRoot(self):
         return self
 
diff --git a/browser/common.py b/browser/common.py
index 7bca5f7..43a0637 100644
--- a/browser/common.py
+++ b/browser/common.py
@@ -97,6 +97,7 @@ class BaseView(GenericView):
         self.setSkin(self.loopsRoot.skinName)
         try:
             if not canAccess(context, 'title'):
+                #raise Unauthorized
                 request.response.redirect('login.html')
         except ForbiddenAttribute:  # ignore when testing
             pass
@@ -246,6 +247,14 @@ class BaseView(GenericView):
 
     # versioning
 
+    @Lazy
+    def useVersioning(self):
+        if 'useVersioning' in self.loopsRoot.options:
+            return True
+        options = getattr(self.controller, 'options', None)
+        if options:
+            return 'useVersioning' in options.value
+
     @Lazy
     def versionId(self):
         context = self.context
@@ -260,6 +269,8 @@ class BaseView(GenericView):
 
     @Lazy
     def versionInfo(self):
+        if not self.useVersioning:
+            return None
         context = self.context
         versionable = IVersionable(context, None)
         if versionable is None:
diff --git a/browser/concept_macros.pt b/browser/concept_macros.pt
index 74d805a..9bac39c 100644
--- a/browser/concept_macros.pt
+++ b/browser/concept_macros.pt
@@ -87,7 +87,8 @@
           <tr>
             <th i18n:translate="label_title">Title</th>
             <th i18n:translate="label_type">Type</th>
-            <th i18n:translate="label_version">V</th>
+            <th i18n:translate="label_version"
+                tal:condition="view/useVersioning">V</th>
             <th i18n:translate="label_size">Size</th>
             <th i18n:translate="label_modifdate">Modification Date</th>
           </tr>
@@ -103,13 +104,15 @@
                   </a>
                 </td>
                 <td><span tal:replace="related/longTypeTitle">Type</span></td>
-                <td style="text-align: center"
-                    tal:define="versionId related/versionId">
-                  <a href="#"
-                     tal:content="versionId"
-                     tal:omit-tag="python: versionId=='1.1'"
-                     tal:attributes="href string:${view/url}/.target${related/uniqueId}?loops.viewName=listversions">1.1</a>
-                </td>
+                <tal:version tal:condition="view/useVersioning">
+                  <td style="text-align: center"
+                      tal:define="versionId related/versionId">
+                    <a href="#"
+                       tal:content="versionId"
+                       tal:omit-tag="python: versionId=='1.1'"
+                       tal:attributes="href string:${view/url}/.target${related/uniqueId}?loops.viewName=listversions">1.1</a>
+                  </td>
+                </tal:version>
                 <td style="text-align: right">
                   <span tal:replace="related/context/sizeForDisplay">Type</span>
                 </td>
diff --git a/browser/node.py b/browser/node.py
index 5bd64ad..134869d 100644
--- a/browser/node.py
+++ b/browser/node.py
@@ -584,6 +584,17 @@ class ViewPropertiesConfigurator(object):
         return setting.get('skinName', {}).get('value', '')
     skinName = property(getSkinName, setSkinName)
 
+    def setOptions(self, options):
+        ann = IAnnotations(self.context)
+        setting = ann.get(configurator.ANNOTATION_KEY, {})
+        setting['options'] = {'value': options}
+        ann[configurator.ANNOTATION_KEY] = setting
+    def getOptions(self):
+        ann = IAnnotations(self.context)
+        setting = ann.get(configurator.ANNOTATION_KEY, {})
+        return setting.get('options', {}).get('value', [])
+    options = property(getOptions, setOptions)
+
 
 class NodeViewConfigurator(configurator.ViewConfigurator):
     """ Take properties from next menu item...
diff --git a/configure.zcml b/configure.zcml
index c366abb..05687ee 100644
--- a/configure.zcml
+++ b/configure.zcml
@@ -80,7 +80,7 @@
     <require
         permission="zope.View"
         attributes="getLoopsRoot getLoopsUri loopsTraverse getConceptManager
-                    getResourceManager getViewManager skinName" />
+                    getResourceManager getViewManager skinName options" />
 
     <require
         permission="zope.ManageContent"
diff --git a/interfaces.py b/interfaces.py
index 134f723..50d5173 100644
--- a/interfaces.py
+++ b/interfaces.py
@@ -498,6 +498,13 @@ class ILoops(ILoopsObject):
         default='',
         required=False)
 
+    options = schema.List(
+        title=_(u'Options'),
+        description=_(u'Additional settings.'),
+        value_type=schema.TextLine(),
+        default=[],
+        required=False)
+
     def getLoopsUri(obj):
         """ Return the relativ path to obj, starting with '.loops/...'.
         """
@@ -642,4 +649,11 @@ class IViewConfiguratorSchema(Interface):
         default=u'',
         required=False)
 
+    options = schema.List(
+        title=_(u'Options'),
+        description=_(u'Additional settings.'),
+        value_type=schema.TextLine(),
+        default=[],
+        required=False)
+
 
diff --git a/search/search.pt b/search/search.pt
index 6ae8da5..0faf31f 100644
--- a/search/search.pt
+++ b/search/search.pt
@@ -50,7 +50,8 @@
             <tr>
               <th i18n:translate="label_title">Title</th>
               <th i18n:translate="label_type">Type</th>
-              <th i18n:translate="label_version">V</th>
+              <th i18n:translate="label_version"
+                  tal:condition="view/useVersioning">V</th>
               <th i18n:translate="label_size">Size</th>
               <th i18n:translate="label_modifdate">Modification Date</th>
             </tr>
@@ -66,13 +67,15 @@
                        tal:content="row/title" />
                   </td>
                   <td tal:content="row/longTypeTitle|row/typeTitle">Type</td>
-                  <td style="text-align: center"
-                      tal:define="versionId row/versionId|string:">
-                    <a href="#"
-                       tal:content="versionId"
-                       tal:omit-tag="python: versionId and versionId=='1.1'"
-                       tal:attributes="href string:${view/url}/.target${row/uniqueId}?loops.viewName=listversions">1.1</a>
-                  </td>
+                  <tal:version condition="view/useVersioning">
+                    <td style="text-align: center"
+                        tal:define="versionId row/versionId|string:">
+                      <a href="#"
+                         tal:content="versionId"
+                         tal:omit-tag="python: versionId and versionId=='1.1'"
+                         tal:attributes="href string:${view/url}/.target${row/uniqueId}?loops.viewName=listversions">1.1</a>
+                    </td>
+                  </tal:version>
                   <td style="text-align: right">
                     <span tal:replace="row/context/sizeForDisplay|string:">Size</span>
                   </td>