[SOLUCIONADO] Esto me graba dos veces los mismos datos

[juandedios]

Hola, quiero guardar el contenido de mi carrito de compra en 2 tablas: "pedidos" y "pedidos_detalles", De guardar si guarda, pero me lo guarda 2 veces, aquí les paso el modelo:

Código PHP:

public function savePedido($cli, $tot, $fec, $est)
{
    $this->db->trans_start();
 
    $dataSave = array(
            'fecha'     => $fec,
            'cliente'   => $cli,
            'total'     => $tot,
            'estado'    => $est
        );
    $this->db->insert('pedidos', $dataSave);
 
    $pedido = 0;
    $this->db->select_max('codigo', 'cod_ped');
    $q = $this->db->get('pedidos');
    $r = $q->result();
    foreach ($r as $row) {
        $pedido = $row->cod_ped;
    }
 
    $i = 1;
    foreach ($this->cart->contents() as $items) {
        $qty = $items['qty'];
        $pro = $items['id'];
 
        $i = 0;
        $opt = array();
        foreach ($this->cart->product_options($items['rowid']) as $option_name => $option_value) {
            $opt[$i] = $option_value;
            $i++;
        }
 
        $ing = $opt[0];
 
        $dataDeta = array(
                'pedido'        => $pedido,
                'cantidad'      => $qty,
                'producto'      => $pro,
                'ingredientes'  => $ing
            );
        $this->db->insert('pedidos_detalles', $dataDeta);
    }
 
    $this->db->trans_complete();
 
    if ($this->db->trans_status() === false) {
        return false;
    } else {
        return true;
    }
}
 

Espero que me puedan ayudar.

Gracias,

[haggenx]

pues, podrían ser muchas cosas, ¿donde te graba doble, en

$this->db->insert('pedidos', $dataSave);

o en

$this->db->insert('pedidos_detalles', $dataDeta);

?, necesitarías revisar como llamas a la función, si no la estás llamando dos veces.

Otra manera sencilla, que no es lo ideal, pero funciona, es que pongas una variable booleana de forma global, cuando llames a la función que inserte los datos, puesto que la variable está en falso permitas que se inserten los datos, inmediatamente la cambias a verdadero. Si por alguna razón se esta llamando la función, aunque no encuentres donde la variable ya usada previamente impediría que se ejecuta el código de nuevo, y así no se pueden duplicar.

[juandedios]

Yo también pensé que la podría estar llamando 2 veces, pero no, en el controlador que llama a esta función va esto:

Código PHP:

public function comprar()
{
    $cli = $this->input->post('txtHiddenCliente');
    $tot = $this->input->post('txtHiddenTotal');
    $fec = date('Y-m-d H:i:s');
    $est = 0;
    $msg = '';
 
    $res = $this->Carrito_model->savePedido($cli, $tot, $fec, $est);
 
    if ($res === false) {
        redirect('carrito/compra_error');
    } else {
        redirect('carrito/compra_ok');
    }
 
    echo $msg;
}
 

Y por cierto, me graba 2 veces en las dos tablas. En mi ejemplo, meto solo un producto, lo cual me tendría que grabar una entrada en "pedidos" y una en "pedidos_detalles", pero me graba 2 veces en ambas tablas.

[xfxstudios]

Prueba cambiando esto:

Código PHP:

$this->db->trans_start();
 

por esto:

Código PHP:

$this->db->trans_begin();
 

y esto:

Código PHP:

if ($this->db->trans_status() === false) {
        return false;
    } else {
        return true;
    }
 

por

Código PHP:

if ($this->db->trans_status()=== FALSE)
{
        $this->db->trans_rollback();
        return false;
}else{
        $this->db->trans_commit();
        return true;
}
 

[juandedios]

Hola xfxstudios, antes lo tenía así como dices y me grababa doble, por eso lo cambié a como está actualmente, pero pasa lo mismo.

[xfxstudios]

entonces tu problema no esta en la funcion si no antes de esta, debes verificar la cadena de codigo que envía los datos a la funcion, puews se estan enviando 2 veces, ya que en la funcion no veo nada del otro mundo ni que refiera a una doble insercion

[haggenx]

los datos a grabar en tu tabla, tienen alguna llave para que no se dupliquen, por ejemplo algún index?, porque si no tienes un index esto es susceptible a duplicarse y tal vez nunca te des cuenta de donde, pero si tienes una llave única no se te pueden duplicar si los datos son exactamente iguales (por ahí puedes controlar el duplicado de datos).

[juandedios]

Tenías razón haggenx, en el JS, donde paso los datos al controller, no había puesto un preventDefault al submit del formulario. Le puse y ahora está de maravilla.

Gracias a todos

[haggenx]

ahhh que bueno que ya funcionó