demonhale — Sun, 11 Jun 2006 05:26:54 +0000

@adjroth

been a great help so far,

The code is correct at checking the files, thats why i said i arrived at the same code using the while earlier, the problem lies in writing the concatenate, im figuring it out now by using string.drop command ...

Thanks for the additional tip, Ill check it out...

adjroth — Sun, 11 Jun 2006 05:13:22 +0000

Quote: Anyways, another question, whats the security issue if I saved a file to html directly rather than on a txt file, and allowing html codes on the textarea,? Any light on this...?

If this is a forum, profile entry, or any other related where other users will view these files, then there is a high security risk. Users may use script, iframe, or activex to run harmful software from your site, or run videos or music.

You may want to use "strip_tags" to strip the html tags. more information on this function can be found here http://us3.php.net/manual/en/function.strip-tags.php

adjroth — Sun, 11 Jun 2006 05:04:16 +0000

yes, you're right. what i mean to do was this ...

checks for "test.txt" then "test1.txt" then "test2.txt" and so on ...

$fileexist = $filename . '.txt';
if (file_exists($fileexist))
{
	$newfile = 0;
	while (file_exists($fileexist))
	{
		$newfile = $newfile + 1;
		$fileexist = $filename.$newfile.'.txt'; //if you get an error here replace with $fileexist = $filename.''.$newfile.'.txt';
	}
}

echo file_exists($fileexist) ? "File Already Exists, new filename: $filename" : 'File Created';

demonhale — Sun, 11 Jun 2006 04:56:47 +0000

@adjroth

Thanks for the rewrite, actually I used the same mehotd as you, using the while loop, the difference is i append filenames like this "test.txt" if duplicate "test1.txt" if duplicate "test11.txt" and so on...

What happens with yours is like this, "test.txt" if duplicate "test1.txt" then becomes "test12.txt" then "test123.txt" ... I think i need to clear $newfile = 0 again at the end... But really good help...

Anyways, another question, whats the security issue if I saved a file to html directly rather than on a txt file, and allowing html codes on the textarea,? Any light on this...?

brady.k — Sat, 10 Jun 2006 18:31:44 +0000

If you're wanting it to return "2" instead of "11", why not add them together instead of concatenating?

adjroth — Sat, 10 Jun 2006 15:33:37 +0000

i added a while loop to keep trying, starts at 1 and keeps going up until file_exists returns true.

$fileexist = $filename . '.txt';
if (file_exists($fileexist))
{
	$newfile = 0;
	while (file_exists($fileexist))
	{
		$newfile = $newfile + 1;
		$filename = $filename.''.$newfile;
		$fileexist = $filename . '.txt';
	}
}

echo file_exists($fileexist) ? "File Already Exists, new filename: $filename" : 'File Created';

demonhale — Sat, 10 Jun 2006 03:28:50 +0000

Kinda figured to non-duplicate but its messy... if anyone got an idea for a cleaner version, it would be good...

I used a filecheck with the getcw command, then if it returns true I apend "1" then "11"

It works but what i intended was "1" then "2" ,

well anyways, it works at the moment so no problem...