How would I detect when a player is outside (or standing in Light Level 15)

The title is pretty self explanatory.

The plugin I am trying to code will detect if someone is outside, then ignite them. Kind of like with the zombies.

I also need to know how to only effect one specific player or permissions group, not everyone.

I would like the code and stuff for this, and a little guidance. I am using Eclipse if that makes any difference.

Just so you can have a look at it and fix some stuff, here is my full code:

Note: I may have some unnecessary code in there, like in the imports and the presence of the playerListener. That is from a removed bit of code.

Code:java

package me.jamengulfer221.burnoutside;
 
package me.jamengulfer221.burnoutside;
 
import java.io.File;
import java.io.FileInputStream;
import java.io.FileOutputStream;
import java.io.IOException;
import java.util.Properties;
import java.util.logging.Logger;
 
import org.bukkit.Location;
import org.bukkit.World;
import org.bukkit.block.Block;
import org.bukkit.command.Command;
import org.bukkit.command.CommandSender;
import org.bukkit.entity.Player;
import org.bukkit.event.Event;
import org.bukkit.event.player.PlayerMoveEvent;
import org.bukkit.plugin.PluginManager;
import org.bukkit.plugin.java.JavaPlugin;
 
public class burnoutside extends JavaPlugin{
	Logger log = Logger.getLogger("Minecraft");
	static String mainDirectory = "plugins/BurnOutside";
	static File config = new File(mainDirectory + File.separator + "config.yml");
	static Properties prop = new Properties();
	public void onEnable() {
		log.info("[BurnOutside] v0.1 Has been successfully enabled!");
 
 
 
		PluginManager pm = this.getServer().getPluginManager();
		final BurnOutsidePlayerListener playerListener = new BurnOutsidePlayerListener();
		pm.registerEvent(Event.Type.PLAYER_MOVE, playerListener, Event.Priority.Normal, this);
	}
	public boolean onCommand(CommandSender sender, Command cmd, String commandLabel, String[] args){
		if(cmd.getName().equalsIgnoreCase("protect")){
			int protect = 1;
			return true;
		} //If this has happened the function will break and return true. if this hasn't happened the a value of false will be returned.
		return false;
	}
	public void onPlayerMove(PlayerMoveEvent event) {
		Player player = event.getPlayer();
		Location loc = player.getLocation();
		World world = loc.getWorld();
		Block block = player.getLocation().getBlock();
 
		if (protect == 1){
			if (world.getHighestBlockYAt(loc) < loc.getY()) {
				&& if(block.getLightLevel() == (byte) 15){
					player.setFireTicks(20 * 60 * 1);
				}
			}
		} else {
			return;
		}
}
	public void onDisable(){
		log.info("[BurnOutside] v0.1 has been successfully disabled");
	}
 
		}
 
 

 

I would say do something like get the block at the players feet, and then create an if statement which checks the light level of that block like this:

if(block.getLightLevel == (byte) 15){
//do something
}

 

Just wondering, could you point me to where I could find out how to get the block at the player's feet Thanks.

Also, if there a way to hook onto or use the code zombies use, because I think glowstone emits light at level 15, but if the glowstone and zombie are outside, the zombie isn't burnt

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

 

http://jd.bukkit.org/apidocs/org/bukkit/entity/Player.html

 

To check if the player is outside:

Code:java

Location loc = player.getLocation();
World world = loc.getWorld();
if (world.getHighestBlockY(loc).getLocation().getY() < loc.getY()) {
    //Do stuff
}

To get the block at the player's feet:

Code:java

Block block = player.getLocation().getBlock();

Or do you mean the block under the player's feet?:

Code:java

Block block = player.getLocation().getBlock().getRelative(BlockFace.DOWN);

 

I will try that, thank you I think that almost all of my problems are solved, all I need to do is ignite the player when all of these conditions are true. (Any way of doing that?)

PS: You just got an internet

 

Code:java

player.setFireTicks(20);

That'll set the player on fire for 1 second to make him burn for 1 minute just use 20 * 60 * 1. And for 2 minutes 20 * 60 * 2 etc. I think you know what I mean

PS: Thanks for the internet

 

You're welcome Is there a way I can target only one player with this (12squares) or a permissions group (necromancers), or will it only effect every player?

Also, I am getting a problem with 'player' as in

Code:java

player.getLocation();

It says "player cannot be resolved" and gives me options like make new variable player and things like that.

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

 

Could you give me your whole code? And where you wanna place this?

 

Oh, ok, that should clarify things

Code:java

package me.jamengulfer221.burnoutside;
 
import java.io.File;
import java.io.FileInputStream;
import java.io.FileOutputStream;
import java.io.IOException;
import java.util.Properties;
import java.util.logging.Logger;
 
import org.bukkit.Location;
import org.bukkit.World;
import org.bukkit.block.Block;
import org.bukkit.event.Event;
import org.bukkit.plugin.PluginManager;
import org.bukkit.plugin.java.JavaPlugin;
 
public class burnoutside extends JavaPlugin{
	Logger log = Logger.getLogger("Minecraft");
	static String mainDirectory = "plugins/BurnOutside";
	static File config = new File(mainDirectory + File.separator + "config.yml");
	static Properties prop = new Properties();
	public void onEnable() {
		log.info("[BurnOutside] v0.1 Has been successfully enabled!");
		Location loc = player.getLocation();
		World world = loc.getWorld();
		Block block = player.getLocation().getBlock();
		if (world.getHighestBlockY(loc).getLocation().getY() < loc.getY()) {
		    && if(block.getLightLevel == (byte) 15){
		    	//do something
		    }
		}
 
		PluginManager pm = this.getServer().getPluginManager();
		final BurnOutsidePlayerListener playerListener = new BurnOutsidePlayerListener();
		pm.registerEvent(Event.Type.PLAYER_MOVE, playerListener, Event.Priority.Normal, this);
	}
	public void onDisable(){
		log.info("[BurnOutside] v0.1 has been successfully disabled");
	}
 
		}
 

Also from now on, I will put the code as I update it into the OP

 

Code:java

 
		Location loc = player.getLocation();
		World world = loc.getWorld();
		Block block = player.getLocation().getBlock();
		if (world.getHighestBlockY(loc).getLocation().getY() < loc.getY()) {
		    && if(block.getLightLevel == (byte) 15){
		    	//do something
		    }

That code needs to be in your PLAYER_MOVE event.

And to make "player" work use:

Code:java

Player player = event.getPlayer();

 

Inside my PLAYER_MOVE event? You mean my player listener?

Hmm, now 'event' is throwing up errors. this is getting weird.

Also, 'getHighestBlockY' Is wanting to be changed into 'getHighestBlockAt' or 'getHighestBlockYAt' iI don't know what is causing this, so I am wondering wether I haven't imported something or called something.

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

 

Use getHighestBlockYAt. Everything should look like this:

Code:java

public void onPlayerMove(PlayerMoveEvent event) {
        //Do stuff
}

But replace the //Do stuff with

Code:java

Player player = event.getPlayer();
Location loc = player.getLocation();
if (loc.getWorld().getHighestBlockYAt(loc) < loc.getY()) {
     player.setFireTicks(20 * 4);
}

That'll set the player on fire for 4 seconds if he's outside.

 

Ok, by the way, I want to have a second if statement that will only ignite when the light level is 15 (I.E. Daytime). Would I use

Code:java

Code:java

 
if (world.getHighestBlockYAt(loc).getLocation().getY() < loc.getY()) {
		    && if(block.getLightLevel == (byte) 15){
		    	player.setFireTicks(20 * 60 * 1);
		    }
[FONT=georgia]		}[/FONT]

Because Eclipse is getting annoyed over the && (which I think is AND). I am wondering if that is how I should put it.

Eventually I would like the whole thing to be stopped with a command (they get burned outside unless they put in the command)

Also, how would I do an individual player or a group?

Also, when I change it to GetHighestBlockYAt, it says it "cannot invoke getLocation() on the primitive type int"

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

 

Then remove the getLocation()

Ok to prevent players from burning with a command you should use a HashMap.

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

 

Ok, I will try. But I am only targeting one player. ever. (12squares). Will that still need a HashMap?

Or have I got HashMaps completely wrong?

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

 

HashMaps can store data but not like int or booleans they can store more than 1 player.

So you could check if the player is in the HashMap and just return