Beginner needs help please

[Michael Norman]

Dear all.

I have the following problem. I have the following code:

<logic:iterate name="form" property="parts" id="part">
<html:radio property="partId" idName="part" value="partId" onChange="changeName(this)" />
<bean:write name="part" property="partName" />
</logic:iterate>

This works fine. BUT... Now I want to change the javascript call from

  
onChange="changeName(this)"

to something like:

  
onChange='<%= "changeName(" + part.partName + ")" %>'

So I want to call "changeName" with the name of the part I just iterated over. The above dosn't work... But what is the correct code for this?

Thanks in advanced for your help

greetings

Norman

[Eric Fletcher]

Try

 
onChange="changeName('<%= part.partName%>')"

HTH,
E

[Merrill Higginson]

You have to remember that your code is in a scriptlet, which means that you need to use java syntax, rather than struts tag syntax.

The following should work:

onChange='<%= "changeName(" + part.getPartName() + ")" %>'

[Eric Fletcher]

Oops, merrill is right, missed the getter. Should look like this:

 
onChange="changeName('<%= part.getPartName()%>')"

Sorry about that!
[ June 10, 2005: Message edited by: Eric Fletcher ]

[Merrill Higginson]

A couple more things on this:

I just noticed that in order for my suggestion to work, you will also have to add type="<<fully qualified name for the Part class>>" to the logic:iterate tag. Otherwise, the part variable will be of type Object, and you will have to cast it before using its getPartName() method

Struts tags have an "all or nothing" policy toward scriptlets, which means that if you substitute an attribute value with a scriptlet, either all of it has to be in a scriptlet, or no part of it should be a scriptlet. Given this policy, I don't think Eric's suggestion will work in this situation.

I hope this helps.
[ June 10, 2005: Message edited by: Merrill Higginson ]

[Michael Norman]

Hey Merrill and Eric.

Thanks for your help, it works now. I had to use the type-attribute of the iterate-tag and the syntax Merrill posted!

Greetings,

Norman