Solved String List...

Hello there!
I'm trying to store a list of players upon login...
Not sure whether my code is correct so far, but this is what I use to store the players:

Code:

                java.util.List<String> staffList = getConfig().getStringList("staff");
                staffList.add(Player);

Could someone please verify that his is correct?
Above this code, I've defined that "Player" is a string that is equal to the player's name.

 

@Raydond123 What is Player? Are you just not following naming conventions?

 

@Skionz I'm pretty sure I am...
Here's what Player is.

Code:

Player player = e.getPlayer();
        String Player = player.getName();

 

@Raydond123 It looks fine besides the naming conventions assuming this is in a class that extends JavaPlugin or you wrote your own getConfig method.

 

@Skionz What do you mean by my own getConfig method?

 

Writing your own getConfig method to get the config from your main class extending JavaPlugin.

 

@WesJD
Uh...
Welp... I don't think I did that.
I just did:

Code:

    public void onEnable() {
        saveConfig();
        getServer().getPluginManager().registerEvents(this, this);
    }

Can someone please tell me what I'm doing wrong?
I've removed the PlayerJoinEvent and then added a new command.
Here's the code for it:

Code:

public boolean onCommand(CommandSender sender, Command cmd, String commandLabel, String[] args) {
        if(commandLabel.equalsIgnoreCase("test")) {
         if(args.length == 0) {
            
         }
         if(args.length == 1) {
            List<String> staffList = getConfig().getStringList("staff");
            staffList.add(args[0]);
            this.getConfig().set("staff", staffList);
         }
        }
        return false;
    }

Okay so, if I did /test test it does nothing, even though I set it to add a string to the stringlist.
What am I doing wrong?

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

 

Don't use commandLabel, use cmd.getName()

The if(args==0) block is completely useless

You don't save the config after you set it

 

OHHHH
THATS WHY THANKS!