[wp-trac] [WordPress Trac] #26886: wp_enqueue_script doesn't load all scripts
WordPress Trac
noreply at wordpress.org
Mon May 27 17:14:37 UTC 2019
#26886: wp_enqueue_script doesn't load all scripts
-------------------------------+------------------------------
Reporter: alfredocubitos | Owner: (none)
Type: defect (bug) | Status: reopened
Priority: normal | Milestone: Awaiting Review
Component: Script Loader | Version: 3.8
Severity: normal | Resolution:
Keywords: reporter-feedback | Focuses:
-------------------------------+------------------------------
Comment (by azaozz):
Replying to [comment:38 emanueleivaldi]:
> we sorted the query parameters at the cdn level in order to increase the
hit ratio on cached pages.
Yeah, that's a good optimization (although not sure how often it gets
used). However this should not happen when the query parameters form an
array, like in this case. Perhaps you should look at the param "names" and
if more than one contain `[` and `]` just leave the query string as-is :)
Assuming the patch from
https://core.trac.wordpress.org/attachment/ticket/45346/45346.diff works
well. Going to commit it and close this ticket with it too. Please feel
free to reopen with more info and examples if it doesn't fix the bug(s)
reported here.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/26886#comment:39>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list