[wp-trac] [WordPress Trac] #58563: Undefined variable $checkout in class-wp-automatic-updater.php line 175

[WordPress Trac]

#58563: Undefined variable $checkout in class-wp-automatic-updater.php line 175
-------------------------------------------------+-------------------------
 Reporter:  jqz                                  |       Owner:  (none)
     Type:  defect (bug)                         |      Status:  new
 Priority:  normal                               |   Milestone:  6.3
Component:  Upgrade/Install                      |     Version:  6.2
 Severity:  normal                               |  Resolution:
 Keywords:  has-patch has-unit-tests needs-      |     Focuses:
  testing-info needs-testing                     |
-------------------------------------------------+-------------------------
Changes (by jqz):

 * keywords:
     has-patch has-unit-tests reporter-feedback needs-testing-info needs-
     testing
     => has-patch has-unit-tests needs-testing-info needs-testing


Comment:

 Replying to [comment:2 costdev]:
 > @jqz Can you provide your `open_basedir` directive (anonymized if
 necessary), and the value of the `ABSPATH` constant? Thanks!

 This are the relevant settings:
 - ABSPATH: `D:\webs\localhost\example.co.uk\sites\ofs3\wordpress/`
 - WP_CONTENT_DIR: `D:\webs\localhost\example.co.uk\sites\ofs3/content`
 - DOCUMENT_ROOT: `D:/webs/localhost/example.co.uk`
 - open_basedir:
 `D:/webs/localhost/example.co.uk/sites/ofs3/;C:\WINDOWS\TEMP/;d:/webs/localhost/`

 > 2. That `WP_Automatic_Updater::is_allowed_dir()` can return `false` for
 all directories on Windows, including `ABSPATH`.

 Looks like `WP_Automatic_Updater::is_allowed_dir()` is not accounting for
 case insensitivity of Windows filenames, nor that the path separator can
 be either `\` or `/`.

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