Merge pull request #60 from plone/confirm-view-for-all-contexts

Allow `confirm-action` for all contexts, instead of only Plone Site
plone · Nov 25, 2016 · 2be2215 · 2be2215
2 parents 9ef9881 + e273918
commit 2be2215
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 1 deletion.
diff --git a/CHANGES.rst b/CHANGES.rst
@@ -14,8 +14,14 @@ New features:
 
 Bug fixes:
 
+- Allow ``confirm-action`` for all contexts, instead of only Plone Site root.
+  This avoids an error when calling it on a subsite.
+  Fixes `issue #51 <https://github.com/plone/plone.protect/issues/51>`_.
+  [maurits]
+
 - Code Style: utf8-headers, import sorting, new style namespace declaration, autopep8
   [jensens]
+
 - Fix #57: Html must contain "body", otherwise plone.protect breaks.
   [jensens]
 

diff --git a/plone/protect/configure.zcml b/plone/protect/configure.zcml
@@ -42,7 +42,7 @@
 
     <browser:page
         name="confirm-action"
-        for="Products.CMFPlone.interfaces.IPloneSiteRoot"
+        for="*"
         class=".views.ConfirmView"
         template="confirm.pt"
         permission="zope2.View"