Package home | Report new bug | New search | Development Roadmap Status: Open | Feedback | All | Closed Since Version 0.4.0
Request #15893 Add deprecated backwards-compatible 'default_acl' config property
Submitted: 2009-02-12 20:36 UTC
From: gauthierm Assigned: schmidt
Status: Closed Package: Services_Amazon_S3
PHP Version: Irrelevant OS: Irrelevant
Roadmaps: (Not assigned)    
Subscription  
Comments Add Comment Add patch


Anyone can comment on a bug. Have a simpler test case? Does it work for you on a different platform? Let us know! Just going to say 'Me too!'? Don't clutter the database with that please !
Your email address:
MUST BE VALID
Solve the problem : 44 + 11 = ?

 
 [2009-02-12 20:36 UTC] gauthierm (Michael Gauthier)
Description: ------------ In CVS, the 'default_acl' config property has changed to just 'acl'. This is a good change, however, lots of code expects the other config property. Since no errors are raised from an incorrect config property, this change easily can go undetected, until you notice ACLs for files are wrong. I propose the 'default_acl' property should still be supported, but should be deprecated. It should be treated the same as 'acl'. Test script: --------------- Expected result: ---------------- Actual result: --------------

Comments

 [2009-02-15 21:42 UTC] schmidt (Christian Schmidt)
-Status: Open +Status: Closed -Assigned To: +Assigned To: schmidt
Fixed in CVS: http://news.php.net/php.pear.cvs/55339 Some time after 0.2.0 has been released, I will add a E_USER_NOTICE about default_acl being deprecated.