Hi,

Having chmodRecursive also in case 1 would be nice too.


Best regards,
Björn Dieding
 

Content & Commerce made easy
GMT +01:00 Hannover, Germany
 
Björn Dieding
         
Phone: +49 (511) 5904576
Mobile: +49 (151) 12169868
 
Email: [EMAIL PROTECTED] 
PayPal: [EMAIL PROTECTED]
MSN: [EMAIL PROTECTED]
YIM: bdieding
AIM: BjoernDieding
ICQ: 176927179
SKYPE: BjoernDieding

> -----Original Message-----
> From: [EMAIL PROTECTED] [mailto:components-
> [EMAIL PROTECTED] On Behalf Of Alexandru Stanoi
> Sent: Friday, November 09, 2007 3:02 PM
> To: Components
> Subject: [Components] ezcBaseFile::copyRecursive()
> 
> Hi all,
> 
> Commit 6220 added the copyRecursive() static method to ezcBaseFile.
> 
> static public function copyRecursive( $source, $destination, $depth =
> -1, $dirMode = 0775, $fileMode = 0664 )
> 
> On IRC it was discussed that maybe there are too many arguments to this
> function and maybe we can split this function into copyRecursive and
> chmodRecursive (so copyRecursive only has 3 arguments). Kore asked me to
> do this, but I kind of forgot about it until now.
> 
> static public function copyRecursive( $source, $destination, $depth = -1 )
> 
> static public function chmodRecursive( $dir, $depth = -1, $dirMode =
> 0775, $fileMode = 0664 )
> 
> 
> Which is the best solution? (You've guessed, voting time again!)
> 
> 1) Keep the copyRecursive method as it is.
> 
> 2) Split it into copyRecursive and chmodRecursive (is there a BC break
> here as we need to remove the last 2 arguments of copyRecursive?).
> 
> Cheers,
> Alex.
> 
> --
> Alexandru Stanoi
> eZ Components System Developer
> eZ Systems | http://ez.no
> --
> Components mailing list
> [email protected]
> http://lists.ez.no/mailman/listinfo/components

Attachment: smime.p7s
Description: S/MIME cryptographic signature

-- 
Components mailing list
[email protected]
http://lists.ez.no/mailman/listinfo/components

Reply via email to