> No Jiri, i mean, when we have ContinueOnError = true , and if we throw any > exception or not, doesn't matter, we should call AcceptChanges (of course if > AcceptChangesDuringUpdate = true) only for those rows, that have no error. > As i said, imagine you have ContinueOnError=true and 2 rows violate > concurrency condition, after calling Update you will know, that 2 rows have > errors (they will have HasError=true) but you won't be able to let the user > correct situation, because you don't know anymore, where those 2 rows > supposed to be updated or deleted when they caused concurrency violation - > cause they have RowVersion = Unchanged...Actually when i think about it...it > is not big problem either, because concurrency exception is important only > for updates (where user can insist on own changes or refresh own data with > changes committed to database before him). If you get concurrency exception > on delete statement you cannot do anything anymore (but may be you can :) ) > . Anyway, i think it might be useful to have the rowversion preserved for > erroneous rows, what the point of calling acceptchanges for rows that have > errors and won't be committed in present state to database? What i'm talkin > about actually this line of code > > if (this.AcceptChangesDuringUpdate) > { > row.AcceptChanges(); > } > > i personally would prefer to have > > if (!row.HasErrors && this.AcceptChangesDuringUpdate) > { > row.AcceptChanges(); > }
Oh, now I understand. I was almost sleeping, so I was thinking wrong. So the main idea behind this is, "when I get error, let the row untouched (without AcceptChanges) so I'll be able to repeat the call to DB", right? > But i'm not so skilled and experienced as you are, that's why i ask what do > you think about this damn concurrency :) No problem. Discussing problems brings us better provider. :) > > P.S. Jiri, can you tell me, why do i get > Error: PROPFIND request failed on '/viewvc/firebird/NETProvider/trunk' > Error: PROPFIND of '/viewvc/firebird/NETProvider/trunk': 301 Moved > Permanently (http://firebird.svn.sourceforge.net) > every time i try to check out from > http://firebird.svn.sourceforge.net/viewvc/firebird/NETProvider/trunk/. i > tried it from several places (with or without proxy) - hopeless :( The http://firebird.svn.sourceforge.net/viewvc/firebird/NETProvider/ is only proxy for web-viewing. -- Jiri {x2} Cincura (Microsoft Student Partner) http://blog.vyvojar.cz/jirka/ | http://www.ID3renamer.com ------------------------------------------------------------------------- This SF.net email is sponsored by: Splunk Inc. Still grepping through log files to find problems? Stop. Now Search log events and configuration files using AJAX and a browser. Download your FREE copy of Splunk now >> http://get.splunk.com/ _______________________________________________ Firebird-net-provider mailing list Firebird-net-provider@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/firebird-net-provider