variable array and a mysql insert statement

viveleroi0

I cannot figure out how to make this work properly... it is a function that will log errors to a database, but the issue is that I am not putting the $Error[''] variables in there correctly.

$Error['error_message']  ="error_message";
$Error['error_page']  = "error_page";
$Error['error_date']  = "error_date";
$Error['error_IP']  = "error_IP";
$Error['error_agent']  = "error_agent";



function Log_Errors($error_msg,$page,$date,$ip,$agent)
	{
golbal $Error_Table;
$Log_Errors="INSERT INTO $Error_Table ("$Error['error_message']", "$Error['error_page']", "$Error['error_date']", "$Error['error_IP']", "$Error['error_agent']") VALUES (\"$error_msg\", \"$page\", \"$date\", \"$ip\", \"$agent\", )";
mysql_query($Log_Errors) or die ("<p>Database Error. We're sorry, but the administrator notification failed.</p>");
	}

LordShryku

You're outside of the function scope with the array elements. You need to put the array inside of the function or declare it global.

viveleroi0

I changed it to this:

function Log_Errors($error_msg,$page,$date,$ip,$agent)
	{
		golbal $Error_Table;
		golbal $Error['error_message'];
		golbal $Error['error_page'];
		golbal $Error['error_date'];
		golbal $Error['error_IP'];

	$Log_Errors="
	INSERT INTO $Error_Table ("$Error['error_message']", "$Error['error_page']", "$Error['error_date']", "$Error['error_IP']", "$Error['error_agent']") 
	VALUES (\"$error_msg\", \"$page\", \"$date\", \"$ip\", \"$agent\", )";

	mysql_query($Log_Errors) or die ("<p>Database Error. We're sorry, but the administrator notification failed.</p>");
}

and I get this error:

Parse error: parse error, unexpected T_VARIABLE in config.php on line 83

Line 83 is the golbal $Error_Table; line

LordShryku

Should be global, not golbal

viveleroi0

oh my - Im such a dork.

But no matter what I do, it seems to hang up on the brackets. Maybe I shoud just make them regular variables.

LordShryku

Or just global $Error;, which would take care of the whole array.

viveleroi0

But how would I put those into the mysql insert query? It seems to not like them there. If I just do:

$Error['error_message']="some string";
$error_message=$Error['error_message'];

then that is just a waste of time - I should just save time by writing:

$error_message="some string";

LordShryku

Concatenation

function Log_Errors($error_msg,$page,$date,$ip,$agent) {
   global $Error_Table;
   global $Error;

   $Log_Errors = "INSERT INTO ".$Error_Table." 
                  (".$Error['error_message'].", ".$Error['error_page'].", 
                   ".$Error['error_date'].", ".$Error['error_IP'].", 
                   ".$Error['error_agent']."
                  ) VALUES (
                   '".$error_msg."', '".$page."', '".$date"', '".$ip."', '".$agent"'
                  )";

   mysql_query($Log_Errors) or die ("<p>Database Error. We're sorry, but the administrator notification failed.</p>");
}