Estoy trabajando en el registro de una transacción en la que tengo que hacer un llamado a la función enviarDinero
en una api externa la cual solo retorna el id de la transacción en el sistema externo o false. (no hay forma de revertir esta operación)
En mi sistema tengo una tabla con un registro que se debe modificar cuando la llamada a la función enviarDinero
sea exitosa.
Entonces plantee esta solución con posibles fallos:
$transaccion_id = 453;
$monto = 53.5;
$destino = "usuariox";
$result = $this->apiexterna->enviarDinero($destino,$monto);
/*
* Parte critica, si el sistema se apaga no habrá ningún registro en la base de datos
* de la operacion a la api externa.
*/
if(!$result) return FALSE;
$data = array("external_transaccion_id", $result);
$this->where("transaccion_id",$transaccion_id);
$this->update("transacciones", $data);
Luego se me ocurrió poner una bandera en la base para que quede al menos un registro de que la operación se intento y que posiblemente se envió el dinero.
$this->db->trans_begin();
$data_update_complete = array("intento"=>1, "fecha_intento"=>date("Y-m-d H:i:s"));
$this->where("transaccion_id",$transaccion_id);
$this->update("transacciones", $data_update_complete);
/*ESTO ES CORRECTO????????*/
if ($this->db->trans_status() === false) {
$this->db->trans_rollback();
return FALSE;
}
$transaccion_id = 453;
$monto = 53.5;
$destino = "usuariox";
$result = $this->apiexterna->enviarDinero($destino,$monto);
/*
* Parte critica, si el sistema se apaga el no habra ningun registro en la base de datos
* de la operacion a la api externa.
*/
if(!$result) return FALSE;
$data = array("external_transaccion_id", $result);
$this->where("transaccion_id",$transaccion_id);
$this->update("transacciones", $data)
if ($this->db->trans_status() === false) {
/* DES HACER LOS CAMBIOS EN LA BASE SI FALLA LA TRANSACCIÓN */
$this->db->trans_rollback();
return FALLSE;
} else {
/* CONFIRMAR CAMBIOS EN LA BASE */
$this->db->trans_commit();
return TRUE;
}
Es correcto utilizar $this->db->trans_status()
más de una vez en codeigniter?
¿Hay alguna forma mas elegante de resolver este problema?
trans_status()
en la documentación oficial de codeigniter hacen uso de el de la misma manera en la que tu lo estas haciendo, excepto por el uso dereturn FALSE;
que en tu caso lo tienes con doble LL (return FALLSE
).