fix: restore presentation value on exception case

javier-godoy · javier-godoy · commit 917dd2ecd283 · 2021-04-27T21:30:34.000-03:00
Close #45
diff --git a/src/main/java/com/flowingcode/vaadin/addons/chipfield/ChipField.java b/src/main/java/com/flowingcode/vaadin/addons/chipfield/ChipField.java
@@ -157,11 +157,13 @@ private void configure() {
                           () -> {
                             if (isAllowAdditionalItems()) {
                               if (newItemHandler == null) {
+                                setPresentationValue(getValue());
                                 throw new IllegalStateException(
                                     "You need to setup a NewItemHandler");
                               }
                               return this.newItemHandler.apply(chipLabel);
                             } else {
+                              setPresentationValue(getValue());
                               throw new IllegalStateException(
                                   "Adding new items is not allowed, but still receiving new items (not present in DataProvider) from client-side. Probably wrong configuration.");
                             }

Original file line number	Diff line number	Diff line change
`@@ -157,11 +157,13 @@ private void configure() {`
`157`	`157`	`() -> {`
`158`	`158`	`if (isAllowAdditionalItems()) {`
`159`	`159`	`if (newItemHandler == null) {`
	`160`	`+ setPresentationValue(getValue());`
`160`	`161`	`throw new IllegalStateException(`
`161`	`162`	`"You need to setup a NewItemHandler");`
`162`	`163`	`}`
`163`	`164`	`return this.newItemHandler.apply(chipLabel);`
`164`	`165`	`} else {`
	`166`	`+ setPresentationValue(getValue());`
`165`	`167`	`throw new IllegalStateException(`
`166`	`168`	`"Adding new items is not allowed, but still receiving new items (not present in DataProvider) from client-side. Probably wrong configuration.");`
`167`	`169`	`}`