Error?

rapmonkey

When I run this line of script:

mysql_query("INSERT INTO " . $prefix . "members (username, displayname, password, email, membergroup) VALUES ('admin', 'GBX Administrator', 'admin', 'Administrator', 'Administrator')", $connection) OR DIE("Gravity Board X experienced an error while trying to create the first Administrator account: " . mysql_error());

I get the error:

Warning: mysql_query(): supplied argument is not a valid MySQL-Link resource in /home/public_html/install.php on line 54
Gravity Board X experienced an error while trying to create the first Administrator account:

I can't find out what's wrong. 🙁

cgraz

What does this print?

echo "Prefix: " . $prefix . "<br>";
$query_p = "INSERT INTO " . $prefix . "members (username, displayname, password, email, membergroup) VALUES ('admin', 'GBX Administrator', 'admin', 'Administrator', 'Administrator')";

print $query_p;

Cgraz

diego25

The SQL string is just ...a string; it's not a MySQL resource, so PHP it's not complaining about that. The only other possible MySQL resource is the $connection variable. That means that your connection to the MySQL database failed.

Diego

rapmonkey

Actually Diego, I have a bunch of queries before that line that use the $connection variable and they work properly.

sijis

cgraz has the best idea.
echo what the sql statement is recieving.

Maybe $prefix has a space?
Maybe you have a misspelled word?

honestly, my instinct tells me it has to do with the $prefix variable.

something you could try is putting the sql query in a $var.

like so:

$strSQL = "INSERT INTO " . $prefix . "members (username, displayname, password, email, membergroup) VALUES ('admin', 'GBX Administrator', 'admin', 'Administrator', 'Administrator')";

mysql_query($strSQL, $connection);

although i'm sure its probably won't make much of a difference.

xblue

just wondering... what about "password" as column name? It's definitely a mysql function, so the documentation about reserved words should mention it in some way.

Funny enough I don't find it in the list of the reserved words nor in the few that are allowed in mysql as exceptions.
http://www.mysql.com/doc/en/Reserved_words.html

Anyway I'd rename the column to make sure it's not failing because of that.

rapmonkey

For one, the $connection and $prefix variables are used earlier in the script and work correctly. Also, the password row in the MySQL table would have not been allowed to install if it would not work.

cgraz

Did you try the printing what I suggested earlier? You haven't mentioned a thing about what that printed, just how everyone suggestions are wrong. If you don't tell us that, then how do you expect to be helped?

Cgraz

rapmonkey

Sorry, I just guess I think it should work how it is.... I mean it's a normal query inserting data into a database, it should work.

cgraz

It's always a little mistake; trust me. Replace your query with the following and tell me what it prints; otherwise I can't help you.

echo "Prefix: " . $prefix . "<br>";
$query_p = "INSERT INTO " . $prefix . "members (username, displayname, password, email, membergroup) VALUES ('admin', 'GBX Administrator', 'admin', 'Administrator', 'Administrator')";

print $query_p;

Cgraz

rapmonkey

Ok, I plugged it in and got:

Prefix: gbx_
INSERT INTO gbx_members (username, displayname, password, email, membergroup) VALUES ('admin', 'GBX Administrator', 'admin', 'Administrator', 'Administrator')

The prefix shown (gbx_) IS correct.

cgraz

Now copy and paste this and see how it works.

$query_p = mysql_query("INSERT INTO " . $prefix . "members (username, displayname, password, email, membergroup) VALUES ('admin', 'GBX Administrator', 'admin', 'Administrator', 'Administrator')") or die(mysql_error());

Cgraz

rapmonkey

It prints the number one: 1.

Not quite sure why that worked and the other didnt...

rapmonkey

Oh, I got it. Stupid me, I was using the $connection variable when already connected to the database.

Also, this was in the install script and the $connection variable isn't used until AFTER the install. :o

Thanks for all your help. 🙂