MYSQL INSERT problem

Question

$body = $_POST['post'];
$submit = $_POST['submit'];
$date = date("Y-m-d");
require('php/connect.php');

if($submit)
{

$query = mysql_query("INSERT INTO news (`id`, `body`, `date`) VALUES (NULL, '".$body."', '".$date."')");
header("Location: index.php");
}

I do not understand why this isn't working, I took the query straight from PHP my admin after writing a line simular myself before hand and it still isn't working, can someone hep?

comment out your header("Location: index.php"); and append or die(mysql_error()); at the end of your query code that will show you what went wrong — afarazit
– afarazit, Commented Apr 22, 2011 at 15:21
Even though I've commented out my header I'm still being redirected to index.php O_O — Aiden Ryan
– Aiden Ryan, Commented Apr 22, 2011 at 15:46

Jakob Alexander Eichler · Accepted Answer · 2011-04-22 15:18:15Z

2

you definitely should escape your input values using mysql_real_escape_string

With mysql_error you can print out an error message but you need the connection identifier as a parameter.

answered Apr 22, 2011 at 15:18

Jakob Alexander Eichler

3,0664 gold badges35 silver badges51 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

Shakti Singh · Accepted Answer · 2011-04-22 15:19:31Z

1

I just suggest to handle mysql error

$query = mysql_query("INSERT INTO news (`id`, `body`, `date`) VALUES (NULL, '".$body."', '".$date."')")
 or trigger_error(mysql_error());

and if id is primary key that can not be null

you should escape user input using mysql_real_escape_string function.

What happen if I put "that's it " value in your $body input, your query will fail.

answered Apr 22, 2011 at 15:19

Shakti Singh

86.7k21 gold badges142 silver badges156 bronze badges

Comments

afarazit · Accepted Answer · 2011-04-22 15:27:11Z

0

Comment out your header("Location: index.php"); and append or die(mysql_error()); at the end of your query code that will show you what went wrong. You should also "mysql_real_escape_string" your user input before inserting it into your database.

$body = mysql_real_escape_string($_POST['post']);
$submit = $_POST['submit'];
$date = date("Y-m-d");
require('php/connect.php');

if($submit)
{

$query = mysql_query("INSERT INTO news (`id`, `body`, `date`) VALUES (NULL, '$body', '$date')") or die(mysql_error());
//header("Location: index.php");
}

answered Apr 22, 2011 at 15:27

afarazit

4,9692 gold badges31 silver badges53 bronze badges

Collectives™ on Stack Overflow

MYSQL INSERT problem

3 Answers 3

Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related