Problems with joining mutliple JS files due to URL generated by Helper

[oscar-b]

It seems that commit 81aa014 broke asset generation, at least on my configuration. (I can't find any test cases that test the actual content of the files being built from the urls generated from the Helper, so I couldn't re-create it that way.)

But, if I add some debug to js_files_controller.php in function get(), I can see that $this->params is wrong:

[url] => Array
    (
        [ext] => html
        [file] => Array
            (
                [0] => jquery-1.4.2.min
            )

        [amp;file] => Array
            (
                [0] => jquery.google-analytics

Obviously, it doesn't pick up the second file. If I change the path from using
&
to just
&
it works as it should.

[markstory]

Not really sure what the issue is here. Without the & the helper would be generating invalid HTML. The & should not be sent by the browser as it should convert the entity back to normal &.

[oscar-b]

Interesting, I guess I found a bug in Google Chrome (9.0.597.84 Linux).

If you click the URL from "view-source:domain.tld" it doesn't handle the & escaping properly. If you do it from the web inspector tool, it fetches the resource by the correct URL.

Strange, but feel free to close :)

[josegonzalez]

I'm on Google Chrome 11.0.696.3 dev, OSX. This issue still exists, and I am preparing a patch to deal with it, since I do not believe it will be going away any time soon, and the majority of my users are on Chrome versions affected by it.

[josegonzalez]

The following commit contains the changes that will "properly" fix this issue. josegonzalez/asset_compress@87ed164

There should also be a bug report on Chromium, but thats a separate issue.

[markstory]

Is there a place I could see this error in action? I tried locally using Chrome 10.0.648.133 on OSX and files downloaded fine when there were &'s in the urls.