On Sunday 30 December 2001 08:00 pm, Casey Duncan wrote:
> This behavior is logical if unintuitive. I propose
> that we can kill two birds with one stone to fix this:
> - Add a new method perhaps: getModificationTime() to
> the API of SimpleItem or even Persistent that returns
> the ZODB modification time or if the object has been
> changed, but not yet commited, the current date/time.
> - Deprecate bobobase_mod_time and perhaps even omit it
> entirely from Zope3.
> This will fix the aforementioned bug and get rid of an
> API anachronism.
sounds good, i played around with an implementation to do the above. i'm not
posting it here causes its inefficient, although i'm happy to email it to
anyone who's interested. it basically added three methods.
1. getModificationTime to App/PersistinentExtra.py
2. is_registered(self,object) to ZODB/Transaction.py
3. get_registration_time(self, object) to ZODB/Transaction.py
add one data structure to Transaction.py, a registration_times dict.
when objects are registered they get a DateTime registered with them that
gets added to registration_times based on their position within the
registered objects list (_objects). another solution might be a weak ref.
keyed dict. so getModificationTime checks if the object is registered in the
current transaction if it is it calls get_registration_time, else it falls
through to bobobase_modification_time.
there are two inefficiencies one is memory because of DateTime for every
registered object and two is computational from the lack of a uniform (across
subclasses) hash function for persistent objects which results in traversal
of the registered object list to identify if an object is registered or not
(and to determine position to figure out the index into registration_times).
a much better solution memory wise would have the transaction.py keep a
DateTime for its initialization and then have registered objects store
registration times as deltas of that. i'm not sure what the better solution
is in terms of the computational issue. but as is this solution doesn't seem
scalable to me until the computational issues can be handled.
> --- Dieter Maurer <[EMAIL PROTECTED]> wrote:
> > I just discovered that the value cataloged for
> > "bobobase_modification_time"
> > is often wrong:
> > The cataloged value is not the time of the
> > current modification
> > but that of the previous modification.
> > I expect that this is because
> > "bobobase_modification_time" is updated
> > when the transaction is commited while the values
> > to be cataloged
> > are determined before the commit.
> > Do not trust the cataloged
> > "bobobase_modification_time"!
> > Dieter
> > _______________________________________________
> > Zope maillist - [EMAIL PROTECTED]
> > http://lists.zope.org/mailman/listinfo/zope
> > ** No cross posts or HTML encoding! **
> > (Related lists -
> > http://lists.zope.org/mailman/listinfo/zope-announce
> > http://lists.zope.org/mailman/listinfo/zope-dev )
> Do You Yahoo!?
> Send your FREE holiday greetings online!
> Zope-Dev maillist - [EMAIL PROTECTED]
> ** No cross posts or HTML encoding! **
> (Related lists -
> http://lists.zope.org/mailman/listinfo/zope )
Zope-Dev maillist - [EMAIL PROTECTED]
** No cross posts or HTML encoding! **
(Related lists -