Details

    • Type: New Feature New Feature
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.0.2-SNAPSHOT
    • Fix Version/s: 1.0.2
    • Labels:
      None

      Description

      The features of this ResourceHandler include the following:

      • relative paths between resources (css files referencing images
        without using #resource['..'])
      • caching resources in the client (disabled if ProjectStage == Development)
      • GZIP compression and local cache in tmp dir (disabled if
        ProjectStage == Development)
      • i18n (supporting country code and language).

      In addition, it does NOT support ValueExpressions in resource files
      for performance reasons.

      The most important feature, in my opinion, is how the resource URL is
      built, e.g. /faces/javax.faces.resource/de_AT/$/some/library/$/my/resource.css

      ... because it permits resources referencing other resources without
      #

      {resource['...']}

      (e.g. css files referencing images or other css
      files). With the standard ResourceHandler this is 1) annoying if you
      have to change the files you get from your designer and 2) a
      performance bottleneck, because resources with ValueExpressions are
      not cached and also regenerated for each request.

      Furthermore, the resource URL contains the locale and thus you have no
      problems with cached resources if a users changes the locale, because
      he/she will get a new URL and thus a new resource (the right one).

        Activity

        Jakob Korherr created issue -
        Hide
        Leonardo Uribe added a comment -

        cool stuff! a good candidate to put on myfaces-commons-utils, so the user can just add the resource handler on its webapp faces-config.

        Show
        Leonardo Uribe added a comment - cool stuff! a good candidate to put on myfaces-commons-utils, so the user can just add the resource handler on its webapp faces-config.
        Hide
        Jakob Korherr added a comment -

        IMO it's better to keep this code in an own module, so that we can keep it lightweight and also deliver the faces-config with the module!

        Show
        Jakob Korherr added a comment - IMO it's better to keep this code in an own module, so that we can keep it lightweight and also deliver the faces-config with the module!
        Hide
        Leonardo Uribe added a comment -

        Yes, even better, so it is just required to add it on the classpath.

        Show
        Leonardo Uribe added a comment - Yes, even better, so it is just required to add it on the classpath.
        Hide
        Jakob Korherr added a comment -

        Exactly!
        I just committed the necessary faces-config.xml

        Show
        Jakob Korherr added a comment - Exactly! I just committed the necessary faces-config.xml
        Jakob Korherr made changes -
        Field Original Value New Value
        Status Open [ 1 ] Resolved [ 5 ]
        Fix Version/s 1.0.2-SNAPSHOT [ 12315968 ]
        Resolution Fixed [ 1 ]
        Jakob Korherr made changes -
        Component/s myfaces-commons-resourcehandler [ 12314322 ]
        Hide
        Jakob Korherr added a comment -

        NOTE that URL style was changed in MFCOMMONS-30

        Show
        Jakob Korherr added a comment - NOTE that URL style was changed in MFCOMMONS-30
        Leonardo Uribe made changes -
        Fix Version/s 1.0.2 [ 12318450 ]
        Fix Version/s 1.0.2-SNAPSHOT [ 12315968 ]
        Leonardo Uribe made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Resolved Resolved
        43m 8s 1 Jakob Korherr 08/Mar/11 18:04
        Resolved Resolved Closed Closed
        216d 2h 43m 1 Leonardo Uribe 10/Oct/11 21:48

          People

          • Assignee:
            Jakob Korherr
            Reporter:
            Jakob Korherr
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development