Solved Protect some blocks...

Dear Bukkit,
I have created a plugin http://dev.bukkit.org/bukkit-plugins/portabletent , and i am trying to protect the blocks of the tents from breaking.
I was thinking of checking for each block placed if it is inlucded in the config, and if it is, then cancel the break event. But then i realised, What if there are 50+ Tents? Would that lagg the server? So i'd like to ask if you have another way that i can protect the blocks

Kind regards

 

Use worldguard or any other area protection plugin. perhaps implement it into your plugin.

 

Hmm, i saw the worldguard api, but i can't find anything there about to create regions :/

 

maybe:
Get the location of the blocks,
Assign them to the player whose tent is

(BlockBreakEvent)
check if the player who tries to break the blocks, contains the blocks of the location.
if not, cancel the event, otherwise break it.

Or maybe you can use WorldGuard regions.

Regards, Greaperc4

 

Hello there, sorry for late response! As i said, the first thing you suggested would cause a massive lag if the tents are many. It would have to process many locations. For example if each tent was 5x5x5, then it would have to scan 125 locations, per tent. If there were 100 tents, it would be 12500 locations! Which i believe would cause lagg ;P
As for worldguard, i am trying to find how to make regions. This is how far i've come, but i can't yet make the region assigned!

Code:java

Location l1 = new Location(player.getWorld(), bloc.getBlockX(), bloc.getBlockY() - 1, bloc.getBlockZ());
Location l2 = new Location(player.getWorld(), bloc.getBlockX() + y, bloc.getBlockY() + TentsConf.getCustomConfig().getConfigurationSection(Building+".Schema").getKeys(false).size(), bloc.getBlockZ() + x);
player.sendMessage(l1 + " |||||| " + l2);
ProtectedCuboidRegion pr = new ProtectedCuboidRegion(size+"_"+player.getName() , PortableTent.convertToSk89qBV(l1), PortableTent.convertToSk89qBV(l2));
pr.setFlag(new StateFlag("build", false), State.DENY);

The locations that are shown in the player.message are correct to create a Region for protection, The flags are good too i believe, but for some reason, worldguard doesn't add the new region to its config.
Edit: I recieve no errors in the console or ingame if you are wondering

 

Good Job,
I'm going into the WorldGuard API soon,
I really want to but.. no time D:

I just searched really fast, and found this:
https://github.com/sk89q/worldguard...uard/protection/GlobalRegionManager.java#L141

If you do get it to work, (creating a region)

Let us know
Thanks,

Regards Greaperc4

 

I've been searching also a little bit of my own, and i found this PortableTent.worldguardPlugin.getRegionManager(player.getWorld()).addRegion(pr);
I am not sure if it works, i am now trying it out, if it doesn't though, i will use yours. Thanks alot for the help!

 

That should work, I think.
Let me know if that worked,

Regards, Greaperc4

 

It didn't worked alone. It was working good, until i realised it didn't saved the config.
So i just did

Code:java

PortableTent.worldguardPlugin.getRegionManager(player.getWorld()).addRegion(pr);
PortableTent.worldguardPlugin.saveConfig();
try {
    PortableTent.worldguardPlugin.getRegionManager(player.getWorld()).save();
} catch (ProtectionDatabaseException e) {
    // TODO Auto-generated catch block
    e.printStackTrace();
}

And now it works <3
Thanks alot

 

You also set custom data inside the blocks and listen on BlockBreakEvent then check if the block has the custom data and if so cancel the event.

That would eliminate the use of worldguard and also require less plugin downloads in order to use your plugin.

 

What do you mean inster custom data?

 

vasil7112 I do know it is possible to store data in blocks. So, maybe setMetaData(data, data);

Not entirely sure how to do it because ive never tried it before

 

Alright, thanks for letting me know

 

Just wanna help. You can use Block.setMetaData("protected",new FixedMetadata(1,this);
then on the Event just do. If(Block.hasMetadata("protected") event.setCancelled(true);
That will give you minimum lag. BUT remember that all MetaData resets after restart.
So it is not perfect

 

I've actually done it with metadata, but i didn't know that they werent saved on restart! Thats a problem i need to solve now :/

 

Yes, i haved the same problem when i made my Door locking plugin. But i solved it by makeing a ArrayList. But if you gonna protect many blocks it will maybe lagg a lot. :/ If you have a answer please post it here so everyone knows how to fix it

 

Well, Probably what i will do is isntead of lagging while playing, i will make it load all the blocks on the plugin enable. So for example it might take some time to load, but atleast they won't require external plugins for protection. Still though, i will offer an option between WorldGuard Protection And MetaData

 

Add the blocks to a config or a sqlite db

 

Well, i will stick with worldguard for now, and see how the users handle it. Later, i will also add the Metadata option(which will be saved on a config)