[Zope-dev] Re: Post-mortem [Was: Zope 2.7.0 b3 regressions]
Brian Lloyd
brian at zope.com
Tue Dec 9 13:23:40 EST 2003
> Summary: absolute_url(1) didn't include the path to the virtual root,
> which broke code that assumed that it could just prepend "/" in all
> cases. I changed it to include the base path, and broke code that
> prepends BASEPATH1.
>
> Since the old behavior existed for two years (including part of the 2.7
> beta), is arguably easier to work with, and works with existing CMF
> code, I will revert to it. This will leave "/"-prepending code broken.
>
> In line with Stefan's original suggestion, I will add a new method with
> alternate behavior. It's not hard to grep for broken code and use the
> new method to make it correct -- Product authors should do so.
Good call. I think it would be best to make sure the docstring of the
new method is clear on its reason for being. I think somewhere there
is an interface file that is used to generate some of the api docs -
ideally that can get updated too.
Brian Lloyd brian at zope.com
V.P. Engineering 540.361.1716
Zope Corporation http://www.zope.com
More information about the Zope-Dev
mailing list