[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