Skip to content

Commit

Permalink
[fc] Repository: plone.recipe.zope2instance
Browse files Browse the repository at this point in the history
Branch: refs/heads/master
Date: 2019-08-29T07:51:37+02:00
Author: Daniel Havlik (dhavlik) <dh@gocept.com>
Commit: plone/plone.recipe.zope2instance@8275f5b

repair incomplete commit from last PR (sry) (#125)

Files changed:
M news/122.bugfix
M src/plone/recipe/zope2instance/ctl.py
  • Loading branch information
icemac committed Aug 29, 2019
1 parent 36c0bd4 commit 9a06861
Showing 1 changed file with 8 additions and 7 deletions.
15 changes: 8 additions & 7 deletions last_commit.txt
Original file line number Diff line number Diff line change
@@ -1,15 +1,16 @@
Repository: plone.app.content
Repository: plone.recipe.zope2instance


Branch: refs/heads/master
Date: 2019-08-28T11:56:04-03:00
Author: Franco Pellegrini (frapell) <frapell@gmail.com>
Commit: https://github.com/plone/plone.app.content/commit/2c07df005b365c6f46719aa962c3c90791961d82
Date: 2019-08-29T07:51:37+02:00
Author: Daniel Havlik (dhavlik) <dh@gocept.com>
Commit: https://github.com/plone/plone.recipe.zope2instance/commit/8275f5ba00486f49b38a3d837c589a745ed88f0d

BUGFIX: self.errors is a list, so .append() should be used
repair incomplete commit from last PR (sry) (#125)

Files changed:
M plone/app/content/browser/contents/rename.py
M news/122.bugfix
M src/plone/recipe/zope2instance/ctl.py

b"diff --git a/plone/app/content/browser/contents/rename.py b/plone/app/content/browser/contents/rename.py\nindex db61450..8e76ec1 100644\n--- a/plone/app/content/browser/contents/rename.py\n+++ b/plone/app/content/browser/contents/rename.py\n@@ -71,7 +71,7 @@ def __call__(self):\n obj = brains[0].getObject()\n title = self.objectTitle(obj)\n if not mtool.checkPermission('Copy or Move', obj):\n- self.errors(_(u'Permission denied to rename ${title}.',\n+ self.errors.append(_(u'Permission denied to rename ${title}.',\n mapping={u'title': title}))\n continue\n \n"
b'diff --git a/news/122.bugfix b/news/122.bugfix\nindex 3ffd825..07cd201 100644\n--- a/news/122.bugfix\n+++ b/news/122.bugfix\n@@ -1 +1 @@\n-Fixed a deprecation warning when importing "makerequest" from Testing.ZopeTestCase.utils. This also fixes a side effect which can bite you if you rely on ZOPETESTCASE environment variable, only being set to "1" during test runs.\n+Fixed a deprecation warning when importing `makerequest` from Testing.ZopeTestCase.utils. This also fixes a side effect which can bite you if you rely on ZOPETESTCASE environment variable, only being set to `1` during test runs.\ndiff --git a/src/plone/recipe/zope2instance/ctl.py b/src/plone/recipe/zope2instance/ctl.py\nindex a07b8c6..c6fd851 100644\n--- a/src/plone/recipe/zope2instance/ctl.py\n+++ b/src/plone/recipe/zope2instance/ctl.py\n@@ -679,7 +679,7 @@ def get_startup_cmd(self, python, more, pyflags=""):\n if not self.options.no_request:\n cmdline += (\n \'from Testing.makerequest import makerequest; \'\n- \'app = Testing.makerequest.makerequest(app); \'\n+ \'app = makerequest(app); \'\n # REQUEST.traverse needs this but no reason not to set\n # this even if we\'re not traversing to an object\n \'app.REQUEST[\\\'PARENTS\\\'] = [app]; \'\n'

0 comments on commit 9a06861

Please sign in to comment.