upload - removing periods from filename

kkiely

someone posted this code online as a way to make sure there is only 1 period in a filename during upload but i cant get it to work, can someone help?

$parts = explode(".", $uploadfilename);
if (count($parts) > 1){
$extension = '.' . $parts[count($parts) - 1]; //note we put the . back in here...
unset($parts[count($parts) - 1]);
$uploadfilename = implode('', $parts);
}

thanks

LordShryku

"can't get it to work"....
Not helping here...
What exactly is your problem? What's it doing? Are youi getting an error message? What's the outcome after running this?

nunomira

hi,

Like you have it, if

$uploadfilename = "some.file.jpg";

in the end

$uploadfilename = "somefile";

is what you get.

If you want

$uploadfilename = "somefile.jpg";

You can't forget the $extension:

$parts = explode(".", $uploadfilename); 
if (count($parts) > 1){ 
	$extension = '.' . $parts[count($parts) - 1];
	unset($parts[count($parts) - 1]); 
	$uploadfilename = implode('', $parts) . $extension; // add the extension 
}

kkiely

sorry to not be more specific.
What is happening is that it is stripping out all the periods as well as the extension
so i upload file "$bad..file.JPG" and it comes out "badfile"

Gary_King

Originally posted by kkiely
sorry to not be more specific.
What is happening is that it is stripping out all the periods as well as the extension
so i upload file "$bad..file.JPG" and it comes out "badfile"

Did nunomira's code helped?

kkiely

wow, it did it. Thanks.
Now what is the best way to make a filename unique? I've been reading that adding a time stamp is a good idea.

nunomira

yes,

something like

$unique = date ("Ymdhis");

is a unique name

Gary_King

Originally posted by nunomira
yes,

something like

$unique = date ("Ymdhis");

is a unique name

Yeah 🙂