#6662 fixed checkbox group code interaction (#6692)

kernel/base/src/main/java/com/twosigma/beakerx/easyform/formitem/widgets/CheckBoxGroupWidget.java

@@ -26,6 +26,7 @@
 import com.twosigma.beakerx.widgets.strings.Label;
 
 import java.util.ArrayList;
+import java.util.Arrays;
 import java.util.Collection;
 import java.util.List;
 import java.util.stream.Collectors;
@@ -56,6 +57,17 @@ public void setHorizontal(final Boolean horizontal) {
     this.horizontal = horizontal;
   }
 
+  @Override
+  public List<String> formatValue(final Object value) {
+    List<String> result = new ArrayList<>();
+    if (value instanceof Object[]) {
+      result = Arrays.stream((Object[]) value).map(i -> (String) i).collect(Collectors.toList());
+    } else if (value instanceof List) {
+      result = (List<String>) value;
+    }
+    return result;
+  }
+
   @Override
   public String getLabel() {
     return this.label.getValue();
@@ -75,6 +87,14 @@ private Collection<String> getValues() {
     return this.checkboxes.stream().map(EasyFormComponent::getWidget).filter(BoolWidget::getValue).map(ValueWidget::getDescription).collect(Collectors.toList());
   }
 
+  @Override
+  public void setValue(Object value) {
+    List<String> descList = (ArrayList<String>) value;
+    checkboxes.stream()
+            .map(c -> c.getWidget())
+            .forEach(w -> w.setValue(descList.contains(w.getDescription())));
+  }
+
   public void setValues(Collection<String> values) {
     values.forEach(item -> {
       CheckBoxWidget checkbox = new CheckBoxWidget(item);
@@ -89,4 +109,9 @@ private void createWidget(Collection<String> values) {
     this.widget = new HBox(asList(label, rightSide));
   }
 
+  @Override
+  protected boolean checkValue(Object value) {
+    return value instanceof Object[] || value instanceof List;
+  }
+
 }

kernel/base/src/main/java/com/twosigma/beakerx/easyform/formitem/widgets/SelectMultipleWidget.java

@@ -70,7 +70,7 @@ public List<String> formatValue(final Object value) {
 
   @Override
   protected boolean checkValue(Object value) {
-    return !(value instanceof String);
+    return value instanceof Object[] || value instanceof List;
   }
 
 }