[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