[wp-trac] [WordPress Trac] #26886: wp_enqueue_script doesn't load all scripts

WordPress Trac noreply at wordpress.org
Mon May 27 16:54:13 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 emanueleivaldi):

 Replying to [comment:37 azaozz]:
 > Replying to [comment:33 emanueleivaldi]:
 > > if you keep the order of the query parameters you can reconstruct the
 original string but if you don't...
 >
 > Why would you "mess up" the order of an array and expect it to work
 properly. It is not the same array any more... Even if we had one script
 per array element, the script loading order would change and it would
 still fail.
 >
 > Anyway, the above patch should fix this.

 Hi, we sorted the query parameters at the cdn level in order to increase
 the hit ratio on cached pages.

 url?a=1&b=1 would not generate the same cache key of url?b=1&a=1

-- 
Ticket URL: <https://core.trac.wordpress.org/ticket/26886#comment:38>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list