Only check for is_writable() pre-update for the files to which we need to write
|Reported by:||nacin||Owned by:||dd32|
|Component:||Upgrade/Install||Keywords:||has-patch fixed-major commit|
Before an update, we check the entire list of files in WordPress, based on a list of all checksums, to see if any need changing. If so, we then check to see if the file is writable. The files_not_writable error code occurs when a file we've determined we need to copy over is not writable, even after chmod 0644.
dd32 and I noticed an error condition where an incomplete update to 3.7 resulted in some about page image files not being created in wp-admin/images. For 3.7.1, files_not_writable was thrown because the checksums did not match for those files. But since a partial build was being used, there was no file to actually copy over in the working directory, even if we could write to it.
We should only perform is_writable() pre-update checks for files to which we need to write. If something goes wonky during a previous update, we can try to make sure that doesn't affect a partial build update to 3.7.1.
Change History (5)
- Owner set to dd32
- Resolution set to fixed
- Status changed from new to closed
- Keywords fixed-major added
- Resolution fixed deleted
- Status changed from closed to reopened