ICEfaces
  1. ICEfaces
  2. ICE-1530

RenderManager throws a NullPointerException if state is null.

          Details

          • Type: Bug Bug
          • Status: Closed
          • Priority: Major Major
          • Resolution: Fixed
          • Affects Version/s: 1.6DR#3
          • Fix Version/s: 1.6DR#4, 1.6
          • Component/s: None
          • Labels:
            None
          • Environment:
            All

            Description

            When the RenderManager attempts to render a Renderable instance, it is possible for that state to be null. Currently, this throws a NullPointerException. It should be handled a bit more elegantly.

              Activity

              Ascending order - Click to sort in descending order
              Hide
              Permalink
              Deryk Sinotte added a comment -

              The validity of the PersistentFacesState is the responsiblity of the application. In certain situations, the state can be null which requires modification in the application (best practices should be documented). Rather than let it toss an NPE, we should check for the state being null and then pass a TransientRenderingException back to the Renderable instance that provides the state. This is the current exception handling design for the RenderManager. I've made the changes to support this decision.

              Show
              Deryk Sinotte added a comment - The validity of the PersistentFacesState is the responsiblity of the application. In certain situations, the state can be null which requires modification in the application (best practices should be documented). Rather than let it toss an NPE, we should check for the state being null and then pass a TransientRenderingException back to the Renderable instance that provides the state. This is the current exception handling design for the RenderManager. I've made the changes to support this decision.
              Hide
              Permalink
              Deryk Sinotte added a comment -

              Assigning to Deryk and marking as FIXED.

              Show
              Deryk Sinotte added a comment - Assigning to Deryk and marking as FIXED.

                People

                • Assignee:
                  Unassigned
                  Reporter:
                  Deryk Sinotte
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  0 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: