ICEfaces
  1. ICEfaces
  2. ICE-8834

NPE thrown in CommandLinkRenderer.getJavascriptHiddenFieldSetters()

          Details

          • Type: Bug Bug
          • Status: Closed
          • Priority: Major Major
          • Resolution: Fixed
          • Affects Version/s: EE-1.8.2.GA_P05
          • Fix Version/s: EE-1.8.2.GA_P06
          • Component/s: ICE-Components
          • Labels:
            None
          • Environment:
            All
          • Salesforce Case Reference:
          • Workaround Exists:
            Yes
          • Workaround Description:
            Initialize the param value to a non-null value.

            Description

            If null is returned for a f:param value in an ice:commandLink, a NPE is thrown. This due to the changes made in ICE-8415.

            Caused by: java.lang.NullPointerException
            at com.icesoft.faces.renderkit.dom_html_basic.CommandLinkRenderer.getJavascriptHiddenFieldSetters(CommandLinkRenderer.java:210)
            at com.icesoft.faces.component.ext.renderkit.CommandLinkRenderer.getJavaScriptOnClickString(CommandLinkRenderer.java:87)
            at com.icesoft.faces.component.ext.renderkit.CommandLinkRenderer.renderOnClick(CommandLinkRenderer.java:69)
            at com.icesoft.faces.renderkit.dom_html_basic.CommandLinkRenderer.encodeBegin(CommandLinkRenderer.java:121)
            at com.icesoft.faces.component.ext.renderkit.CommandLinkRenderer.encodeBegin(CommandLinkRenderer.java:50)
            at javax.faces.component.UIComponentBase.encodeBegin(UIComponentBase.java:515)
            at com.icesoft.faces.renderkit.dom_html_basic.DomBasicRenderer.encodeParentAndChildren(DomBasicRenderer.java:355)
            at com.icesoft.faces.renderkit.dom_html_basic.DomBasicRenderer.encodeParentAndChildren(DomBasicRenderer.java:364)
            at com.icesoft.faces.renderkit.dom_html_basic.DomBasicRenderer.encodeParentAndChildren(DomBasicRenderer.java:364)
            at com.icesoft.faces.renderkit.dom_html_basic.DomBasicRenderer.encodeParentAndChildren(DomBasicRenderer.java:364)
            at com.icesoft.faces.renderkit.dom_html_basic.DomBasicRenderer.encodeParentAndChildren(DomBasicRenderer.java:364)
            at com.icesoft.faces.renderkit.dom_html_basic.DomBasicRenderer.encodeParentAndChildren(DomBasicRenderer.java:364)
            at com.icesoft.faces.renderkit.dom_html_basic.GroupRenderer.encodeChildren(GroupRenderer.java:95)
            at javax.faces.component.UIComponentBase.encodeChildren(UIComponentBase.java:527)
            at com.icesoft.faces.application.D2DViewHandler.renderResponse(D2DViewHandler.java:490)
            3. Text File
              ice-8834.patch
              1 kB
              Deryk Sinotte

              Activity

              Ascending order - Click to sort in descending order
              Hide
              Permalink
              Arran Mccullough added a comment -

              Attached test case that shows issue. Load welcomeICEfaces.iface to see error.

              Show
              Arran Mccullough added a comment - Attached test case that shows issue. Load welcomeICEfaces.iface to see error.
              Hide
              Permalink
              Subodh Godbole added a comment -

              Some thoughts -

              • It used to work perfectly with EE P04 and this issue got introduced with EE P05.
              • This JIRA is marked with - Workaround Exists: Yes and Workaround Description: Initialize the param value to a non-null value.
              • But I believe for migrating customers from EE P04 to EE P05 (like us), it is not feasible to add null check all over the source code to check and pass non-null value
              • Hence in my opinion this issue makes a kind of showstopper for migration to EE P05
              Show
              Subodh Godbole added a comment - Some thoughts - It used to work perfectly with EE P04 and this issue got introduced with EE P05. This JIRA is marked with - Workaround Exists: Yes and Workaround Description: Initialize the param value to a non-null value. But I believe for migrating customers from EE P04 to EE P05 (like us), it is not feasible to add null check all over the source code to check and pass non-null value Hence in my opinion this issue makes a kind of showstopper for migration to EE P05
              Hide
              Permalink
              Deryk Sinotte added a comment -

              Apologies. This issue was already noted and fixed as per ICE-8749 but the fix was only applied to the ICEfaces 3 trunk. I've applied the relevant fix to the 1.8 and 3.0 maintenance streams now and attached a patch to this case.

              Show
              Deryk Sinotte added a comment - Apologies. This issue was already noted and fixed as per ICE-8749 but the fix was only applied to the ICEfaces 3 trunk. I've applied the relevant fix to the 1.8 and 3.0 maintenance streams now and attached a patch to this case.
              Hide
              Permalink
              Carmen Cristurean added a comment -

              Verified fix in IE6/7/8/9/FF18/Chrome24 with ICEfaces EE-1.8.GA Build1.

              Show
              Carmen Cristurean added a comment - Verified fix in IE6/7/8/9/FF18/Chrome24 with ICEfaces EE-1.8.GA Build1.

                People

                • Assignee:
                  Deryk Sinotte
                  Reporter:
                  Arran Mccullough
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  4 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: