On 2008-12-12 14:24:09 +0100, Martijn Faassen <faassen@startifact.com> said:
Hey,
Christian Zagrodnick wrote: [snip]
That's good. One thing which is not good is that you deprecated the use of ITerms from zope.app.form. I'd just leave the reference/import there like we did with ISite in zope.app.component.
Why is such a deprecation warning bad? Wouldn't this encourage people to update their code?
A deprecation warning isn't bad. But I think we should not deprecate the use of ITerms from zope.app.form. I don't see a gain in this API change. -- Christian Zagrodnick · cz@gocept.com gocept gmbh & co. kg · forsterstraße 29 · 06112 halle (saale) · germany http://gocept.com · tel +49 345 1229889 4 · fax +49 345 1229889 1 Zope and Plone consulting and development