copy()

zingbats

<?
$image_name = $_FILES['imagefile']['name'];
$image_size = $_FILES['imagefile']['size'];
$image_type = $_FILES['imagefile']['type'];
?>
<?
$image_size_kb = $image_size / 1024;
$image_size_kb_rounded = round($image_size_kb, 2);
if ($image_size <= 20480) {
print "The file size is $image_size_kb_rounded kb The minimum size it is allowed to be is 20kb";
} else {
?>
<?
if (file_exists("$image_name")) {
print "It seems that the file is already on the server, sorry.";
} else {
?>
<? 
if(isset($Submit)) 
{
copy ($_FILES['imagefile']['tmp_name'], /usr/local/psa/home/vhosts/tycooneden.com/httpdocs/$_FILES['imagefile']['name']) or die ("Could not copy");  //LINE 21
}
?>

Gives the error..
Parse error: parse error in /usr/local/psa/home/vhosts/tycooneden.com/httpdocs/index.php(72) : eval()'d code on line 21

I have pointed out line 21

jonno946

You havent ended the if statement correctly.

You are missing a }.

zingbats

That isn't my whole script!

feldon23

Well, if you aren't closing the if statement within that section of code, you've got some scary code man!

zingbats

I Shall post the whole code 😃

<?
$image_name = $_FILES['imagefile']['name'];
$image_size = $_FILES['imagefile']['size'];
$image_type = $_FILES['imagefile']['type'];
?>
<?
$image_size_kb = $image_size / 1024;
$image_size_kb_rounded = round($image_size_kb, 2);
if ($image_size <= 20480) {
print "The file size is $image_size_kb_rounded kb The minimum size it is allowed to be is 20kb";
} else {
?>
<?
if (file_exists("$image_name")) {
print "It seems that the file is already on the server, sorry.";
} else {
?>
<? 
if(isset($Submit)) 
{
copy ($_FILES['imagefile']['tmp_name'], /usr/local/psa/home/vhosts/tycooneden.com/httpdocs/$_FILES['imagefile']['name']) or die ("Could not copy"); 
}
?>
<HTML>
<HEAD>
<TITLE></TITLE>
</HEAD>
<BODY>

<FORM METHOD=POST ACTION="savedownload.php">
<FONT FACE=ARIAL SIZE=4><B>New Download</B></FONT><HR>
<TABLE border=0 cellPadding=2 cellSpacing=0 width="100%" height="210">

  <TR>
    <TD style="WIDTH: 25%" width="25%" height="22"><FONT face=Arial size=2>Name</FONT></TD>
    <TD height="22"><FONT face=Arial size=2>
    <INPUT name=Name value="<?=$image_name?>" style="WIDTH: 100%" size="20"></FONT></TD>
    </TR>
  <TR>
    <TD vAlign=top height="132"><FONT face=Arial size=2>Description:</FONT></TD>
    <TD height="132"><TEXTAREA cols=60 name=Description rows=8 style="WIDTH: 100%"></TEXTAREA></TD>
    </TR>
  <TR>
    <TD height="22"><FONT face=Arial size=2>Category</FONT></TD>
    <TD height="22"><SELECT name=Categories style="WIDTH: 50%" WIDTH=50%> 
<?

		include "common.h";
				// Get categories from database and select the appropriate one
				$con=mysql_connect($DB_SERVER, $DB_USER, $DB_PASS);
				if (!$con) {
					echo mysql_error();
					exit;
				}
				if (!mysql_select_db($DB_NAME, $con)) {
					echo mysql_error();
					mysql_close($con);
					exit;
				}

				$rst = mysql_query("SELECT * FROM Download_Cat ORDER BY CatName", $con);
				if (!$rst) {
					echo mysql_error();
					mysql_close($con);
					exit;
				}

				while ($row = mysql_fetch_row($rst)) {

					echo "\t\t\t<OPTION ";
					if ($Category==$row[0]) 
						{echo "selected "; } 
					echo "VALUE=" . $row[0] . ">" . $row[1] . "</OPTION>\n";

				}			

				mysql_free_result($rst);

			?>	

</SELECT></TD>
</TR>
  <TR>
    <INPUT type="hidden" id=text2 name=FileName size=80 
      style="WIDTH: 100%" value="http://www.tycooneden.com/uploads/<?=$image_name?>"></FONT>
    </TR>
  <TR>
    <INPUT type="hidden" id=text3 name=FileSize size=10 value="<?=$image_size?>"></FONT>
    </TR>
  </TABLE>
<HR>
<INPUT TYPE="submit" VALUE="Save">
</FORM>
</BODY>
</HTML>
<?
}
?>
<?
}
?>

feldon23

copy ($_FILES['imagefile']['tmp_name'], /usr/local/psa/home/vhosts/tycooneden.com/httpdocs/$_FILES['imagefile']['name']) or die ("Could not copy");

Missing quotes around /usr/local/psa/home/vhosts/tycooneden.com/httpdocs/

I highly recommend wrapping complex variables like $FILES['imagefile']['name'] in curly braces like {$FILES['imagefile']['name']}.

And I don't know if you can do a die() like that; never tried it.