David
From: "Mark" <[EMAIL PROTECTED]> Reply-To: "Struts Developers List" <[EMAIL PROTECTED]> To: [EMAIL PROTECTED] Subject: Html taglib indexed+mapactionform bug? Date: Wed, 05 Mar 2003 23:06:54 -0600
I wanted to post this here before opening a case so that I could foster some discussion beforehand.
Ok, This is a bit hard for me to describe, please be patient. I have indeed tried to find this in the archives but to no avail.
I will also admit that there are probably a miniscule number of people who might be affected by this, but here goes nonetheless
This problem occurs when you use an index html: field with a map based form.
In my application, I have to show a series of "forms" on one screen that essentially enable the user to configure more than one similar items at a time. So, if they add 7 "cars" to their order form, they can configure each car on one screen (color, make, model, etc). This is hypotethical of course, but the analogy is good. So instead of going through 7 screens, they can do it all on one. For this, I use indexed fields and a Hashmap/MapActionForm which I borrowed from the Struts user guides.
In the instructions, we are told to use this style of property fetching for MapActionForms:
<html:text property="value(color)"/>
Adding to the mix Indexed fields,
<html:text indexed="true" property="value(color)"/>
Our desirable html output should be:
<input type="text" name="value(color_0)">
However, using the combination above, our actual html html field is rendered like so:
<input type="text" name="org.apache.struts.taglib.html.BEAN[0].value(color)">
<input type="text" name="org.apache.struts.taglib.html.BEAN[1].value(color)">
... and so on, incrementing the BEAN[x] for each iteration
Thus, this is incorrect. The MapActionForm gets no values.
I have remedied this problem with a hack, and submit it to you guys for review and possibly "doing it better" since my way probably isnt optimal.
The problem we need to solve is, the property="value(firstname)" needs to have a counter appended to whats inside the () rather than prepending something on the left.
Since whats inside the parens is our "fieldname" and to accomplish an indexed field, we need to append the index value, what we really want is
value(color_0) value(color_1)
etc etc
I've had to fix several files to remedy this:
BaseFieldTag.java BaseHandlerTag.java CheckboxTag.java RadioTag.java SelectTag.java TextareaTag.java
However, I think all the html tags are affected, and should be fixed if the dev team determines this is indeed a bug.
I had to fix BaseHandlerTag like so:
previous: // this code builds the name org.apache.struts.taglib.html.BEAN[0]. if (name != null) handlers.append(name); handlers.append("["); handlers.append(iterateTag.getIndex()); handlers.append("]"); if (name != null) handlers.append(".");
new:
if (name != null) { handlers.append(name); handlers.append("_"); } handlers.append("_"); handlers.append(iterateTag.getIndex());
A diff from BaseFieldTag should give you the fix:
E:\jakarta-struts-1.1-b3-src\src\share\org\apache\struts\taglib\html>diff BaseFieldTag.java ../html-prev/BaseFieldTag.java
142,160c142,145
< // if we are using an indexed field, we need to do something
< // special for map based properties
< if( indexed )
< {
< // are we using a map based property?
< if(this.property.indexOf("(")>0)
< {
< // figure out our prefix and suffix based on where the first ( is found.
< // the goal here is to render our field like value(firstname_0) .. value(firstname_1) etc
< // rather than the original struts way of org.apache.struts.taglib.html.BEAN[0].value(firstname)
< String prefix=this.property.substring(0,this.property.indexOf(")"));
< results.append(prefix);
< prepareIndex(results,null);
< results.append(")");
< } else // otherwise just proceed as normal
< prepareIndex( results, name );
< } else
< results.append(this.property);
<
---
> // * @since Struts 1.1
> if (indexed)
> prepareIndex(results, name);
> results.append(property);
Regards, Mark Williamson
--------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
_________________________________________________________________
Add photos to your e-mail with MSN 8. Get 2 months FREE*. http://join.msn.com/?page=features/featuredemail
--------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
