WorldEdit Selection Problem?

Okay so I have been trying to do a selection with worldedit and i did this.

Code:java

    public boolean onCommand(CommandSender sender, Command cmd, String label, String[] args)
    {
        if(cmd.getName().equalsIgnoreCase("rset")){
            if(sender.hasPermission("rb.set")){
                if (sender instanceof Player) {
                    WorldEditPlugin worldEdit = (WorldEditPlugin) Bukkit.getServer().getPluginManager().getPlugin("WorldEdit");
                    Selection selection = WorldEditPlugin.getSelection(Player);
                    if (selection != null) {
                        World world = selection.getWorld();
                        Location min = selection.getMinimumPoint();
                        Location max = selection.getMaximumPoint();
 
                        // Do something with min/max
                    } else {
                        // No selection available
                    }
                }
            }
        }
    }
}

But player is not a variable. What would i do for the variable?
Also how could I do a randomization of the blocks after the you enter the command?

 

PolarCraft
You're using 'Player'. Of course it's not a variable, because you didn't determine it. And because you're spelling it as a class name ("Player"), it even handles it as the Bukkit "Player" class. You would have to retrieve your own player variable after checking if the player is a sender (you're already doing that, so that is fine), like so:

Code:

Player player = (Player) sender;

This, by the way, is some réally basic Java, so I'd advise you getting to know the basics better first.

 

Rprrr ... Already tried that should of said that. Error when doing that.
"Cannot make a static reference to the non-static method getSelection(Player) from the type WorldEditPlugin"

 

PolarCraft Don't use Player in the

Code:

 Selection selection = WorldEditPlugin.getSelection(Player);

use (Player) sender

 

sgavster What would i use then?

 

PolarCraft

 

PolarCraft
You did not read my post (properly) and you do not understand the basics of Java... What I said, does work, you're simply doing it wrong.

 

PolarCraft
Set a variable for Player:

Code:

if(sender instanceof Player) {
    Player player = (Player)sender;
    //code
}

then get the Selection with the variable:

Code:java

Selection sel = WorldEditPlugin.getSelection(player);

 

sgavster AoH_Ruthless
That's what I told him already, but he didn't read properly.

 

Rprrr sgavster AoH_Ruthless Already did that. Hints this.

Code:java

    public boolean onCommand(CommandSender sender, Command cmd, String label, String[] args)
    {
        if(cmd.getName().equalsIgnoreCase("rset")){
            if(sender.hasPermission("rb.set")){
                if (sender instanceof Player) {
                    Player player = (Player) sender;
                    WorldEditPlugin worldEdit = (WorldEditPlugin) Bukkit.getServer().getPluginManager().getPlugin("WorldEdit");
                    Selection selection = WorldEditPlugin.getSelection(player);
                    if (selection != null) {
                        World world = selection.getWorld();
                        Location min = selection.getMinimumPoint();
                        Location max = selection.getMaximumPoint();

 

PolarCraft
Now you're using the 'WorldEditPlugin' class and not the variable of that class (worldEdit). That's probably the mistake (I cannot be entirely sure about this being the issue, because I've never worked with the WorldEdit API, but that would make sense to me).

 

Rprrr Nor have I worked with the api of worldedit. Just looked at there wiki and they said to do this.

 

Rprrr
I know you said that and I know that's what you meant, but even so I found it slightly unclear so I reiterated it in a different manner.

PolarCraft
Call line 7 in your onEnable();
then in line 8 use worldEdit instead of WorldEditPlugin. Also, just use your common sense. Why would you initialize a variable but never use it?

PS: You might want to add checks in your onEnable() to see if WorldEdit is on the server, and if not, disable the plugin.

 

AoH_Ruthless worldEdit cannot be resolved
Also I did this for the check.

Code:java

        WorldEditPlugin worldEdit = (WorldEditPlugin) Bukkit.getServer().getPluginManager().getPlugin("WorldEdit");
        if (worldEdit == null) {
            Bukkit.getLogger().log(Level.SEVERE, "WorldEdit not found!");
            return; 
        }

 

PolarCraft
Wait, my mistake, set it in both the onCommand and onEnable.

 

AoH_Ruthless How could i make a timer that is incorporated into changing blocks within the amount set in the config.yml?

 

PolarCraft
Can you give me an example of what you want? I don't really understand what you are asking.

 

Like when you type /rset it enables a timer which changes the selected blocks to different blocks set in the config.yml.

 

PolarCraft
You need to bring the worldEdit variable into scope, move it outside of the onEnable.

 

xTrollxDudex Already did that. But can you help me with the question at hand?

 

PolarCraft
Loop through the blocks in the selection and get Material.matchMaterial(...) the string from the config

 

xTrollxDudex Now since the worldedit takes min, max how would i set it so it takes both of those?