[RESOLVED] OOP INSERT Query will not input data

Radon3k

I'm not the best with OOP concepts but I am trying to learn. I have a MySQL Connection class which also has a query function in it. I have a simple form and I just want the data entered into the database. Records are being added to the database however not the actual information from the forms. Any help would be appreciated.

//MySQL query function:
function query($sql)
  {
    $query = mysql_query($sql, $this->db);
    return $query;
  }

// submit function (different class):
function submitInvoice()
	{
		$connection = new mysqlConnection();
		$connection->connect();

	if($_POST['submit'] == "Submit")
	{
		$qry = $connection->query("INSERT INTO invoices (id, date, item, ownership, amount) VALUES('', NOW(), '$this->item', '$this->ownership', '$this->amount')")  or die(mysql_errno().' : '.mysql_error());
		if ($qry)
		{
			echo($qry . "<br />");
			echo("Invoice added.");
		} else {
			echo("Error adding invoice.");
		}
	}
}

I have the echo($qry) bit in there to see what the query is echo'ing out, and the only thing that it puts out is '1' and 'Invoice added'.

bpat1434

And where do you populate "$this->ownership", "$this->amount", "$this->item"? Where's that?

~Brett

Radon3k

Well that's one thing I'm not sure about. I figured that they would be populated via the form names however I'm not sure how to get the information from the form to populate those variables. In the variable declarations at the top of that class I tried making it something like var $item = $_POST['item']; but that doesn't work either. Do you have any suggestions on how to make this work? Thanks. 🙂

bpat1434

well, you could use a function like:

<?php

function setVals($own, $item, $amount)
{
  $this->ownership = $own;
  $this->item = $item;
  $this->amount = $amount;
}

?>

Use it in this form:

<?php
// New object declaration
$obj->setVals($_POST['ownership'], $_POST['item'], $_POST['amount']);
$obj->submitInvoice();
?>

~Brett

Radon3k

Thanks, this seems to have done the trick! 🙂