Insert query is successful but database error comes up? - php

Question

Product_model.php
public function addProduct($info)
{
$sql_array = array(
$info['name'],
$info['SKU'],
$info['supplier_id'],
$info['type_id'],
$info['brand_id'],
$info['added_on'],
$info['initial_cost_price']);
$this->db->trans_start();
//INSERT
$sql = "INSERT INTO
product(
name,
SKU,
supplier_id,
type_id,
brand_id,
added_on,
initial_cost_price
)
VALUES(
?,
?,
?,
?,
?,
?,
?
)
";
$query = $this->db->query($sql, $sql_array);
$insert_id = $this->db->insert_id();
$this->addProductVariant($insert_id, $info);
$this->db->trans_complete();
if ($this->db->trans_status() === FALSE)
{
return 'transaction failed';
} return TRUE;
}
public function addProductVariant($product_id, $info)
{
$sql_array = array(
$product_id,
$info['variant_name'],
$info['variant_SKU'],
$info['barcode'],
$info['wholesale_price'],
$info['buy_price'],
$info['retail_price'],
$info['description'],
$info['initial_quantity'],
$info['added_on'],
$info['last_updated']
);
$sql = "INSERT INTO
product_variant(product_id,
name,
SKU,
barcode,
wholesale_price,
buy_price,
retail_price,
description,
qty_on_hand,
created_on,
last_updated
)
VALUES(?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?)";
$query = $this->db->query($sql, $sql_array);
}
When I hit submit on the form, the page reloads and shows this error:
Error Number: 1048
Column 'name' cannot be null
INSERT INTO product_variant(product_id, name, SKU, barcode, wholesale_price, buy_price, retail_price, description, qty_on_hand, created_on, last_updated ) VALUES(71, NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL, '2017-09-17 10:45:12', '2017-09-17 10:45:12')
Filename: D:/xampp/htdocs/svcc_inv/system/database/DB_driver.php
Line Number: 691
But when I check the database, the inserts are successful. The new records are there.
I have tried echoing the values in the $sql_array and I get correct results. I don't know why the error says they're null. I'm hypothesizing could it be because I split the whole process into two methods, one of which is inside the other, but I don't see yet how that could be.

Answer 1

In your example $info['variant_name'] is NULL, you need to dump $info from the start of addProduct function what does it contain, like so :
public function addProduct($info)
{
var_dump($info);
exit;
...
}
Also, I you want to use $this->db->trans_status() then you need to run transactions manually, using $this->db->trans_begin() and not $this->db->trans_start(), like so :
$this->db->trans_begin();
$sql_array = array(
$info['name'],
$info['SKU'],
$info['supplier_id'],
$info['type_id'],
$info['brand_id'],
$info['added_on'],
$info['initial_cost_price']);
$this->db->trans_start();
//INSERT
$sql = "INSERT INTO
product(
name,
SKU,
supplier_id,
type_id,
brand_id,
added_on,
initial_cost_price
)
VALUES(
?,
?,
?,
?,
?,
?,
?
)
";
$query = $this->db->query($sql, $sql_array);
$insert_id = $this->db->insert_id();
$this->addProductVariant($insert_id, $info);
if ($this->db->trans_status() === false) {
$this->db->trans_rollback();
return 'transaction failed';
} else {
$this->db->trans_commit();
return true;
}