Looping a block of code.

[Red Blaze]

<?php
$username = $_SESSION['MM_Username'];
  $times = $_POST['times'];

for ($counter = 1; $counter < $times + 1; $counter++)
{
if ($HTTP_POST_FILES['file$counter']['size'] <= 0)
	{
		print "<b>File Not Saved</b><BR>";
		print "Because your file was over 2 MB, it wasn't recieved.<br>";
	}
	else
	{
$tempFile = $HTTP_POST_FILES['file$counter']['tmp_name'];
$destination = "/****/*******/public_html/photos/$username/" .
	$HTTP_POST_FILES['file$counter']['name'];
copy($tempFile, $destination);
		print "<b>Your File has been accepted</b><br>";
	}
}

?>

$times is the number of times I want the block of code to repeat itself.
'file$counter' has to be in order, such as 1, 2, 3... all the way to the specified number in the $times value.

Now, the loop works fine, however, no matter what it won't upload anything.

[noppid]

Does the $username folder exist and is it writable?

And, using copy() it expects to find the $HTTP_POST_FILES['file$counter']['tmp_name'] in the current folder when it is actually buffered in memory and not on disk if I'm reading the functions correcty.

[Andrew]

I think you might need to change all instances of this:

PHP Code:

$HTTP_POST_FILES['file$counter'] 


To this since $counter is a variable:

PHP Code:

$HTTP_POST_FILES['file' . $counter] 


[Red Blaze]

--CURRENTLY TESTING ANDREW'S POST--

In my case $username = Red Blaze. And that Directory does exist.

I tested it like this:

if ($HTTP_POST_FILES['file1']['size'] <= 0)
	{
		print "<b>File Not Saved</b><BR>";
		print "Because your file was over 2 MB, it wasn't recieved.<br>";
	}
	else
	{
$tempFile = $HTTP_POST_FILES['file1']['tmp_name'];
$destination = "/****/*******/public_html/photos/$username/" .
	$HTTP_POST_FILES['file1']['name'];
copy($tempFile, $destination);
		print "<b>Your File has been accepted</b><br>";
	}

That's without the looping. It only uploaded the file that came from form "file1". I don't really want to add the block of code 10 times (which is the limit of uploading forms, btw). The forms are using the same looping function.

This is the code in the uploading forms page.

  <?php
  $times = $_POST['times'];

for ($counter = 1; $counter < $times + 1; $counter++)
{
print "<tr>
      <td align=\"right\">File $counter</td>
        <td><input name=\"file$counter\" type=\"file\" id=\"file$counter\" /></td>
    </tr>";
}

  ?>

[noppid]

Originally Posted by Andrew

I think you might need to change all instances of this:

PHP Code:

$HTTP_POST_FILES['file$counter'] 


To this since $counter is a variable:

PHP Code:

$HTTP_POST_FILES['file' . $counter] 


Good catch!

[Andrew]

Thanks - I made that mistake so many times when I first started coding that it's become the first thing I check now when my scripts don't work

[Red Blaze]

lol Andrew thanks a ton. Now I learned my lesson.