wp_enqueue_script shouldn't convert source URL ampersands to character entities

[filosofo]

As ​Mike Schinkel pointed out, when a script source URL has ampersands in it, they get converted to their character entities.

Instead, clean_url() should not use the default "display" argument.

[sivel]

I do not think this should be done. Not encoding the ampersands will break xhtml validation and I have tested that it does indeed works as expected while encoded.

From my email to wp-hackers:

Okay, I put together a quick plugin to prove to myself that either the
use of & does work or does not work. In short I found that it
*does* work.

I'll even provide my plugin code so that anyone else who wants can test it.

The code can be found at ​http://gist.github.com/71796.txt

Save this as a .php file and place in your plugins directory. Once
activated navigate to your blog and you should see a javascript alert
that either says 'IT WORKED!' or 'IT DIDN'T WORK!" followed by some
extra data. It won't do the alert in the admin as I figured it might
be annoying to try to deal with the alert while activating and
deactivating.

[filosofo]

sivel is right. His proof plugin shows that it works correctly, and it is ​invalid xhtml

I apologize for not verifying this myself.

[SergeyBiryukov]

#23502 was marked as a duplicate.