feat: improve output with better-entity usage (#3114)

Author: dordsor21

worldedit-core/src/main/java/com/fastasyncworldedit/core/util/ExtentTraverser.java

@@ -64,7 +64,7 @@ public boolean setNext(T next) {
             ReflectionUtils.setFailsafeFieldValue(field, root, next);
             return true;
         } catch (Throwable e) {
-            e.printStackTrace();
+            LOGGER.error(e);
             return false;
         }
     }
@@ -80,7 +80,7 @@ public ExtentTraverser<T> last() {
     }
 
     @Nullable
-    public <U extends Extent> U findAndGet(Class<U> clazz) {
+    public <U extends Extent> U findAndGet(Class<? super U> clazz) {
         ExtentTraverser<U> traverser = find(clazz);
         return (traverser != null) ? traverser.get() : null;
     }
@@ -97,7 +97,7 @@ public <U extends Extent> ExtentTraverser<U> find(Class<U> clazz) {
             }
             return null;
         } catch (Throwable e) {
-            e.printStackTrace();
+            LOGGER.error(e);
             return null;
         }
     }
@@ -114,7 +114,7 @@ public <U extends Extent> ExtentTraverser<U> find(Object object) {
             }
             return null;
         } catch (Throwable e) {
-            e.printStackTrace();
+            LOGGER.error(e);
             return null;
         }
     }
@@ -135,7 +135,7 @@ public ExtentTraverser<T> next() {
             }
             return null;
         } catch (Throwable e) {
-            e.printStackTrace();
+            LOGGER.error(e);
             return null;
         }
     }

worldedit-core/src/main/java/com/sk89q/worldedit/function/operation/ForwardExtentCopy.java

@@ -42,11 +42,11 @@
 import com.google.common.collect.ImmutableList;
 import com.google.common.collect.Lists;
 import com.sk89q.worldedit.WorldEditException;
-import com.sk89q.worldedit.entity.BaseEntity;
 import com.sk89q.worldedit.entity.Entity;
 import com.sk89q.worldedit.entity.metadata.EntityProperties;
 import com.sk89q.worldedit.extent.AbstractDelegateExtent;
 import com.sk89q.worldedit.extent.Extent;
+import com.sk89q.worldedit.extent.clipboard.Clipboard;
 import com.sk89q.worldedit.function.CombinedRegionFunction;
 import com.sk89q.worldedit.function.RegionFunction;
 import com.sk89q.worldedit.function.RegionMaskingFilter;
@@ -63,11 +63,9 @@
 import com.sk89q.worldedit.regions.CuboidRegion;
 import com.sk89q.worldedit.regions.FlatRegion;
 import com.sk89q.worldedit.regions.Region;
-import com.sk89q.worldedit.util.Location;
 import com.sk89q.worldedit.util.formatting.text.Component;
 import com.sk89q.worldedit.util.formatting.text.TextComponent;
 import org.apache.logging.log4j.Logger;
-import org.jetbrains.annotations.Nullable;
 
 import java.util.Collection;
 import java.util.Collections;
@@ -483,15 +481,19 @@ public static Collection<Entity> getEntities(Extent source, Region region) {
             extent = clip.getExtent();
         }
         IQueueExtent<IQueueChunk> queue = null;
-        if (Settings.settings().EXPERIMENTAL.IMPROVED_ENTITY_EDITS) {
+        if (Settings.settings().EXPERIMENTAL.IMPROVED_ENTITY_EDITS && !(source instanceof Clipboard)) {
             ParallelQueueExtent parallel = new ExtentTraverser<>(extent).findAndGet(ParallelQueueExtent.class);
             if (parallel != null) {
                 queue = parallel.getExtent();
             } else {
                 queue = new ExtentTraverser<>(extent).findAndGet(SingleThreadQueueExtent.class);
             }
             if (queue == null) {
-                LOGGER.warn("Could not find IQueueExtent instance for entity retrieval, OncePerChunkExtent will not work.");
+                LOGGER.warn("Could not find IQueueExtent from `" + source.getClass().getName() +
+                        "` instance for entity retrieval, OncePerChunkExtent will not work.");
+                if (Settings.settings().ENABLED_COMPONENTS.DEBUG) {
+                    ExtentTraverser.printNestedExtents(source);
+                }
             }
         }
         if (queue == null) {