$PHP_SELF and $REMOTE_ADDR in functions?

Anon

On my site, I have a file with my commonHeader and commonFooter, which includes my HTML, etc. Works great. Although, I am having some problems.

Whenever I try to get $REMOTE_ADDR or $PHP_SELF in that header function, I get nothing. How can I make this return the correct values.

Furthermore, although not a big issue, is I am having trouble getting session variables in these common functions. Help? Thanks.

petrb

Post your syntax to show how you are trying to getenv()...

--ph

Anon

OK...

function commonHeader($title, $header) {
@session_start();

// update user list
db_Connect();
$timestamp=time();
$timeout=time()-300; // value in seconds (300 = 5mins)
$result=mysql_db_query("urbandig", "INSERT INTO activity VALUES ('$timestamp', '$REMOTE_ADDR', '$PHP_SELF')");
$result=mysql_db_query("urbandig", "DELETE FROM activity WHERE timestamp<$timeout");

mysql_close(); ?>

...php and blah blah.

It's pretty standard...

jmakeig

You've got scoping problems:

Inside the function commonHeader() the variables $PHP_SELF and $REMOTE_ADDR are local (and thus empty before you set them). Put

global $PHP_SELF; global $REMOTE_ADDR;

inside the function to let PHP know that you would like to use the global versions of these variables and not the local versions.

Justin

petrb

I may be grasping here, but try this before you call commonHeader()...

$add = getenv($REMOTE_ADDR);
$scrname = getenv($PHP_SELF);

echo $add$scrname; //debug

if that returns the strings, good - then wipe the echo and use these variables instead in your insert statement.

--ph

petrb

Yep, Justin is right, but you don't really have to declare the envvars as global if you define them outside the function. I think I took an end curly for granted there.

--ph

Anon

Hmmm, yes. $REMOTE_ADDR works but $PHP_SELF does not. ...also, any ideas about the sessions thing?

I may get all my questions answered (I've been wondering about this stuff for awhile now...) 😉 Thanks!

Anon

you can also roll your globals into one line like this:

global $REMOTE_ADDR, $PHP_SELF, $SessionID;

etc.

btw i made up that sessionID var, it's not real 🙂