wp-includes references should be wiped off

[steak]

I think all references to wp-includes should be wiped off and use the WPINC WP constant. This may allow for a future renaming of the wp-includes folder if needed.

E.g in script-loader.php:

$scripts->add( 'jquery-ui-core', '/wp-includes/js/jquery/ui.core.js', array('jquery'), '1.7.1' );

To:

$scripts->add( 'jquery-ui-core', '/' . WPINC. '/js/jquery/ui.core.js', array('jquery'), '1.7.1' );

[wojtek.szkutnik]

Patch attached. Although, I have a few questions:
wp-admin/includes/update-core.php - it is unclear to me, if the wp-includes references should stay or not

wp-admin/network.php and wp-includes/rewrite.php I need strong testing and feedback on these rewrites, can anyone confirm it works as expected?

wp-includes/js/thickbox/thickbox.js two references to wp-includes in a js file, how can these two be handled?

[anantshri]

Hi This should be done.... if no one is working on it i can try creating a patch, as per my initail scan of the whole folder structure only 1-2 files can have some problem other then that all could be modified.

will start working if no objection is recieved.

[wonderboymusic]

In 28903:

Use the WPINC constant when loading class-phpass.php

Props wojtek.szkutnik
See #14157.

[wonderboymusic]

In 28904:

WP_Filesystem_Base->abspath() should use the WPINC constant

Props wojtek.szkutnik
See #14157.

[wonderboymusic]

In 28905:

wlwmanifest_link() should use the WPINC constant

Props wojtek.szkutnik
See #14157.

[nacin]

[28905] is using WPINC directly in a URL, versus a filesystem path. It should instead be using includes_url().

[wonderboymusic]

In 28906:

Admin screens should use the WPINC constant

Props wojtek.szkutnik
See #14157.

[wonderboymusic]

In 28907:

WP_Scripts->in_default_dir() should use the WPINC constant

Props wojtek.szkutnik
See #14157.

[SergeyBiryukov]

Replying to nacin:

[28905] is using WPINC directly in a URL, versus a filesystem path. It should instead be using includes_url().

Also, string concatenation should probably not be replaced with commas, it seems less readable.

[wonderboymusic]

In 28908:

Use includes_url() in wlwmanifest_link().

Props nacin.
See #14157.

[wonderboymusic]

14157-script-loader.diff​ uses includes_url( ..., 'relative' ) all over script-loader.php

[wonderboymusic]

In 28911:

Use includes_url( ..., 'relative' ) in script-loader.php in lieu of hard-coding /wp-includes/.... everywhere.

Fixes #14157.

[wonderboymusic]

#28686 was marked as a duplicate.

[nacin]

[28911] should be reverted. This is much slower, and the script loader has an internal implementation where it detects /wp-includes/ or /wp-admin/ at the start of a script and then switches it out with a base url. Some other checks specifically depends on wp-includes and wp-admin.

[wonderboymusic]

In 28914:

Revert [28911] for performance concerns.

See #14157.

[SergeyBiryukov]

[28911] also broke wp-admin/load-styles.php, which calls wp_default_styles() without loading wp-includes/link-template.php, so includes_url() is unavailable there. See #28686.

[SergeyBiryukov]

What's left to do here?

[lancewillett]

Found a case in wp-admin/includes/ms.php: https://core.trac.wordpress.org/browser/trunk/src/wp-admin/includes/ms.php#L810

var tb_pathToImage = "../../wp-includes/js/thickbox/loadingAnimation.gif";

Should be changed to:

var tb_pathToImage = '<?php echo includes_url( 'js/thickbox/loadingAnimation.gif' ); ?>';

Not sure if the relative context as second argument is needed.

[SergeyBiryukov]

#29335 was marked as a duplicate.

[voldemortensen]

Fixing hardcoding in wp-admin/includes/ms.php

[wonderboymusic]

In 29741:

Make the URL output in _thickbox_path_admin_subfolder() use includes_url().

Props voldemortensen.
Fixes #14157.

[johnbillion]

#35825 was marked as a duplicate.