No ... el ROLLBACK lo tienes que hacer al final de todo el proceso...
//Inicializo el objeto
objConn=new Conn(.....)
//Inicializo la variable de control
error=0;
//Inicializo la transacció
objConn-> instrucción_segura ("BEGIN");
//Hago n operaciones con la bbdd
INSERTs UPDATEs, DELETEs
//Modifico la variable error en funcion del resultado de las operaciones
//Si NO hay error
//Hago otras operaciónes (p.e. mando un mail)
//Tambien registro el posible error en la misma variable
//Cierro la transacción
//Si hay error lanzo el ROLLBACK (este deshara TODAS las operaciones hechas a la bbdd desde el BEGIN)
//Si NO hay error lanzo el COMMIT
Obviamente puedes abortar el proceso con algun break o GO TO si al encontrar el primer error ya invalida todo el proceso...
En realidad como el control del error lo hacemos con una variable externa podemos controlar qualquier error no solo los de bbdd ... así es como si el envio del mail fuera una operación mas dentro de la transacción y por tanto lanzare el ROLLBACK si falla... como si hubiese fallado una operacion de la BBDD.
El ROLLBACK es para deshacer aquello que se ha hecho no lo que no se ha hecho, luego en tu codigo hay algo que no tiene sentido
PRIMERA OPERACION CON LA BBDD
$resultado_insert1 = $Data->qryNoRes($INSERT1);
if (!$resultado_insert1) $Data-> instrucción_segura ("ROLLBACK");
Si FALLA hago ROLLBACK
esto no tiene sentido puesto que si ha fallado no hay nada que deshacer (ROLLBACK--> rodar para atras ->retroceder), en todo caso si la primera operacion te falla lo que tienes que evitar es que se hagan las demas, especialmente el envio del mail, puesto que si este se realiza no habra ROLLBACK posible para retrocederlo... (Si tendria sentido des de el punto de vista de cerrar la transacción)
No se si me he explicado, el sentido del tratamento de transacciones es el de poder "deshacer" aquello que se haya hecho no lo que no se ha hecho!!!
Si despues de una serie de operaciones hechas se produce algo que las hace invalidas deshacerlas TODAS para dejar la BBDD en el estado anterior...
Quim
Edito: Como no estoy seguro de saber explicarme te lo doy hecho
Código PHP:
Ver original<?php
/*
*Supongamos que
*qryNoRes() retorna true si no hay error
* false se se produce un error
* Send() retorna true si no hay error
* false se se produce un error
*/
$Data = new CBaseDatos(st_host,st_user,st_pass,st_db);
$Data-> instruccion_segura ("BEGIN");
$error=false;
if ($Data->qryNoRes($INSERT1)){
if ($Data->qryNoRes($INSERT2)){
// Si los dos se ejecutan correctamente
if ($Mail->Send()){
$Data->instruccion_segura ("COMMIT");
echo "Todo correcto";
}else{
//Si hay error en el mail
$error=true;
}
}else{
//Si hay error en el segundo INSERT
$error=true;
}
}else{
//Si hay error en el primer INSERT
$error=true;
}
if($error) $Data-> instruccion_segura ("ROLLBACK");
?>