ICEfaces
  1. ICEfaces
  2. ICE-11189

showcase ace:buttonGroup > Group in Row demo has state issue when refreshing page

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 4.1
    • Fix Version/s: 4.2
    • Component/s: ACE-Components, Sample Apps
    • Labels:
      None
    • Environment:
      Tomcat 8. all browsers, ICEfaces 4 trunk r49588. This issue was also present as far back as ICEfaces 4 trunk continuous build 2129

      Description

      Showcase ace:buttonGroup > Group in Row demo
      When you first you make selections in the rows the mutuallyExclusive=true attribute is respected. However, when you refresh the page, all of the previous selections made are now rendered. This is not an issue in the QA test app /ICE-11123-2.jsf found at:
      http://dev.icesoft.com/svn/repo/qa/trunk/Regression-Icefaces4/Sparkle/Nightly/buttonGroup
      To reproduce:
      1.) Use the showcase ace:buttonGroup > Group in Row demo.
      2.) In any row, select each button or checkbox, note that mutuallyExclusive = true.
      3.) Refresh the page, all buttons previously selected will now show as selected. See attached screenshot 1.PNG

        Issue Links

          Activity

          Liana Munroe created issue -
          Liana Munroe made changes -
          Field Original Value New Value
          Attachment 1.PNG [ 22386 ]
          Liana Munroe made changes -
          Fix Version/s 4.2 [ 12870 ]
          Liana Munroe made changes -
          Assignee Judy Guglielmin [ judy.guglielmin ]
          Hide
          Judy Guglielmin added a comment - - edited

          The only fix appears to be using:-
          <ace:ajax render="@form" execute="@form"/>
          for each of the buttons.
          The buttonGroup is not a naming container, so you have to submit the entire form. Weshould probably consider whether the buttonGroup should be a naming container so that we could use:-
          <ace:ajax render="frm2:rpt:#

          {group.row}:radioGroup" execute="#{frm2:rpt:#{group.row}

          :radiogroup"/>

          This would mean that you only submit the group and render the group..... (in this example, it would be correct).

          for now, the only way to make this work and to ensure that all members of the group are submitted to be state-saved is executing and rendering the entire form for each submit.

          Show
          Judy Guglielmin added a comment - - edited The only fix appears to be using:- <ace:ajax render="@form" execute="@form"/> for each of the buttons. The buttonGroup is not a naming container, so you have to submit the entire form. Weshould probably consider whether the buttonGroup should be a naming container so that we could use:- <ace:ajax render="frm2:rpt:# {group.row}:radioGroup" execute="#{frm2:rpt:#{group.row} :radiogroup"/> This would mean that you only submit the group and render the group..... (in this example, it would be correct). for now, the only way to make this work and to ensure that all members of the group are submitted to be state-saved is executing and rendering the entire form for each submit.
          Judy Guglielmin made changes -
          Link This issue blocks ICE-11193 [ ICE-11193 ]
          Ken Fyten made changes -
          Link This issue blocks ICE-11193 [ ICE-11193 ]
          Hide
          Ken Fyten added a comment -

          For the nested use-case where buttons are inside the ace:buttonGroup, the resolution depends on making the buttonGroup a JSF naming container, captured in ICE-11193.

          For the none-nested use-case, the solution is to scope the ace:ajax call for each button press to execute + render the entire form, or at least all of the buttonGroup and associated buttons.

          Show
          Ken Fyten added a comment - For the nested use-case where buttons are inside the ace:buttonGroup, the resolution depends on making the buttonGroup a JSF naming container, captured in ICE-11193 . For the none-nested use-case, the solution is to scope the ace:ajax call for each button press to execute + render the entire form, or at least all of the buttonGroup and associated buttons.
          Ken Fyten made changes -
          Link This issue depends on ICE-11193 [ ICE-11193 ]
          Ken Fyten made changes -
          Affects Version/s 4.1 [ 11375 ]
          Affects Version/s 4.2 [ 12870 ]
          Ken Fyten made changes -
          Security Private [ 10001 ]
          Ken Fyten made changes -
          Fix Version/s EE-4.2.0.GA [ 13071 ]
          Fix Version/s 4.2 [ 12870 ]
          Hide
          Judy Guglielmin added a comment -

          rev 50706

          Show
          Judy Guglielmin added a comment - rev 50706
          Judy Guglielmin made changes -
          Status Open [ 1 ] Resolved [ 5 ]
          Resolution Fixed [ 1 ]
          Hide
          Liana Munroe added a comment -

          Verified ICEfaces 4 trunk r50730. Tomcat 8, MS Edge 38, IE 11, 10, Chrome 56, FF 47.

          Show
          Liana Munroe added a comment - Verified ICEfaces 4 trunk r50730. Tomcat 8, MS Edge 38, IE 11, 10, Chrome 56, FF 47.
          Ken Fyten made changes -
          Fix Version/s 4.2 [ 12870 ]
          Fix Version/s EE-4.2.0.GA [ 13071 ]
          Ken Fyten made changes -
          Status Resolved [ 5 ] Closed [ 6 ]

            People

            • Assignee:
              Judy Guglielmin
              Reporter:
              Liana Munroe
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: