[wp-trac] [WordPress Trac] #53108: `get_theme_file_uri()` fails with query params
WordPress Trac
noreply at wordpress.org
Thu Apr 29 01:25:31 UTC 2021
#53108: `get_theme_file_uri()` fails with query params
-------------------------+------------------------------
Reporter: zulaica | Owner: (none)
Type: enhancement | Status: new
Priority: normal | Milestone: Awaiting Review
Component: Themes | Version: 5.7.1
Severity: normal | Resolution:
Keywords: | Focuses:
-------------------------+------------------------------
Changes (by dd32):
* type: defect (bug) => enhancement
* component: General => Themes
Comment:
The functionality currently isn't designed to handle query arguments, it's
intentionally to find a URI for a ''file''.
I would say the expected use-case currently is something like this:
{{{
// preferably:
$uri = add_query_arg( 'id', '41142aba6c0214da8ec1', get_theme_file_uri(
'js/script.js' ) );
// will probably work most of the time, unless a plugin has filtered the
URI to include query args:
$uri = get_theme_file_uri( 'js/script.js' ) . '?id=41142aba6c0214da8ec1';
}}}
Marking this as an enhancement instead as a result, the fact that this
works for parent themes is kind of an unexpected side effect of simply not
checking to see if the file exists (ie. `get_theme_file_url( '404.js' )`
returns a URI even though the file doesn't exist).
--
Ticket URL: <https://core.trac.wordpress.org/ticket/53108#comment:1>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list