Solved hasPermission() working backwards!

Hi everybody!
Today I have a little trouble with the hasPermission() tag.

In my code, I defined that:

Code:

THREAD SOLVED

But, when I run it on the server if the player has got the "rank.user" permission can use the command, but Op players can't.

My question is, what I'm doing wrong? It doesn't supposed that if the player has the rank.user permission return an error message?

PS: I did't use quotation marks inside the hasPermission(-HERE-) because I'm calling a class named "rank" who has got a String named "user". The same with the sendMessage().

PS 2: I'm sorry about my english but isn't my native language

 

Check to see if the player is OP

Code:

if(player.isOp()) {

 

1.In your plugin.yml have you registered the permission and defaulted it to op?
2.In order to better assist you with this problem, we need to see the rank.user
3.In this case for a ranking system, it would be better to use an ENUM (if you're not already and for some reason are using an enum lower case)

 

1. No, I did not registered the permission 'cause in other Bukkit versions the code that I use worked perfectly fine.
2. The class "rank" have just a list of String = "text". But, here is the code:

Code:

THREAD SOLVED

I use this method because, if I want to rename some of the server ranks I will not have to modify class by class of the plugin.
3. I'm not using ENUM or enum, as you can see in the code posted above, I use a simple class with public Strings to access them from any class of the server, this is because, if I use enum instead of class I can't put dots in the permissions.
PS: I'm coding a private plugin for my server that is because I don't use individual permissions as "server.fly.hasPerm", for example.

 

@mtrack260 Have you tried Final? And it would be even more helpful if you posted more of the code, like the declaration of the player object and I assume the onCommand() method is where this is located

Code:java

package me.frank.repository;
public class Rank {
    public static final String user = "rank.user";
    public static final String vip = "rank.vip";
    public static final String mod = "rank.mod";
    public static final String dev = "rank.dev";
    public static final String admin = "rank.admin";
    public static final String master = "rank.master";
}

 

@Jayfeather311
@mtrack260

And now we could agree to write them in Uppercase. That way you can see they are constants after a short glance

 

Code:java

 

@mtrack260 Due to the fact that you do not have a constructor, I know the problem. Add this line to the top of your "fly" class. I am substituting "Main" as the class name for the one that extends JavaPlugin, change it to how yours is setup.

Code:java

Main plugin;

Then create a constructor shown below.

Code:java

public fly(Main passedPlugin) {
    plugin = passedPlugin;
    plugin.getServer().getPluginManager().registerEvents(this, plugin);
}

Then in your main class in your onEnable() method, remove the register events if you have it there and add this

Code:java

new fly(this);

 

I tried, and it keeps working backwards.

 

@mtrack260
Have you tried putting the permission in directly instead of getting it from another class?

And please start your class names with an upper-case letter.

 

@Jayfeather311
How would this affect anything? Do you even know what constructors are for? And with your example, why isn't that field private and final?

 

That's just an example... But i wouldn't register events like that.

 

Quoting from your lastest code:

Code:

if(p.hasPermission(rank.user)) {
            p.sendMessage(text.flySuff + text.noPerm);
            p.playSound(p.getLocation(), sounds.errorSound, 1, 1);
            return true;
        } else { ...

So, you're checking if the player has the permission, and if they have... tell them they don't? Isn't it supposed to be the other way around?