Author: Alex.Kolonitsky Date: 2010-12-08 07:19:29 -0500 (Wed, 08 Dec 2010) New Revision: 20454 Modified: trunk/ui/output/ui/src/main/java/org/richfaces/component/AbstractPanelMenu.java trunk/ui/output/ui/src/main/resources/META-INF/pn.faces-config.xml Log: RF-9890 panelMenu: itemChangeListener never fired and activeItem not put to model Modified: trunk/ui/output/ui/src/main/java/org/richfaces/component/AbstractPanelMenu.java =================================================================== --- trunk/ui/output/ui/src/main/java/org/richfaces/component/AbstractPanelMenu.java 2010-12-08 12:16:21 UTC (rev 20453) +++ trunk/ui/output/ui/src/main/java/org/richfaces/component/AbstractPanelMenu.java 2010-12-08 12:19:29 UTC (rev 20454) @@ -30,6 +30,7 @@ import javax.el.ValueExpression; import javax.faces.component.UIComponent; +import javax.faces.component.UIOutput; import javax.faces.context.FacesContext; import javax.faces.event.AbortProcessingException; import javax.faces.event.FacesEvent; @@ -39,7 +40,7 @@ * @author akolonitsky * @since 2010-10-25 */ -public abstract class AbstractPanelMenu extends AbstractActionComponent implements ItemChangeSource { +public abstract class AbstractPanelMenu extends UIOutput implements ItemChangeSource { public static final String COMPONENT_TYPE = "org.richfaces.PanelMenu"; @@ -77,7 +78,7 @@ String previous = (String) getValue(); setActiveItem(activeItem); setSubmittedActiveItem(null); - if (previous != null && !previous.equalsIgnoreCase(activeItem)) { + if (previous == null || !previous.equalsIgnoreCase(activeItem)) { queueEvent(new ItemChangeEvent(this, previous, activeItem)); } } Modified: trunk/ui/output/ui/src/main/resources/META-INF/pn.faces-config.xml =================================================================== --- trunk/ui/output/ui/src/main/resources/META-INF/pn.faces-config.xml 2010-12-08 12:16:21 UTC (rev 20453) +++ trunk/ui/output/ui/src/main/resources/META-INF/pn.faces-config.xml 2010-12-08 12:19:29 UTC (rev 20454) @@ -1712,18 +1712,6 @@ <component-class>org.richfaces.component.html.HtmlPanelMenu</component-class> <property> - <property-name>action</property-name> - <property-class>javax.faces.el.MethodBinding</property-class> - </property> - <property> - <property-name>actionExpression</property-name> - <property-class>javax.el.MethodExpression</property-class> - </property> - <property> - <property-name>actionListener</property-name> - <property-class>javax.faces.el.MethodBinding</property-class> - </property> - <property> <description></description> <property-name>disabled</property-name> <property-class>boolean</property-class>