Help with seeing invisible players

Today I made a plugin that makes players go invisible.. That works completely fine.. exempt that I want it so all players with the permission FairiHide.see can see the players that are invisible..

Could someone please help me and provide a code and where to put it please.

Current code:

Code:java

package me.Ricecutter0.FairiHide;
 
import java.util.logging.Logger;
 
import org.bukkit.Bukkit;
import org.bukkit.ChatColor;
import org.bukkit.command.Command;
import org.bukkit.command.CommandSender;
import org.bukkit.entity.Player;
import org.bukkit.plugin.java.JavaPlugin;
import org.bukkit.potion.PotionEffect;
import org.bukkit.potion.PotionEffectType;
 
public class Main extends JavaPlugin {
 
    Logger l = Bukkit.getLogger();
 
    @Override
    public void onEnable()
    {   
        l.info("Plugin starting");
    }
 
    @Override
    public void onDisable()
    {
        l.info("Plugin stopping");
    }
 
    public boolean onCommand(CommandSender sender, Command cmd, String commandLabel, String [] args)
    {
        if(commandLabel.equalsIgnoreCase("hides"))
        {
            Player p = (Player) sender;
            if(p.hasPermission("FairiHide.hides"))
            {
                p.sendMessage(ChatColor.GOLD + "Hiding from the world..");
                p.addPotionEffect(new PotionEffect(PotionEffectType.INVISIBILITY, Integer.MAX_VALUE, 0));
            }
            else p.sendMessage(ChatColor.DARK_RED + "Permission Denied");
        }
 
        if(commandLabel.equalsIgnoreCase("show"))
        {
            Player p = (Player) sender;
            if(p.hasPermission("FairiHide.show"))
            {
                p.sendMessage(ChatColor.BLUE + "Revealing yourself");
                for(PotionEffect pe : p.getActivePotionEffects())
                p.removePotionEffect(pe.getType());
            }
            else p.sendMessage(ChatColor.DARK_RED + "Permission Denied");
 
        }
        return false;
 
    }
 
}
 

Bump
I'd really appreciate it if someone could help

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

 

Don't use commandLabel. Use cmd.getName(). Also why do you keep declaring Player p?

 

I do Player p because it's easier to type. Lol. And Do you know how to solve my problem?

 

1.

Tell me how you are supposed to learn anything from copy-pasting and spoon-fed?
2. Giving a player Invisibility hides the player to anyone. You could use the methods Player.showPlayer(Player) and Player.hidePlayer(Player) instead. Read more about these methods here and here.

 

If you are looking to remove the player from a player (This makes the player not able to be seen or attacked by the player it is being hidden from), use the hidePlayer(Player player) and showPlayer(Player player) methods.

http://jd.bukkit.org/rb/apidocs/org/bukkit/entity/Player.html#hidePlayer(org.bukkit.entity.Player)

http://jd.bukkit.org/rb/apidocs/org/bukkit/entity/Player.html#showPlayer(org.bukkit.entity.Player)

Used:
PlayerToHideFrom.hidePlayer(HidingPlayer)
PlayerToShowTo.showPlayer(PlayerToShow)

 

I understand why you did it, but you declare it twice in your code. You should only declare it once.
Also, due to the nature of this plugin, you should check to see if the sender is player first, that way you won't throw a NPE from the console.