Saving and loading of a Location

My problem is that I want to set a Location and then look for a play who clicked on the block on that Location: My try was:

Code:java

@EventHandler
    public void onClick (PlayerInteractEvent event) {
 
        if (event.getAction() == Action.RIGHT_CLICK_BLOCK) {
            Player p = event.getPlayer(); 
            Location loc = event.getClickedBlock().getLocation();
            if (event.getPlayer().getItemInHand().getType() == Material.GOLD_INGOT) {
                Utils.SetPoint(loc, p, "Yellow", Core.locations, Core.locationFile);
                p.sendMessage(Core.prefix + " §bYellow Location successfully set!");
            }
        }
    }

Code:java

public static void SetPoint(Location loc, Player p, String path, FileConfiguration cfg, File file){
        if(!p.isOp()||!p.hasPermission("SetPoint.*")){
            p.sendMessage("§4No Permission §c(SetPoint.*)");
            return;
        } else {
            cfg.set(path+".world", loc.getWorld().getName());
            cfg.set(path+".X", loc.getX());
            cfg.set(path+".Y", loc.getY());
            cfg.set(path+".Z", loc.getZ());
            try {
                cfg.save(file);
            } catch (IOException e) {
                System.err.println(e.getMessage());
            }
        }
    }

and:

Code:java

public static Location GetLocation(Plugin pl,  String path, FileConfiguration cfg){
        if(!cfg.contains(path)){
            return null;
        } else {
            World world = pl.getServer().getWorld((String) cfg.get(path+".world"));
            int X = cfg.getInt(path+".X");
            int Y = cfg.getInt(path+".Y");
            int Z = cfg.getInt(path+".Z");
 
            if(world == null){
                return null;
            }
            Location point = new Location(world, X, Y, Z);
            return point;
        }
    }

And at last the method of clicking:

Code:java

public void onInteract (PlayerInteractEvent event) {
        Player p = event.getPlayer();
        Block clicked = event.getClickedBlock();
        Location bloc = clicked.getLocation();
        try {
            blueloc = Utils.GetLocation(plugin, "BlueObsidian", Core.locs);
            redloc = Utils.GetLocation(plugin, "RedObsidian", Core.locs);
            greenloc = Utils.GetLocation(plugin, "GreenObsidian", Core.locs);
            yellowloc = Utils.GetLocation(plugin, "YellowObsidian", Core.locs);
        } catch (Exception ex ) {
            ex.printStackTrace();
        }
 
        if (event.getAction() == Action.RIGHT_CLICK_BLOCK||event.getAction() == Action.LEFT_CLICK_BLOCK) {
            if (clicked.getType() == Material.ENCHANTMENT_TABLE) {
                if (p.getItemInHand().getType() != Material.STICK) {
                    event.setCancelled(true);
                    event.getPlayer().sendMessage(Core.prefix + " §bPlease click with a stick");
                    return;
                }
                if (yloc == yellowloc) {
                    Utils.destroyStick(plugin, bloc, "Yellow");
                    event.setCancelled(true);
                    p.getInventory().remove(Material.STICK);
                    p.updateInventory();
                } else {
                    event.setCancelled(true);
                    event.getPlayer().sendMessage(Core.prefix + " §cPlease destroy the stick on an altar");
                }
            }
        }
    }

 

I think Material.GOLD_INGOT() returns an integer value. Instead of comparing it to getItemInHand.getType() compare it to getItemInHand.getTypeId(). I am not at my development PC though, so I am not able to test what I just said.

 

StevenMG there is no java error or any in the console, its just not working at clicking on an enchantment table