From 34405359c0b6c78155dd7769545f38cfc13e98c1 Mon Sep 17 00:00:00 2001
From: Joacim Breiler <joacim@breiler.com>
Date: Mon, 23 Oct 2023 19:36:48 +0200
Subject: [PATCH] Fixed problem with resizing entities when typing zero as
 width or height.

---
 .../nbp/designer/entities/AbstractEntity.java |  8 ++---
 .../nbp/designer/entities/EntityGroup.java    | 35 +++++++++----------
 2 files changed, 20 insertions(+), 23 deletions(-)

diff --git a/ugs-platform/ugs-platform-plugin-designer/src/main/java/com/willwinder/ugs/nbp/designer/entities/AbstractEntity.java b/ugs-platform/ugs-platform-plugin-designer/src/main/java/com/willwinder/ugs/nbp/designer/entities/AbstractEntity.java
index 6d4311675..f6a1a4d7c 100644
--- a/ugs-platform/ugs-platform-plugin-designer/src/main/java/com/willwinder/ugs/nbp/designer/entities/AbstractEntity.java
+++ b/ugs-platform/ugs-platform-plugin-designer/src/main/java/com/willwinder/ugs/nbp/designer/entities/AbstractEntity.java
@@ -85,12 +85,12 @@ public Size getSize() {
 
     @Override
     public void setSize(Size size) {
-        if (size.getWidth() < 0.1) {
-            size = new Size(0.1, size.getHeight());
+        if (size.getWidth() <= 0) {
+            size = new Size(0.0001, size.getHeight());
         }
 
-        if (size.getHeight() < 0.1) {
-            size = new Size(size.getWidth(), 0.1);
+        if (size.getHeight() <= 0) {
+            size = new Size(size.getWidth(), 0.0001);
         }
 
         Size currentSize = getSize();
diff --git a/ugs-platform/ugs-platform-plugin-designer/src/main/java/com/willwinder/ugs/nbp/designer/entities/EntityGroup.java b/ugs-platform/ugs-platform-plugin-designer/src/main/java/com/willwinder/ugs/nbp/designer/entities/EntityGroup.java
index b557a8b29..ecefd7408 100644
--- a/ugs-platform/ugs-platform-plugin-designer/src/main/java/com/willwinder/ugs/nbp/designer/entities/EntityGroup.java
+++ b/ugs-platform/ugs-platform-plugin-designer/src/main/java/com/willwinder/ugs/nbp/designer/entities/EntityGroup.java
@@ -32,7 +32,6 @@ This file is part of Universal Gcode Sender (UGS).
 import java.util.List;
 import java.util.Optional;
 import java.util.concurrent.CopyOnWriteArrayList;
-import java.util.stream.Collectors;
 import java.util.stream.Stream;
 
 /**
@@ -57,6 +56,10 @@ public void render(Graphics2D graphics, Drawing drawing) {
 
     @Override
     public void setSize(Size size) {
+        if(size.getWidth() == 0 || size.getHeight() == 0) {
+            return;
+        }
+
         Size originalSize = getSize();
         scale(size.getWidth() / originalSize.getWidth(), size.getHeight() / originalSize.getHeight());
     }
@@ -250,35 +253,31 @@ public void removeAll() {
     }
 
     public List<Entity> getChildrenAt(Point2D p) {
-        List<Entity> result = this.children
+        return this.children
                 .stream()
                 .flatMap(s -> {
-                    if (s instanceof EntityGroup) {
-                        return ((EntityGroup) s).getChildrenAt(p).stream();
+                    if (s instanceof EntityGroup entityGroup) {
+                        return entityGroup.getChildrenAt(p).stream();
                     } else if (s.isWithin(p)) {
                         return Stream.of(s);
                     } else {
                         return Stream.empty();
                     }
-                }).collect(Collectors.toList());
-
-        return Collections.unmodifiableList(result);
+                }).toList();
     }
 
     public List<Entity> getChildrenIntersecting(Shape shape) {
-        List<Entity> result = this.children
+        return this.children
                 .stream()
                 .flatMap(s -> {
-                    if (s instanceof EntityGroup) {
-                        return ((EntityGroup) s).getChildrenIntersecting(shape).stream();
+                    if (s instanceof EntityGroup entityGroup) {
+                        return entityGroup.getChildrenIntersecting(shape).stream();
                     } else if (s.isIntersecting(shape)) {
                         return Stream.of(s);
                     } else {
                         return Stream.empty();
                     }
-                }).collect(Collectors.toList());
-
-        return Collections.unmodifiableList(result);
+                }).toList();
     }
 
     /**
@@ -315,17 +314,15 @@ public final List<Entity> getAllChildren() {
             return Collections.emptyList();
         }
 
-        List<Entity> result = this.children
+        return this.children
                 .stream()
                 .flatMap(s -> {
-                    if (s instanceof EntityGroup) {
-                        return ((EntityGroup) s).getAllChildren().stream();
+                    if (s instanceof EntityGroup entityGroup) {
+                        return entityGroup.getAllChildren().stream();
                     } else {
                         return Stream.of(s);
                     }
-                }).collect(Collectors.toList());
-
-        return Collections.unmodifiableList(result);
+                }).toList();
     }
 
     @Override