Ticket #7861 (closed defect (bug): fixed)

Opened 3 months ago

Last modified 3 months ago

"Unable to locate WordPress directory." on core update & PHPDoc for WP_Filesystem_Base

Reported by: DD32 Assigned to: anonymous
Priority: normal Milestone: 2.7
Component: General Version: 2.7
Severity: normal Keywords: has-patch
Cc:

Description

Attached patch does 2 things:

  • Moves sanitization for windows paths around a bit
  • Fixes trailing slash problems which caused "Unable to locate WordPress directory." on core upgrade attempts with FTP
  • Added PHPDoc for most of the class functions. (Hopefully the docs are inline with the rest of WP..)

Attachments

7861.diff (6.9 kB) - added by DD32 on 10/10/08 09:32:18.
7861.2.diff (7.1 kB) - added by DD32 on 10/10/08 09:43:12.
7861.3.diff (1.1 kB) - added by ShaneF on 10/11/08 02:21:01.

Change History

10/10/08 09:32:18 changed by DD32

  • attachment 7861.diff added.

10/10/08 09:32:52 changed by DD32

  • PHPDoc also adds correct sources for the helper functions

10/10/08 09:43:12 changed by DD32

  • attachment 7861.2.diff added.

10/10/08 09:44:25 changed by DD32

attachment 7861.2.diff added.

  • Adds a check for situations where the FTP account is rooted to the base of the WordPress install (Which some security concious people do, and some webhosts do(when WordPress is installed into the base of the public_html folder))

10/10/08 10:02:47 changed by azaozz

  • status changed from new to closed.
  • resolution set to fixed.

(In [9117]) Fix for "Unable to locate WordPress directory" on core update and PHPDoc for class-wp-filesystem-base.php, props DD32, fixes #7861

10/10/08 10:40:04 changed by azaozz

(In [9118]) Check if the FTP account is rooted to the base of the WordPress install, props DD32, see #7861

10/11/08 02:20:00 changed by ShaneF

  • status changed from closed to reopened.
  • resolution deleted.

10/11/08 02:21:01 changed by ShaneF

  • attachment 7861.3.diff added.

10/12/08 10:44:07 changed by DD32

attachment 7861.3.diff added.

That fixes the issue outlined by Viper007Bond in #7872: 

Downloading update from http://wordpress.org/nightly-builds/wordpress-latest.zip 
Unpacking the core update
Verifying the unpacked files
The update could not be unpacked
Installation Failed

10/12/08 18:09:45 changed by azaozz

  • status changed from reopened to closed.
  • resolution set to fixed.

(In [9128]) Core update: fix for "The update could not be unpacked" error, props ShaneF, fixes #7861