Opened 18 years ago
Closed 18 years ago
#7059 closed defect (bug) (fixed)
WP_filesystem changes to work with new directory constants
| Reported by: | DD32 | Owned by: | |
|---|---|---|---|
| Priority: | normal | Milestone: | 2.6 |
| Component: | Administration | Version: | 2.6 |
| Severity: | normal | Keywords: | has-patch needs-testing |
| Cc: | Focuses: |
Description
Thanks to #6938 and [7999] the WP Filesystem abstraction classes need a update in order for the plugin upgrader to continue working.
- All Fielsystem classes now extend the WP_Filesystem_Base class
- The code now has a few methods to get the various folders needed: wp_content_dir(), wp_plugins_dir(), abspath(), wp_themes_dir()
- The find_folder() method should locate any real path on the server hopefully. Be warned there will be cases where it'll locate the wrong folder if the requested folder doesnt exist.
- The gethchmod() and getnumchmodfromh() methods which were common have been moved to the base calss
- The upgrader code has a few extra error cases builtin (Eg. Copy Failed messages will now indicate what copy process failed)
- A lot of whitespace additions to the classes
Warning: Tested in a few setups, Needs a full testing with a variety of custom setups.
Attachments (4)
Change History (13)
#3
@
18 years ago
- Resolution fixed
- Status closed → reopened
(In [8009]) Make WP_Filesystem work with new directory constants. Props DD32. fixes #7059
The patch included an extra file wp-admin/includes/class-wp-filesystem-base.php which wasnt commited.
Thanks for the catch on the typo's Azaozz, I really need a spellcheck plugin for Dreamweaver that understands literal strings..
Note:
See TracTickets
for help on using tickets.
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
oops, a extra function sliped in :)