Dropdown menu (h:selectOneMenu) cleared when button (h:commandButton) pressed

[bedek_bedkowski]

Hi,

I've problem - when on my page I click button Search drop down menu is cleared - when I click Search again it gets populated again!

Action class:

@Name("search")
public class SearchAction {
     
     // without @Out annotation
     @Factory("list")
     public Map<String,Integer> getList() {
          System.out.println("In-method getList");
          Map<String,Integer> list = new TreeMap<String,Integer>();
          list.put("one", 1);
          list.put("two", 2);          
          return list;
     }
     public void it(){          
          System.out.println("In-method it");
     }
}

Page:

<!DOCTYPE composition PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN"
    "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
<ui:composition xmlns="http://www.w3.org/1999/xhtml"
    xmlns:s="http://jboss.com/products/seam/taglib"
    xmlns:ui="http://java.sun.com/jsf/facelets"
    xmlns:f="http://java.sun.com/jsf/core"
    xmlns:h="http://java.sun.com/jsf/html"
    xmlns:rich="http://richfaces.org/rich"
    template="layout/template.xhtml">

<ui:define name="body">
    <h:form id="search" styleClass="edit">
        <h:selectOneMenu>
         <s:selectItems
                  value="#{list}"/>
        </h:selectOneMenu>
    <h:commandButton id="search"  value="Search" action="#{search.it}"/>
    </h:form>    
</ui:define>
</ui:composition>

[chris.simons]

Your not injecting list, so why would you expect it to be available again?

You could also put an action parameter on your pages.xml for this particular page and set action="#{search.getList()}"

My point being that list is not on an entity and you're not outjecting or injecting list as an object.

[bedek_bedkowski]

Hi,

I think you're wrong - using @Factory in way as above is outjecting automaticly.

To be clear drop down list is rendered when I enter my page for a first time, then when I press Search button it is cleared (not populated and there is no exception thrown) and then when I click once again Search it is populated again and so on.

Please take a look here http://chiralsoftware.com/jboss-seam-book/unwrap.seam, this document says:

The @Factory annotation can be used in two different ways:

1. On a getter method, like this: @Factory public String getTimeString()


2. On a method with a void return type, like this: @Factory(componentName) public void generateResult() in conjunction with an @Out annotation

Example which I gave above is related with first option.

I've also made few tests - take a look here: http://www.seamframework.org/Community/SelectItemsAndFactory

[bedek_bedkowski]

Hi,

The problem was not related with Chris Simons comment about not outjecting variable back to page - this is not needed since @Factory with getter method returning value don't need additional @Out.

Resolution:

In page code to generate list of options I've used Seam tag:

         <s:selectItems
                  value="#{list}"/>

while this is causing problems and to fix I've just changed to JSF tag:

         <f:selectItems
                  value="#{list}"/>

and all works fine.

The only thing is that I'm not sure why this is working but it sure is :)