ICEfaces
  1. ICEfaces
  2. ICE-1460

RowSelector doesn't work correctly with the GlassFish implementation of JSF 1.2

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Cannot Reproduce
    • Affects Version/s: 1.6DR#3
    • Fix Version/s: 1.6DR#4
    • Component/s: Framework
    • Labels:
      None
    • Environment:
      any
    • Assignee Priority:
      P2

      Description

      Contribution from Dan Leahu:

      *****

      RowSelector doesn't work correctly with the GlassFish implementation of JSF 1.2 because this method:

      public void setSelectionListener(MethodBinding selectionListener) {
             this.selectionListener = selectionListener;
         }

      is never called.

      We've solved this by creating a new listener type called SelectionListener and modified the method createMetaRuleset present in com.sun.facelets.tag.jsf.ComponentHandler.java to include rules for this new SelectionListener:

      if (SelectionSource.class.isAssignableFrom(type)) {
                 m.addRule(SelectionSourceRule.Instance);
             }

      Also created for the proxy class MethodExpressionSelectionListener

      public class MethodExpressionSelectionListener implements SelectionListener {
      ...
      }

      which knows how to call our SelectionListener.

        Activity

        Hide
        Mark Collette added a comment -

        My assumption is that this was fixed in ICEfaces 1.6.0 DR3. But however it is, the ice:rowSelector.selectionListener and ice:rowSelector.selectionAction attributes work properly in Glassfish, with JSF RI 1.2.

        Show
        Mark Collette added a comment - My assumption is that this was fixed in ICEfaces 1.6.0 DR3. But however it is, the ice:rowSelector.selectionListener and ice:rowSelector.selectionAction attributes work properly in Glassfish, with JSF RI 1.2.

          People

          • Assignee:
            Mark Collette
            Reporter:
            Philip Breau
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: