[carmelraj]

The value is inserting... suppose i insert the same data or values it will be updated it won't get insert. But this update is not working plz help me

$sql="INSERT INTO order_details(co_id,company_name,bookingf_no,amount,discount,amount_payable,amount_collected,balance_payable_on,balance_payed,executive,receipt_no,receipt_date,no_of_iss,s_size,d_size,remarks)VALUES('$id','$_POST[company_name]',$_POST[bookingf_no],'$_POST[amount]','$_POST[discount]','$_POST[amount_payable]','$_POST[amount_collected]','$_POST[balance_payable_on]','$_POST[balance_payed]','$_SESSION[User]','$_POST[receipt_no]','$_POST[receipt_date]','$_POST[no_of_issues]','$_POST[s_size]','$_POST[d_size]','$_POST[remarks]') ON DUPLICATE KEY UPDATE SET  VALUES('".$id."','$_POST[company_name]',$bookingf_no,$amount,$discount,'$_POST[amount_payable]','$_POST[amount_collected]','$_POST[balance_payable_on]','$_POST[balance_payed]','$_SESSION[User]','$_POST[receipt_no]','$_POST[receipt_date]','$_POST[no_of_issues]','$_POST[s_size]','$_POST[d_size]','$_POST[remarks]')";

[CTphpnwb]

First, NEVER put user supplied data directly into a query unless you're using prepared statements. Next, associative arrays should have the index be quoted. While PHP will let you get away with not doing so, the index may not point to the intended item.

[Atli]

Your SQL syntax is wrong.

The INSERT ... ON DUPLICATE KEY UPDATE syntax should look more like this:

INSERT INTO mytbl(id, val, last_updated)
VALUES (123, 'updated value', NOW())
ON DUPLICATE KEY UPDATE
    val = VALUES(val),
    last_updated = VALUES(last_update);

There is no SET clause, and you need to specify each column you want updated, just like in an UPDATE query.
The VALUES() function there allows you to fetch the value that should have been inserted for the column you specify.

CTphpnwb, on 07 May 2011 - 01:02 PM, said:

Next, associative arrays should have the index be quoted. While PHP will let you get away with not doing so, the index may not point to the intended item.

It's OK inside a string. One of those ancient exceptions that sounded like a good idea at the time
I don't like it though. Much prefer to use:

$str = "{$arr['elem']}";