Where did I go wrong?

[borrmaskin]

Tried to do a nice mailing function.
Just to send everything in a $message[]
Unfortunately it seemes to de something wrong with it...

function Message($head, $message, $owner) {
foreach ($message as $the_message) {
$new_message="$the_message\n$new_message";
}

$query="select email from org where owner = $owner";
$result=Query($query);
$row=mysql_fetch_array($result)
$recipient=$row["email"];

mail("[e-mail]", "$head","$new_message","From: [email][e-mail][/email]");
IF ($message > NULL) Message($head, $message, $dork);

}

[xblue]

$result=Query($query); 

Do you have a function called query? If not, I'd think this is supposed to be mysql_query($query)?

And I can't see you are doing anything to $message within the function, so do you really mean to call the function again recursively with exactly the same $message parameter?

Also, I don't see $dork defined anywhere.

Maybe you could explain a little more detailed what you are trying to do?

BTW, unless owner is a numeric field in you database, you need to put single quotes around $owner.

[borrmaskin]

Thanks for the help.
unfortunately it did nothing for me.
I get a parse error about10 rows before this function.
But i know it is this function that does it.
If i remove this function the error goes away...

And i get a MySQL error message from about 100 rows after this.
Saying that $owner is empty.
"SELECT * FROM sida where owner = order by id"
Does the function "swallow" the var?

I figure that since it defined outside functions, it will always be avaliable outside functions...

#insert the head of the message, the array message, and the owners id
function Message($head, $message, $owner) { 
#conver the array message to a readeble text
foreach ($message as $the_message) { 
$new_message="$the_message\n$new_message"; 
} 
#get the email of the recipient $owner
$query="select email from org where owner = $owner"; 
#using the fancy function Query() that i made.
$result=Query($query); 
$row=mysql_fetch_array($result) 
$recipient=$row["email"]; 
#send the mail
mail("$recipient", "$head","$new_message","From: [email][e-mail][/email]"); 
}

IF ($message > NULL) Message($head, $message, $owner);

[xblue]

you should have mentioned the parse error in your first post (makes it far easier to help if you let us know what is going wrong).

you forgot the semicolon in this line:
$row=mysql_fetch_array($result)

a you aware of the possibility to include your code in [PHP] [/PHP]? You can still do it in your last post using the edit button, it's easier to read your code then.