[wp-trac] [WordPress Trac] #26886: wp_enqueue_script doesn't load all scripts
WordPress Trac
noreply at wordpress.org
Mon Feb 23 23:31:28 UTC 2015
#26886: wp_enqueue_script doesn't load all scripts
-------------------------------+------------------------------
Reporter: alfredocubitos | Owner:
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 alturic):
Replying to [comment:16 dd32]:
> > Let me ask you this, is it possible if breaks when more than two
load[] Query String lines occur?
> It's possible that your server configuration is setup to break requests
like that, yes. It could be some kind of security plugin kicking in or
something.
So if it's literally WP that's "breaking" (note, I use that loosely) and I
know the exact code that's causing it, down to the char length, how can it
be the "server" outside of saying "because other's don't have the issue"?
> > Lastly, is there anything to worry about simply changing that from 128
to something higher? 300? 3000? etc...
> Yes, Servers which limit the maximum query var value length, using some
kind of security plugin..
So basically if I know (when changing that to a crazy value), it then
works correctly, nothing with WP at least will break putting that higher
so WP doesn't concat the string?
> > `/wp-admin/load-scripts.php?c=0&load%5B%5D=jquery-core,...`
> Also worth noting, that if you want to take that URL from the page
source (or network inspector in some browsers) and request that using the
browser (by putting it into the address bar) you need to properly decode
`load%5B%5D=` to `load[]=` otherwise it'll seemingly not load anything (or
load something wrong). In other words - `load%5B%5D=` is valid (and
correct) in the page source, but not when placed into the address bar.
Yea, that's been my point, it loads perfectly fine if I a.) convert it to
load[] in the url and b.) put those load[]'s not in the middle of script-
names. :P
--
Ticket URL: <https://core.trac.wordpress.org/ticket/26886#comment:17>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list