World Guard support

I made a plugin but people can break in worldguard protected areas. Can anyone help me out, I have checked the worldguard api it didn't help I hate to ask this but anyone have any code?

This is my current code:

Code:java

public static boolean canBuild(Location loc){
        boolean canBuild = false;
        RegionManager rm = WorldGuard.getRegionManager(loc.getWorld());
        ApplicableRegionSet region = rm.getApplicableRegions(loc);
        if((region.getFlag(DefaultFlag.BUILD) != null) && ((StateFlag.State)region.getFlag(DefaultFlag.BUILD)).equals(StateFlag.State.ALLOW)){
            canBuild = true;
        }else if(region.getFlag(DefaultFlag.BUILD) != null){
            canBuild = false;
        }else{
            canBuild = true;
        }
        System.out.println("" + canBuild);
        return canBuild;
    }

Code:java

if(!WorldGuard.canBuild(e.getBlock().getLocation())){
    e.getBlock().getDrops().clear();
        ItemStack drops = new ItemStack(Material.IRON_INGOT);
    player.getInventory().addItem(drops);
    e.setBlock().setType(Material.AIR);

Tried true and false didn't work.

 

Bump

 

TotallyNotWalshy JordyPwner Normally the build flag of a region should not be set, building is disabled by default for all players that are not an owner/member of the region. So that is why you method of checking does not work. In one of my plugins I wanted to check if a player was in PVP area, I did this with the following:

Code:java

RegionManager manager = worldGuard.getRegionManager(Bukkit.getWorld("world"));
ApplicableRegionSet regions = manager.getApplicableRegions(player.getLocation());
boolean pvpAllowed = regions.allows(DefaultFlag.PVP, worldGuard.wrapPlayer(player));

You already have the first two lines, after that you need the allows() method to check if the set of regions allows the specified action. You need to use DefaultFlag.BUILD instead of PVP, it also seems that you don't have access to the player object, you will need that for using the allows() method, so you probably want to change the Location argument to a Player argument.

 

nlthijs48 This is my code

Code:java

public static boolean canBuild(Player player){
        Location loc = player.getLocation();
        RegionManager manager = WorldGuard.getRegionManager(loc.getWorld()); //NPE!!!
        ApplicableRegionSet regions = manager.getApplicableRegions(loc);
        boolean canBuild = regions.allows(DefaultFlag.BUILD, WorldGuard.wrapPlayer(player));
 
        return canBuild;
    }

But it is throwing a npe, any idea how to fix?

 

TotallyNotWalshy

Debug the NPE.

 

TotallyNotWalshy Are you sure it is that line causing the NPE? If it is then loc is null, which is not possible if I'm correct or WorldGuard is null, which is more likely. Assuming WorldGuard is a variable (named incorrectly, should be starting with a lowercase character) then that is null. If you show the complete class I can help you more with it.

 

nlthijs48

Code:

28) [custom.jar:git-Bukkit-1.7.9-R0.1-10-g8688bd4-b3092jnks]
Caused by: java.lang.NullPointerException
        at com.murderxtreme.test.WorldGuard.canBuild(WorldGuard.java:17) ~[?
:?]
        at com.murderxtreme.test.Dummy.onWorldTest(Dummy.java:14) ~[?:?]
        at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method) ~[?:1.7.0
_51]
        at sun.reflect.NativeMethodAccessorImpl.invoke(Unknown Source) ~[?:1.7.0
_51]
        at sun.reflect.DelegatingMethodAccessorImpl.invoke(Unknown Source) ~[?:1
.7.0_51]
        at java.lang.reflect.Method.invoke(Unknown Source) ~[?:1.7.0_51]
        at org.bukkit.plugin.java.JavaPluginLoader$1.execute(JavaPluginLoader.ja
va:292) ~[custom.jar:git-Bukkit-1.7.9-R0.1-10-g8688bd4-b3092jnks]
 
        ... 15 more

 

TotallyNotWalshy Can you post the code of WorldGuard.java?

 

nlthijs48

Code:java

package com.murderxtreme.test;
 
import org.bukkit.Location;
import org.bukkit.entity.Player;
 
import com.sk89q.worldguard.bukkit.WorldGuardPlugin;
import com.sk89q.worldguard.protection.ApplicableRegionSet;
import com.sk89q.worldguard.protection.flags.DefaultFlag;
import com.sk89q.worldguard.protection.managers.RegionManager;
 
public class WorldGuard {
 
    public static WorldGuardPlugin WorldGuard;
 
    public static boolean canBuild(Player player){
        Location loc = player.getLocation();
        RegionManager manager = WorldGuard.getRegionManager(loc.getWorld()); //NPE!!!
        ApplicableRegionSet regions = manager.getApplicableRegions(loc);
        boolean canBuild = regions.allows(DefaultFlag.BUILD, WorldGuard.wrapPlayer(player));
 
        return canBuild;
    }
}
 

 

TotallyNotWalshy Your WorldGuard variable is null, you have never assigned something to it. And you still need to change the name of that to 'worldGuard', following the conventions.

 

nlthijs48 Ah I am super dumb the class and the WorldGuard worldGuard class are the same, that's probably why.

So what should I assign it to?

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

 

TotallyNotWalshy You need to get the WorldGuardPlugin instance through Bukkit, check this page to see how that works. Because you don't have a reference to your main class you need to use 'Bukkit.getServer()' instead of 'getServer()' or give a reference to your main class with the constructor.