Details

    • Type: Improvement Improvement
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.1.14
    • Fix Version/s: 1.1.15
    • Component/s: impl
    • Labels:
      None
    • Environment:

      Operating System: All
      Platform: All

    • Issuezilla Id:
      277

      Description

      The facelets.REFRESH_PERIOD init param allows a minimum of one second for the
      view compilation interval.

      One second is too long.

      While view caching improves performance in production environments, it slows it
      down in a development environment where the UI is in a state of continuous
      evolution, refactoring, and change.

      It would be good for rapid page development to be able to disable the view cache
      entirely. In other words, tell the Facelets compiler to re-compile the view
      every time it is rendered.

      For example, the facelets.DISABLE_VIEW_CACHING init param be used to turn off
      this feature. It could have a default value of false to remain consistent with
      existing Facelets behavior.

        Activity

        Hide
        jhook added a comment -

        Does just specifying '0' work here?

        Show
        jhook added a comment - Does just specifying '0' work here?
        Hide
        Ian Hlavats added a comment -

        Zero works.

        View compilation seems to be a bit snappier.

        Thanks for the tip!

        Show
        Ian Hlavats added a comment - Zero works. View compilation seems to be a bit snappier. Thanks for the tip!
        Hide
        jhook added a comment -

        In looking at the code, we should just add a case for zero and do an immediate
        return without checking the filesystem

        Show
        jhook added a comment - In looking at the code, we should just add a case for zero and do an immediate return without checking the filesystem
        Hide
        Ian Hlavats added a comment -

        Sounds good (and easy to fix).

        What's the ETA on a patch?

        Show
        Ian Hlavats added a comment - Sounds good (and easy to fix). What's the ETA on a patch?
        Hide
        Ryan Lubke added a comment -

        This appears to have been implemented:

        protected boolean needsToBeRefreshed(DefaultFacelet facelet)

        { // if set to 0, constantly reload-- nocache if (this.refreshPeriod == 0) return true; . . . }
        Show
        Ryan Lubke added a comment - This appears to have been implemented: protected boolean needsToBeRefreshed(DefaultFacelet facelet) { // if set to 0, constantly reload-- nocache if (this.refreshPeriod == 0) return true; . . . }

          People

          • Assignee:
            youngm
            Reporter:
            Ian Hlavats
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: