Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Critical Critical
    • Resolution: Fixed
    • Affects Version/s: 2.0.3
    • Fix Version/s: 2.1.15, 2.2.0-m06
    • Component/s: ajax
    • Labels:
      None
    • Environment:

      Operating System: All
      Platform: All

    • Issuezilla Id:
      1,817
    • Status Whiteboard:
      Hide

      size_medium importance_medium

      Show
      size_medium importance_medium
    • Tags:

      Description

      The action of a commandLink is not called via ajax when it is located in a
      double nested <ui:repeat>. It works in a non-nested <ui:repeat>. Using a double
      nested <h:dataTable> works instead.

      Example:
      <ui:repeat var="items" value="#

      {someBean.items}

      ">
      <ui:repeat var="subitem" value="#

      {item.subitems}

      ">
      <h:commandLink action="#

      {subitem.action}

      ">
      <f:ajax />
      </h:commandLink>
      </ui:repeat>
      </ui:repeat>

      1. changebundle-1817.txt
        14 kB
        rogerk
      2. new example.zip
        1.27 MB
        adriaaaaan
      3. WebApplication1.zip
        2.04 MB
        adriaaaaan

        Activity

        Hide
        rogerk added a comment -

        Thanks for the detailed analysis. I verified that clearing the datamodel in visitTree appears to work. I'll need to run this through all our tests to make sure it does not break something else.

        Show
        rogerk added a comment - Thanks for the detailed analysis. I verified that clearing the datamodel in visitTree appears to work. I'll need to run this through all our tests to make sure it does not break something else.
        Hide
        rogerk added a comment -

        Changes.

        Show
        rogerk added a comment - Changes.
        Hide
        rogerk added a comment -

        Committed to MOJARRA_2_1X_ROLLING:
        Sending jsf-ri/src/main/java/com/sun/faces/facelets/component/UIRepeat.java
        Adding test/agnostic/ajax/src/main/java/com/sun/faces/test/agnostic/ajax/Issue1817Bean.java
        Adding test/agnostic/ajax/src/main/java/com/sun/faces/test/agnostic/ajax/Issue1817Item.java
        Adding test/agnostic/ajax/src/main/webapp/issue1817.xhtml
        Adding test/agnostic/ajax/src/test/java/com/sun/faces/test/agnostic/ajax/Issue1817IT.java
        Transmitting file data .....
        Committed revision 10936.

        Show
        rogerk added a comment - Committed to MOJARRA_2_1X_ROLLING: Sending jsf-ri/src/main/java/com/sun/faces/facelets/component/UIRepeat.java Adding test/agnostic/ajax/src/main/java/com/sun/faces/test/agnostic/ajax/Issue1817Bean.java Adding test/agnostic/ajax/src/main/java/com/sun/faces/test/agnostic/ajax/Issue1817Item.java Adding test/agnostic/ajax/src/main/webapp/issue1817.xhtml Adding test/agnostic/ajax/src/test/java/com/sun/faces/test/agnostic/ajax/Issue1817IT.java Transmitting file data ..... Committed revision 10936.
        Hide
        rogerk added a comment -

        Committed to trunk:
        Sending jsf-ri/src/main/java/com/sun/faces/facelets/component/UIRepeat.java
        Adding test/agnostic/ajax/src/main/java/com/sun/faces/test/agnostic/ajax/Issue1817Bean.java
        Adding test/agnostic/ajax/src/main/java/com/sun/faces/test/agnostic/ajax/Issue1817Item.java
        Adding test/agnostic/ajax/src/main/webapp/issue1817.xhtml
        Adding test/agnostic/ajax/src/test/java/com/sun/faces/test/agnostic/ajax/Issue1817IT.java
        Transmitting file data .....
        Committed revision 10937.

        Show
        rogerk added a comment - Committed to trunk: Sending jsf-ri/src/main/java/com/sun/faces/facelets/component/UIRepeat.java Adding test/agnostic/ajax/src/main/java/com/sun/faces/test/agnostic/ajax/Issue1817Bean.java Adding test/agnostic/ajax/src/main/java/com/sun/faces/test/agnostic/ajax/Issue1817Item.java Adding test/agnostic/ajax/src/main/webapp/issue1817.xhtml Adding test/agnostic/ajax/src/test/java/com/sun/faces/test/agnostic/ajax/Issue1817IT.java Transmitting file data ..... Committed revision 10937.
        Hide
        rogerk added a comment -

        Committed.

        Show
        rogerk added a comment - Committed.

          People

          • Assignee:
            rogerk
            Reporter:
            Mathias Werlitz
          • Votes:
            22 Vote for this issue
            Watchers:
            17 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: