for() and mysql statement

Question

I use the Paypal IPN. When the user buys articles, I use this code to generate codes they can redeem. However, it's not working:

public function clave(){
       $Valores = "ABCDEFGHIJKLMNOPQRSTUVWXYZ1234567890";
       $ValorTemporal = "";
        for($i=0;$i<10;$i++) {
        $ValorTemporal .= substr($Valores,rand(0,37),1);

        }
        return $ValorTemporal;
    }

    public function cupon($items){
                $mysqli = $this->connection();
            for ($i = 1; $i <= $items; $i++) {
        $yz= $this->clave();
    $sqli.$i = $mysqli->query("INSERT INTO ventas (id_venta, id_usuario,id_producto ,used,cupon) VALUES ('$txn_id','$username','$item_name','$used','$yz')");

    }

            return true;
    }

$items is the number of products I don't know if I'm using the for() statement correctly.

what is this? $sqli.$i also $txn_id $username $item_name $used are undefined in you cupon function — M0rtiis
– M0rtiis, Commented Sep 2, 2015 at 1:57

M0rtiis · Accepted Answer · 2015-09-02 02:28:26Z

public function cupon($items){
    global $txn_id, $username, $item_name, $used;

    $mysqli = $this->connection();
    for ($i = 1; $i <= $items; $i++) {
        $yz = $this->clave();
        $mysqli->query("
            INSERT INTO `ventas` 
                (`id_venta`, `id_usuario`, `id_producto`, `used`, `cupon`) 
            VALUES 
                ('$txn_id', '$username', '$item_name', '$used', '$yz')
        ");
    }
    return true;
}

OR little better

public function cupon($items, $txn_id, $username, $item_name, $used){
    $mysqli = $this->connection();
    for ($i = 1; $i <= $items; $i++) {
        $yz = $this->clave();
        $mysqli->query("
            INSERT INTO `ventas` 
                (`id_venta`, `id_usuario`, `id_producto`, `used`, `cupon`) 
            VALUES 
                ('$txn_id', '$username', '$item_name', '$used', '$yz')
        ");
    }
    return true;
}

Collectives™ on Stack Overflow

for() and mysql statement

1 Answer 1

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

Comments

Your Answer

Sign up or log in

Post as a guest

Related