From 3c9b0fa3eaee2834361afb95f6954333ed0b30cb Mon Sep 17 00:00:00 2001 From: Jesse Boyd Date: Wed, 17 Oct 2018 23:42:30 +1100 Subject: [PATCH] Pull boy0001/WorldEdit#9 --- .../java/com/boydti/fawe/util/MainUtil.java | 24 +++- .../worldedit/command/SchematicCommands.java | 22 +--- .../extent/clipboard/io/ClipboardFormat.java | 109 ++++++++++++------ 3 files changed, 101 insertions(+), 54 deletions(-) diff --git a/core/src/main/java/com/boydti/fawe/util/MainUtil.java b/core/src/main/java/com/boydti/fawe/util/MainUtil.java index 7f2e2f98..a117b71d 100644 --- a/core/src/main/java/com/boydti/fawe/util/MainUtil.java +++ b/core/src/main/java/com/boydti/fawe/util/MainUtil.java @@ -11,6 +11,7 @@ import com.github.luben.zstd.ZstdInputStream; import com.github.luben.zstd.ZstdOutputStream; import com.sk89q.jnbt.*; import com.sk89q.worldedit.entity.Entity; +import com.sk89q.worldedit.extent.clipboard.io.ClipboardFormat; import com.sk89q.worldedit.history.changeset.ChangeSet; import com.sk89q.worldedit.util.Location; import java.awt.Graphics2D; @@ -34,6 +35,7 @@ import java.util.concurrent.atomic.AtomicLong; import java.util.function.Consumer; import java.util.regex.Pattern; import java.util.zip.*; +import javax.annotation.Nullable; import javax.imageio.ImageIO; import net.jpountz.lz4.*; @@ -880,9 +882,29 @@ public class MainUtil { return res; } - public static boolean isInSubDirectory(File dir, File file) { + public static File resolve(File dir, String filename, @Nullable ClipboardFormat format, boolean allowDir) { + if (format != null) { + if (!filename.matches(".*\\.[\\w].*")) { + filename = filename + "." + format.getExtension(); + } + return MainUtil.resolveRelative(new File(dir, filename)); + } + if (allowDir) { + File file = MainUtil.resolveRelative(new File(dir, filename)); + if (file.exists() && file.isDirectory()) return file; + } + for (ClipboardFormat f : ClipboardFormat.values()) { + File file = MainUtil.resolveRelative(new File(dir, filename + "." + f.getExtension())); + if (file.exists()) return file; + } + return null; + } + + public static boolean isInSubDirectory(File dir, File file) throws IOException { if (file == null) return false; if (file.equals(dir)) return true; + file = file.getCanonicalFile(); + dir = dir.getCanonicalFile(); return isInSubDirectory(dir, file.getParentFile()); } diff --git a/core/src/main/java/com/sk89q/worldedit/command/SchematicCommands.java b/core/src/main/java/com/sk89q/worldedit/command/SchematicCommands.java index 492f5bde..6dc64dda 100644 --- a/core/src/main/java/com/sk89q/worldedit/command/SchematicCommands.java +++ b/core/src/main/java/com/sk89q/worldedit/command/SchematicCommands.java @@ -191,20 +191,6 @@ public class SchematicCommands extends MethodCommands { player.print(BBC.getPrefix() + "Remapped schematic"); } - private File resolve(File dir, String filename, @Nullable ClipboardFormat format) { - if (format != null) { - if (!filename.matches(".*\\.[\\w].*")) { - filename = filename + "." + format.getExtension(); - } - return MainUtil.resolveRelative(new File(dir, filename)); - } - for (ClipboardFormat f : ClipboardFormat.values()) { - File file = MainUtil.resolveRelative(new File(dir, filename + "." + f.getExtension())); - if (file.exists()) return file; - } - return null; - } - @Command(aliases = {"load"}, usage = "[] ", desc = "Load a schematic into your clipboard") @Deprecated @CommandPermissions({"worldedit.clipboard.load", "worldedit.schematic.load", "worldedit.schematic.upload", "worldedit.schematic.load.other"}) @@ -255,12 +241,12 @@ public class SchematicCommands extends MethodCommands { String extension = filename.substring(filename.lastIndexOf('.') + 1, filename.length()); format = ClipboardFormat.findByExtension(extension); } - f = resolve(dir, filename, format); + f = MainUtil.resolve(dir, filename, format, false); } if (f == null || !f.exists()) { if (!filename.contains("../")) { dir = this.worldEdit.getWorkingDirectoryFile(config.saveDir); - f = resolve(dir, filename, format); + f = MainUtil.resolve(dir, filename, format, false); } } if (f == null || !f.exists() || !MainUtil.isInSubDirectory(working, f)) { @@ -385,7 +371,7 @@ public class SchematicCommands extends MethodCommands { @Command(aliases = {"move", "m"}, usage = "", desc = "Move your loaded schematic", help = "Move your currently loaded schematics", min = 1, max = 1) @CommandPermissions({"worldedit.schematic.move", "worldedit.schematic.move.other"}) - public void move(final Player player, final LocalSession session, final CommandContext args) throws WorldEditException { + public void move(final Player player, final LocalSession session, final CommandContext args) throws WorldEditException, IOException { final LocalConfiguration config = this.worldEdit.getConfiguration(); final File working = this.worldEdit.getWorkingDirectoryFile(config.saveDir); final File dir = Settings.IMP.PATHS.PER_PLAYER_SCHEMATICS ? new File(working, player.getUniqueId().toString()) : working; @@ -431,7 +417,7 @@ public class SchematicCommands extends MethodCommands { @Command(aliases = {"delete", "d"}, usage = "", desc = "Delete a saved schematic", help = "Delete a schematic from the schematic list", min = 1, max = 1) @CommandPermissions({"worldedit.schematic.delete", "worldedit.schematic.delete.other"}) - public void delete(final Player player, final LocalSession session, final CommandContext args) throws WorldEditException { + public void delete(final Player player, final LocalSession session, final CommandContext args) throws WorldEditException, IOException { final LocalConfiguration config = this.worldEdit.getConfiguration(); final File working = this.worldEdit.getWorkingDirectoryFile(config.saveDir); final File dir = Settings.IMP.PATHS.PER_PLAYER_SCHEMATICS ? new File(working, player.getUniqueId().toString()) : working; diff --git a/core/src/main/java/com/sk89q/worldedit/extent/clipboard/io/ClipboardFormat.java b/core/src/main/java/com/sk89q/worldedit/extent/clipboard/io/ClipboardFormat.java index f4c314cb..6ad63092 100644 --- a/core/src/main/java/com/sk89q/worldedit/extent/clipboard/io/ClipboardFormat.java +++ b/core/src/main/java/com/sk89q/worldedit/extent/clipboard/io/ClipboardFormat.java @@ -49,6 +49,7 @@ import com.sk89q.worldedit.event.extent.PlayerSaveClipboardEvent; import com.sk89q.worldedit.extension.platform.Actor; import com.sk89q.worldedit.extent.clipboard.BlockArrayClipboard; import com.sk89q.worldedit.extent.clipboard.Clipboard; +import com.sk89q.worldedit.extent.clipboard.ClipboardFormats; import com.sk89q.worldedit.session.ClipboardHolder; import com.sk89q.worldedit.world.registry.WorldData; import java.io.*; @@ -59,6 +60,7 @@ import java.nio.channels.Channels; import java.nio.channels.ReadableByteChannel; import java.util.*; import java.util.concurrent.ConcurrentHashMap; +import java.util.regex.Pattern; import java.util.zip.GZIPInputStream; import java.util.zip.GZIPOutputStream; import java.util.zip.ZipEntry; @@ -303,16 +305,25 @@ public enum ClipboardFormat { }); } - public MultiClipboardHolder loadAllFromInput(Actor player, WorldData worldData, String input, boolean message) throws IOException { + public static MultiClipboardHolder loadAllFromInput(Actor player, WorldData worldData, String input, boolean message) throws IOException { checkNotNull(player); checkNotNull(input); + ClipboardFormat format = null; WorldEdit worldEdit = WorldEdit.getInstance(); LocalConfiguration config = worldEdit.getConfiguration(); if (input.startsWith("url:")) { + if (!player.hasPermission("worldedit.schematic.load.web")) { + if (message) BBC.NO_PERM.send(player, "worldedit.schematic.load.web"); + return null; + } URL base = new URL(Settings.IMP.WEB.URL); input = new URL(base, "uploads/" + input.substring(4) + ".schematic").toString(); } if (input.startsWith("http")) { + if (!player.hasPermission("worldedit.schematic.load.asset")) { + if (message) BBC.NO_PERM.send(player, "worldedit.schematic.load.asset"); + return null; + } URL url = new URL(input); URL webInterface = new URL(Settings.IMP.WEB.ASSETS); if (!url.getHost().equalsIgnoreCase(webInterface.getHost())) { @@ -322,64 +333,89 @@ public enum ClipboardFormat { MultiClipboardHolder clipboards = loadAllFromUrl(url, worldData); return clipboards; } else { - if (input.contains("../") && !player.hasPermission("worldedit.schematic.load.other")) { - if (message) BBC.NO_PERM.send(player, "worldedit.schematic.load.other"); + if (Settings.IMP.PATHS.PER_PLAYER_SCHEMATICS && Pattern.compile("[0-9a-f]{8}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{12}").matcher(input).find() && !player.hasPermission("worldedit.schematic.load.other")) { + BBC.NO_PERM.send(player, "worldedit.schematic.load.other"); return null; } File working = worldEdit.getWorkingDirectoryFile(config.saveDir); - File dir = new File(working, (Settings.IMP.PATHS.PER_PLAYER_SCHEMATICS ? (player.getUniqueId().toString() + File.separator) : "") + input); - if (!dir.exists()) { - dir = new File(dir + "." + getExtension()); - } - if (!dir.exists()) { - if ((!input.contains("/") && !input.contains("\\")) || player.hasPermission("worldedit.schematic.load.other")) { - dir = new File(worldEdit.getWorkingDirectoryFile(config.saveDir), input); + File dir = Settings.IMP.PATHS.PER_PLAYER_SCHEMATICS ? new File(working, player.getUniqueId().toString()) : working; + File f; + if (input.startsWith("#")) { + String[] extensions; + if (format != null) { + extensions = new String[] { format.getExtension() }; + } else { + extensions = ClipboardFormats.getFileExtensionArray(); } - if (!dir.exists()) { - dir = new File(dir + "." + getExtension()); + f = player.openFileOpenDialog(extensions); + if (f == null || !f.exists()) { + if (message) player.printError("Schematic " + input + " does not exist! (" + f + ")"); + return null; + } + } else { + if (Settings.IMP.PATHS.PER_PLAYER_SCHEMATICS && Pattern.compile("[0-9a-f]{8}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{12}").matcher(input).find() && !player.hasPermission("worldedit.schematic.load.other")) { + if (message) BBC.NO_PERM.send(player, "worldedit.schematic.load.other"); + return null; + } + if (format == null && input.matches(".*\\.[\\w].*")) { + String extension = input.substring(input.lastIndexOf('.') + 1, input.length()); + format = ClipboardFormat.findByExtension(extension); + } + f = MainUtil.resolve(dir, input, format, true); + } + if (f == null || !f.exists()) { + if (!input.contains("../")) { + dir = worldEdit.getWorkingDirectoryFile(config.saveDir); + f = MainUtil.resolve(dir, input, format, true); } } - if (!dir.exists()) { + if (f == null || !f.exists() || !MainUtil.isInSubDirectory(working, f)) { + if (message) player.printError("Schematic " + input + " does not exist! (" + ((f == null) ? false : f.exists()) + "|" + f + "|" + (f == null ? false : !MainUtil.isInSubDirectory(working, f)) + ")"); + return null; + } + if (format == null && f.isFile()) { + format = ClipboardFormat.findByFile(f); + if (format == null) { + BBC.CLIPBOARD_INVALID_FORMAT.send(player, f.getName()); + return null; + } + } + if (!f.exists()) { if (message) BBC.SCHEMATIC_NOT_FOUND.send(player, input); return null; } - if (!dir.isDirectory()) { - ByteSource source = Files.asByteSource(dir); - URI uri = dir.toURI(); - return new MultiClipboardHolder(uri, worldData, new LazyClipboardHolder(dir.toURI(), source, this, worldData, null)); + if (!f.isDirectory()) { + ByteSource source = Files.asByteSource(f); + URI uri = f.toURI(); + return new MultiClipboardHolder(uri, worldData, new LazyClipboardHolder(f.toURI(), source, format, worldData, null)); } - URIClipboardHolder[] clipboards = loadAllFromDirectory(dir, worldData); + URIClipboardHolder[] clipboards = loadAllFromDirectory(f, worldData); if (clipboards.length < 1) { if (message) BBC.SCHEMATIC_NOT_FOUND.send(player, input); return null; } - return new MultiClipboardHolder(dir.toURI(), worldData, clipboards); + return new MultiClipboardHolder(f.toURI(), worldData, clipboards); } } - public URIClipboardHolder[] loadAllFromDirectory(File dir, WorldData worldData) { - if (worldData == null) { - try { - worldData = WorldEdit.getInstance().getServer().getWorlds().get(0).getWorldData(); - } catch (Throwable ignore) { - } - } - File[] files = dir.listFiles(new FileFilter() { - @Override - public boolean accept(File pathname) { - return pathname.getName().endsWith(".schematic"); - } + public static URIClipboardHolder[] loadAllFromDirectory(File dir, WorldData worldData) { + HashSet extensions = new HashSet<>(Arrays.asList(ClipboardFormats.getFileExtensionArray())); + File[] files = dir.listFiles(pathname -> { + String input = pathname.getName(); + String extension = input.substring(input.lastIndexOf('.') + 1, input.length()); + return (extensions.contains(extension.toLowerCase())); }); LazyClipboardHolder[] clipboards = new LazyClipboardHolder[files.length]; for (int i = 0; i < files.length; i++) { File file = files[i]; ByteSource source = Files.asByteSource(file); - clipboards[i] = new LazyClipboardHolder(file.toURI(), source, this, worldData, null); + ClipboardFormat format = ClipboardFormat.findByFile(file); + clipboards[i] = new LazyClipboardHolder(file.toURI(), source, format, worldData, null); } return clipboards; } - public MultiClipboardHolder loadAllFromUrl(URL url, WorldData worldData) throws IOException { + public static MultiClipboardHolder loadAllFromUrl(URL url, WorldData worldData) throws IOException { List clipboards = new ArrayList<>(); try (ReadableByteChannel rbc = Channels.newChannel(url.openStream())) { try (InputStream in = Channels.newInputStream(rbc)) { @@ -387,7 +423,10 @@ public enum ClipboardFormat { ZipEntry entry; byte[] buffer = new byte[8192]; while ((entry = zip.getNextEntry()) != null) { - if (entry.getName().endsWith(".schematic")) { + String filename = entry.getName(); + String extension = filename.substring(filename.lastIndexOf('.') + 1, filename.length()); + ClipboardFormat format = findByExtension(filename); + if (format != null) { FastByteArrayOutputStream out = new FastByteArrayOutputStream(); int len = 0; while ((len = zip.read(buffer)) > 0) { @@ -395,7 +434,7 @@ public enum ClipboardFormat { } byte[] array = out.toByteArray(); ByteSource source = ByteSource.wrap(array); - LazyClipboardHolder clipboard = new LazyClipboardHolder(url.toURI(), source, this, worldData, null); + LazyClipboardHolder clipboard = new LazyClipboardHolder(url.toURI(), source, format, worldData, null); clipboards.add(clipboard); } }