Using f:setPropertyActionListener

[Julien Martin]

Hello,

I am trying to pass two parameters i.e. (nomComposantARejouer, typeFileARejouer) to an action method (gestionnaireMessagesController.rejouerMessage) using the setPropertyActionListener (we use jsf 1.2).

Here is the relevant jsp code:

<h:column>
<h:columnHeaderFacet>
<h:columnHeader value="#{msgs['pilotage.coordinateur.libelle.rejouer']}" />
</h:columnHeaderFacet>
<h:commandLink id="rejouer"
value="#{msgs['pilotage.coordinateur.libelle.rejouer']}"
action="#{gestionnaireMessagesController.rejouerMessage}">
<f:setPropertyActionListener
target="#{gestionnaireMessagesController.nomComposantARejouer}"
value="#{gestionnaireMessagesController.nomComposant}" />
<f:setPropertyActionListener
target="#{gestionnaireMessagesController.typeFileARejouer}"
value="#{gestionnaireMessagesController.typeFile}" />
</h:commandLink>
</h:column>

However, I always get a NPE because both parameters are null when used in the action method:

	public String rejouerMessage() {
		log.debug("-->"+nomComposantARejouer);//null
		ParamResultatMessagesDTO message= (ParamResultatMessagesDTO) messagesTableau.getRowData();
		log.debug("MessageId: " + message.getMessageId());
		try {
			Pager p = Pager.getInstance();
			ParamRejouerMessageDTO prm = new ParamRejouerMessageDTO();
			prm.setMessageId(message.getMessageId());
			prm.setFileGet(nomsFilesMap.get(nomComposantARejouer).get(typeFileARejouer));
			prm.setFilePut(nomsFilesMap.get(nomComposantARejouer).get("TASKQ"));
			RejouerMessageService serv = (RejouerMessageService) this.getService(ServiceCst.REJOUER_MESSAGE_SERVICE);
			serv.rejouerMessage(prm);
		} catch (BusinessException e) {
			this.addMessage(e);
		} catch (ServiceException e) {
			this.addMessage(e);
		}

		
		return chargerPage(); // TODO Navigation case.
	}

FYI, the variables

nomComposant 

and

typeFile

can be displayed without problem and are not null.

Also ignore the h:columnHeaderFacet tags. They are inhouse tags that I've renamed.

I am not sure what I am getting wrong. Can anyone please help?

Regards,

[Yeray Santana Borges]

I think that your code is doing something wrong in the getter and setter of nomComposant and typeFile. Could you write them?, what is the scope of your bean and how are you initializing the nomComposant and typeFile variables?

In my opinion you should remove the setPropertyActionListener, I think that you don't need them.
When you are clicking on commandLink "rejouer", first you are getting the value of nomComposant to assign it to nomComposantARejouer (getters and setter will be called before your code enter in rejouerMessage() method), maybe you can use directly the value of nomComposant in your backed bean when you are in rejouerMessage(), something like this:

   
    <h:commandLink id="rejouer"  
    value="#{msgs['pilotage.coordinateur.libelle.rejouer']}"  
    action="#{gestionnaireMessagesController.rejouerMessage}">     
    </h:commandLink>  
 

public String rejouerMessage() {  
        log.debug("-->"+nomComposantARejouer);//null  
        ParamResultatMessagesDTO message= (ParamResultatMessagesDTO) messagesTableau.getRowData();  
        log.debug("MessageId: " + message.getMessageId());  
        try {  
            Pager p = Pager.getInstance();  
            ParamRejouerMessageDTO prm = new ParamRejouerMessageDTO();  
            prm.setMessageId(message.getMessageId());  
            prm.setFileGet(nomsFilesMap.get(this.nomComposant).get(this.typeFile));  
            prm.setFilePut(nomsFilesMap.get(this.nomComposant).get("TASKQ"));  
            RejouerMessageService serv = (RejouerMessageService) this.getService(ServiceCst.REJOUER_MESSAGE_SERVICE);  
            serv.rejouerMessage(prm);  
        } catch (BusinessException e) {  
            this.addMessage(e);  
        } catch (ServiceException e) {  
            this.addMessage(e);  
        }  
  
          
        return chargerPage(); // TODO Navigation case.  
    }  

[Julien Martin]

Thank you for your reply Yeray.
I finally went on to use f:param which works well in my case.
J.