From: Oliver H. <Ol...@ou...> - 2004-08-25 06:00:19
|
Keith, Attached is a patch for a nasty bug that I've been chasing today...=20 Essentially DefaultFormModel should not call commit on its form object holder. This is the responsibility of the surrounding CompoundFormModel or whatever it was that produced the form object holder. Your comment (see patch) indicated you thought there was a problem and I've certainly confirmed it. The problem is essentially that all forms that share a single form object should commit *before* the form object its self is committed. With the current code the form object will be committed at a random time and certainly not last. Ollie |