[ZCM] [ZC] 388/ 2 Resolve "Common::is_acquired() broken"

Collector: Zope Bugs, Features, and Patches ... zope-coders-admin at zope.org
Tue Oct 21 10:30:32 EDT 2003


Issue #388 Update (Resolve) "Common::is_acquired() broken"
 Status Resolved, Zope/bug medium
To followup, visit:
  http://collector.zope.org/Zope/388

==============================================================
= Resolve - Entry #2 by mcdonc on Oct 21, 2003 10:30 am

 Status: Pending => Resolved

This patch appears to be applied as of 10/21/2003.
________________________________________
= Request - Entry #1 by tejay on May 9, 2002 9:32 pm

The method calls objectIds() on the aq_base of the parent.  Some product may have objects that overload objectIds() making it require on acquision for functionality!  Zope2.5.1 breaks the BizarShop product becuase of this.  My patch is:

*** Common.py   Fri May 10 11:30:52 2002
--- Common.py.old       Fri May 10 11:31:28 2002
***************
*** 77,90 ****
      if not hasattr(ob, 'aq_parent'):
          return 0
 
!     parent = ob.aq_parent
      absId  = absattr(ob.id)
 
      if hasattr(parent,'_objects'):
          if absId+' ' in parent.objectIds():
              return 0
 
!     if hasattr(aq_base(parent), absId):
          return 0
      if hasattr(aq_base(ob), 'isTopLevelPrincipiaApplicationObject') and \
              ob.isTopLevelPrincipiaApplicationObject:
--- 77,90 ----
      if not hasattr(ob, 'aq_parent'):
          return 0
 
!     parent = aq_base(ob.aq_parent)
      absId  = absattr(ob.id)
 
      if hasattr(parent,'_objects'):
          if absId+' ' in parent.objectIds():
              return 0
 
!     if hasattr(parent, absId):
          return 0
      if hasattr(aq_base(ob), 'isTopLevelPrincipiaApplicationObject') and \
              ob.isTopLevelPrincipiaApplicationObject:
==============================================================




More information about the Zope-Collector-Monitor mailing list