CODEWITHSUNDEEP
jsf
diegohsi
diegohsi

Reputation: 166

selectItems dont convert to myEnum in JSF1.1

I did several tests and conversion error occurs whenever I looked several posts from @BalusC, and errors persist. also used the GenericEnumConverter the @BalusC

my enum

public enum EnumTipoPapel {

CONTATO(1,"Contato"),
DIRETOR(2,"Diretor"),
PASSAGEIRO(3,"Passageiro"),
COMANDANTE(4,"Comandante"),
ACOMPANHANTE(5,"Acompanhante"),
VISITANTE(6, "Visitante"),
SECRETARIA_GERAL(7,"Secretaria Geral"),
ANIVERSARIANTE(99,"Aniversariante");

private int codigo;
private String descricao;

EnumTipoPapel(int codigo, String descricao){
    this.codigo = codigo;
    this.descricao = descricao;
}

/**
 * @return the codigo
 */
public int getCodigo() {
    return codigo;
}

@Override
public String toString() {
    return descricao;
}

}

My jsf page

<h:selectManyCheckbox id="cadastrarPessoaPapeis" styleClass="select_table" value="#{cadastrarPessoasBean.papeis}">
     <f:converter converterId="tipoPapelEnum"/>
     <f:selectItems value="#{cadastrarPessoasBean.selectedPapeis}"/>
</h:selectManyCheckbox>

my bean

public class CadastrarPessoasBean extends AbstractBean {
private IPessoa iPessoa = (IPessoa) BradescoSpringUtils.getBeanById("pessoaBO");

private SelectItem[] papeis;
private EnumTipoPapel[] selectedPapeis;

public CadastrarPessoasBean() {
    this.initValues();
}

/**
 * Realiza o load de todos os campos necessario para o carregamento da tela
 */
public void initValues() {
    loadPapeis();
}


/*********************************************************************************
 * Componentes de tela
 ********************************************************************************/


/*********************************************************************************
 * get and set
 ********************************************************************************/


public SelectItem[] getPapeis() {
    return papeis;
}

public void setPapeis(SelectItem[] papeis) {
    this.papeis = papeis;
}

/*********************************************************************************
 * Acoes da tela
 ********************************************************************************/

public EnumTipoPapel[] getSelectedPapeis() {
    return selectedPapeis;
}

public void setSelectedPapeis(EnumTipoPapel[] selectedPapeis) {
    this.selectedPapeis = selectedPapeis;
}

/**
 * realiza o load dos papeis na pagina
 */
private void loadPapeis() {
    this.papeis = new SelectItem[] {
        new SelectItem(EnumTipoPapel.ANIVERSARIANTE),
        new SelectItem(EnumTipoPapel.DIRETOR),
        new SelectItem(EnumTipoPapel.PASSAGEIRO),
        new SelectItem(EnumTipoPapel.COMANDANTE),
        new SelectItem(EnumTipoPapel.SECRETARIA_GERAL)
    };
}

}

my converter 

public class TipoPapelConverter implements Converter {

public Object getAsObject(FacesContext context, UIComponent component, String value) {
    return EnumTipoPapel.valueOf(value);
}

public String getAsString(FacesContext context, UIComponent component, Object value) {
    return ((EnumTipoPapel) value).name();
}

}

my faces-beam config

<converter>
    <converter-id>tipoPapelConverter</converter-id>
    <converter-class>....converter.TipoPapelConverter</converter-class>
</converter>
<converter>
    <converter-id>genericEnumConverter</converter-id>
    <converter-class>...converter.GenericEnumConverter</converter-class>
</converter>

exceptions

Value is not an enum: class javax.faces.model.SelectItem

Upvotes: 0

Views: 139

Answers (1)

BalusC
BalusC

Reputation: 1109570

There are several mistakes:

  1. You should not bind SelectItem instances to <h:selectManyCheckbox value>. They are supposed to be bound to <f:selectItem(s) value> only.

  2. You should perform null checks in the converter.

All in all, this kickoff example should do:

<h:selectManyCheckbox value="#{bean.selectedPaperTypes}" converter="paperTypeConverter">
     <f:selectItems value="#{bean.availablePaperTypes}" />
</h:selectManyCheckbox>

with in bean:

private PaperType[] selectedPaperTypes;
private SelectItem[] availablePaperTypes;

public void init() {
    PaperType[] paperTypes = PaperType.values();
    SelectItem[] availablePaperTypes = new SelectItem[paperTypes.length];

    for (int i = 0; i < paperTypes.length; i++) {
        availablePaperTypes[i] = new SelectItem(paperTypes[i]);
    }
}

and in converter:

public Object getAsObject(FacesContext context, UIComponent component, String submittedValue) {
    return (submittedValue != null) ? PaperType.valueOf(value) : null;
}

public String getAsString(FacesContext context, UIComponent component, Object modelValue) {
    return (value instanceof PaperType) ? ((PaperType) value).name() : null;
}

Upvotes: 1

Related Questions