[WICKET-2806] Javascript looking for related radio in group is not working if css class is already assigned to radio

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.4.7
• Fix Version/s: 1.4.8, 1.5-M1
• Component/s: wicket
• Environment: Windows XP
• Assignee: Igor Vaynberg
• Reporter: Per Newgro
• Created: 26/Mar/10 10:28
• Updated: 28/Mar/10 22:33
• Resolved: 26/Mar/10 14:15

Description

We added a RadioGroup and two Radios. We often use multiple css classes for a component. Everything is working, until we add the class attribute to the radios. Then the click on radio is ignored.

The sad thing is that the tests for behavior with WicketTester are working. This problem is only related to the browsers. So we can't provide a testcase.
Sorry you have to check it manually.

We traced it to the line 73 in AjaxFormChoiceComponentUpdatingBehavior

<code>
asb.append(" if (!(inputNode.className.indexOf('wicket-'markupId)===0)&&!(inputNode.id.indexOf(markupId'-')===0)) continue;\n");
</code>

That means that the className has to start with the appropriate "wicket-markupId" value. But it will be added to the end of the class attribute value in
line 226 of Radio. So we get "abc wicket-markupId".

It is working with
asb.append(" if (!(inputNode.className.indexOf('wicket-'markupId)>=0 || inputNode.id.indexOf(markupId'-')>=0)) continue;\n");

Alternatively the class for identifying in script has to be added at beginning
Radio line 226:
clazz = marker + " " + clazz;