issue reading fields from form in struts2

[Juan Luis Prieto]

I have an issue reading the files from a form that I have contained in to a jsp web. I use this web as a login page so the form has 3 fields, username, password, url.
all with a different name to avoid reading issues. The issue is that only the first field is been read doesn't matter how you call it. The other 2 values will remain as NULL.

This is the form, and the files involved on the reading.

connect.jsp

						<s:form action="ConnectAction" validate="true" method="POST">
					        <tr>
					          <td colspan="2">
					            <strong>Connect to SongStore</strong>
					          </td>
					        </tr>
					        <tr>
					          <td colspan="2">
					            <s:actionerror />
					            <s:fielderror />
					          </td>
					          <s:textfield name="songStoreUri" label="User"/>
					          <s:password name="password" label="Password"/>
					          <s:textfield name="username" label="Song Store Uri" size="60"/>
					          <s:submit value="Connect" name="connect" id="connect" />
					    </s:form>

connectAction.java

  

package org.ws.songstore.actions;

import org.apache.struts2.dispatcher.DefaultActionSupport;
import org.ws.songstore.connect.Connect;

public class ConnectAction extends DefaultActionSupport {

	private static final long serialVersionUID = 47839074883409332L;
	private String songStoreUri;
	private String username;
	private String password;
	Connect connect = new Connect();
	
	public String execute() throws Exception{
		try{
			
			setUsername(username);
			setPassword(password);
			setSongStoreUri(songStoreUri);

			System.out.println(username);
			System.out.println(password);
			System.out.println(songStoreUri);
			
			return SUCCESS;
		}catch (Exception e) {
			// TODO: handle exception
			return ERROR;
		}
	}
	
	public String getSongStoreUri() {
        return this.songStoreUri;
    }
	
	public void setSongStoreUri(String uri) {
		songStoreUri = uri;
		connect.setSongStoreUri(uri);
		connect.storeUri(uri);
	}
	
	public String getUsername() {
        return this.username;
    }
	
	public void setUsername(String username) {
		songStoreUri = username;
		connect.setUsername(username);
		connect.storeUsername(username);
	}
	
	public String getPassword() {
        return this.password;
    }
	
	public void setPassword(String password) {
		songStoreUri = password;
		connect.setPassword(password);
		connect.storePassword(password);
	}
	
}

connect.java

  

package org.ws.songstore.connect;

import java.io.BufferedWriter;
import java.io.File;
import java.io.FileWriter;
import java.io.IOException;
import java.net.URI;
import java.util.Properties;

import misc.BTUtilsNG;

import org.apache.muse.ws.addressing.EndpointReference;
import org.apache.muse.ws.addressing.soap.SoapFault;
import org.w3c.dom.Element;
import org.ws.songstore.proxy.WsSongStoreProxy;
import org.ws.songstore.utils.FileUtils;

/**
 */
public class Connect  {
	
    // ---- songStoreUri property ----

    /**
     * <p>Field to store SongStore Uri.</p>
     * <p/>
     */
	public String username = null;
    public String password = null;
    public String songStoreUri = null;

    
    public String getUsername() {
        return username;
    }
    
    public String getPassword() {
        return password;
    }
    
    public String getSongStoreUri() {
        return songStoreUri;
    }
    
    

    public void setUsername(String username) {
        this.username = username;
    }
    
    public void setPassword(String password) {
        this.password = password;
    }
    
    public void setSongStoreUri(String uri) {
        this.songStoreUri = uri;
    }
    

    // ---- Store properties ----
    
    static EndpointReference calleeEpr ;
    static WsSongStoreProxy factoryProxy, callerProxy, calleeProxy;
    static boolean printTrace = true;
    static String connectionFile = "connection.properties";
    static FileUtils futils = new FileUtils();
    
    public void storeUri(String uri){
    	String propertyName = "songStore.uri";
    	try{
    		futils.writeProperty(propertyName, uri, connectionFile);
	    }catch (Exception e){//Catch exception if any
	    	System.out.println("Error: " + e.getMessage());
	    }
	}
    
    public void storeUsername(String username){
    	String propertyName = "songStore.username";
    	try{
    		futils.appendProperty(propertyName, username, connectionFile);
	    }catch (Exception e){//Catch exception if any
	    	System.out.println("Error: " + e.getMessage());
	    }
	}
    
    public void storePassword(String password){
    	String propertyName = "songStore.password";
    	try{
    		futils.appendProperty(propertyName, password, connectionFile);
	    }catch (Exception e){//Catch exception if any
	    	System.out.println("Error: " + e.getMessage());
	    }
	}
    
    
    private String getUri(){
    	return futils.getProperty("songStore.uri", connectionFile);
    }
    
	private static EndpointReference elemToEpr(Element elem) throws SoapFault {
	return new EndpointReference(elem);
}
    
    /*
     * Creates the instance and returns the proxy.
     * 
     * @return WsSongStoreProxy
     */
    public WsSongStoreProxy getConnectionProxy() throws SoapFault {
    	String uri = getUri();
    	System.out.println("WsSongStoreUri: "+uri);
    	try{
    		callerProxy = new WsSongStoreProxy(new EndpointReference(URI.create(uri)));
    		callerProxy = new WsSongStoreProxy (elemToEpr (callerProxy.createOperation()));
    		callerProxy.setTrace(printTrace);
    		return callerProxy;
    	}catch (Exception e){
    		System.out.println("Error in getConnectionProxy connect.java: "+ e.getMessage());
    		return null;
    	}
	}
    
    /*
     * Destroy the SongStore instance
     */
    public void destroyConnectionProxy() throws SoapFault, IOException {
		callerProxy.destroy();
		new WsSongStoreProxy(calleeEpr).destroy();
	}
    
}

[David Newton]

Your action's setters all set the same action property.

[Juan Luis Prieto]

thanks for it, I didn't pay attention to that basic issue. Dumb!!