Help i have an error i cant solve

im creating a protect player on teleport plugin but i am getting the can not pass on PleyerTeleportEvent error
here is my code for the main class

Code:java

public void onEnable(){
        getLogger().info(prefix+"PVPDelay Enabling");
        getConfig().options().copyDefaults(true);
        saveConfig();
        delay=getConfig().getInt("Seconds", delay);
        Bukkit.broadcastMessage(prefix+"PvpDelay Enabled");
        Bukkit.getServer().getPluginManager().registerEvents(new Pvpdelaylistener(this), this);

and here's the code of the listener class

Code:java

public class Pvpdelaylistener implements Listener {
 
    public static int delay;
    public static Player player;
    public static Entity damager;
      private PvpDelaystart plugin;
      public Pvpdelaylistener(PvpDelaystart instance){
          this.plugin = instance;
      }
 
    @SuppressWarnings("unused")
    @EventHandler(priority = EventPriority.LOW)
    public void onPlayerTeleport(final PlayerTeleportEvent e){
        Player player= ((Player)e).getPlayer();
        delay=plugin.getConfig().getInt("Seconds:",delay);
        Bukkit.broadcastMessage(prefix+"Got player teleport");
 
  }
    @EventHandler(priority = EventPriority.HIGH)
    public void OnPlayerDamage(final EntityDamageByEntityEvent event){
        Entity damager= event.getDamager();
        Bukkit.broadcastMessage(prefix+"Got all varriables");
        while(delay>0){
            if(player.hasPermission("pvpdelay.protected")||player.isOp()){
                player.sendMessage(prefix+"Your protected");
                Bukkit.broadcastMessage(prefix+"entered loop");
                        player.sendMessage(prefix +"You Cant pvp for another"+delay+"Seconds");
                        event.setCancelled(true);
                        player.sendMessage(prefix+"You have "+delay+"until you can take damage");
                        if(damager instanceof Player){
                            event.setCancelled(true);
                            ((Player) damager).sendMessage(prefix+"You are under Pvpdelay Protection you may mot pvp for anoter "+delay+" Seconds");
                        }
                        Bukkit.broadcastMessage(prefix+"exeted loops");
        }
            delay--;
      }
    }
 
}
 

 

You cast the event to a Player...

Code:java

Player player= ((Player)e).getPlayer();

 

Thank you i must have read over that 25 times and never caught it

Superckl1
any idea why im getting cant pass on entitydamagebyentityevent now? some other mistake i cant see when i read over this

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

 

You'll have to post the error here.

 

Superckl1
I get this error

Code:

[19:04:32 ERROR]: Could not pass event EntityDamageByEntityEvent to PvpDelay v1.
1
org.bukkit.event.EventException
        at org.bukkit.plugin.java.JavaPluginLoader$1.execute(JavaPluginLoader.ja
va:320) ~[craftbukkit.jar:git-Bukkit-1.7.2-R0.2-15-g2f3dbd3-b2995jnks]
        at org.bukkit.plugin.RegisteredListener.callEvent(RegisteredListener.jav
a:62) ~[craftbukkit.jar:git-Bukkit-1.7.2-R0.2-15-g2f3dbd3-b2995jnks]
        at org.bukkit.plugin.SimplePluginManager.fireEvent(SimplePluginManager.j
ava:481) [craftbukkit.jar:git-Bukkit-1.7.2-R0.2-15-g2f3dbd3-b2995jnks]
        at org.bukkit.plugin.SimplePluginManager.callEvent(SimplePluginManager.j
ava:466) [craftbukkit.jar:git-Bukkit-1.7.2-R0.2-15-g2f3dbd3-b2995jnks]
        at org.bukkit.craftbukkit.v1_7_R1.event.CraftEventFactory.callEvent(Craf
tEventFactory.java:98) [craftbukkit.jar:git-Bukkit-1.7.2-R0.2-15-g2f3dbd3-b2995j
nks]
        at org.bukkit.craftbukkit.v1_7_R1.event.CraftEventFactory.callEntityDama
geEvent(CraftEventFactory.java:386) [craftbukkit.jar:git-Bukkit-1.7.2-R0.2-15-g2
f3dbd3-b2995jnks]
        at org.bukkit.craftbukkit.v1_7_R1.event.CraftEventFactory.handleEntityDa
mageEvent(CraftEventFactory.java:414) [craftbukkit.jar:git-Bukkit-1.7.2-R0.2-15-
g2f3dbd3-b2995jnks]
        at net.minecraft.server.v1_7_R1.EntityLiving.damageEntity(EntityLiving.j
ava:656) [craftbukkit.jar:git-Bukkit-1.7.2-R0.2-15-g2f3dbd3-b2995jnks]
        at net.minecraft.server.v1_7_R1.EntityHuman.damageEntity(EntityHuman.jav
a:746) [craftbukkit.jar:git-Bukkit-1.7.2-R0.2-15-g2f3dbd3-b2995jnks]
        at net.minecraft.server.v1_7_R1.EntityPlayer.damageEntity(EntityPlayer.j
ava:446) [craftbukkit.jar:git-Bukkit-1.7.2-R0.2-15-g2f3dbd3-b2995jnks]
        at net.minecraft.server.v1_7_R1.EntityArrow.h(EntityArrow.java:228) [cra
ftbukkit.jar:git-Bukkit-1.7.2-R0.2-15-g2f3dbd3-b2995jnks]
        at net.minecraft.server.v1_7_R1.World.entityJoinedWorld(World.java:1338)
[craftbukkit.jar:git-Bukkit-1.7.2-R0.2-15-g2f3dbd3-b2995jnks]
        at net.minecraft.server.v1_7_R1.World.playerJoinedWorld(World.java:1319)
[craftbukkit.jar:git-Bukkit-1.7.2-R0.2-15-g2f3dbd3-b2995jnks]
        at net.minecraft.server.v1_7_R1.World.tickEntities(World.java:1207) [cra
ftbukkit.jar:git-Bukkit-1.7.2-R0.2-15-g2f3dbd3-b2995jnks]
        at net.minecraft.server.v1_7_R1.WorldServer.tickEntities(WorldServer.jav
a:480) [craftbukkit.jar:git-Bukkit-1.7.2-R0.2-15-g2f3dbd3-b2995jnks]
        at net.minecraft.server.v1_7_R1.MinecraftServer.u(MinecraftServer.java:6
37) [craftbukkit.jar:git-Bukkit-1.7.2-R0.2-15-g2f3dbd3-b2995jnks]
        at net.minecraft.server.v1_7_R1.DedicatedServer.u(DedicatedServer.java:2
50) [craftbukkit.jar:git-Bukkit-1.7.2-R0.2-15-g2f3dbd3-b2995jnks]
        at net.minecraft.server.v1_7_R1.MinecraftServer.t(MinecraftServer.java:5
45) [craftbukkit.jar:git-Bukkit-1.7.2-R0.2-15-g2f3dbd3-b2995jnks]
        at net.minecraft.server.v1_7_R1.MinecraftServer.run(MinecraftServer.java
:457) [craftbukkit.jar:git-Bukkit-1.7.2-R0.2-15-g2f3dbd3-b2995jnks]
        at net.minecraft.server.v1_7_R1.ThreadServerApplication.run(SourceFile:6
17) [craftbukkit.jar:git-Bukkit-1.7.2-R0.2-15-g2f3dbd3-b2995jnks]
Caused by: java.lang.NullPointerException
        at us.Cheapcoders.PvpDelay.Pvpdelaylistener.OnPlayerDamage(Pvpdelayliste
ner.java:36) ~[?:?]
        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:318) ~[craftbukkit.jar:git-Bukkit-1.7.2-R0.2-15-g2f3dbd3-b2995jnks]
        ... 19 more

 

GhostHack your 'delay' is always null, so it gives a NPE.

 

Primitive integers can't be null...

GhostHack
Can you post the whole class, imports and everything? We need the code on line 36.

 

Superckl1 The EntityDamageEventByEntity could be fired before the PlayerTeleportEvent, causing it to be null. Also, there could be no value in the config.

 

my main class

Code:java

 
 
import org.bukkit.command.Command;
import org.bukkit.command.CommandSender;
import org.bukkit.entity.Entity;
import org.bukkit.entity.Player;
import org.bukkit.event.EventHandler;
import org.bukkit.event.EventPriority;
import org.bukkit.event.entity.EntityDamageByEntityEvent;
import org.bukkit.event.entity.EntityTeleportEvent;
import org.bukkit.Bukkit;
import org.bukkit.ChatColor;
import org.bukkit.OfflinePlayer;
import org.bukkit.plugin.PluginManager;
import org.bukkit.plugin.java.JavaPlugin;
 
 
 
@SuppressWarnings("unused")
public class PvpDelaystart extends JavaPlugin {
    public static String prefix = (ChatColor.AQUA+"[pvp"+ChatColor.RED+"delay]"+ChatColor.GREEN);
    public static int delay;
    public static boolean over;
 
 
    public void onEnable(){
        getLogger().info(prefix+"PVPDelay Enabling");
        getConfig().options().copyDefaults(true);
        saveConfig();
        delay=getConfig().getInt("Seconds", delay);
        Bukkit.broadcastMessage(prefix+"PvpDelay Enabled");
        Bukkit.getServer().getPluginManager().registerEvents(new Pvpdelaylistener(this), this);
        getLogger().info(" "+delay);
    }
    public void onDisable(){
        getLogger().info(prefix+"PvpDelay Disabled");
    }
 
    public static String help;
    @Override
    public boolean onCommand(CommandSender sender, Command cmd, String label, String[] args){
        if(sender.hasPermission("pvpdelay.reload")||sender.isOp()){
            if(cmd.getName().equalsIgnoreCase("pdreload")){
                reloadConfig();
                sender.sendMessage(prefix+"Config.yml reloaded");
                getLogger().info(prefix+"Pvpdelay Config.yml reloaded");
                return true;
            }
        }
        return false;
    }
 
}
 
 
 

there is a value in the config it is set at 15 i checked that
listener class

Code:java

 
 
import org.bukkit.ChatColor;
import org.bukkit.entity.Entity;
import org.bukkit.entity.Player;
import org.bukkit.event.EventHandler;
import org.bukkit.event.EventPriority;
import org.bukkit.event.Listener;
import org.bukkit.event.entity.EntityDamageByEntityEvent;
import org.bukkit.event.player.PlayerTeleportEvent;
 
public class Pvpdelaylistener implements Listener {
    public static String prefix = (ChatColor.AQUA+"[pvp"+ChatColor.RED+"delay]"+ChatColor.GREEN);
    public static int delay;
    public static Player player;
    public static Entity damager;
    public static boolean protect;
      private PvpDelaystart plugin;
      public Pvpdelaylistener(PvpDelaystart instance){
          this.plugin = instance;
      }
 
    @SuppressWarnings("unused")
    @EventHandler(priority = EventPriority.LOW)
    public void onPlayerTeleport(final PlayerTeleportEvent e){
        Player player= (Player)e.getPlayer();
        protect=true;
        plugin.getLogger().info(prefix+"Got player teleport "+delay);
 
  }
    @EventHandler(priority = EventPriority.LOW)
    public void OnPlayerDamage(final EntityDamageByEntityEvent event){
        Entity damager= event.getDamager();
        delay=plugin.getConfig().getInt("Seconds:");
        plugin.getLogger().info(prefix+"Got all varriables");
        while(protect==true){
        if(delay>0){
            if(player.hasPermission("pvpdelay.protected")||player.isOp()){
                player.sendMessage(prefix+"Your protected");
                plugin.getLogger().info(prefix+"entered loop");
                        player.sendMessage(prefix +"You Cant pvp for another"+delay+"Seconds");
                        event.setCancelled(true);
                        player.sendMessage(prefix+"You have "+delay+"until you can take damage");
                        if(damager instanceof Player){
                            event.setCancelled(true);
                            ((Player)damager).sendMessage(prefix+"You are under Pvpdelay Protection you may mot pvp for anoter "+delay+" Seconds");
                        }
                        plugin.getLogger().info(prefix+"exeted loops");
        }else if(delay==0){
            protect=false;
        }
            delay--;
      }
    }
    }       
 
}
 

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

 

You don't understand. They can't be null, as in it's impossible. They are defaulted to 0 by the JVM.

GhostHack
Why do you have a static reference to a player...

 

config file holds the following
#This Is Where The Seconds of Delay will be set default is 15 seconds, or 300 ticks.
#More Configurations can/will be added later
Seconds: 15

Superckl1
i posted both my main and my listener

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

 

Yeah, I didn't see your post. The teleport event isn't fired before the damage event, so the player is null.

 

how can i fix it the only way i know wont work id have to make it a static Superckl1

 

You would have to use a Set to keep track of players. If you don't know how to properly use a Set, I suggest you take the time to learn some more about Java.

 

GhostHack,

Code:java

    public static String prefix = (ChatColor.AQUA+"[pvp"+ChatColor.RED+"delay]"+ChatColor.GREEN);
 

shouldn't there be spaces between the pluses in line 13?? Idk it just looks wierd...

 

i know sets ive programmed several things and im currently working on a java rpg game for a competition, but this is the first bukkit plugin ive done

MoeMix
really there should be but its not essential to have them

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