Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: current
    • Fix Version/s: milestone 1
    • Component/s: www
    • Labels:
      None
    • Environment:

      Operating System: All
      Platform: All

    • Issuezilla Id:
      27

      Description

      Including ajaxZone tags immediately around a commandButton does not send
      proper "partial render" params. For example:

      <%@ taglib uri="http://java.sun.com/jsf/core" prefix="f" %>
      <%@ taglib uri="http://java.sun.com/jsf/html" prefix="h" %>
      <%@ taglib prefix="jsfExt" uri="http://java.sun.com/jsf/extensions/dynafaces" %>

      <f:view>
      <h:form id="form">
      <jsfExt:ajaxZone id="zone2">
      <h:commandButton id="_0" value="press me"
      actionListener="#

      {game.select}

      "/>
      </jsfExt:ajaxZone>
      </h:form>
      </f:view>

      Does not send the proper partial render params.

        Activity

        Hide
        Ed Burns added a comment -

        edward-burns-powerbook-g4-15:~/Projects/J2EE/workareas/jsf-extensions-
        trunk/code edburns$ svn diff run-time/avatar/src/main/java/com/sun/faces/
        extensions/avatar/renderkit/AjaxZoneRenderer.java
        Index: run-time/avatar/src/main/java/com/sun/faces/extensions/avatar/
        renderkit/AjaxZoneRenderer.java
        ===================================================================
        — run-time/avatar/src/main/java/com/sun/faces/extensions/avatar/
        renderkit/AjaxZoneRenderer.java (revision 294)
        +++ run-time/avatar/src/main/java/com/sun/faces/extensions/avatar/
        renderkit/AjaxZoneRenderer.java (working copy)
        @@ -39,6 +39,7 @@
        import javax.el.MethodExpression;
        import javax.el.ValueExpression;
        import javax.faces.FacesException;
        +import javax.faces.component.ActionSource;
        import javax.faces.component.EditableValueHolder;

        import javax.faces.component.UIComponent;
        @@ -291,7 +292,8 @@
        new Util.TreeTraversalCallback() {
        public boolean takeActionOnNode(FacesContext context,
        UIComponent curNode) throws FacesException {
        boolean keepGoing = true;

        • if (curNode instanceof EditableValueHolder) {
          + if (curNode instanceof EditableValueHolder ||
          + curNode instanceof ActionSource) { keepGoing = false; }

          return keepGoing;
          edward-burns-powerbook-g4-15:~/Projects/J2EE/workareas/jsf-extensions-
          trunk/code edburns$

        Show
        Ed Burns added a comment - edward-burns-powerbook-g4-15:~/Projects/J2EE/workareas/jsf-extensions- trunk/code edburns$ svn diff run-time/avatar/src/main/java/com/sun/faces/ extensions/avatar/renderkit/AjaxZoneRenderer.java Index: run-time/avatar/src/main/java/com/sun/faces/extensions/avatar/ renderkit/AjaxZoneRenderer.java =================================================================== — run-time/avatar/src/main/java/com/sun/faces/extensions/avatar/ renderkit/AjaxZoneRenderer.java (revision 294) +++ run-time/avatar/src/main/java/com/sun/faces/extensions/avatar/ renderkit/AjaxZoneRenderer.java (working copy) @@ -39,6 +39,7 @@ import javax.el.MethodExpression; import javax.el.ValueExpression; import javax.faces.FacesException; +import javax.faces.component.ActionSource; import javax.faces.component.EditableValueHolder; import javax.faces.component.UIComponent; @@ -291,7 +292,8 @@ new Util.TreeTraversalCallback() { public boolean takeActionOnNode(FacesContext context, UIComponent curNode) throws FacesException { boolean keepGoing = true; if (curNode instanceof EditableValueHolder) { + if (curNode instanceof EditableValueHolder || + curNode instanceof ActionSource) { keepGoing = false; } return keepGoing; edward-burns-powerbook-g4-15:~/Projects/J2EE/workareas/jsf-extensions- trunk/code edburns$
        Hide
        Ed Burns added a comment -

        fix checked in.

        Show
        Ed Burns added a comment - fix checked in.

          People

          • Assignee:
            Ed Burns
            Reporter:
            rogerk
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: