Tracker log

An overview of recent activity in the tracker.

Tres Seaver added a new response to »Use adapter rather than utility to replace script processing«:

Patch applied to head:

  https://dev.plone.org/[…]/CMFPaypalHelper?rev=41349

  https://dev.plone.org/[…]/CMFPaypalHelper?rev=41348

(in two pieces due to missing added files)

Added 1746 day(s) ago.

Tres Seaver added a new response to »''default_cancel_path'' seems identical to ''default_return_path''«:

Patch applied to head:

  https://dev.plone.org/[…]/CMFPaypalHelper?rev=41349

  https://dev.plone.org/[…]/CMFPaypalHelper?rev=41348

(in two pieces due to missing added files)

Added 1746 day(s) ago.

Tres Seaver added a new response to »Spurious Plone dependency«:

Patch applied to head:

  https://dev.plone.org/[…]/CMFPaypalHelper?rev=41349

  https://dev.plone.org/[…]/CMFPaypalHelper?rev=41348

(in two pieces due to missing added files)

Added 1746 day(s) ago.

Tres Seaver added a new response to »Tool is missing class initialization«:

Patch applied to head:

  https://dev.plone.org/[…]/CMFPaypalHelper?rev=41349

  https://dev.plone.org/[…]/CMFPaypalHelper?rev=41348

(in two pieces due to missing added files)

Added 1746 day(s) ago.

Tres Seaver added a new response to »Deprecation warning in recent CMF«:

Patch applied to head:

  https://dev.plone.org/[…]/CMFPaypalHelper?rev=41349

  https://dev.plone.org/[…]/CMFPaypalHelper?rev=41348

(in two pieces due to missing added files)

Added 1746 day(s) ago.

New issue »Deprecation warning in recent CMF« added by Tres Seaver



The 'product_name' parameter to 'Products.CMFCore.utils.ToolInot' is now ignored,
and deprectaed. The attached patch removes it from CMFPaypalHelper's product initialization.

Added 1753 day(s) ago.

New issue »Use adapter rather than utility to replace script processing« added by Tres Seaver



Because utilities, even local ones, may not have access to the portal
object when processing an update from Paypal, the attached patch makes
the component used in place of the script an adapter, whose context
is the site, rather than a utility.

It should be applied *after* applying the patch from

 http://plone.org/products/cmfpaypalhelper/issues/2

Added 1758 day(s) ago.

New issue »Tool is missing class initialization« added by Tres Seaver



The tool makes security declarations, but fails to to the class initialization
which would make them effective.

Added 1758 day(s) ago.

New issue »Spurious Plone dependency« added by Tres Seaver



CMFPaypalHelper 0.1.2 contains a spurious import dependency on the CMFPlone product.
The following patch agains CMFPaypalHelper 0.1.2 removes that dependency, making it
possible to use the product from within any CMF-based site. It also cleans up a number
of coding style / impelmentation issues:

o Fixes http://plone.org/products/cmfpaypalhelper/issues/1

o Normalizes imports

o Defers side-efffects from product import time to product initialization
  time

o Trims lines to < 80 characters

o Uses the stock Zope2 pattern for declaring properties

o Adds and tests a tool interface

o Moves permissions out into a separate module

o Adds unit tests

o Makes the 'script' delegation defer to a utility, if registered

o Uses a utility to test the callback to Paypal without doing the
  actual unwanted / faux request.

Added 1759 day(s) ago.

New issue »''default_cancel_path'' seems identical to ''default_return_path''« added by Juan Pablo Di Lelle



I think that:
        cancel_url = html_quote(return_url)
should be:
        cancel_url = html_quote(cancel_url)
in line 119 of PaypalTool.py (in function makePaypalButton).

Otherwise the makePaypalButton always generates a bogus cancel path.

Added 2004 day(s) ago.