Parse error: syntax error, unexpected T_STRING

Gnuff

Hello, I get this error:
Parse error: syntax error, unexpected T_STRING in /home/snodigfi/public_html/borders/inc.config.php on line 12

<?php

mysql_pconnect("localhost","snodigfi_*****","*****") or die(mysql_error());
mysql_select_db("snodigfi_borders") or die(mysql_error());

$site_com = 'Borders';




//hourly money gain
(SELECT * FROM states WHERE password='$password' AND statename='$statename')
$statename = mysql_fetch_array($selection);

//money amount to add for each round
$add_money = 1000;

//time now, and round time in seconds (1h=60*60seconds=3600)
$time_now = time();
$round = 3600;

//time of last update+time needed must be smaller then time now to update
if (($statename[lastupdate]+$round) <= $time_now) {

//see how many rounds (hours) were there from last update
$nr_rounds = floor(($time_now-$statename[lastupdate])/$round);

//calculate how much money user gained
$all_money = $nr_rounds * $add_money;

//calculate how many rounds in seconds (how many hours in seconds)
$add_time = $nr_rounds * $round;

//lets update users table
mysql_query("UPDATE states SET lastupdate=lastupdate+'$add_time', money=money+'$all_money' WHERE id = '$statename[id]'") or die(mysql_error());

//here you can refresh page (so you can see progress) or select user again using login
}

?>

I cant figure out what the problem is, great if someone would be able to help me, thank you.

bradgrafelman

Your problem is that you have a (presumably) MySQL query in the middle of your PHP script; you can't simply throw a MySQL query in a random spot and expect PHP to know what to do with it. Instead, build the query in a string and execute the query using [man]mysql_query/man.

Pehape

is this the line 12 :

(SELECT FROM states WHERE password='$password' AND statename='$statename')

$selection = mysql_query ("SELECT * FROM states WHERE password='$password' AND statename='$statename'")

bradgrafelman

Yes, that is the line I was referring to.

I don't understand the rest of your post... did you resolve your issue? Is there a new problem?

Gnuff

I changed the thing "Perhape" said, now I got this error:

Parse error: syntax error, unexpected T_VARIABLE in /home/snodigfi/public_html/borders/inc.config.php on line 15

bradgrafelman

Don't forget the semicolon at the end of that line.

Pehape

@
I just tried to solve Gnuff problem 🙂
and Gnuff, I just guessing your problem by using your $selection to keep your executed query. ( i assume that it is the variable you are using too, as you suddenly use that in your fetch_array function ) .. and don't forget the semicolon as mr.bradgrafelman said.
sorry if i'm wrong.

bradgrafelman

Oops, sorry, didn't notice the change in posters... :o

Also note that array indexes, Gnuff, in your case, should be strings; $array[name] doesn't have a string as an index but a constant. Since I doubt you've defined these constants, you need to use strings as your indexes (example: $array['name']). More information here: [man]language.types.array[/man].

floating

Hello. I get this same error.

Here's some background. I am trying to install a php-script. When i first run the install, it went nicely through saying that the installation was ok. Then I went to the admin page, which gave me an error "Warning: mysql_fetch_array(): supplied argument is not a valid MySQL result resource". Also, when I go to my sqlyog to look at the database and the newly created tables, I can't find them. It didn't create any tables(or I can't see them). After that I was guided in IRC to append $query = mysql_query($sql)|| die mysql_error(); the bolded part in these lines.( || die mysql_error(); was not in original install.php-file) After that when I ran the install, I got that T_STRING problem on that line.

#
Parse error: syntax error, unexpected T_STRING in C:\Program Files\Apache Software Foundation\Apache2.2\htdocs\wlmod\install.php on line 27
#

#
here are lines 26 starting with $sql and line 27 starting with $query:
#

 $sql = "CREATE TABLE $playerstable (player_id int(10) DEFAULT '0' NOT NULL auto_increment, name varchar(20) DEFAULT '' NOT NULL, passworddb varchar(10), approved  varchar(10) DEFAULT 'no', mail varchar(50), icq varchar(15), aim varchar(40), msn varchar (100), country varchar(40), rating int(10) DEFAULT '1500', games int(10) DEFAULT '0', wins int(10) DEFAULT '0', losses int(10) DEFAULT '0', points int(10) DEFAULT '0', totalwins int(10) DEFAULT '0', totallosses int(10) DEFAULT '0', totalpoints int(10) DEFAULT '0', totalgames int(10) DEFAULT '0', rank int(10) DEFAULT '0', streakwins int(10) DEFAULT '0', streaklosses int(10) DEFAULT '0', ip varchar(100), PRIMARY KEY (player_id))";
#

#
$query = mysql_query($sql)|| die mysql_error();

My guess is that the line 26 has some syntax problems, or my mysql 6.x is some how messed up or non-compliant(to not make those tables)

This script is rather old, and I tried to e-mail the maker of the script aswell, but I thought I'll ask here too.

thx

bradgrafelman

The problem is that you must surround [man]die/man's parameter with parenthesis.

floating

Thank you \o/

The login page opens now, I enter nick and a password and then.. New problem is:

Creating tables...

Invalid default value for 'player_id'

edit:

Alright. I probably posted that one too quickly, since I found a solution to that problem from google \o/ http://www.webmasterworld.com/databases_sql_mysql/3440204.htm

lets see how far i get next.. 🙂