Remove unnecessary adapter

collective · Apr 13, 2016 · 9309c5d · 9309c5d
1 parent 6bfb8d3
commit 9309c5d
Show file tree

Hide file tree

Showing 4 changed files with 7 additions and 24 deletions.
diff --git a/CHANGES.rst b/CHANGES.rst
@@ -6,6 +6,12 @@ There's a frood who really knows where his towel is.
 1.1b2 (unreleased)
 ^^^^^^^^^^^^^^^^^^
 
+- The new version of plone.app.layout fix the ``<base>`` TAG to use the request ACTUAL_URL
+  variable instead of the context ``absolute_url``.
+  This remove the need to adapt the ``renderBase`` method to use ``cover URL + '/'`` as
+  ``<base>`` URL.
+  [rodfersou]
+
 - Handle `AssertionError` on upgrade step to profile 13 to avoid failures when a cover object has duplicated tiles on it.
   Now, an error message will be logged and the object will be skipped;
   you must manually remove the duplicated tiles (closes #619).

diff --git a/setup.py b/setup.py
@@ -56,7 +56,7 @@
           'plone.app.iterate',
           'plone.app.jquery >=1.7.2',
           'plone.app.jquerytools >=1.5.1',
-          'plone.app.layout',
+          'plone.app.layout >=2.3.14',
           'plone.app.linkintegrity',
           'plone.app.lockingbehavior',
           'plone.app.registry',

diff --git a/src/collective/cover/adapters.py b/src/collective/cover/adapters.py
diff --git a/src/collective/cover/configure.zcml b/src/collective/cover/configure.zcml
@@ -57,14 +57,6 @@
     <browser:page attribute="jsonByType" name="jsonbytype" />
   </browser:pages>
 
-  <browser:page
-      for="collective.cover.content.ICover"
-      name="plone"
-      class=".adapters.PloneView"
-      permission="zope.Public"
-      allowed_interface="Products.CMFPlone.browser.interfaces.IPlone"
-      />
-
   <class class=".widgets.selectpreview.SelectPreviewWidget">
     <require
         permission="zope.Public"