Opened 3 years ago
Last modified 2 years ago
#14157 new enhancement
wp-includes references should be wiped off
| Reported by: |
|
Owned by: | |
|---|---|---|---|
| Priority: | lowest | Milestone: | Future Release |
| Component: | General | Version: | |
| Severity: | minor | Keywords: | has-patch needs-testing dev-feedback gsoc |
| Cc: | wojtek.szkutnik@…, anantshri |
Description
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' );
Attachments (1)
Change History (7)
comment:1
steak
— 3 years ago
- Summary changed from wp-include references should be wiped off to wp-includes references should be wiped off
comment:2
scribu
— 3 years ago
- Milestone changed from Awaiting Review to Future Release
- Priority changed from normal to lowest
wojtek.szkutnik
— 3 years ago
comment:4
wojtek.szkutnik
— 3 years ago
- Cc wojtek.szkutnik@… added
- Keywords has-patch needs-testing dev-feedback added; needs-patch removed
comment:5
wojtek.szkutnik
— 3 years ago
- Keywords gsoc added
comment:6
anantshri
— 2 years ago
- Cc anantshri added
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.
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?