[Testbot] Plone 4.3 - Python 2.7 - Build # 2038 - Still failing! - 2 failure(s)

jenkins at plone.org jenkins at plone.org
Wed Mar 12 10:09:42 UTC 2014


-------------------------------------------------------------------------------
Plone 4.3 - Python 2.7 - Build # 2038 - Still Failing!
-------------------------------------------------------------------------------

http://jenkins.plone.org/job/plone-4.3-python-2.7/2038/


-------------------------------------------------------------------------------
CHANGES
-------------------------------------------------------------------------------

Repository: Products.CMFPlone
Branch: refs/heads/4.3.x
Date: 2014-03-05T16:26:51+01:00
Author: Jan Van Hees (jakke) <Jakkevanhees at gmail.com>
Commit: https://github.com/plone/Products.CMFPlone/commit/5033eac4238430b1564efdc82f3c519c83e20f3f

use getPhysicalPath instead of absolute_url_path

Files changed:
M Products/CMFPlone/PloneTool.py

diff --git a/Products/CMFPlone/PloneTool.py b/Products/CMFPlone/PloneTool.py
index 48d15d1..b93539f 100644
--- a/Products/CMFPlone/PloneTool.py
+++ b/Products/CMFPlone/PloneTool.py
@@ -1242,7 +1242,8 @@ def deleteObjectsByPaths(self, paths, handle_errors=True, REQUEST=None):
                 # To avoid issues with the check for acquisition,
                 # relative paths should not be part of the API anymore.
                 # Plone core itself does not use relative paths.
-                if not path.startswith(portal.absolute_url_path()):
+                #if not path.startswith(portal.absolute_url_path()):
+                if not path.startswith("/".join(portal.getPhysicalPath())):
                     msg = (
                         'Path {} does not start '
                         'with path to portal'.format(path)


Repository: Products.CMFPlone
Branch: refs/heads/4.3.x
Date: 2014-03-12T10:23:02+01:00
Author: Godefroid Chapelle (gotcha) <gotcha at bubblenet.be>
Commit: https://github.com/plone/Products.CMFPlone/commit/5a930503bd9c2ccea855e9a168c4ea7618d5b682

Merge pull request #190 from spereverde/plonetool_physicalpathfix

use getPhysicalPath instead of absolute_url_path

Files changed:
M Products/CMFPlone/PloneTool.py

diff --git a/Products/CMFPlone/PloneTool.py b/Products/CMFPlone/PloneTool.py
index 48d15d1..b93539f 100644
--- a/Products/CMFPlone/PloneTool.py
+++ b/Products/CMFPlone/PloneTool.py
@@ -1242,7 +1242,8 @@ def deleteObjectsByPaths(self, paths, handle_errors=True, REQUEST=None):
                 # To avoid issues with the check for acquisition,
                 # relative paths should not be part of the API anymore.
                 # Plone core itself does not use relative paths.
-                if not path.startswith(portal.absolute_url_path()):
+                #if not path.startswith(portal.absolute_url_path()):
+                if not path.startswith("/".join(portal.getPhysicalPath())):
                     msg = (
                         'Path {} does not start '
                         'with path to portal'.format(path)




-------------------------------------------------------------------------------
-------------- next part --------------
A non-text attachment was scrubbed...
Name: CHANGES.log
Type: application/octet-stream
Size: 2431 bytes
Desc: not available
URL: <http://lists.plone.org/pipermail/plone-testbot/attachments/20140312/7ec74ccb/attachment-0002.obj>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: build.log
Type: application/octet-stream
Size: 78235 bytes
Desc: not available
URL: <http://lists.plone.org/pipermail/plone-testbot/attachments/20140312/7ec74ccb/attachment-0003.obj>


More information about the Testbot mailing list