Tags
People
Badges
Help-
CORE FRAMEWORK
FEATURED COMPONENTS
-
RESOURCES
-
LEARN ZK
RESOURCES
I have a radiogroup:
<radiogroup onCheck="@command('optionSelected')">
<radio label="Option 1" checked="@bind(vm.option1)" />
<radio label="Option 2" checked="@bind(vm.option2)" />
</radiogroup>
If I select an option, true is pushed down to the model, but the other option (deselected) is not pushing false down to the model. After clicking both options, both boolean properties in the model are set to true. The same problem exists if I bind to the "selected" attribute of radio.
What am I doing wrong?
Summary so far: It seems like the zk framework does not handle a binding of the 'checked' and the 'selected' property of the 'radio' component correctly. There are other options/workarounds.
Edit: Here's my view model to clarify what I would like to achieve.
public class TestVM
{
private boolean m_option1 = false;
private boolean m_option2 = false;
@Command
public void optionSelected()
{
if (isOption1())
{
// handle option 1
}
else if (isOption2())
{
// handle option 2
}
else
{
throw new IllegalStateException("unexpected");
}
}
public boolean isOption1()
{
return m_option1;
}
public void setOption1(boolean option1)
{
m_option1 = option1;
}
public boolean isOption2()
{
return m_option2;
}
public void setOption2(boolean option2)
{
m_option2 = option2;
}
}
Binding to the selectedItem of the radiogroup works for me:
<radiogroup selectedItem="@bind(vm.selection)">
<radio label="Label1" value="value1"/>
<radio label="Label2" value="value2"/>
</radiogroup>
Note that the corresponding VM property must be of type String. A commmand is not needed here.
This is my current working solution that I would like to replace (I believe this is what Matze2 suggests.
ZUL:
<radiogroup selectedItem="@bind(vm.selectedOption)" onCheck="@command('optionSelected')">
<radio label="Option 1" value="option1"/>
<radio label="Option 2" value="option2"/>
</radiogroup>
And the model:
public class TestVM
{
private String m_selectedOption = null;
@Command
public void optionSelected()
{
if ("option1".equals(m_selectedOption))
{
// handle option 1
}
else if ("option2".equals(m_selectedOption))
{
// handle option 2
}
else
{
throw new IllegalStateException("unexpected");
}
}
public String getSelectedOption()
{
return m_selectedOption;
}
public void setSelectedOption(String selectedOption)
{
m_selectedOption = selectedOption;
}
}
Sort of. You can use internally also an enum if you want and convert it on the fly in the getter (name()) and setter (valueOf). And I would merge the optionSelected() into setSelectedOption() method (no idea how you would call it otherwise). It's not the cleanest code, but short and understandable.
Matze2 ( 2013-02-26 00:29:05 +0800 )editAnd here's how it probably should be done (the clean solution requiring a bit more typing effort):
<radiogroup model="@load(vm.options)" selectedItem="@bind(vm.selectedOption)" onCheck="@command('optionSelected')">
<template name="model">
<radio label="${each.label}" />
</template>
</radiogroup>
And the model:
public class TestVM
{
public enum Option
{
OPTION_1("Option 1"), //
OPTION_2("Option 2"), //
;
private final String m_label;
private FilterOption(String label)
{
m_label = label;
}
public String getLabel()
{
return m_label;
}
}
private Option m_selectedOption = FilterOption.OPTION_1;
public List<Option> getOptions()
{
return Arrays.asList(Option.values());
}
@Command
public void optionSelected()
{
switch(getSelectedOption())
{
// handle options
}
}
}
Asked: 2013-02-20 17:59:35 +0800
Seen: 122 times
Last updated: Feb 26 '13
For the radio element, which loses the selection, no check-event is sent. You have to monitor this status change itself. During the processing of the command optionSelected, you can reset vm.option1/2.
khcyt ( 2013-02-21 07:34:25 +0800 )editToo bad, from my point of view that is the job of the framework. So I guess there is no point in binding to the radio components checked property :-(. During the processing of the command 'optionSelected' I wanted to use vm.option1/2, not maintain them lol.
RetoH ( 2013-02-21 22:31:07 +0800 )edit