On Mar 19, 2007, at 5:20 PM, Ed Leafe wrote:

>       That sounds wrong. New, unchanged records in the cursor should be
> discarded. I wonder if any of the memento optimizations that were
> done recently overlooked this case.
>
>       I'll look into it some more and get back to you.

        OK, I think I found the problem. The new code is assuming that a new  
row is "changed"; it's not even bothering to check its memento:

        def isChanged(self, allRows=True):
                if allRows:
                        return len(self._mementos) > 0 or len(self._newRecords) 
> 0

        The code also shows that the memento for the new row doesn't include  
the default values, which is wrong. Guess I have to dig a lot deeper  
to straighten this out.

-- Ed Leafe
-- http://leafe.com
-- http://dabodev.com



_______________________________________________
Post Messages to: [email protected]
Subscription Maintenance: http://leafe.com/mailman/listinfo/dabo-dev

Reply via email to