I don't recall the original discussion but this was added a few months ago and was needed for something. What you need is for the last line to be changed to:<br><br><div>
<span style="white-space: pre;"> </span>group hasCallback </div><div><span style="white-space: pre;"> </span>ifTrue: [ super value: (group
callbackValue: anObject) ]<br><span style="white-space: pre;"> </span>ifFalse: [ super value: anObject ]<br></div><br>Is that right?<br><br>Julian<br><br><div class="gmail_quote">On Sun, Apr 18, 2010 at 11:26 AM, Nick Ager <span dir="ltr"><<a href="mailto:nick.ager@gmail.com">nick.ager@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">Hi,<div><br></div><div>I'm defining set of radio buttons in Seaside 3. My initial attempt was:</div>
<div><br></div><div><div> html radioGroup</div><div><span style="white-space: pre;"> </span>selected: 6; </div>
<div><span style="white-space: pre;"> </span>with: [ :group |</div><div><span style="white-space: pre;"> </span><span style="white-space: pre;"> </span>optionDescriptions doWithIndex: [:each :index|</div>
<div><span style="white-space: pre;"> </span>html radioButton</div><div><span style="white-space: pre;"> </span>group: group;</div><div><span style="white-space: pre;"> </span>value: index;</div>
<div><span style="white-space: pre;"> </span>title: each</div><div><span style="white-space: pre;"> </span><span style="white-space: pre;"> </span>]</div>
<div><span style="white-space: pre;"> </span>]</div><div><br></div><div>This generated a set of radio buttons without the "value" attribute. I found the cause in:</div><div><br></div><div>
<div>WARadioButtonTag>>value: anObject</div><div><span style="white-space: pre;"> </span>group isNil</div><div><span style="white-space: pre;"> </span>ifTrue: [ ^ super value: anObject ].</div>
<div><span style="white-space: pre;"> </span>group hasSelection </div><div><span style="white-space: pre;"> </span>ifTrue: [ self selected: (group isSelected: anObject) ].</div><div>
<span style="white-space: pre;"> </span>group hasCallback </div><div><span style="white-space: pre;"> </span>ifTrue: [ super value: (group callbackValue: anObject) ]</div></div><div>
<br></div><div>so the fix was to define a callback for radioGroup. As I'm using Ajax to read the value from the radio buttons, I defined an empty callback. I think I can see the rational for this line of code, but in my case it was counterintuitive. </div>
</div><div><br></div><div>Is the check for the callback really necessary? Could it be removed?</div><div><br></div><div>Thanks</div><div><br></div><font color="#888888"><div>Nick</div>
</font><br>_______________________________________________<br>
seaside mailing list<br>
<a href="mailto:seaside@lists.squeakfoundation.org">seaside@lists.squeakfoundation.org</a><br>
<a href="http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside" target="_blank">http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside</a><br>
<br></blockquote></div><br>