[Checkins] SVN: zope.app.apidoc/trunk/src/zope/app/apidoc/codemodule/module.py If there is something the same name beneath, then module should have priority.

Christian Theune ct at gocept.com
Fri Apr 24 01:35:25 EDT 2009


Hi there,

On Thu, 2009-04-23 at 11:11 -0400, Yusei Tahara wrote:
> Log message for revision 99417:
>   If there is something the same name beneath, then module should have priority.
>   
> 
> Changed:
>   U   zope.app.apidoc/trunk/src/zope/app/apidoc/codemodule/module.py
> 
> -=-
> Modified: zope.app.apidoc/trunk/src/zope/app/apidoc/codemodule/module.py
> ===================================================================
> --- zope.app.apidoc/trunk/src/zope/app/apidoc/codemodule/module.py	2009-04-23 04:59:07 UTC (rev 99416)
> +++ zope.app.apidoc/trunk/src/zope/app/apidoc/codemodule/module.py	2009-04-23 15:11:16 UTC (rev 99417)
> @@ -124,6 +124,11 @@
>                      names.append(name)
>  
>          for name in names:
> +            # If there is something the same name beneath, then module should
> +            # have priority.
> +            if name in self._children:
> +                continue
> +
>              attr = getattr(self._module, name, None)
>              if attr is None:
>                  continue

Is there a chance you can elaborate on that comment a little bit more? I
looked at the code but didn't understand whats going on (that function
is hard to read already, I know). Also: any chance to a get (small) test
that demonstrates what you changed there?

Christian

-- 
Christian Theune · ct at gocept.com
gocept gmbh & co. kg · forsterstraße 29 · 06112 halle (saale) · germany
http://gocept.com · tel +49 345 1229889 7 · fax +49 345 1229889 1
Zope and Plone consulting and development
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: This is a digitally signed message part
Url : http://mail.zope.org/pipermail/checkins/attachments/20090424/3bedb49a/attachment.bin 


More information about the Checkins mailing list