Skip to content

Commit d63b6e5

Browse files
committed
WIP: improve updateNode logic
1 parent 77c3bc8 commit d63b6e5

File tree

1 file changed

+4
-3
lines changed
  • src/main/java/com/flowingcode/vaadin/addons/orgchart

1 file changed

+4
-3
lines changed

src/main/java/com/flowingcode/vaadin/addons/orgchart/OrgChart.java

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -677,15 +677,16 @@ public void updateNode(Integer nodeId, OrgChartItem newDataItem) {
677677
if (newDataItem.getClassName() != null) {
678678
nodeToUpdate.setClassName(newDataItem.getClassName());
679679
}
680-
nodeToUpdate.setHybrid(newDataItem.isHybrid());
681-
680+
if (nodeToUpdate.isHybrid() != newDataItem.isHybrid()) {
681+
nodeToUpdate.setHybrid(newDataItem.isHybrid());
682+
}
682683
if (newDataItem.getData() != null) {
683684
newDataItem.getData().forEach(nodeToUpdate::setData);
684685
}
685686

686687
// Call the client-side JS function to update the visual representation
687688
String newDataJson = convertToJsonObj(newDataItem);
688-
this.getElement().executeJs("this.updateNode($0, $1)", nodeId.toString(), newDataJson);
689+
this.getElement().executeJs("this.updateNode($0, $1)", nodeId, newDataJson);
689690
}
690691
}
691692

0 commit comments

Comments
 (0)