Solved How to prevent people from creating nether portals?

Hello.

I'm trying to create a plugin to stop people from creating nether portals if they don't have the permission "portalpermission.create". I have de-opped myself and there are no errors in the console but the nether portal gets ignited. Also, the plugin does get enabled because it is green in the /plugins list. Any ideas how to fix this?

Code:java

 
package me.astrophylite.portalpermission;
 
import java.io.File;
 
import org.bukkit.Bukkit;
import org.bukkit.Material;
import org.bukkit.block.Block;
import org.bukkit.command.Command;
import org.bukkit.command.CommandSender;
import org.bukkit.configuration.file.FileConfiguration;
import org.bukkit.configuration.file.YamlConfiguration;
import org.bukkit.entity.Player;
import org.bukkit.event.EventHandler;
import org.bukkit.event.Listener;
import org.bukkit.event.block.Action;
import org.bukkit.event.player.PlayerInteractEvent;
import org.bukkit.inventory.ItemStack;
import org.bukkit.plugin.java.JavaPlugin;
 
public class PortalPermission extends JavaPlugin implements Listener {
 
    FileConfiguration config;
    File cfile;
 
    public void onEnable() {
        getConfig().options().copyDefaults(true);
        config = getConfig();
        saveConfig();
        cfile = new File(this.getDataFolder(), "config.yml");
 
        Bukkit.getServer().getPluginManager().registerEvents(this, this);
    }
 
    public boolean onCommand(CommandSender sender, Command cmd, String label, String[] args) {
        if(cmd.getName().equalsIgnoreCase("portalreload")) {
            if(!sender.hasPermission("portalpermission.reload")) {
                MessageManager.sendNoPermissionMessage(sender, "portalpermission.reload");
                return true;
            }
 
            config = YamlConfiguration.loadConfiguration(cfile);
            MessageManager.sendMessage(sender, "&aYou successfully reloaded PortalPermissions' configuration file!");
            return true;
        }
        return true;
    }
 
    @EventHandler
    public void onPlayerInteract(PlayerInteractEvent e) {
        Player p = e.getPlayer();
        Block b = e.getClickedBlock();
        if(!(e.getAction() == Action.RIGHT_CLICK_BLOCK)) return;
        if(!(p.getItemInHand().equals(new ItemStack(Material.FLINT_AND_STEEL)))) return;
        if(!(b.getType() == Material.OBSIDIAN)) return;
        if(p.hasPermission("portalpermission.create")) return;
 
        e.setCancelled(true);
        MessageManager.sendMessage(p, "&cYou are not allowed to create a nether portal!");
        return;
    }
}
 
 

Any help will be appreciated,
Astrophylite.

 

Check for interactevent, then if the player has a flint and steel in his hands -> cancel the event

 

I don't want to prevent them from using flint&steel completely, just when trying to ignite a portal.

Astrophylite.

 

Then check if they clicked on obsidian. If you want to, ypu can check if there's a portal structure arpund that 1 obsidian block.

 

Look in the code? I've already checked if they clicked on the obsidian and it's not working.

Code:java

 
Block b = e.getClickedBlock();
if(!(b.getType() == Material.OBSIDIAN)) return;
 

Astrophylite.

 

If you want to check if something doesn not equals with something, use != instead of !(this == that)

Why do you return at the end of the method.

Debug, check where it stops from executing.

 

Ok, I will try the other way of checking. I didn't realize I did that and I will begin to debug now.

Astrophylite.

EDIT: Plugin works now, thanks for the tips!