[pLog-svn] r5224 - plog/branches/lifetype-1.2/class/gallery/dao

[Jon Daley]

 	I have been playing around with a wordpress plugin (I'll probably 
port it to LT at some point, it mostly just a javascript thing) and one of 
the things that people complain about over there is using a "/" instead of 
DIRECTORY_SEPARATOR, to be compatible with Windows.
 	I haven't heard anyone complaining about that with LT.  I see that 
you just modified a string that has a "/" in it, and presumably, you are 
testing on windows, so I guess that must mean that "/" work fine on 
windows?

On Thu, 29 Mar 2007, mark at devel.lifetype.net wrote:

> Author: mark
> Date: 2007-03-29 14:35:18 -0400 (Thu, 29 Mar 2007)
> New Revision: 5224
>
> Modified:
>   plog/branches/lifetype-1.2/class/gallery/dao/galleryresourcestorage.class.php
> Log:
> Fixed a bug reported by hlps9999.
>
> Modified: plog/branches/lifetype-1.2/class/gallery/dao/galleryresourcestorage.class.php
> ===================================================================
> --- plog/branches/lifetype-1.2/class/gallery/dao/galleryresourcestorage.class.php	2007-03-29 17:24:50 UTC (rev 5223)
> +++ plog/branches/lifetype-1.2/class/gallery/dao/galleryresourcestorage.class.php	2007-03-29 18:35:18 UTC (rev 5224)
> @@ -244,7 +244,7 @@
> 			if( $config->getValue( "resources_naming_rule" ) == "encoded_file_name" )
> 				$origFile = $tmpFolder."/".rawurlencode($upload->getFileName());
> 			else
> -				$origFile = $tmpFolder."/".$upload->basename(getFileName());
> +				$origFile = $tmpFolder."/".basename($upload->getFileName());
>
> 			//do not use storeFile method because I have change filename in $tmpFolder.
> 			//$destFile = $this->storeFile( $resourceId, $ownerId, $origFile, RESOURCE_STORAGE_STORE_MOVE );