[wp-trac] [WordPress Trac] #62722: Fix all ABSPATH direct access errors
WordPress Trac
noreply at wordpress.org
Fri Dec 20 12:09:05 UTC 2024
#62722: Fix all ABSPATH direct access errors
------------------------------------+------------------------------
Reporter: bor0 | Owner: (none)
Type: defect (bug) | Status: new
Priority: normal | Milestone: Awaiting Review
Component: General | Version:
Severity: normal | Resolution:
Keywords: has-patch dev-feedback | Focuses:
------------------------------------+------------------------------
Changes (by bor0):
* keywords: => has-patch dev-feedback
Comment:
The attached file looks to address all the ABSPATH issues. Note that there
are other issues (such as `add_action` undefined, etc.) but I'd propose
addressing those as separate trac issues.
{{{
$ grep ABSPATH ~/dev/log/error_log | grep -o '/[^ ]*.php' | uniq
/opt/homebrew/var/www/wp-settings.php
/opt/homebrew/var/www/wp-admin/includes/class-wp-privacy-data-export-
requests-list-table.php
/opt/homebrew/var/www/wp-admin/includes/class-wp-upgrader.php
/opt/homebrew/var/www/wp-admin/includes/nav-menu.php
/opt/homebrew/var/www/wp-admin/includes/class-wp-privacy-data-removal-
requests-list-table.php
/opt/homebrew/var/www/wp-admin/includes/template.php
/opt/homebrew/var/www/wp-includes/functions.php
/opt/homebrew/var/www/wp-includes/blocks/require-dynamic-blocks.php
/opt/homebrew/var/www/wp-includes/class-wp-customize-setting.php
/opt/homebrew/var/www/wp-includes/class-wp-customize-panel.php
/opt/homebrew/var/www/wp-includes/class-simplepie.php
/opt/homebrew/var/www/wp-includes/cache.php
/opt/homebrew/var/www/wp-includes/class-IXR.php
/opt/homebrew/var/www/wp-includes/meta.php
/opt/homebrew/var/www/wp-includes/ms-blogs.php
/opt/homebrew/var/www/wp-includes/Requests/library/Requests.php
/opt/homebrew/var/www/wp-includes/wp-diff.php
/opt/homebrew/var/www/wp-includes/class-wp-customize-section.php
/opt/homebrew/var/www/wp-includes/class-wp-customize-control.php
/opt/homebrew/var/www/wp-includes/nav-menu-template.php
/opt/homebrew/var/www/wp-includes/default-widgets.php
/opt/homebrew/var/www/wp-includes/class-wp-http.php
/opt/homebrew/var/www/wp-includes/ms-settings.php
/opt/homebrew/var/www/wp-includes/script-loader.php
$ patch -p0 < ~/Desktop/62722.patch
patching file 'wp-admin/includes/class-wp-privacy-data-export-requests-
list-table.php'
patching file 'wp-admin/includes/class-wp-privacy-data-removal-requests-
list-table.php'
patching file 'wp-admin/includes/class-wp-upgrader.php'
patching file 'wp-admin/includes/nav-menu.php'
patching file 'wp-admin/includes/template.php'
patching file 'wp-includes/Requests/library/Requests.php'
patching file 'wp-includes/blocks/require-dynamic-blocks.php'
patching file 'wp-includes/cache.php'
patching file 'wp-includes/class-IXR.php'
patching file 'wp-includes/class-simplepie.php'
patching file 'wp-includes/class-wp-customize-control.php'
patching file 'wp-includes/class-wp-customize-panel.php'
patching file 'wp-includes/class-wp-customize-section.php'
patching file 'wp-includes/class-wp-customize-setting.php'
patching file 'wp-includes/class-wp-http.php'
patching file 'wp-includes/default-widgets.php'
patching file 'wp-includes/functions.php'
patching file 'wp-includes/meta.php'
patching file 'wp-includes/ms-blogs.php'
patching file 'wp-includes/ms-settings.php'
patching file 'wp-includes/nav-menu-template.php'
patching file 'wp-includes/script-loader.php'
patching file 'wp-includes/wp-diff.php'
patching file wp-settings.php
$ > ~/dev/log/error_log # empty error log
$ find . -name '*.php' | sed 's|^\./||' | xargs -I {} echo
"http://localhost:8080/{}" > urls.txt # generate urls
$ xargs -P 10 -n 1 curl -s -o /dev/null < urls.txt # visit each url
$ grep ABSPATH ~/dev/log/error_log | grep -o '/[^ ]*.php' | uniq
$
}}}
That is, after applying the patch, no ABSPATH errors are reported.
@SergeyBiryukov I'd like to get your attention on this ticket - since
you're blazingly fast at helping out :)
--
Ticket URL: <https://core.trac.wordpress.org/ticket/62722#comment:1>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list