Indexed tags and form beans

I have a requirement for users to update multiple rows at the same time. My problem is that the form bean I get in my action class
does not accurately represent the html form that is submitted. In the example below, the bean's headerId and personsCode elements
are sometimes null when they always contain a value in the html, and the destination radio button element has a value in the bean in many cases even if it has not been selected.
I've used <logic:iterate> and the indexed attribute, to generate the HTML:

<html:form name="bonusDestinationForm" method="post" type="com.mgms.zodiac.crew.form.BonusDestinationForm" action="/bonus/setBonusDestinationAction.do">
<table>
  <tr>
    <td align="center"><html:submit property="submit" value="Update"/></td>
  </tr>
</table>
<br><br>
<table width="90%">
<html:hidden name="getBonusDestinationsForm" property="rank"/> 
<html:hidden name="getBonusDestinationsForm" property="year"/> 
<html:hidden name="getBonusDestinationsForm" property="unallocatedOnly"/> 
<logic:iterate name="bonusDestinationForm" id="person" property="bonusDestinations" type="com.mgms.zodiac.dto.BonusPaymentTO">
  <html:hidden name="person" property="personsCode" indexed="true"/> 
  <html:hidden name="person" property="headerId" indexed="true"/> 
  <tr>
    <td>
      <table class="formatted" width="100%">
        <tr>
          <th style="width:40%">
            <h4><bean:write name="person" property="rank"/> 
              <bean:write name="person" property="name"/> 
              <bean:write name="person" property="surname"/> (<bean:write name="person" property="personsCode"/> :wink: 
            </h4>
          </th>
          <%Iterator it = person.getBonusDestinationList().iterator();
            KeyValuePairTO dto = null;
            while (it.hasNext()) {
              dto = (KeyValuePairTO) it.next(); %>
          <th style="width:15%">
            <html:radio name="person" property="destination" value="<%=dto.getValue()%>" indexed="true" style="background-color:transparent"/> 
            <%=dto.getKey()%>
          </th>
           <% } %>
        </tr>
      </table>
    </td>
    <td style="vertical-align:middle"><a href="#top">back to top</a></td>
  </tr>
  <tr>
    <td>
      <table width="100%" cellspacing="0">
        <tr>
        <logic:iterate name="person" id="detail" property="allotments" type="com.mgms.zodiac.dto.AllotmentTO">
          <td width="45%">
            <%@ include file="bonus-payment-location.jsp" %>
          </td>
          <td width="5%"></td>
        </logic:iterate>
        <logic:equal name="person" property="allotmentsCount" value="1">
          <td width="45%"></td>
          <td width="5%"></td>
        </logic:equal>
        </tr>
      </table>
    <br>
    </td>
    <td></td>
  </tr>
</logic:iterate>
</table>
</html:form>

which gives me ...

<form name="bonusDestinationForm" method="post" action="/zodiac/bonus/setBonusDestinationAction.do">
<table>
  <tr>
    <td align="center"><input type="submit" name="submit" value="Update"></td>
</tr>
</table>
<br><br>
<table width="90%">
<input type="hidden" name="rank" value="C/E"> 
<input type="hidden" name="year" value="2003"> 
<input type="hidden" name="unallocatedOnly" value="N"> 
<input type="hidden" name="person[0].personsCode" value="2317"> 
<input type="hidden" name="person[0].headerId" value="38444"> 
<tr>
  <td>
    <table class="formatted" width="100%">
      <tr>
        <th style="width:40%">
          <h4>C/E 
            JOE 
            BLOGGS (2317)
          </h4>
        </th>

<th style="width:15%">
  <input type="radio" name="person[0].destination" value="1202315" style="background-color:transparent"> 
            1st Allotment
        </th>

<th style="width:15%">
          <input type="radio" name="person[0].destination" value="1202317" style="background-color:transparent"> 
            2nd Allotment
        </th>

</tr>
    </table>
  </td>
  <td style="vertical-align:middle"><a href="#top">back to top</a></td>
</tr>
....etc
</table>

My form bean looks like this:

public class BonusDestinationForm extends ActionForm {
private List bonusDestinations = new ArrayList();
private String rank = null;
private String year = null;
private String unallocatedOnly = null;

public void setBonusDestinations(List list) {

bonusDestinations = list;
}

public List getBonusDestinations() {

return bonusDestinations;
}

public BonusPaymentTO getPerson(int index) {

BonusPaymentTO dto = null;
if (index >= bonusDestinations.size()) {
dto = new BonusPaymentTO();
bonusDestinations.add(dto);
}
else {
dto = (BonusPaymentTO) bonusDestinations.get(index);
}
return dto;
}
public void reset(ActionMapping arg0, HttpServletRequest arg1) {
// TODO Auto-generated method stub
bonusDestinations = new ArrayList();
}
/**
 * @return
 */
public String getRank() {
return rank;
}
/**
 * @return
 */
public String getUnallocatedOnly() {
return unallocatedOnly;
}
/**
 * @return
 */
public String getYear() {
return year;
}
/**
 * @param string
 */
public void setRank(String string) {
rank = string;
}
/**
 * @param string
 */
public void setUnallocatedOnly(String string) {
unallocatedOnly = string;
}
/**
 * @param string
 */
public void setYear(String string) {
year = string;
}
}

and the relevant bits of struts-config.xml ...

Could someone please tell me where I've gone wrong - or let me know if there's a better way of doing it ?
Many thanks,
Neil Richards