ICEfaces
  1. ICEfaces
  2. ICE-11268

ace:colorEntry dynamic example causes duplicate update to client/browser

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 4.2
    • Fix Version/s: EE-4.2.0.GA
    • Component/s: ACE-Components
    • Labels:
      None
    • Environment:
      ace tabSet
    • Assignee Priority:
      P2

      Description

      When using the dynamic swatches example in showcase (there are three examples for ace:colorEntry-> Swatches each in a tabPane, the dynamic example is in the third or last tabPane):-
      choose up to 4 colors and click on CreateSwatch button.
      a new colorEntry component appears with the Swatch created from the up to 4 input colorEntry components previously.
      Choose a color and it shows the selected color as it should.
      Click on "Reset" button and perform the same steps again (choose different or more or less colors but make sure the color is different than the last one chosen or the value change will not fire!)>
      Create new swatch
      Now choose a color from the new swatch created and a response containing the update returns from the server and not only is the ace:colorEntry component updated, but a duplicate is created too.

      I'll attach an screenshot, but tried, ace:tabPane cache="none" as well as various attributes on the ace:tabSet. Tried tabSetProxy with forms within the tabSet, ace:ajax with various execute and render values. Replaced c:forEach with individual components.....
      still get the update duplicated into the client.

      After commenting out the ace:tabSet and ace:tabPanes, I was able to reproduce the same issue. After reload of the page, the duplicate disappears.... leading me to wonder if the original markup should be destroyed before a re-render can occur in some cases by the javascript(?)

        Activity

        Hide
        Judy Guglielmin added a comment -

        reopening the component is not available now as the component is not re-initialized properly.

        Show
        Judy Guglielmin added a comment - reopening the component is not available now as the component is not re-initialized properly.
        Hide
        Mircea Toma added a comment -

        Ok, I now can reproduce and understand what the problem is.

        Show
        Mircea Toma added a comment - Ok, I now can reproduce and understand what the problem is.
        Hide
        Judy Guglielmin added a comment -

        two screen shots ...one showing correct rendering and the incorrect one showing the duplicated update to the page.

        Show
        Judy Guglielmin added a comment - two screen shots ...one showing correct rendering and the incorrect one showing the duplicated update to the page.
        Hide
        Mircea Toma added a comment -

        The issue is caused by ColorEntryRenderer because is rendering two elements with the exact same "id" attribute.

        Modified the renderer to assign an ID attribute value for the swatch markup that does not coincide with the ID of the root element.

        Show
        Mircea Toma added a comment - The issue is caused by ColorEntryRenderer because is rendering two elements with the exact same "id" attribute. Modified the renderer to assign an ID attribute value for the swatch markup that does not coincide with the ID of the root element.
        Hide
        Carmen Cristurean added a comment -

        Verified ICEfaces4 trunk r 51396 on MsEdge, IE11, FF49 and Tomcat7.

        Show
        Carmen Cristurean added a comment - Verified ICEfaces4 trunk r 51396 on MsEdge, IE11, FF49 and Tomcat7.

          People

          • Assignee:
            Mircea Toma
            Reporter:
            Judy Guglielmin
          • Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: