[wp-trac] [WordPress Trac] #58940: site-health.php page Fatal error on version: 6.3-RC2

[WordPress Trac]

#58940: site-health.php page Fatal error on version: 6.3-RC2
-----------------------------------------+-----------------------------
 Reporter:  utsav72640                   |       Owner:  (none)
     Type:  defect (bug)                 |      Status:  new
 Priority:  normal                       |   Milestone:  6.3
Component:  Site Health                  |     Version:
 Severity:  normal                       |  Resolution:
 Keywords:  reporter-feedback has-patch  |     Focuses:  administration
-----------------------------------------+-----------------------------

Comment (by SergeyBiryukov):

 Hi there, thanks for the ticket!

 I was able to reproduce the issue with `define( 'FS_METHOD', 'ftpext' )`.

 As noted above, [55556] / #33058 does not appear to be related, however
 the error comes from the Site Health test added in [55720] / #51857, so
 this was indeed introduced in 6.3.

 I don't think the `WP_Filesystem_FTPext` class should be patched here, I
 believe this is an issue with the test itself, specifically that we don't
 check the result of the `WP_Filesystem()` call, or whether the
 `$wp_filesystem` object is ready for use and does not have any errors. In
 my case, the issue was caused by missing credentials.

 Looking closer, `_wp_delete_all_temp_backups()` appears to have the same
 issue.

 [attachment:"58940.diff"] attemps to use the stored credentials if
 available, and displays an error message otherwise.

-- 
Ticket URL: <https://core.trac.wordpress.org/ticket/58940#comment:15>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform