Allright! Now I see the red and it all makes sense for me. why didn't you go the way of adding another property for customerId to your form? just wondering?
ATTA
Hi,
Here is the FormController' code. The code in red solved the problem.
package com._4s_.invoice.web.action;
import java.text.NumberFormat;
import java.text.SimpleDateFormat;
import java.util.Date;
import java.util.HashMap;
import java.util.Map;
import javax.servlet.ServletException;
import javax.servlet.http.HttpServletRequest;
import javax.servlet.http.HttpServletResponse;
import org.apache.commons.logging.Log;
import org.apache.commons.logging.LogFactory;
import org.springframework.beans.propertyeditors.CustomDa teEditor;
import org.springframework.beans.propertyeditors.CustomNu mberEditor;
import org.springframework.validation.BindException;
import org.springframework.web.bind.ServletRequestDataBin der;
import org.springframework.web.servlet.ModelAndView;
import org.springframework.web.servlet.mvc.SimpleFormCont roller;
import org.springframework.web.servlet.view.RedirectView;
import com._4s_.invoice.model.Customer;
import com._4s_.invoice.model.Invoice;
import com._4s_.invoice.service.CustomerManager;
import com._4s_.invoice.service.InvoiceManager;
/**
* @author smagied
*
* TODO To change the template for this generated type comment go to
* Window - Preferences - Java - Code Style - Code Templates
*/
public class AddInvoiceFormController extends SimpleFormController {
private static Log log = LogFactory.getLog(AddInvoiceFormController.class);
private InvoiceManager mgr = null;
private CustomerManager customerManager = null;
public void setInvoiceManager(InvoiceManager invoiceManager){
this.mgr = invoiceManager;
}
public InvoiceManager getInvoiceManager(){
return this.mgr;
}
public void setCustomerManager(CustomerManager customerManager){
this.customerManager = customerManager;
}
public CustomerManager getCustomerManager(){
return this.customerManager;
}
/**
* *Set up a custom property editor for converting Longs
*
*/
public void initBinder(HttpServletRequest request,
ServletRequestDataBinder binder){
NumberFormat nf = NumberFormat.getNumberInstance();
binder.registerCustomEditor(Long.class, null, new CustomNumberEditor(Long.class, nf,true));
binder.registerCustomEditor(Date.class,null, new CustomDateEditor(new SimpleDateFormat("yyyy-MM-dd"), true));
}
public ModelAndView onSubmit(HttpServletRequest request,
HttpServletResponse response,
Object command, BindException errors)
throws Exception{
if (log.isDebugEnabled()){
log.debug("entering 'onSubmit' method....");
}
log.error("<<<<<<<<<<<<<<<<<<<<<<<<<<onSubmit:>>>>>>>>>>>>>>>>>>>>>>>>>>>");
Invoice invoice = (Invoice) command;
String customerId = request.getParameter("selectCustomer");
Customer customer = customerManager.getCustomer(customerId);
invoice.setCustomer(customer);
mgr.saveInvoice(invoice);
return new ModelAndView(new RedirectView(getSuccessView()));
}
protected Object formBackingObject (HttpServletRequest request)
throws ServletException{
log.error("<<<<<<<<<<<<<<<<<<<<<<<<<<formBackingObject:>>>>>>>>>>>>>>>>>>>>>>>>>>>");
String invoiceId = request.getParameter("id");
if ((invoiceId != null) && !invoiceId.equals("")) {
Invoice invoice= mgr.getInvoice(new Long(invoiceId));
return invoice;
}else {
return new Invoice()
}
}
protected Map referenceData(HttpServletRequest request)
throws ServletException {
Map model = new HashMap();
model.put("customers", customerManager.getCustomers());
return model;
}
}
Here is the JSP page:
<%@ include file="/taglibs.jsp"%>
<html>
<title>Add Invoice</title>
<spring:bind path="invoice.*">
<c:if test="${not empty status.errorMessages}">
<div class="error">
<c:forEach var="error" items="${status.errorMessages}">
<c:out value="${error}" escapeXml="false"/><br />
</c:forEach>
</div>
</c:if>
</spring:bind>
<p>Please fill in the invoice details:</p>
<form method="POST">
<spring:bind path="invoice.id">
<input type="hidden" name="id" value="${status.value}"/>
</spring:bind>
<table>
<tr>
<td>Invoice-Number:</td>
<td>
<spring:bind path="invoice.invoiceNumber">
<input type="text" name="invoiceNumber" value="${status.value}"/>
</spring:bind>
</td>
</tr>
<tr>
<td>Date:</td>
<td>
<spring:bind path="invoice.date">
<input type="text" name="date" value="${status.value}"/>
</spring:bind>
</td>
</tr>
<tr>
<td>Customer:</td>
<td>
<spring:bind path="invoice.customer">
<select name="selectCustomer" value="${status.value}">
<option value="-1">-- Select --</option>
<c:forEach items="${customers}" var="customer">
<option value="${customer.id}"${customer.id == status.value ? ' selected' : ''}>${customer.firstName}</option>
</c:forEach>
</select>
</spring:bind>
</td>
</tr>
<tr>
<td>Total Price:</td>
<td>
<spring:bind path="invoice.totalPrice">
<input type="text" name="totalPrice" value="${status.value}"/>
</spring:bind>
</td>
</tr>
</table>
<p><input type="submit" name="submit" value="Submit"/>
</form>
</html>
Hope it's helpful.
Thanks a lot for your help.
Sherihan.