Re: [OJB-developers] PersistentFieldDefaultImpl NPE
Brought to you by:
thma
From: Jakob B. <jbr...@ho...> - 2002-06-07 13:30:10
|
hi leandro, with the current state of the test objects i prefer to comment this line. afaik it was no problem before your changes, but i do not know the reason it fails now. jakob ----- Original Message ----- From: "Leandro Rodrigo Saad Cruz" <le...@ib...> To: "Jakob Braeuchi" <jbr...@ho...> Cc: <obj...@li...> Sent: Thursday, June 06, 2002 11:10 PM Subject: Re: [OJB-developers] PersistentFieldDefaultImpl NPE > Hi. I believe I was the last one to change this file. > When I commited this changes the ant target junit was failing due to > SQLExceptions. I can comment this line if you want. > > My opinion is that every testcase should be ok at all times ! > Can we work on that ? > > On Thu, 2002-06-06 at 17:51, Jakob Braeuchi wrote: > > hi , > > > > i just loaded the newest version of PersistentFieldDefaultImpl and now i get > > NPE caused by the statement > > > > logger.debug("Setting field:"+f.getName()+" in obj:"+obj+" value:"+value); > > > > in method set(obj, value). the npe is mainly caused by the toString() > > implementation of obj. > > ie. obj is a ProductGroupWithCollectionProxy and toString() fails bcause > > allArticlesInGroup() is null. > > the point is that we just try to set allArticlesInGroup for > > ProductGroupWithCollectionProxy in PersistenFieldDefaultImpl . > > > > the simplest fix is to comment the logger call. but it may be cleaner to fix > > the various toString() in the test objects. > > > > jakob > > > > _______________________________________________________________ > > > > Don't miss the 2002 Sprint PCS Application Developer's Conference > > August 25-28 in Las Vegas -- http://devcon.sprintpcs.com/adp/index.cfm > > > > _______________________________________________ > > Objectbridge-developers mailing list > > Obj...@li... > > https://lists.sourceforge.net/lists/listinfo/objectbridge-developers > -- > Leandro Rodrigo Saad Cruz > IT - Inter Business Tecnologia e Servicos (IB) > http://www.ibnetwork.com.br > |