Unwanted behavior: Action method being called several times

Question

Hello,
I have a managed bean that calls a session ejb which itself does a search. The problem is that it is supposed to retrieve a resultset ONCE and the displayed behavior is that it never stop doing so. It fetches the resultset N TIMES.
Here is the managed bean and the method to look at is the performSearch method:
package arcoiris;
import java.util.ArrayList;
import java.util.List;
import javax.faces.model.SelectItem;
public class SearchManagedBean {
    //Collected from search form
    private String keyword;
    private String country;
    private Integer[] postcode;
    private boolean commentExists;
    private Integer rating;
    private boolean websiteExists;
    //Used to populate form
    private List<SelectItem> availableCountries;
    private List<SelectItem> availablePostcodes;
    private List<SelectItem> availableRatings;
    //Retrieved from ejb tier
    private List<EstablishmentLocal> retrievedEstablishments;
    //Service locator
    private arcoiris.ServiceLocator serviceLocator;
    //Constructor
    public SearchManagedBean() {
        System.out.println("within constructor SearchManagedBean");
    //Getters and setters
    public String getKeyword() {
        return keyword;
    public void setKeyword(String keyword) {
        this.keyword = keyword;
    public String getCountry() {
        return country;
    public void setCountry(String country) {
        this.country = country;
    public boolean isCommentExists() {
        return commentExists;
    public void setCommentExists(boolean commentExists) {
        this.commentExists = commentExists;
    public Integer getRating() {
        return rating;
    public void setRating(Integer rating) {
        this.rating = rating;
    public boolean isWebsiteExists() {
        return websiteExists;
    public void setWebsiteExists(boolean websiteExists) {
        this.websiteExists = websiteExists;
    public List<SelectItem> getAvailableCountries() {
        List<SelectItem> countries = new ArrayList<SelectItem>();
        SelectItem si_1 = new SelectItem();
        SelectItem si_2 = new SelectItem();
        SelectItem si_3 = new SelectItem();
        si_1.setValue("2");
        si_1.setLabel("ecuador");
        si_2.setValue("1");
        si_2.setLabel("colombia");
        si_3.setValue("3");
        si_3.setLabel("peru");
        countries.add(si_1);
        countries.add(si_2);
        countries.add(si_3);
        return countries;
    public void setAvailableCountries(List<SelectItem> countries) {
        this.availableCountries = availableCountries;
    public List<SelectItem> getAvailablePostcodes() {
        List<SelectItem> postcodes = new ArrayList<SelectItem>();
        SelectItem si_1 = new SelectItem();
        SelectItem si_2 = new SelectItem();
        SelectItem si_3 = new SelectItem();
        si_1.setValue(new Integer(75001));
        si_1.setLabel("75001");
        si_2.setValue(new Integer(75002));
        si_2.setLabel("75002");
        si_3.setValue(new Integer(75003));
        si_3.setLabel("75003");
        postcodes.add(si_1);
        postcodes.add(si_2);
        postcodes.add(si_3);
        return postcodes;
    public void setAvailablePostcodes(List<SelectItem> availablePostcodes) {
        this.availablePostcodes = availablePostcodes;
    public List<SelectItem> getAvailableRatings() {
        List<SelectItem> ratings = new ArrayList<SelectItem>();
        SelectItem si_1 = new SelectItem();
        SelectItem si_2 = new SelectItem();
        SelectItem si_3 = new SelectItem();
        si_1.setValue(new Integer(1));
        si_1.setLabel("1");
        si_2.setValue(new Integer(2));
        si_2.setLabel("2");
        si_3.setValue(new Integer(3));
        si_3.setLabel("3");
        ratings.add(si_1);
        ratings.add(si_2);
        ratings.add(si_3);
        return ratings;
    public void setAvailableRatings(List<SelectItem> availableRatings) {
        this.availableRatings = availableRatings;
    public Integer[] getPostcode() {
        return postcode;
    public void setPostcode(Integer[] postcode) {
        this.postcode = postcode;
    public List<EstablishmentLocal> getRetrievedEstablishments() {
        return retrievedEstablishments;
    public void setRetrievedEstablishments(List<EstablishmentLocal> retrievedEstablishments) {
        this.retrievedEstablishments = retrievedEstablishments;
    //Business methods
    public String performSearch(){
        System.out.println("performSearchManagedBean begin");
        SearchRequestDTO searchRequestDto = new SearchRequestDTO(this.keyword,this.country,this.postcode,this.commentExists,this.rating, this.websiteExists);
        System.out.println("java bean "+this.keyword+" "+this.country+" "+this.postcode+" "+this.commentExists+" "+this.rating+" "+this.websiteExists);
        SearchSessionLocal searchSession = lookupSearchSessionBean();
        List<EstablishmentLocal> retrievedEstablishments = searchSession.performSearch(searchRequestDto);
        this.setRetrievedEstablishments(retrievedEstablishments);
        System.out.println("performSearchManagedBean end");
        return "success";
    private arcoiris.ServiceLocator getServiceLocator() {
        if (serviceLocator == null) {
            serviceLocator = new arcoiris.ServiceLocator();
        return serviceLocator;
    private arcoiris.SearchSessionLocal lookupSearchSessionBean() {
        try {
            return ((arcoiris.SearchSessionLocalHome) getServiceLocator().getLocalHome("java:comp/env/ejb/SearchSessionBean")).create();
        } catch(javax.naming.NamingException ne) {
            java.util.logging.Logger.getLogger(getClass().getName()).log(java.util.logging.Level.SEVERE,"exception caught" ,ne);
            throw new RuntimeException(ne);
        } catch(javax.ejb.CreateException ce) {
            java.util.logging.Logger.getLogger(getClass().getName()).log(java.util.logging.Level.SEVERE,"exception caught" ,ce);
            throw new RuntimeException(ce);
}Here is the object request passed:
package arcoiris;
import java.io.Serializable;
public class SearchRequestDTO implements Serializable {
    private String keyword;
    private String country;
    private Integer[] postcode;
    private boolean commentExists;
    private Integer rating;
    private boolean websiteExists;
    public SearchRequestDTO() {
    public SearchRequestDTO(String keyword, String country, Integer[] postcode, boolean commmentExists, Integer rating, boolean websiteExists){
        this.keyword=keyword;
        this.country= country;
        this.postcode = postcode;
        this.commentExists = commentExists;
        this.rating = rating;
        this.websiteExists = websiteExists;
    public String getKeyword() {
        return keyword;
    public void setKeyword(String keyword) {
        this.keyword = keyword;
    public String getCountry() {
        return country;
    public void setCountry(String country) {
        this.country = country;
    public Integer[] getPostcode() {
        return postcode;
    public void setPostcode(Integer[] postcode) {
        this.postcode = postcode;
    public boolean isCommentExists() {
        return commentExists;
    public void setCommentExists(boolean commmentExists) {
        this.commentExists = commmentExists;
    public Integer getRating() {
        return rating;
    public void setRating(Integer rating) {
        this.rating = rating;
    public boolean isWebsiteExists() {
        return websiteExists;
    public void setWebsiteExists(boolean websiteExists) {
        this.websiteExists = websiteExists;
}and here is the session ejb:
package arcoiris;
import java.sql.Connection;
import java.sql.ResultSet;
import java.sql.SQLException;
import java.sql.Statement;
import java.util.ArrayList;
import java.util.List;
import javax.ejb.*;
import javax.naming.Context;
import javax.naming.InitialContext;
import javax.naming.NamingException;
import javax.sql.DataSource;
public class SearchSessionBean implements SessionBean, SearchSessionLocalBusiness {
    private SessionContext context;
    private arcoiris.ServiceLocator serviceLocator;
    // <editor-fold defaultstate="collapsed" desc="EJB infrastructure methods. Click the + sign on the left to edit the code.">
    // TODO Add code to acquire and use other enterprise resources (DataSource, JMS, enterprise bean, Web services)
    // TODO Add business methods or web service operations
     * @see javax.ejb.SessionBean#setSessionContext(javax.ejb.SessionContext)
    public void setSessionContext(SessionContext aContext) {
        context = aContext;
     * @see javax.ejb.SessionBean#ejbActivate()
    public void ejbActivate() {
     * @see javax.ejb.SessionBean#ejbPassivate()
    public void ejbPassivate() {
     * @see javax.ejb.SessionBean#ejbRemove()
    public void ejbRemove() {
    // </editor-fold>
     * See section 7.10.3 of the EJB 2.0 specification
     * See section 7.11.3 of the EJB 2.1 specification
    public void ejbCreate() {
        // TODO implement ejbCreate if necessary, acquire resources
        // This method has access to the JNDI context so resource aquisition
        // spanning all methods can be performed here such as home interfaces
        // and data sources.
    // Add business logic below. (Right-click in editor and choose
    // "EJB Methods > Add Business Method" or "Web Service > Add Operation")
    public List<EstablishmentLocal> performSearch(SearchRequestDTO searchRequestDto) {
        System.out.println("within performSearch");
        List<EstablishmentLocal> establishments = new ArrayList<EstablishmentLocal>();
        try {
            Context ctx = new InitialContext();
            DataSource ds = (DataSource)ctx.lookup("java:/guia");
            Connection con = ds.getConnection();
            Statement stm = con.createStatement();
            String query = buildQueryFromSearchRequestDTO(searchRequestDto);
            ResultSet rs = stm.executeQuery(query);
            int rowCount = 0;
            while(rs.next()){
                rowCount = rowCount + 1;
                System.out.println("while");
                try {
                    EstablishmentLocal establishment = lookupEstablishmentBean().findByEstablishmentId(rs.getInt("establishment_id"));
                    establishments.add(establishment);
                    System.out.println("est name "+establishment.getEstablishmentName());
                } catch (FinderException ex) {
                    ex.printStackTrace();
                } catch (SQLException ex) {
                    ex.printStackTrace();
            System.out.println("rowCount = "+rowCount);
            rs.close();
            stm.close();
            con.close();
        catch (SearchRequestMalformedException ex){
            ex.printStackTrace();
        }catch (NamingException ex) {
            ex.printStackTrace();
        }catch (SQLException ex){
            ex.printStackTrace();
        //TODO implement performSearch
        return null;
    private String buildQueryFromSearchRequestDTO(SearchRequestDTO searchRequestDto) throws SearchRequestMalformedException {
        StringBuffer sb = new StringBuffer();
        //Head of the query
        sb.append("SELECT ");
        sb.append("e.establishment_id, ");
        sb.append("e.establishment_name, ");
        sb.append("avg(r.rating) as avg_rating ");
        sb.append("FROM ");
        sb.append("establishment e, ");
        sb.append("rating r, ");
        sb.append("comment com, ");
        sb.append("establishment_country ec, ");
        sb.append("country_ref cou ");
        sb.append("where ");
        sb.append("e.establishment_id=ec.establishment_id and ");
        sb.append("ec.country_id=cou.country_id and ");
        sb.append("e.establishment_id=com.establishment_id and ");
        sb.append("r.establishment_id=e.establishment_id and ");
        //Conditional parts of the query
        //If user wants comment to be present
        if(searchRequestDto.isCommentExists()){
            sb.append("e.establishment_id=com.establishment_id and ");
        //If user has typed in a keyword
        if(!searchRequestDto.getKeyword().equals("") && !searchRequestDto.getKeyword().equals(" ")&& searchRequestDto.getKeyword()!=null){
            sb.append("e.establishment_name like '%"+ searchRequestDto.getKeyword() +"%' and ");
        //If user has choosen a country
        if(searchRequestDto.getCountry()!=null && !searchRequestDto.getCountry().equals("") && !searchRequestDto.getCountry().equals(" ")&& searchRequestDto.getCountry()!=null){
            sb.append("ec.country_id = " + searchRequestDto.getCountry() + " and ");
        //If user wants website to be present
        if(searchRequestDto.isWebsiteExists()){
            sb.append("e.establishment_website is not null and ");
/* conditions
si utilisateur a renseign� un premier code postal "e.establishment_postcode = 'postcode_one' --si racine
        si autre code postal "or e.establishment_postcode = 'postcode_two'
        ainsi de suite
and--vient de si racine
        //Tail of the query
        sb.append("0=0 ");
        sb.append("group by e.establishment_id ");
        //If the user wants a minimum rating
        if(searchRequestDto.getRating() != null && searchRequestDto.getRating().intValue()!=0){
            sb.append("having avg(r.rating) >= " + searchRequestDto.getRating());
        sb.append(" ");
        sb.append("order by e.establishment_id ");
        System.out.println("****************************");
        System.out.println(sb);
        System.out.println("****************************");
        return sb.toString();
    private arcoiris.ServiceLocator getServiceLocator() {
        if (serviceLocator == null) {
            serviceLocator = new arcoiris.ServiceLocator();
        return serviceLocator;
    private EstablishmentLocalHome lookupEstablishmentBean() {
        try {
            return (arcoiris.EstablishmentLocalHome) getServiceLocator().getLocalHome("java:comp/env/ejb/EstablishmentBean");
        } catch(javax.naming.NamingException ne) {
            java.util.logging.Logger.getLogger(getClass().getName()).log(java.util.logging.Level.SEVERE,"exception caught" ,ne);
            throw new RuntimeException(ne);
}I just don't understand why it does so. The peformSearch should be executed ONCE only.
Here is my face config file:
<?xml version='1.0' encoding='UTF-8'?>
<!DOCTYPE faces-config PUBLIC
  "-//Sun Microsystems, Inc.//DTD JavaServer Faces Config 1.1//EN"
  "http://java.sun.com/dtd/web-facesconfig_1_1.dtd">
<faces-config>
    <managed-bean>
        <managed-bean-name>SearchManagedBean</managed-bean-name>
        <managed-bean-class>arcoiris.SearchManagedBean</managed-bean-class>
        <managed-bean-scope>request</managed-bean-scope>
    </managed-bean>
    <navigation-rule>
       <description></description>
        <from-view-id>/welcomeJSF.jsp</from-view-id>
        <navigation-case>
        <from-outcome>success</from-outcome>
        <to-view-id>index.jsp</to-view-id>
        </navigation-case>
    </navigation-rule>
</faces-config>Can anyone help?
Thanks in advance,
Julien.

Answer 1

Just reactivating my thread hoping someone will reply to it....
Julien.