Tapestry 5
  1. Tapestry 5
  2. TAP5-1756

Let the asset path prefix be configurable

    Details

    • Type: Improvement Improvement
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 5.3, 5.4
    • Fix Version/s: 5.3.1, 5.4
    • Component/s: tapestry-core
    • Labels:
      None

      Description

      Sometimes the asset path should be configurable to fulfill customer expectations. The asset path should be a configuration symbol, but now it is hard coded into the org.apache.tapestry5.internal.services.RequestConstants.ASSET_PATH_PREFIX variable.

        Activity

        Hide
        Massimo Lusetti added a comment -

        Just to set the correct fix version

        Show
        Massimo Lusetti added a comment - Just to set the correct fix version
        Hide
        Christian Riedel added a comment -

        you should change the fix version to 5.3.1. I'm missing this issue in the release notes

        Show
        Christian Riedel added a comment - you should change the fix version to 5.3.1. I'm missing this issue in the release notes
        Hide
        Hudson added a comment -

        Integrated in tapestry-trunk-freestyle #637 (See https://builds.apache.org/job/tapestry-trunk-freestyle/637/)
        TAP5-1756: Assets path can now be configured using tapestry.asset-path-prefix.

        tawus : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1211946
        Files :

        • /tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/SymbolConstants.java
        • /tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/internal/services/AssetDispatcher.java
        • /tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/internal/services/RequestConstants.java
        • /tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/internal/services/assets/AssetPathConstructorImpl.java
        • /tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/services/TapestryModule.java
        • /tapestry/tapestry5/trunk/tapestry-core/src/test/groovy/org/apache/tapestry5/integration/app3/PageCatalogTests.groovy
        • /tapestry/tapestry5/trunk/tapestry-core/src/test/java/org/apache/tapestry5/internal/services/AssetDispatcherTest.java
        • /tapestry/tapestry5/trunk/tapestry-core/src/test/java/org/apache/tapestry5/internal/services/ClasspathAssetAliasManagerImplTest.java
        • /tapestry/tapestry5/trunk/tapestry-core/src/test/java/org/apache/tapestry5/internal/services/ContextAssetFactoryTest.java
        Show
        Hudson added a comment - Integrated in tapestry-trunk-freestyle #637 (See https://builds.apache.org/job/tapestry-trunk-freestyle/637/ ) TAP5-1756 : Assets path can now be configured using tapestry.asset-path-prefix. tawus : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1211946 Files : /tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/SymbolConstants.java /tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/internal/services/AssetDispatcher.java /tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/internal/services/RequestConstants.java /tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/internal/services/assets/AssetPathConstructorImpl.java /tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/services/TapestryModule.java /tapestry/tapestry5/trunk/tapestry-core/src/test/groovy/org/apache/tapestry5/integration/app3/PageCatalogTests.groovy /tapestry/tapestry5/trunk/tapestry-core/src/test/java/org/apache/tapestry5/internal/services/AssetDispatcherTest.java /tapestry/tapestry5/trunk/tapestry-core/src/test/java/org/apache/tapestry5/internal/services/ClasspathAssetAliasManagerImplTest.java /tapestry/tapestry5/trunk/tapestry-core/src/test/java/org/apache/tapestry5/internal/services/ContextAssetFactoryTest.java

          People

          • Assignee:
            Taha Hafeez
            Reporter:
            Ville Virtanen
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development