renaming uploaded image fie

[IG88]

I have this function to upload User images, but I need it to either rename or simply add a time() or date() stamp to the file name.

 

Any suggestions?

 

	function CreateUserImage()
{
	global $modx;

	$imageAttributes = str_replace(', ', ',', $this->UserImageSettings);
	$imageAttributes = explode(',', $imageAttributes);

	if ($_FILES['photo']['size'] >= $imageAttributes[0])
	{
		$sizeInKb = round($imageAttributes[0] / 1024);
		$sizeError = str_replace('[+000+]', $sizeInKb, $this->LanguageArray[28]);
		return $this->FormatMessage($sizeError);
	}

	$userImage = $modx->config['base_path'].strtolower(str_replace(' ', '-', basename( $_FILES['photo']['name'])));
	if (!move_uploaded_file($_FILES['photo']['tmp_name'], $userImage))
	{
		return $this->FormatMessage($this->LanguageArray[29]);
	}

	// License and registration ma'am. I need to se an ID!
	if ($modx->getLoginUserID())
	{
		$currentWebUser = $modx->getWebUserInfo($modx->getLoginUserID());
		if ($this->Type == 'manager')
		{
			$currentWebUser['username'] = $_POST['username'];
		}
	}
	else
	{
		$currentWebUser['username'] = $this->Username;
		if ($this->Username == '' || empty($this->Username))
		{
			$currentWebUser['username'] = $_POST['username'];
		}
	}

[.josh]

http://www.php.net/rename

[IG88]

Thank you, but that does not actually help.

 

The line I am dealing with here is

 

$userImage = $modx->config['base_path'].strtolower(str_replace(' ', '-', basename( $_FILES['photo']['name'])));

[.josh]

echo out $userImage to give an example of what it looks like.

[IG88]

I have added

echo $userImage = $modx->config['base_path'].strtolower(str_replace(' ', '-', basename( $_FILES['photo']['name'])));

 

and got nothing. I also tried echo htmlentities(var_dump($userImage));

 

I think I am just echoing wrong.

 

[.josh]

echo $userImage;

 

should do the trick.  If it's not echoing anything, then I don't think that renaming your file is what you should be worrying about right now..

 

[IG88]

Doing that just echos out the filepath and name of the file.

 

/home/site/public_html/assets/snippets/webloginpe/userimages/8_5_12[1].gif

[.josh]

Well that was the point.  I needed to see what to work with. You do want to rename 8_5_12[1].gif to something else right?

 

$temp = explode(".", $userImage);
$temp[0] .= time();
$userImage = implode(".", $temp);

[IG88]

Yes that is exactly what I need to do. Adding the server time would look ugly but serve the purpose. There will be numerous seperate members uploading so it needs to be something random that will not repeat.

 

I am just not sure why I can not get the time() to work.

 

Thank you.

[.josh]

I am just not sure why I can not get the time() to work.

 

so that codeblock didn't work? As in...it's still producing the original file, as if time() were just ""?

[IG88]

It is not working.

 

Very interesting though, when I right click the image and look at the properties the location shows this really long string with a hash at the end.

 

http://www.site.com/image.php?src=/assets/snippets/webloginpe/userimages/bride_of_bob.jpg&w=100&h=100&aoe=1&hash=35dd829cf2b4ac5f79e8

 

Here is the code I was using.

		$userImage = $modx->config['base_path'].'assets/snippets/webloginpe/userimages/'.strtolower(str_replace(' ', '-', basename($_FILES['photo']['name'])));
	$temp = explode(".", $userImage);
               $temp[0] .= time();
               $userImage = implode(".", $temp);


	if (!move_uploaded_file($_FILES['photo']['tmp_name'], $userImage))
	{
		return $this->FormatMessage($this->LanguageArray[29]);
	}

[IG88]

Any one have any ideas?

[IG88]

Just hoping that some one who knows an answer will see this.