2 things (Syntax and max lines)

MoldRat

What is wrong with this syntax?

	if ($action == 'na'){
    mysql_query("INSERT INTO `modded` (`modbod`,`board`,`topic`,`message`,`modby`,`moddedat`,`postedat`,`action`,`punishment`,`reason`,`moduser`) VALUES ('$messbod','$board','$topic','$message','$modby','".date('m/d/Y H:i:s')"','$postdat','No Action','$punishment','$reason','$uname')");
    }

How would I make it so when you enter box 'sig' that it checks to make sure there isnt more than 2 lines.

MoldRat

wait 3 things.

How can I make it so instead of checking for individual characters, it only accepts a-z, A-Z, and 0-9?

sdjensen

use this statement after the last ) and before the ; to see the why the statement is failing.

or die(mysql_error());

rIkLuNd

Problem No.1:

// try this
if ($action == 'na')
{ 
	$sql = "INSERT INTO `modded`(`modbod`,`board`,`topic`,`message`,"; 
    $sql .=  "`modby`,`moddedat`,`postedat`,`action`,";
    $sql .= "`punishment`,`reason`,`moduser`) VALUES('$messbod',";
    $sql .= "'$board','$topic','$message','$modby',";
    $sql .= "'".date('m/d/Y H:i:s')"','$postdat','No Action',";
    $sql .= "'$punishment','$reason','$uname')";

mysql_query($sql);
}

Problem No.2:

// find all the linebreaks
preg_match_all("/\n/",$str,$lines);
if(count($lines[0]) > 1)
{
     die('max lines exceeded');
}

rIkLuNd

Originally posted by MoldRat
wait 3 things.

How can I make it so instead of checking for individual characters, it only accepts a-z, A-Z, and 0-9? [/B]

// this code will also accept spaces and new-lines
$str = "this is a string that won't be accepted...";
if(preg_match('/^[a-zA-Z0-9\s]+$/',$str))
{
     echo 'the string passed the test';
}
else
{
     echo 'the string was invalid';
}

MoldRat

I tried die(mysql_error()); but it didnt say anything at all.
I tried it the other way and it said there error was in

$sql .= "'".date('m/d/Y H:i:s')"','$postdat','No Action','$punishment','$reason','$uname')";

kevinsequeira

'".date('m/d/Y H:i:s')"',' ure missing a .

'".date('m/d/Y H:i:s')."','

reg
kevin

rIkLuNd

There is a dot missing!!

$sql .= "'".date('m/d/Y H:i:s')."','$postdat','No Action','$punishment','$reason','$uname')";

EDIT: I did not see Kevins post..

sdjensen

the problem could be that you are trying to put more vaiables into the db than there are fields.

How is your db designed?

rIkLuNd

Originally posted by sdjensen
the problem could be that you are trying to put more vaiables into the db than there are fields.

When I count them, I get 11 fields and 11 values.. Did you mean like that or that he's got a field in his query that isn't in the table?

sdjensen

Originally posted by sdjensen
use this statement after the last ) and before the ; to see the why the statement is failing.
or die(mysql_error());
[/B]

to quote myself, this line will give the information why the insert statement fails.

It can because one of the fields are expection a number but are receiving a string.
Also if a fieldname is spelled wrong.

Try to replace all the ` with a normal '

rIkLuNd

My insertion queries usually looks like this:

$sql = "INSERT INTO table(field1,field2,field3) VALUES('value1','value2','value3')";

MoldRat

Ok, its not giving me errors, but its not entering data into the database, ie

		if ($action == 'dt'){
    $sql=mysql_query("INSERT INTO `modded` (`modbod`,`board`,`topic`,`message`,`modby`,`moddedat`,`postedat`,`action`,`punishment`,`reason`,`moduser`) VALUES ('$messbod','$board','$topic','$message','$modby','".date('m/d/Y H:i:s')."','$postdat','Topic Deleted','$punishment','$reason','$uname')");
    $sql=mysql_query("DELETE FROM `messages` WHERE messageid='$message'");
	$sql=mysql_query("DELETE FROM `topics` WHERE topicid='$topic'");
	}

Its not entering stuff, or deleting it.

rIkLuNd

Maybe this is only like this when it's showed here at PHPbuilder, but you cannot have a new-line inside the name of a column in the query like it is above..

Jack_Slocum

Lose the ` around your table name and field names. You can use ' if you'd like but it's not needed.

MoldRat

Its only like that when I paste it here.
And the backticks arent the problem.

Jack_Slocum

error_reporting(E_ALL);

Put that line right before your code and let us know what you get.