Michael,

Yup. You might want to consider naming the transaction, just in case any of those processes you call have any embedded transactions (or get them sometime in the future) within them.

=SQLEXEC(THIS.oSqlconnection.nConnHandle,"BEGIN TRANSACTION supertrans")

when you commit or rollback you provide the same transaction name in the command.

>Do I have this right?
>
>

>=SQLEXEC(THIS.oSqlconnection.nConnHandle,"BEGIN TRANSACTION")
>THIS.oFromWarehouse.Dotransfer()
>THIS.oToWarehouse.Dotransfer()
>THIS.UpdateCounters()
>THIS.Makexfer()
>THIS.AddVelocity()
>IF THIS.lSuccess
>	=SQLEXEC(THIS.oSqlconnection.nConnHandle,"COMMIT TRANSACTION")
>ELSE
>	=SQLEXEC(THIS.oSqlconnection.nConnHandle,"ROLLBACK TRANSACTION")
>ENDIF
>