[Zope-dev] Re: SVN: zope.component/ Merge wichert-utility-factories branch to trunk

Wichert Akkerman wichert at wiggy.net
Thu Jul 24 13:45:46 EDT 2008


Jacob Holm wrote:
> Hello
>
> This checkin contains the following change to 
> zope.component/trunk/src/zope/component/registry.py
> which looks wrong to me (it's a noop).   It looks to me as if the [:2] 
> should move outside the ")" following it.  I am surprised that this 
> doesn't break any tests, as it means that reregistrations are not 
> detected as such.
>
> -        if (self._utility_registrations.get((provided, name))
> +        if (self._utility_registrations.get((provided, name)[:2]) 

Excellent catch, fixed.

Wichert.

-- 
Wichert Akkerman<wichert at wiggy.net>    It is simple to make things.
http://www.wiggy.net/                  It is hard to make things simple.



More information about the Zope-Dev mailing list