[wp-trac] [WordPress Trac] #10889: filesystem's put_content() methods have inconsistent arguments -- causes .maintenance file to not be created
WordPress Trac
wp-trac at lists.automattic.com
Fri Dec 11 09:33:59 UTC 2009
#10889: filesystem's put_content() methods have inconsistent arguments -- causes
.maintenance file to not be created
-------------------------------+--------------------------------------------
Reporter: Denis-de-Bernardy | Owner: dd32
Type: defect (bug) | Status: new
Priority: normal | Milestone: 3.0
Component: Filesystem | Version: 2.8.4
Severity: normal | Keywords: needs-patch
-------------------------------+--------------------------------------------
Comment(by dd32):
Darn, forgot about this ticket.
> I'm not sure where the discussion is on the remainder of the patch.
Denis is correct, the put_contents() function signature varies between
classes, causing some upgrades to proceed without a .maintainence file.
I'll take a look at it now and post a patch, You can decide if it should
still wait for 3.0.
Attached patch also fixes the "fix" that was used elsewhere, since
put_contents() didnt do the chmod correctly, another call to chmod was
made. At present, on the Direct transport, this means Chmod() will be
called twice.
--
Ticket URL: <http://core.trac.wordpress.org/ticket/10889#comment:9>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software
More information about the wp-trac
mailing list