Hi Suman, I would be interested to know if you are updating any other properties in your subclass, that after removing dynamic-update=true, all works. This is because if I update a property on the subclass then any processing in the PreUpdate event is persisted, but if the subclass is only modified in the PreUpdate event then nothing is persisted.
On Apr 6, 7:58 am, Suman <[email protected]> wrote: > Hi Scott, > > I have similar issue and I was about to ask the same question here. > > Further to your situation, I am not sure whether it's a bug or > feature, In my mapping, when I have dynamic-update = "true" in > subclass the audit columns in base class, the generated sql doesn't > include audited column (ModifiedBy & ModifiedDate which are properties > of base class) but if I remove dynamic-update = "true" it works. So > essentially it points me to some reflection + dynamic-update issue. > Just wondering if that's what's happening in your side as well. > > Suman > > On Apr 5, 6:27 pm, Scott Findlater <[email protected]> wrote: > > > > > > > > > Please may I ask for clarification on the semantics/ usage of > > PreUpdate vs OnSaveOrUpdate listeners based on NH-2596 (http:// > > 216.121.112.228/browse/NH-2596). > > > After reading Ayende's blog posts on NHibernate > > IPreUpdateEventListener & IPreInsertEventListener (http://ayende.com/ > > Blog/archive/2009/04/29/nhibernate-ipreupdateeventlistener-amp- > > ipreinserteventlistener.aspx) with an audit trail example I adopted > > this method myself. During my learning I raised NH-2617 (http:// > > 216.121.112.228/browse/NH-2617) issue, where I found PreUpdate event > > data changes on an entity which is only modified in the PreUpdate > > event is not persisted. > > > My issue was closed as a duplicate of NH-2596. However, NH-2596 is > > also closed as “not an issue” with the solution to use the > > OnSaveOrUpdate event listeners on the basis that pre-insert and pre- > > update listeners are not intended to be used to change the values of > > the entity. Instead they should be used to check-values (for that > > reason they return "veto"). > > > I am alittle confused as to the best practice usages/ semantics of > > PreUpdate vs OnSaveOrUpdate now. Would someone be able to clarify why > > the use of PreInsert and PreUpdate is used by many for Audit trailing > > but incorrectly? Or why is the OnSaveOrUpdate listener suggested in > > the closure of NH-2596 and this non peristance is not an issue. > > > Many thanks, Scott. -- You received this message because you are subscribed to the Google Groups "nhusers" group. To post to this group, send email to [email protected]. To unsubscribe from this group, send email to [email protected]. For more options, visit this group at http://groups.google.com/group/nhusers?hl=en.
