Fixed ObjectNode.java. Thanks, Oliver! Regards, Peter
> -----Original Message----- > From: Oliver Zeigermann [mailto:[EMAIL PROTECTED] > Sent: Thursday, October 16, 2003 13:40 > To: Slide Developers Mailing List > Subject: Additional ObjectNode bug > > > There seems to be another bug in ObjectNode: > > In getChildren() there is a check for childrenCache being null. If it > is, the cache will be recomputed. The cache will be reset on every > addition or removal of a binding or a child which is correct. > Unfortunately when copyObject() is called like from > MacroImpl.copyObject(...) bindings are reset, but not the children > cache. This leads to inconsisten data. An obvious fix would be to add > > childrenCache = null; > > at line 416 after ObjectNode result = null; > > The problem occasionally occured with the tx file store while moving > resources. As it turned out it was a combination of non-transactional > and arbitrarily resetting caching in StandardStore and this > minor error. > > I suppose nobody noticed before as it only occurs when binding is > disabled. Also, slidestore.file.XMLFileDescriptorsStore does some > additional unbounded internal caching and thus does not > reveal this error. > > Oliver > > Nevermann, Dr., Peter wrote: > > Thanks again, Martin. Sorry, I was in a hurry when I did > the 1st fix. > > > > Regards, > > Peter > > > > > >>-----Original Message----- > >>From: Martin Holz [mailto:[EMAIL PROTECTED] > >>Sent: Tuesday, September 16, 2003 13:14 > >>To: [EMAIL PROTECTED] > >>Subject: Re: ObjectNode bug > >> > >> > >>"Nevermann, Dr., Peter" <[EMAIL PROTECTED]> writes: > >> > >> > >>>Fixed. > >> > >>Unfortunatly not. You have to check the return value of > parent() now. > >> > >>For example > >> ObjectNode p = new SubjectNode( new > >>UriPath(uri).parent().toString() ); > >>at line 544 of ObjectNode.java will cause a NPE, if uri is '/'. > >> > >> > >> > >>------------------------------------------------------------ > --------- > >>To unsubscribe, e-mail: [EMAIL PROTECTED] > >>For additional commands, e-mail: [EMAIL PROTECTED] > >> > > > > > > > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: [EMAIL PROTECTED] > For additional commands, e-mail: [EMAIL PROTECTED] >
