On Wednesday, October 12, 2011 09:34:26 am Paul McNett wrote:
> On 10/12/11 7:56 AM, John Fabiani wrote:
> > -1
> > If I recall this caused an issue with my code. Could you provide the
> > commit number and I could check into it further.
>
> Let me svn that for you:
>
> {{{
> mac:~ pmcnett$ cd dabo
> mac:dabo pmcnett$ cd dabo
> mac:dabo pmcnett$ cd biz
> mac:biz pmcnett$ svn blame dBizobj.py | grep -i updateinternals
> 6850 JacekK def appendDataSet(self, ds, updateInternals=False):
> 6850 JacekK self._CurrentCursor.appendDataSet(ds,
> updateInternals=updateInternals)
> mac:biz pmcnett$ svn diff -r 6849:6850 | grep -in2 updateinternals
> 44-
> 45-- def appendDataSet(self, ds):
> 46:+ def appendDataSet(self, ds, updateInternals=False):
> 47- """
> 48- Appends the rows in the passed dataset to this bizobj's
dataset. No
> checking --
> 52- """
> 53-- self._CurrentCursor.appendDataSet(ds)
> 54:+ self._CurrentCursor.appendDataSet(ds,
> updateInternals=updateInternals) 55-
> 56-
> mac:biz pmcnett$ svn log -r 6850
> ------------------------------------------------------------------------
> r6850 | JacekK | 2011-09-20 05:19:10 -0700 (Tue, 20 Sep 2011) | 4 lines
>
> - Finally, I decided to remove isAnyChanged() method call from isChanged.
> This avoids false positives for all child cursors when only one grandchild
> cursor is changed.
> The isAnyChanged method remains unchanged and is still fast but not so
> precise as isChanged.
> - The appendDataSet method gains extended functionality and allows to
> update cursor internals: temporary key value and new flag.
> ------------------------------------------------------------------------
>
> }}}
>
> So your commit number is 6850, on Sept. 20.
>
> Paul
thanks Paul.
I guess I will remove my objection. This does not appear to be the commit
that broke my code. I think it had something to do with the fact I open an
empty form and that was resolved by adding a flag.
Johnf
_______________________________________________
Post Messages to: [email protected]
Subscription Maintenance: http://leafe.com/mailman/listinfo/dabo-users
Searchable Archives: http://leafe.com/archives/search/dabo-users
This message:
http://leafe.com/archives/byMID/[email protected]
