Error while copying world folder

I'm getting an internal error on the command. I'm trying to copy the world folder (CollideTemplate) and paste it with the name of the player.

My code:

Code:java

 
                    if (args.length == 1) {
                        if (args[0].equalsIgnoreCase("start")) {
                            if (!player.hasPermission("collide.start")) {
                                getLogger().fine("Player " + player.getName() + " has no collide.start permission.");
                                player.sendMessage(ChatColor.BLUE + "[Collide]" + ChatColor.RED + " You do not have permission to use this command!");
                                return false;
                            }
                            World template = Bukkit.getWorld("CollideTemplate");
                            World playerworld = Bukkit.getWorld(player.getName() + "-world");
                            String templateName = "CollideTemplate";//enter template world's name
                            String newWorldName = player.getName() + "-world";//name of the world thats going to be made
 
                            File serverFolder = new File(System.getProperty("user.dir"));//server directory
                            File templateFolder = new File(serverFolder + File.separator + templateName);//template world directory
                            File newWorldFolder = new File(serverFolder + File.separator + newWorldName);//new player world directory
 
                            if (template!=null) {Bukkit.unloadWorld(template, true);}
                            if (template==null) {player.sendMessage(ChatColor.BLUE + "[Collide] " + ChatColor.RED + "ERROR #101: Unable to find base template world! Report to a MUOSU staff member." + " Current world: " + ChatColor.GOLD + player.getWorld().getName()); return false;}
 
                            if (!(playerworld==null)){
                                sender.sendMessage(ChatColor.YELLOW + "" + ChatColor.GOLD + "" + ChatColor.YELLOW + "");
                                sender.sendMessage(ChatColor.YELLOW + "---" + ChatColor.GOLD + "Collide Base Generation " + ChatColor.GREEN + "In Process!" + ChatColor.YELLOW + "---");
                                sender.sendMessage(ChatColor.BLUE + "[Collide] " + ChatColor.RED + "ERROR #102");
                                sender.sendMessage(ChatColor.BLUE + "[Collide] " + ChatColor.RED + "You already have a base!");
                                sender.sendMessage(ChatColor.BLUE + "[Collide] " + ChatColor.RED + "Use /collide home or /collide delete.");
                                sender.sendMessage(ChatColor.YELLOW + "---" + ChatColor.GOLD + "Collide Base Generation " + ChatColor.RED + "Failed!" + ChatColor.YELLOW + "---");
                                return false;
                            }            
 
                            sender.sendMessage(ChatColor.YELLOW + "" + ChatColor.GOLD + "" + ChatColor.YELLOW + "");
                            sender.sendMessage(ChatColor.YELLOW + "---" + ChatColor.GOLD + "Collide Base Generation " + ChatColor.GREEN + "In Process!" + ChatColor.YELLOW + "---");
                            sender.sendMessage(ChatColor.BLUE + "[Collide] " + ChatColor.RED + "Creating folders... Please wait!");//create folders
 
                            if (!newWorldFolder.exists()){
                            newWorldFolder.mkdir();
                            new File(newWorldFolder.getPath() + File.separator + "data").mkdirs();
                            new File(newWorldFolder.getPath() + File.separator + "players").mkdirs();
                            new File(newWorldFolder.getPath() + File.separator + "region").mkdirs();
                            }
 
                            sender.sendMessage(ChatColor.BLUE + "[Collide] " + ChatColor.RED + "Copying world data... Please wait!");//copy files into the folders
                            try {
                            Files.copy(new File(templateFolder.getPath() + File.separator + "data"), new File(newWorldFolder.getPath() + File.separator + "data"));
                            Files.copy(new File(templateFolder.getPath() + File.separator + "region"), new File(newWorldFolder.getPath() + File.separator + "region"));
                            Files.copy(templateFolder, newWorldFolder);
                            } catch (IOException e) {
                            e.printStackTrace();
                            sender.sendMessage(ChatColor.BLUE + "[Collide] " + ChatColor.RED + "ERROR! Please wait!");
                            sender.sendMessage(ChatColor.BLUE + "[Collide] " + ChatColor.RED + "There was an error on world copy! Please report the following to staff:");
                            getLogger().info("Server: "+serverFolder.getPath());
                            getLogger().info("From: "+templateFolder.getPath());
                            getLogger().info("To: "+newWorldFolder.getPath());
                            sender.sendMessage(ChatColor.BLUE + "[Collide] " + ChatColor.GOLD + "Server: " + ChatColor.RED + serverFolder.getPath());
                            sender.sendMessage(ChatColor.BLUE + "[Collide] " + ChatColor.GOLD + "From: " + ChatColor.RED + templateFolder.getPath());
                            sender.sendMessage(ChatColor.BLUE + "[Collide] " + ChatColor.GOLD + "To: " + ChatColor.RED + newWorldFolder.getPath());
                            sender.sendMessage(ChatColor.YELLOW + "---" + ChatColor.GOLD + "Collide Base Generation " + ChatColor.RED + "Failed!" + ChatColor.YELLOW + "---");
                            return true;//ends command on error
                            }
 
                            sender.sendMessage(ChatColor.BLUE + "[Collide] " + ChatColor.RED + "Preventing problems... Please wait!");//remove the copied id so it can load properly
                            File CopiedWorldUID = new File(newWorldFolder + File.separator + "uid.dat");
                            if (CopiedWorldUID.exists()) {CopiedWorldUID.delete();}
 
                            sender.sendMessage(ChatColor.BLUE + "[Collide] " + ChatColor.RED + "Loading world... Please wait!");//tells Bukkit to create the world with the same data as the template
                            Bukkit.createWorld(new WorldCreator(templateName).copy(template));//this will let Bukkit load it and have the same seed for generation
 
 
                            sender.sendMessage(ChatColor.BLUE + "[Collide] " + ChatColor.RED + "Your base has been created!");
                            sender.sendMessage(ChatColor.BLUE + "[Collide] " + ChatColor.RED + "Use /collide home.");
                            sender.sendMessage(ChatColor.YELLOW + "---" + ChatColor.GOLD + "Collide Base Generation " + ChatColor.GREEN + "Complete!" + ChatColor.YELLOW + "---");
                        }
                    }
 

Error:

Code:

2013-07-09 13:13:34 [INFO] Thunder_Waffe issued server command: /collide start
2013-07-09 13:13:35 [SEVERE] java.io.FileNotFoundException: C:\Users\Toshiba\Documents\zServers\Collide Server\CollideTemplate\data (Access is denied)
2013-07-09 13:13:35 [SEVERE]     at java.io.FileInputStream.open(Native Method)
2013-07-09 13:13:35 [SEVERE]     at java.io.FileInputStream.<init>(Unknown Source)
2013-07-09 13:13:35 [SEVERE]     at com.google.common.io.Files$1.getInput(Files.java:100)
2013-07-09 13:13:35 [SEVERE]     at com.google.common.io.Files$1.getInput(Files.java:97)
2013-07-09 13:13:35 [SEVERE]     at com.google.common.io.ByteStreams.copy(ByteStreams.java:116)
2013-07-09 13:13:35 [SEVERE]     at com.google.common.io.Files.copy(Files.java:231)
2013-07-09 13:13:35 [SEVERE]     at com.google.common.io.Files.copy(Files.java:277)
2013-07-09 13:13:35 [SEVERE]     at com.gmail.thunderwaffemc.Collide.onCommand(Collide.java:238)
2013-07-09 13:13:35 [SEVERE]     at org.bukkit.command.PluginCommand.execute(PluginCommand.java:44)
2013-07-09 13:13:35 [SEVERE]     at org.bukkit.command.SimpleCommandMap.dispatch(SimpleCommandMap.java:189)
2013-07-09 13:13:35 [SEVERE]     at org.bukkit.craftbukkit.v1_6_R2.CraftServer.dispatchCommand(CraftServer.java:523)
2013-07-09 13:13:35 [SEVERE]     at net.minecraft.server.v1_6_R2.PlayerConnection.handleCommand(PlayerConnection.java:964)
2013-07-09 13:13:35 [SEVERE]     at net.minecraft.server.v1_6_R2.PlayerConnection.chat(PlayerConnection.java:882)
2013-07-09 13:13:35 [SEVERE]     at net.minecraft.server.v1_6_R2.PlayerConnection.a(PlayerConnection.java:839)
2013-07-09 13:13:35 [SEVERE]     at net.minecraft.server.v1_6_R2.Packet3Chat.handle(SourceFile:49)
2013-07-09 13:13:35 [SEVERE]     at net.minecraft.server.v1_6_R2.NetworkManager.b(NetworkManager.java:296)
2013-07-09 13:13:35 [SEVERE]     at net.minecraft.server.v1_6_R2.PlayerConnection.e(PlayerConnection.java:118)
2013-07-09 13:13:35 [SEVERE]     at net.minecraft.server.v1_6_R2.ServerConnection.b(SourceFile:37)
2013-07-09 13:13:35 [SEVERE]     at net.minecraft.server.v1_6_R2.DedicatedServerConnection.b(SourceFile:30)
2013-07-09 13:13:35 [SEVERE]     at net.minecraft.server.v1_6_R2.MinecraftServer.t(MinecraftServer.java:590)
2013-07-09 13:13:35 [SEVERE]     at net.minecraft.server.v1_6_R2.DedicatedServer.t(DedicatedServer.java:226)
2013-07-09 13:13:35 [SEVERE]     at net.minecraft.server.v1_6_R2.MinecraftServer.s(MinecraftServer.java:486)
2013-07-09 13:13:35 [SEVERE]     at net.minecraft.server.v1_6_R2.MinecraftServer.run(MinecraftServer.java:419)
2013-07-09 13:13:35 [SEVERE]     at net.minecraft.server.v1_6_R2.ThreadServerApplication.run(SourceFile:582)
2013-07-09 13:13:35 [INFO] [Collide] Server: C:\Users\Toshiba\Documents\zServers\Collide Server
2013-07-09 13:13:35 [INFO] [Collide] From: C:\Users\Toshiba\Documents\zServers\Collide Server\CollideTemplate
2013-07-09 13:13:35 [INFO] [Collide] To: C:\Users\Toshiba\Documents\zServers\Collide Server\Thunder_Waffe-world

Any help is appreciated! Thanks!

 

There's a runtime file lock on certain files and folders used by CraftBukkit, including worlds. You can't edit or move world directories after the worlds have been loaded, which occurs before your plugin's onEnable() method is called. Override the onLoad() method and put your file operations there instead.

 

> (Access is denied)

CollideTemplate/data is a directory, you cannot open a directory with FileInputStream.

 

LinearLogic How can I do this?
tills13 Any fix for this?

 

well ... just throwing an idea out there, but:

Code:java

public void copy(File curDir, String destination) {
    for (File file : curDir.listFileS()) {
        if (file.isDirectory()) copy(file, destination + file.getName());
        else Files.copy(file.getAbsolutePath(), new FileOutputStream(new File(destination + file.getName())));
    }   
}

I believe that will work.

 

If you aren't actually editing the world folder or its contents, the above method should suffice. Otherwise, put the same method in onLoad() instead of onEnable() to circumnavigate the file lock issue.

 

That doesn't actually work...

This does, though:

Code:java

public static void copy(File curDir, String destination) {
        try {
            for (File file : curDir.listFiles()) {
                if (file.isDirectory()) {
                    if (!new File(destination + File.separator + file.getName()).exists()) new File(destination + File.separator + file.getName()).mkdir();
                    copy(file, destination + File.separator + file.getName());
                } else {
                    Path path = FileSystems.getDefault().getPath(curDir.getPath(), file.getName());
                    if (!new File(destination).exists()) new File(destination).mkdir();
                    Files.copy(path, new FileOutputStream(new File(destination + File.separator + file.getName())));
                }
            }   
        } catch (FileNotFoundException e) {
            System.out.println("File not found: " + e.getMessage());
        } catch (IOException e) {
            System.out.println("IO error: " + e.getMessage());
        }
    }

 

I get an error in eclipse on "copy" from:

Code:java

 
else Files.copy(file.getAbsolutePath(), new FileOutputStream(new File(destination + file.getName())));
 

Saying: The method copy(InputSupplier<? extends InputStream>, File) in the type Files is not applicable for the arguments (String, FileOutputStream)

Any idea?

 

I just posted an update.

Import these:

import java.io.File;
import java.io.FileOutputStream;
import java.io.FileNotFoundException;
import java.io.IOException;
import java.nio.file.Files;
import java.nio.file.Path;
import java.nio.file.FileSystems;

EDIT by Moderator: merged posts, please use the edit button instead of double posting.

 

If that doesn't work (but it looks just fine), below is what I'm using in a number of my plugins. It simply copies file contents use a byte buffer instead of the java.nio utils (I like to minimize imports and dependencies where I can), but both should perform on par.

Code:

private static void copyDir(File source, File target) throws IOException {
    if(source.isDirectory()) {
        if(!target.exists())
          target.mkdirs();
        String files[] = source.list();
        for (String file : files) {
          File srcFile = new File(source, file);
          File destFile = new File(target, file);
          copyDir(srcFile, destFile);
        }
    } else {
        InputStream in = new FileInputStream(source);
        OutputStream out = new FileOutputStream(target);
        byte[] buffer = new byte[1024];
        int length;
        //copy the file content in bytes
        while ((length = in.read(buffer)) > 0)
            out.write(buffer, 0, length);
        in.close();
        out.close();
    }
}

 

But I want to copy it on command? Is there any way to fix my code without having to replace it or doing something entirely new?

 

Are you trying to copy the world folder elsewhere?

 

No. I'm trying to copy the template (CollideTemplate) from my bukkit worlds and paste it in the same directory but rename it as <PlayerName>-world as seen in my code.

PS. I appreciate the help.

 

Yeah. So you would just replace this block in yours:

Code:java

try {
  Files.copy(new File(templateFolder.getPath() + File.separator + "data"), new                      File(newWorldFolder.getPath() + File.separator + "data"));
    Files.copy(new File(templateFolder.getPath() + File.separator + "region"), new   File(newWorldFolder.getPath() + File.separator + "region"));
 Files.copy(templateFolder, newWorldFolder);
} catch (IOException e) {
 e.printStackTrace();
 sender.sendMessage(ChatColor.BLUE + "[Collide] " + ChatColor.RED + "ERROR! Please wait!");
 sender.sendMessage(ChatColor.BLUE + "[Collide] " + ChatColor.RED + "There was an error on  world copy! Please report the following to staff:");
 getLogger().info("Server: "+serverFolder.getPath());
 getLogger().info("From: "+templateFolder.getPath());
 getLogger().info("To: "+newWorldFolder.getPath());
 sender.sendMessage(ChatColor.BLUE + "[Collide] " + ChatColor.GOLD + "Server: " + ChatColor.RED + serverFolder.getPath());
 sender.sendMessage(ChatColor.BLUE + "[Collide] " + ChatColor.GOLD + "From: " + ChatColor.RED + templateFolder.getPath());
 sender.sendMessage(ChatColor.BLUE + "[Collide] " + ChatColor.GOLD + "To: " + ChatColor.RED + newWorldFolder.getPath());
 sender.sendMessage(ChatColor.YELLOW + "---" + ChatColor.GOLD + "Collide Base Generation " + ChatColor.RED + "Failed!" + ChatColor.YELLOW + "---");
 return true;//ends command on error
}

with a simple call to either:

Code:

copy(templateFolder, newWorldFolder.getPath());

or

Code:

copyDir(templateFolder, newWorldFolder.);

depending on which method you'd like to use.

 

ThunderWaffeMC, I went ahead and wrote a plugin that achieves what I think you're trying to do (when I run /collide start, it creates a world folder in my server directory called LinearLogic-world, based off of a world folder called TemplateWorld). I cut out some of what you had for expediency. Let me know if this helps:

Code:java

public class Collide extends JavaPlugin {
 
    public boolean onCommand(CommandSender sender, Command cmd, String label, String[] args) {
        if (!cmd.getName().equalsIgnoreCase("collide"))
            return true;
        if (args.length == 1 && args[0].equalsIgnoreCase("start")) {
            if (!(sender instanceof Player)) {
                sender.sendMessage(ChatColor.RED + "This command is only available to players!");
                return true;
            }
 
            Player player = (Player) sender;
            if (!player.hasPermission("collide.start")) {
                getLogger().fine("Player " + player.getName() + " has no collide.start permission.");
                player.sendMessage(ChatColor.BLUE + "[Collide]" + ChatColor.RED + " You do not have permission to use this command!");
                return true;
            }
 
            // As far as I can tell, you only retrieve the World objects in order to unload them,
            // which won't alleviate the issue anyway, so I skip straight to the file operations:
            File templateWorldFolder = new File(getServer().getWorldContainer(), "world");
            if (!templateWorldFolder.exists() || !templateWorldFolder.isDirectory()) {
                // The template world is missing - abort the operation
                player.sendMessage(ChatColor.RED + "Could not locate the template world - contact your administrator.");
                return true;
            }
            File playerWorldFolder = new File(getServer().getWorldContainer(), player.getName() + "-world");
            if (playerWorldFolder.exists()) {
                // The world has already been created
                player.sendMessage(ChatColor.RED + "You already have a base!");
                return true;
            }
            try {
                copyDir(templateWorldFolder, playerWorldFolder);
            } catch (IOException e) {
                player.sendMessage(ChatColor.RED + "Failed to create base world - error whilse copying template directory!");
                return true;
            }
            player.sendMessage(ChatColor.GREEN + "Successfully created your base world!");
            return true;
        }
        // Handle other subcommands here...
        return true;
    }
 
    private static void copyDir(File source, File target) throws IOException {
        if(source.isDirectory()) {
            if(!target.exists())
              target.mkdirs();
            String files[] = source.list();
            for (String file : files) {
              File srcFile = new File(source, file);
              File destFile = new File(target, file);
              copyDir(srcFile, destFile);
            }
        } else {
            InputStream in = new FileInputStream(source);
            OutputStream out = new FileOutputStream(target);
            byte[] buffer = new byte[1024];
            int length;
            //copy the file content in bytes
            while ((length = in.read(buffer)) > 0)
                out.write(buffer, 0, length);
            in.close();
            out.close();
        }
    }
}

 

WOW! Thanks so much! Is there a way to not copy some files? I only want to copy the 3 folders inside the world folder (data, region and players) and not the uid.dat. Or I could delete the uid.dat after creation but is there a way to not copy the uid.dat?

 

Here's my method modified to account for something like that:

Code:

public static void copy(File curDir, String destination) {
        ArrayList<String> ignore = new ArrayList<String>(Arrays.asList("uid.dat", "level.dat", "session.dat"));
        try {
            for (File file : curDir.listFiles()) {
                System.out.println(file.getName());
                if (!ignore.contains(file.getName())) {
                    if (file.isDirectory()) {
                        if (!new File(destination + File.separator + file.getName()).exists()) new File(destination + File.separator + file.getName()).mkdir();
                        copy(file, destination + File.separator + file.getName());
                    } else {
                        Path path = FileSystems.getDefault().getPath(curDir.getPath(), file.getName());
                        if (!new File(destination).exists()) new File(destination).mkdir();
                        Files.copy(path, new FileOutputStream(new File(destination + File.separator + file.getName())));
                    }
                }
            }   
        } catch (FileNotFoundException e) {
            System.out.println("File not found: " + e.getMessage());
        } catch (IOException e) {
            System.out.println("IO error: " + e.getMessage());
        }
    }

Notice the ArrayList line with the files to ignore and the if statement ignore.contains(file.getName())

It works with directories, as well.

Here's LinearLogic's function modified:

Code:

private static void copyDir(File source, File target) {
        try {
            ArrayList<String> ignore = new ArrayList<String>(Arrays.asList("uid.dat", "level.dat", "session.dat", "data"));
            if (!ignore.contains(source.getName())) {
                if(source.isDirectory()) {
                    if(!target.exists())
                      target.mkdirs();
                    String files[] = source.list();
                    for (String file : files) {
                      File srcFile = new File(source, file);
                      File destFile = new File(target, file);
                      copyDir(srcFile, destFile);
                    }
                } else {
                    InputStream in = new FileInputStream(source);
                    OutputStream out = new FileOutputStream(target);
                    byte[] buffer = new byte[1024];
                    int length;
                    //copy the file content in bytes
                    while ((length = in.read(buffer)) > 0)
                        out.write(buffer, 0, length);
                    in.close();
                    out.close();
                }
            }
        } catch (IOException e) {
 
        }
    }

EDIT by Moderator: merged posts, please use the edit button instead of double posting.

 

Thanks very much! Much appreciated!