I have an Event object with an EventType:

(snipped of Event.java)
private Long id;
private EventType eventType;
private String title;
private String intro;

And a snipped of the mapping file (Event.hbm.xml):

<many-to-one name="eventType"
class="com.egoactive.lego.event.model.EventType"
column="EVENT_TYPE_ID"
not-null="true"
lazy="true"/>

I have 1 JSP/Struts issue

How t enter a new event via a webform:
<%
Event event = new Event();
request.setAttribute("element", event);

//retrieve all evenTypes
List eventTypeList = EventTypeService.getInstance().getList();
request.setAttribute("eventTypes", eventTypeList);
%>
<html:form action="deleteEvent-submit.do" method="post">

<!--create option list--->
<html:select name="element" property="eventType" >
<html:options collection="eventTypes" property="id"
labelProperty="value" />
</html:select>
<html:text name="element" property="title" />
<html:submit property="submit"/>
</html:form>

The problem is the html:select bit..what should the proporty be? Or should I define my html:option bit different?

Thanks In advance.
Niels

I think it should be different. The HTML select submits EventType id only therefore receiving property should be of id type, not the EventType type.

<!--create option list--->
<html:select name="element" property="eventTypeId" >
<html:options collection="eventTypes" property="id"
labelProperty="value" />
</html:select>

On the struts form you can do something like this:

setEventTypeId( id){
getEvent().setEventType( daoLayer.getEventTypeByID(id) );
}

Thanks...I am coming there but ..

Event event = new Event();
request.setAttribute("element", event);

So I don't have a eventTypeId in my Event object,

what I can do:
<html:select name="element" property="eventType.id" >
<html:options collection="eventTypes" property="id" labelProperty="value" />
</html:select>

But how should I map this field (eventType.id) in my struts form?

Something like...

private EventType eventType;

public EventType getEventType() {
return eventType;
}

..but I want the id of the EventType..

Thanks

I suggest using Strut's ActionForm as a helper structure that holds relevant data, in this case we could code like this:

In the Action that extends BaseAction (assuming that base action takes care of redirecting to given functions ):

public ActionForward createNew(ActionMapping mapping, ActionForm actionForm, HttpServletRequest request, HttpServletResponse response) throws Exception {
EventForm form = (EventForm) actionForm;
form.setModeCreate();
form.setEvent( new Event() );
form.setEventTypes( getListOfTypesSomehow() );
return mapping.findForward("crud");
}

public ActionForward saveNew(ActionMapping mapping, ActionForm actionForm, HttpServletRequest request, HttpServletResponse response) throws Exception {
EventForm form = (EventForm) actionForm;
Event evt = form.getEvent( );
eventDao.saveEvent( evt );
form.setModeEdit();
return mapping.findForward("crud");
}


EventForm extends CRUDForm{
Event event;
List eventTypes;

Event Event()....
setEvent()....

....

Integer getEventTypeId(){
getEvent().getEventTYpe().getId();
}

void setEventTypeId( Integer id){
EventType et = findEventTypeInTheListById( id, getEventTypes() );
getEvent().setEventType( et );
}

}

Note that the code assumes that eventTyoe field of the Event is never NULL, if that is nullable then we need guards here and there.