| 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) | ||
[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.