[wp-trac] [WordPress Trac] #10522: timeout for ftp connections as a constant in config.php

[WordPress Trac]

#10522: timeout for ftp connections as a constant in config.php
--------------------------+-------------------------------------------------
 Reporter:  ntm           |       Owner:  dd32                                   
     Type:  defect (bug)  |      Status:  new                                    
 Priority:  normal        |   Milestone:  2.9                                    
Component:  Filesystem    |     Version:  2.8.2                                  
 Severity:  normal        |    Keywords:  ftp connection timeout, plugin upgrade,
--------------------------+-------------------------------------------------

Comment(by ntm):

 I have thought about my last patch file for the
 [http://core.trac.wordpress.org/browser/trunk/wp-admin/includes/class-wp-
 filesystem-ftpext.php class-wp-filesystem-ftpext.php] file during the last
 days. Maybe it was a little bit to much to the additional error message in
 there as part of this ticket. Maybe it is more correct to open a separate
 ticket for this issue.

 Should I open a new ticket for that?

 (I have made another patch without it that message but with the described
 changes for class-wp-filesystem-ftpext.php and file.php.)


 Further I'm not sure if my changes (in class-wp-filesystem-ftpext.php) are
 ok or not.

 DD32 has said:
 >i'd use the define directly there, ie:
 >
 {{{
 if ( defined('TIMEOUT..') )
 $this->timeout = TIMEOUT...;
 }}}

 Should I create a patch which changes only class-wp-filesystem-ftpext.php
 like this (but with the constant name FTP_CONNECT_TIMEOUT)?

 What should i do?

-- 
Ticket URL: <http://core.trac.wordpress.org/ticket/10522#comment:10>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software