[wp-trac] Re: [WordPress Trac] #10170: File Permissions on Creation
(Plugin but might be Theme as well)
WordPress Trac
wp-trac at lists.automattic.com
Sun Jun 28 11:05:12 GMT 2009
#10170: File Permissions on Creation (Plugin but might be Theme as well)
-------------------------------------+--------------------------------------
Reporter: hakre | Owner: dd32
Type: defect (bug) | Status: reopened
Priority: normal | Milestone: 2.8.1
Component: Filesystem | Version: 2.8
Severity: normal | Resolution:
Keywords: has-patch tested commit |
-------------------------------------+--------------------------------------
Comment(by dd32):
IMO, Theres no need to add verbose comments like that. A simple "Octal
representation of a unix permission" would fit better.
I'm adding a patch which strips out the permission property, since its
useless, And makes the permission a required arguement of ::chmod() (I
mean, You call it to be very specific, chmod to THIS permission set). The
method setDefaultPermission() has never really been used by WP, nor is it
actually useable.. So.. removed that as well.
My patch needs testing, As i've not actually tested it.
--
Ticket URL: <http://core.trac.wordpress.org/ticket/10170#comment:13>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software
More information about the wp-trac
mailing list