[Stefan H. Holek]
> I am pretty sure this should say "isinstance(connection, basestring)",
> not "type".
>
>      def __call__(self, connection=None):
>          db, aname, version_support = self._stuff
>
>          if connection is None:
>              connection=db.open()
>          elif isinstance(type, basestring):
>              connection=db.open(connection)
>
>          return connection.root()[aname]

Ya, I don't know why this is even part of ZODB, but that code is clearly
senseless.  OTOH, I see that change was made more than a year ago, and
you're the first to notice <wink>.

_______________________________________________
For more information about ZODB, see the ZODB Wiki:
http://www.zope.org/Wikis/ZODB/

ZODB-Dev mailing list  -  ZODB-Dev@zope.org
http://mail.zope.org/mailman/listinfo/zodb-dev

Reply via email to