Tienda online - Notificacion URL se ejecuta varias veces en una sola compra!!

[alfredo49]

Buenos días:

A ver si me podeis ayudar con una cosa, estoy actualizando el codigo de un par de TPV que tenemos y después de leer y hablar con los de ruralvía he decidido cambiar una cosa.

Los clientes dueños de la TPV se "quejaron" de que una ventana emergente "La operación se esta procesando" tapaba el contenido de la aceptación de pago con tarjeta, las 2 ventanas son generadas por ruralvía por lo que poco se podía hacer, al tapar el contenido hacía que compras no se finalizaran al usuario no ver el botón aceptar de atrás.

Antiguamente la web le cedia el control a la TPV pasandole los datos necesarios para efectuar el pago, y esta a su vez retornaba con la información de si el pago había sido exitoso o no a confirmacion_TPV que era la pagina que recogía los datos , envíaba el correo al cliente con el resultado y demás. Al taparse la ventana de aceptar un % muy alto de veces esta página nunca llegaba a ejecutarse por lo que era un problema bastante grande.

Bien, desde caja rural me dijeron que para eliminar esa ventana emergente, debería programar una pagina llamada notificacion_TPV ésta se ejecutaría automaticamente nada más efectuar el pago y que este archivo .php estaría oculto para el user, en esta ventana lo que se haría sería hacer lo que se hacia antiguamente en confirmacion_TPV (marcar pedido en la base de datos, enviar correo y demás), por lo que confirmacion_TPV (si visible por el usuario) simplemente sería una página de informacion de cara al usuario, pero que a nivel operacional no haría nada.


Todo esto te lo cuento porque en una de las 2 tiendas online todo fué perfecto, pero hay una de las 2 que por algún motivo la página notificacion_TPV se me "buclea" unas 3-4 veces, siempre ese número exacto y es que es imposible porque no hay ningún for, while ni ninguna sentencia de bucle, esto conlleva el problema de que envía 3-4 correos, intenta meter el producto 3-4 veces en la base de datos etc etc, he hablado con los de ruralvia pero me dicen que esa página no tiene nada de especial para tener ese comportamiento, me he asegurado de quitar todo lo que tenga que ver con flujo de salida de datos ya que es una página oculta para el usuario, pero así se me buclea..

Había pensando que igual el usuario le diera varias veces al botón de comprar, pero es que pase siempre.. nose.. aparte si fuera así no sabría decirle que sólo lo haga una vez, con una variable global booleana.. o como porque tengo miedo que se ponga a comprar justo detrás en la misma sesion y no le funcione.

Se que hace el bucle porque tengo salidas a un fichero de texto con eventos a modo de checkpoint para saber una traza de por donde va, y se que se llega a ejecutar hasta el final del codigo , pero por algún motivo cuando acaba vuelve a ejecutar el php entero.

El codigo le he quitado llamadas y funciones a demas para dejarlo todo en el mismo archivo .php

¿Alguien puede saber el motivo?

El codigo de notificacion_TPV

Código PHP:

<?php session_start();
    ini_set('arg_separator.output', '&amp;');
    ini_set('url_rewriter.tags', 'a=href,area=href,frame=src,input=src,fieldset=');
    
    @include_once("inc/funciones.inc.php");
    @include_once("inc/declaraciones.php");
    
    //Variables generales
    $error=0;                           
    $mensaje="";    
    $file = fopen("logTPV.txt", "a+");  //fichero donde se guardará
 
    /********************** GESTION TRANSFERENCIA ******************************/
    // Valores constantes del comercio  
    $fuc=xxxxx;
    $idterminal=xxx;
    $idioma=x;
    $CIP=xxxxxxx;
    $moneda=978;    
    $name='xxxxxxxxxxxxxxx';
    $url='http://xxxxx/confirmacion_tpv.php'; //URL destino
 
    //Valores recibidos
    $importe = $_GET['importe'];
    $compra = $_GET['compra'];
    $moneda = $_GET['moneda'];
    $numpedido = $_GET['numpedido'];
    $fecha = $_GET['fecha'];
    $fentrega = $_GET['fentrega'];
    $firma = $_GET['firma'];
 
    $mensaje = $_GET['Ds_Amount'].$_GET['Ds_Order'].$_GET['Ds_MerchantCode'].$_GET['Ds_Currency'].$_GET['Ds_Response'].$clave;
    
    $sha = new SHA; 
 
    fwrite($file, "CODIGO PAGO: ".$numpedido."\r\n");       
    fwrite($file, "FECHA PAGO PEDIDO: ".$fecha."\r\n");
    fwrite($file, "FECHA ENTREGA: ".$fentrega."\r\n");
    fwrite($file, "IMPORTE: ".$importe."\r\n"); 
    fwrite($file, "ESTADO PEDIDO A NO PAGADO POR DEFECTO \r\n");
    $sql = "UPDATE tienda_pedidos SET estado ='3' WHERE num_pedido = '".$numpedido."'";
    ejecutarSQL($sql);
        
    
    if ($compra=="si"){
        fwrite($file, "COMPRA = SI, ponemos estado a PAGADO \r\n"); 
        $cadena = '0'.$CIP.$numpedido.$importe.$fecha;
        $digest1 = $sha->hash_string($cadena);
        $calculo_firma = $sha->hash_to_string( $digest1 );
        
        $sql = "UPDATE tienda_pedidos SET estado ='2' WHERE num_pedido = '".$numpedido."'";
        ejecutarSQL($sql);
        
        // DESCONTAR STOCK
        $id_pedido = dame_valor("id","tienda_pedidos",1,1, " AND num_pedido LIKE '".$numpedido."'");    
        $_SESSION["id_pedido"] = $id_pedido;
        
        $lineas = new objeto("id,entidad,identificador,titulo,precio,cantidad,imagen","tienda_pedidos_lineas",1,1,"AND id_pedido='".$id_pedido."'");
 
        $total_precio = 0;
        
    
        
    }
    else{
        if ($compra=="no"){
            fwrite($file, "COMPRA = NO \r\n");
            $cadena = '1'.$CIP.$numpedido.$importe.$fecha;
            $digest1 = $sha->hash_string($cadena);
            $calculo_firma = $sha->hash_to_string( $digest1 ); //firma generada no fraudulenta tiene que coincidir con $firma (esto se hace para evitar que introduzcan la URL de compra exitosa directamente sin pagar)            
        }
        else{
            fwrite($file, "COMPRA = INDETERMINADO \r\n");
            $error=1;
        }
    }
    
    /***************** FIN GESTION TRANSFERENCIA ******************************/
        
 
//Comprobamos si la firma es valida => respuesta del tpv del banco autentica
if ($calculo_firma==$firma){
    fwrite($file, "CALCULO FIRMA = FIRMA, OK \r\n");
    if ($compra == "si") {
        fwrite($file, "COMPRA = SI, DENTRO DE FIRMA, OK \r\n");
    
        
        if(!$error){
            
            //Destruimos la variable carrito de sesión
           include("apps/carrito/resumen_mail.php");
           
       
           
           
           /*FIN ENVIO CORREO */
            
            //unset($_SESSION['carrito']);                                      
            //unset($_SESSION["id_pedido"]);
            fwrite($file, "TODO OK,ENVIAMOS CORREO \r\n");                          
        }
    }else{
            fwrite($file, "COMPRA = NO, DENTRO DE LA FIRMA, FALLO \r\n");
            //unset($_SESSION['carrito']);                                      
            //unset($_SESSION["id_pedido"]);
            
    }
}else{
    fwrite($file, "CALCULO FIRMA NO COINCIDE CON FIRMA, FALLO \r\n");
 echo "Error calculo firma";
 }
 
//$carro = new carrito("carrito");
//$carro->vaciar();
//unset($_SESSION["id_pedido"]);
fwrite($file, "CERRAMOS LA COMPRA \r\n");
fclose($file);
exit();
 
//Implementacion de la clase SHA
?>
 

Gracias.

Saludos

[luis010182]

para mi el problema esta en la pagina que hace el llamado a notificacion_TPV.

veo que guardas en un archivo los pasos de la operacion.
Como se guardan?

PASO1
PASO2
PASO3
PASO4
PASO1
PASO2
PASO3
PASO4

O

PASO1
PASO1
PASO2
PASO2
PASO3
PASO3
PASO4
PASO4

[alfredo49]

No no, se guardan
PASO 1
PASO 2
PASO 3
PASO 4
PASO 1
PASO 2
PASO 3
PASO 4
..........

Se ejecuta la pagina entera recursivamente