MyFaces Core
  1. MyFaces Core
  2. MYFACES-3335

jsf.getViewState() create a new _AjaxUtils object, but it is singleton

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 2.0.10, 2.1.4
    • Component/s: JSR-314
    • Labels:
      None

      Description

      jsf.getViewState() create a new _AjaxUtils object, but it is singleton. It should just grab the object, because when jsf.js is loaded, this instance is created

      1. MYFACES-3335-1.patch
        0.7 kB
        Leonardo Uribe

        Activity

        Hide
        Werner Punz added a comment -

        Ok this bug gave me the incentive to rework the entire viewstate subystem

        Show
        Werner Punz added a comment - Ok this bug gave me the incentive to rework the entire viewstate subystem
        Hide
        Werner Punz added a comment -

        Ok there is more to it, I rely on the standard error handlers in the encodesubmittablefields, I have to change that, this could be related to: https://issues.apache.org/jira/browse/MYFACES-3334 because if you call encode submittable fields and raise an error by issuing it wrongly, you will run into the issue of not having an error handler there because there is no context available which hosts the error handlers.

        This needs to be fixed.

        Show
        Werner Punz added a comment - Ok there is more to it, I rely on the standard error handlers in the encodesubmittablefields, I have to change that, this could be related to: https://issues.apache.org/jira/browse/MYFACES-3334 because if you call encode submittable fields and raise an error by issuing it wrongly, you will run into the issue of not having an error handler there because there is no context available which hosts the error handlers. This needs to be fixed.
        Hide
        Werner Punz added a comment -

        Thanks Leo, I have missed this at my refactoring, you are right, it has been a singleton since my last refactoring and a new is totally out of scope there.

        Show
        Werner Punz added a comment - Thanks Leo, I have missed this at my refactoring, you are right, it has been a singleton since my last refactoring and a new is totally out of scope there.

          People

          • Assignee:
            Leonardo Uribe
            Reporter:
            Leonardo Uribe
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development