Thanks Erik for the extra info. I was thinking that was the problem I was having. One idea that I had was to check in my action class to see if any check boxes were on the request and if not then call the set method myself setting it to null. Does anyone know of a better way to do this.
Thanks Jeff -----Original Message----- From: Erik Hatcher [mailto:[EMAIL PROTECTED]] Sent: Tuesday, October 02, 2001 8:49 AM To: [EMAIL PROTECTED] Subject: Re: multibox and reset() "ERIK" Jeff, No, my patch did not fix this particular case. Here is what is happening in your scenario: checkboxes do not submit a value if they are unchecked, therefore Struts has no way of knowing of their existence on the original HTML form and thus does not call any setters. The reset method is where you implement setting all your checkboxes (and likely other HTML field types) to their default (false, in your case) value. My patch was for the case where the ActionForm bean was created by a JSP page rather than ActionServlet, and reset was not called at that point. In your case, the ActionForm bean is being created (in the last request you speak of) by ActionServlet, so reset should be called - this is all assuming you are going through a Struts action for that request. My patch has been committed to Struts CVS, so its available in the nightly builds (I presume, unless those builds are running on a different branch or something). Erik ----- Original Message ----- From: "Krueger, Jeff" <[EMAIL PROTECTED]> To: <[EMAIL PROTECTED]> Sent: Tuesday, October 02, 2001 7:07 AM Subject: RE: multibox and reset() "ERIK" > Erik, > > I believe your patch will solve my problem but I'm not sure. I have > a page with many multibox tags on it. When I select several of them my form > bean is correctly populated with a string array. If I come back to the page > all the correct boxes are checked. If I remove all the checks and click > submit then my string array is left to what it was before. I assume this is > because there is no mutliboxes being submitted to the server, therefore the > setMethod is never getting called. Is that what you patch fixed, if so do > you know the status of that making it into a build? > > Thanks > > Jeff > > > -----Original Message----- > From: Erik Hatcher [mailto:[EMAIL PROTECTED]] > Sent: Friday, September 14, 2001 8:24 PM > To: [EMAIL PROTECTED] > Subject: Re: multibox and reset() > > > reset is currently not called if <html:form> creates the form bean, which > happens if you go to the JSP page directly without hitting ActionServlet > first. > > I've submitted a patch to fix this, but it has not been committed yet. > > Erik > > > ----- Original Message ----- > From: "Renaud Waldura" <[EMAIL PROTECTED]> > To: <[EMAIL PROTECTED]> > Cc: <[EMAIL PROTECTED]> > Sent: Friday, September 14, 2001 10:12 AM > Subject: Re: multibox and reset() > > > > The signature for reset() is the following: > > > > void reset(ActionMapping, HttpServletRequest) > > > > I don't know whether setting the array reference is null is enough, or you > > truly need to create a zero-length array like in your example below. > > > > I set mine to null, it seemed to work. > > > > > > > > > > ----- Original Message ----- > > From: "Dirk Jaeckel" <[EMAIL PROTECTED]> > > To: <[EMAIL PROTECTED]> > > Sent: Friday, September 14, 2001 6:26 AM > > Subject: multibox and reset() > > > > > > > Hi! > > > > > > > > > I am having trouble resetting a StringArray that is connected to a > > <html:multibox>-tag. > > > > > > HTML-Example: > > > > > > <html:form action="/map" method="GET"> > > > <html:multibox value="parking" property="layer"/> > > > <html:multibox value="petrol" property="layer"/> > > > > > > <html:image property="layers" page="/img/update.gif"/> > > > > > > </html:form> > > > > > > > > > Corresponding ActionForm: > > > > > > public class MapForm extends ActionForm { > > > > > > private String[] _on = new String[0]; > > > > > > public void reset() { > > > _on = new String[0]; > > > } > > > > > > public void setLayer(String in) { > > > _on = in; > > > } > > > > > > public String[] getLayer() { > > > return _on; > > > > > > } > > > } > > > > > > The Manual told me to use the reset()-method to set The Array to the > > length 0, but reset() is never called. > > > > > > > > > Dirk > > > > > > > > > > >
