Ok having trouble with a downloaded file, it times out (30secs) I went through the code and found this line and when I comment it out the script seems to still work fine and the error goes away I have never really seen code used like this before so was wondering if I was missing anything here is the code...

  do
{
} while (($_REQUEST['option'] == 'new_item'));

Thats basicly it I have no idea what the use of it is

Pretty much this code says do nothing while the superglobal 'option' is equal to 'new_item'. It seems like you are timing out because 'option' is always equal to new_item, causing your script to enter an infinite loop. Is there a part of this code missing? There are some cases where an infinite loop can be a good thing but this doesn't seem like one.

Originally Posted by NullPointer

Pretty much this code says do nothing while the superglobal 'option' is equal to 'new_item'. It seems like you are timing out because new_item is always equal to new_item, causing your script to enter an infinite loop. Is there a part of this code missing? There are some cases where an infinite loop can be a good thing but this doesn't seem like one.

Nice explanation. I suppose this is the cause.

Thanks for the help,

Nah there is none of the code missing thats all it does, the rest of the script seems to be working totaly fine even tho I removed that loop, I still can't figure out why that would be used there I think there is loads of easier ways to do that.

Thanks again

That looks like it was written by a lazy coder. Even though superglobal vars can be overwritten and changed, its always better to treat them as static vars. Also the $_REQUEST SG var a mixed array of user input (get, post and cookie) and therfore should never change in theory during the course of execution of the script.