Combine scripts does not work
-----------------------------

                 Key: TAP5-1416
                 URL: https://issues.apache.org/jira/browse/TAP5-1416
             Project: Tapestry 5
          Issue Type: Bug
          Components: tapestry-core
    Affects Versions: 5.2.4
            Reporter: Donny Nadolny


In any Tapestry 5.2.4 in production mode, multiple javascript files are 
supposed to be combined in to one virtual file to reduce the number of HTTP 
requests on the page. This bug (regression from 5.1.0.5) is that this doesn't 
happen, even if combine scripts is explicitly enabled.

To reproduce, take any application (eg the maven quickstart one) and add the 
following line to the page:
@IncludeJavaScriptLibrary({"context:test.js", "context:test2.js"})
public class Index {
//rest of page
}

In src/main/webapp added two files, test.js and test2.js

In AppModule, make sure that production mode is not set to false.

In 5.1.0.5, going to that page would result in one virtual asset for the 
javascript files. In 5.2.4, they are all served separately.


Additional details:
This happens even if SymbolConstants.COMBINE_SCRIPTS is explicitly set to true 
in the AppModule (according to the docs, it should default to true if 
production mode is true).
This also happens using the new @Import annotation instead of 
@IncludeJavaScriptLibrary

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to