[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