[wp-trac] Re: [WordPress Trac] #6245: FTP Class Incompatibilities
WordPress Trac
wp-trac at lists.automattic.com
Sun Mar 16 07:19:54 GMT 2008
#6245: FTP Class Incompatibilities
-------------------------------+--------------------------------------------
Reporter: DD32 | Owner: anonymous
Type: defect | Status: new
Priority: normal | Milestone: 2.5
Component: Administration | Version: 2.5
Severity: normal | Resolution:
Keywords: ftp needs-testing |
-------------------------------+--------------------------------------------
Comment (by DD32):
>attachment 6245.2.diff added.
> needs-testing: New code for searching for wordpress install via ftp.
Also introduces FTP_BASE constant to fit in with the other FTP_ constants
Calling testers, Calling testers!
This code should fail gracefully in event that the folder cannot be found,
so you should be able to test ok.
This code should also handle cases where the wordpress install which is
being delt with, is inside another, eg:
{{{
/home/.../dd32.id.au/wordpress/ <- Main WordPress blog
/home/.../dd32.id.au/wordpress/devblog/ <- A WordPress blog in a subfolder
}}}
Previously, updating a plugin in the devblog would find the main blog
instead, as it just assumed the first blog it found was correct.[[BR]]
New code should find the Devblog instead.
As an added bonus, It should also be much faster in locating it for the
ajority of WordPress installs.
You may also install (version 1.4) http://dd32.id.au/files/wp-filesystem-
tester.zip and follow through that way, As long as this patch isnt
commited(just yet) "FS" will be the current code, and will fail on certain
setups (And you'll see it does a lot of folder changing), "FS2" in the
above plugin uses the new function, and should locate it with much fewer
iterations needed.[[BR]]
If the paths are not correctly displayed on that plugin, I need to hear
from you.
--
Ticket URL: <http://trac.wordpress.org/ticket/6245#comment:1>
WordPress Trac <http://trac.wordpress.org/>
WordPress blogging software
More information about the wp-trac
mailing list