Solved [URGENT] [HELP] A lotta things Not Working

I'm nervous about posting my full code.
DON'T COPY-PASTE THIS INTO ONE OF YOUR PLUGINS!

Code:java

package me.TheEpicButterStudios.InstaTnT;
import java.util.ArrayList;
import org.bukkit.ChatColor;
import org.bukkit.Location;
import org.bukkit.Material;
import org.bukkit.command.Command;
import org.bukkit.command.CommandSender;
import org.bukkit.entity.Entity;
import org.bukkit.entity.Player;
import org.bukkit.entity.TNTPrimed;
import org.bukkit.event.EventHandler;
import org.bukkit.event.EventPriority;
import org.bukkit.event.Listener;
import org.bukkit.plugin.java.JavaPlugin;
 
//InstaTnT code by TheEpicButterStudios
//Distributed on [url]http://dev.bukkit.org/bukkit-plugins/instatnt/[/url]
//under the GNU General Public License, version 3.
//All code by TheEpicButterStudios, cannot be edited and
//redistributed. Should only be downloaded from BukkitDev.
// This is a comment =D
public class InstaTnT extends JavaPlugin implements Listener
{
/*
* @author TheEpicButterStudios
*/
    @EventHandler(priority=EventPriority.HIGH)
    public void tntPlace(Listener event)
    {
        getServer().getPluginManager().registerEvents(event, this);
      if(((Location) event).getBlock().getType() == Material.TNT)      {
        ((Location) event).getBlock().setType(Material.AIR);
        Entity tnt = ((Player) event).getPlayer().getWorld().spawn(((Location) event).getBlock().getLocation(), TNTPrimed.class);
        ((TNTPrimed)tnt).setFuseTicks(0);
      }
    }
 
 
 
 
 
 
 
 
 
    public final ArrayList<Player> InstaTnTUsers = new ArrayList<Player>();
 
    public void OnEnable()
        {
            getLogger().info("[InstaTnT] has been enabled!");
            getLogger().info("[InstaTnT] by TheEpicButterStudios!");
            getLogger().info("Running version rd-100001913");
            getLogger().warning("You shouldn't be running this build!");
        }
 
        public void OnDisable()
        {
            getLogger().info("[InstaTnT] Has Been Disabled.");
            getLogger().info("[InstaTnT] Good Night!");
        }
 
        public boolean OnCommand(CommandSender sender, Command cmd, String commandLabel, String args[]) {
        if(player.hasPermission("instatnt.explode")); {
            if(commandLabel.equalsIgnoreCase("InstaBoom")){
            toggleInstaTnT(sender);
 
    }  else ((Player) sender).sendMessage(ChatColor.RED + "You do not have access to that command.");
        return false;
        }
 
 
        }
        private void toggleInstaTnT(CommandSender sender)
        {
 
            if(!enabled((Player)sender) )
            {
                InstaTnTUsers.add((Player) sender);
                ((Player) sender).sendMessage(ChatColor.YELLOW + "[InstaTnT] Enabled for YOU!");
            }
            else
            {
 
                InstaTnTUsers.remove((Player)sender);
                ((Player) sender).sendMessage(ChatColor.RED + "[InstaTnT] Disabled. See Ya!");
            }
        }
        public boolean enabled(Player player)
        {
            return InstaTnTUsers.contains(player);
        }
        public static Player player;
 
    }

Help appreciated. Thanks!

 

um would you mind being a bit more specific with what you need help with? is the plugin not loading, is it throwing errors at certain points? if so what points. etc etc

 

thecrystalflame
Problems:
* Not logging to console
* Command not working
* Block replace not working

 

Because we're totally going to steal your code and release it? It's 94 lines and doesn't even work.. Be confident nobody will steal it.

 

Why are you registering events inside of an event instead of in onEnable()?

Code:java

 
@EventHandler(priority=EventPriority.HIGH)
public void tntPlace(Listener event)
{
getServer().getPluginManager().registerEvents(event, this);
//Other things
}
 

Also, don't store players in an ArrayList. It is smarter to store the players names (Strings) in an ArrayList so that if the player logs off the GC can unload them.

Code:java

if(player.hasPermission("instatnt.explode")); {
            if(commandLabel.equalsIgnoreCase("InstaBoom")){
            toggleInstaTnT(sender);

You are also checking to see if the player has permission to use a command before checking to make sure it is the correct command.

 

Most of us can create this plugin: with less lines, cleaner code, and it will actually work in less than 10 minutes.

Not to be offending or anything, but you may want to learn a bit more about bukkit and java before, erm, throwing together code.

 

Also, the license you say your code is under and the statements your saying are sorta conflicting. The GPL license is all about open source and the ability to edit and make changes. You might want to read what a license entails before slapping it onto your code. http://www.gnu.org/copyleft/gpl.html

 

Sour
Haha, didn't even read what license he put at the top. So technically, we can copy his code and release it! Oh, wait, that's right.. it needs to be scrapped and redone.. Seriously, I didn't read his code until now, it makes absolutely no sense at all.

 

half of which are annotations, blank lines, or imports

 

A Listener event? Casting events to locations? Where did you get that from.

 

You should probably @Override onEnable and onDisable for your plugin to enable first lol

 

TheEpicButterStudios You're doing OnEnable and OnDisable, this will not work. You have to use onEnable and onDisable. (Yes it is case-sensitive)

TheEpicButterStudios Well I couldn't resis and wrote a snippet that should work:

Code:java

package me.TheEpicButterStudios.InstaTnT;
 
import java.util.ArrayList;
import org.bukkit.ChatColor;
import org.bukkit.Location;
import org.bukkit.Material;
import org.bukkit.command.Command;
import org.bukkit.command.CommandSender;
import org.bukkit.entity.Entity;
import org.bukkit.entity.Player;
import org.bukkit.entity.TNTPrimed;
import org.bukkit.event.EventHandler;
import org.bukkit.event.EventPriority;
import org.bukkit.event.Listener;
import org.bukkit.plugin.java.JavaPlugin;
import org.bukkit.event;
 
public class InstaTnT extends JavaPlugin implements Listener{
 
    public List<String> instaTnTers = new ArrayList<String>();
 
    public void onDisable(){
        instaTnTers.clear();
    }
 
    public void onEnable(){
        getServer().getPluginManager().registerEvents(this, this);
    }
 
    @EventHandler
    public void onBlockPlace(BlockPlaceEvent event){
        if(instaTnTers.conatins(event.getPlayer().getName())){
            if(event.getBlock().getType().equals(Material.TNT)){
                event.getPlayer().getWorld().spawnEntity(event.getBlock().getLocation(), EntityType.PRIMED_TNT);
            }
        }
    }
 
    public boolean onCommand(CommandSender sender, Command command, String label, String[] args){
        if(sender instanceof Player){
            Player player = (Player) sender;
            if(player.hasPermission("instatnt.explode")){ //I would rather use something like instatnt.use but okay...
                if(instaTnTers.contains(player.getName()){
                    instaTnTers.remove(player.getName());
                    player.sendMessage(ChatColor.RED + "Insta TNT disabled!");
                }else{
                    instaTnTers.add(player.getName());
                    player.sendMessage(ChatColor.GREEN + "Insta TNT enabled!");
                }
            }else{
                player.sendMessage(ChatColor.RED + "You don't have permission to use this command!");
            }
        }else{
            sender.sendMessage(ChatColor.RED + "You do not have access to that command!");
        }
    }
}

It's written in Notepad++ so imports may be wrong. Also, I didn't check for the label but I'm sure you know how to do that yourself.
I wrote this code for educational purposes. If you're planning on just copy-pasting your way trough it then you won't learn anything. You may consider this code as an example of how it should be done. Please for the sake of everything thats holy, learn how to make plugins first before just pushing some code together. As about the "Don"t copy this inside your own plugins!" part, there are over 100 insta-tnt plugins out there, so I'm sure no one will, but let your rule also count for this code. Don't just copy it, learn from it.
- CaptainBern

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

 

CaptainBern Thanks!
Why does everyone else have to be so NEGATIVE...
I will admit, I am a beginner...

 

TheEpicButterStudios You're welcome, happy coding.

 

TheEpicButterStudios
I need to make a text file containing a few paragraphs about what I'm about to say right now, as I say it a million times a day to people.

Learn Java before getting into 3rd party APIs(as said in my signature). It is well worth it.