My plugin doesn't work.

Code:java

 
String prefix;
public String getPlayerPrefix(Player player) {
return prefix;
}
//snipped stuff
String g = (prefix);
 

You define the string prefix, return the (empty) prefix value in a function (that you don't seem to call) and then use the (still empty) value to assign to 'g' which you use to print the prefix.

There is no way prefix can have a value with the code you have shown.

To integrate with Vault, you need to:
1. Put Vault as a dependency in your plugin.yml
2. Check in your onEnable that Vault is loaded (optional as Bukkit should ensure this if you have the dependency)
3. Set up the chat provider (see https://github.com/MilkBowl/Vault#implementing-vault)
4. Remove your getPlayerPrefix function (and the prefix variable)
5. Replace the line "String g = (prefix);" with "String g = chat.getPlayerPrefix(p);"

That should do it

 

LOL, the cut was already registered! The problem was I didn't know how to use the function! Thanks! Now it works perfectly.

Actually, now it is giving a nullpointerexception again:

16:02:22 [INFO] boss86741[/127.0.0.1:60275] logged in with entity id 830 at ([world] -2052.326800969757, 66.0, 1280.271361218455)

Code:

16:02:26 [INFO] boss86741 issued server command: /shout hi
16:02:27 [SEVERE] null
org.bukkit.command.CommandException: Unhandled exception executing command 'shout' in plugin ShoutCraft v0.1.6
at org.bukkit.command.PluginCommand.execute(PluginCommand.java:46)
at org.bukkit.command.SimpleCommandMap.dispatch(SimpleCommandMap.java:186)
at org.bukkit.craftbukkit.CraftServer.dispatchCommand(CraftServer.java:502)
at net.minecraft.server.NetServerHandler.handleCommand(NetServerHandler.java:985)
at net.minecraft.server.NetServerHandler.chat(NetServerHandler.java:903)
at net.minecraft.server.NetServerHandler.a(NetServerHandler.java:858)
at net.minecraft.server.Packet3Chat.handle(Packet3Chat.java:44)
at net.minecraft.server.NetworkManager.b(NetworkManager.java:290)
at net.minecraft.server.NetServerHandler.d(NetServerHandler.java:113)
at net.minecraft.server.ServerConnection.b(SourceFile:39)
at net.minecraft.server.DedicatedServerConnection.b(SourceFile:30)
at net.minecraft.server.MinecraftServer.r(MinecraftServer.java:595)
at net.minecraft.server.DedicatedServer.r(DedicatedServer.java:222)
at net.minecraft.server.MinecraftServer.q(MinecraftServer.java:493)
at net.minecraft.server.MinecraftServer.run(MinecraftServer.java:426)
at net.minecraft.server.ThreadServerApplication.run(SourceFile:856)
Caused by: java.lang.NullPointerException
at com.boss86741.plugins.ShoutCraft.ShoutCraftCommands.onCommand(ShoutCraftCommands.java:50)
at org.bukkit.command.PluginCommand.execute(PluginCommand.java:44)
... 15 more

Cruel, huh? The problem is at this line of code:

Code:java

if (player.hasPermission("shoutcraft.shout")) {
    String plName = player.getName();
    String g = chat.getPlayerPrefix(player);

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

 

player is null?

 

don't think so...

the command is executed from the player

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

 

then chat is null. What is line 50 anyway?

 

Code:java

String g = chat.getPlayerPrefix(player);

 

then chat is null

 

how do I fix that?

 

post your current code. I need to know what 'chat' is

 

Code:java

package com.boss86741.plugins.ShoutCraft;
 
import java.lang.String;
 
import net.milkbowl.vault.chat.Chat;
 
import org.bukkit.command.Command;
import org.bukkit.command.CommandSender;
import org.bukkit.Bukkit;
import org.bukkit.ChatColor;
import org.bukkit.command.CommandExecutor;
import org.bukkit.entity.Player;
import org.bukkit.plugin.RegisteredServiceProvider;
 
public class ShoutCraftCommands implements CommandExecutor {
    private ShoutCraft plugin;
 
    public ShoutCraftCommands(ShoutCraft plugin) {
        this.plugin = plugin;
    }
 
    Player player = null;
 
    public static Chat chat = null;
 
    private boolean setupChat()
    {
        RegisteredServiceProvider<Chat> chatProvider = Bukkit.getServer().getServicesManager().getRegistration(net.milkbowl.vault.chat.Chat.class);
        if (chatProvider != null) {
            chat = chatProvider.getProvider();
        }
 
        return (chat != null);
    }
 
 
// Below is the command code.   
 
    @Override
    public boolean onCommand(CommandSender sender, Command cmd, String label, String[] args) {
        if(cmd.getName().equalsIgnoreCase("shout")) {
            if (sender instanceof Player) {
                player = (Player) sender;
            }
                if (player == null) {
                    sender.sendMessage(ChatColor.RED + "Can't be used from the console.");
                } else {
                    if (player.hasPermission("shoutcraft.shout")) {
                        String plName = player.getName();
                        String g = chat.getPlayerPrefix(player);
 
                        String message = "";
 
                        if (args.length > 0) {
                            for (int i = 1; i < args.length; ++i) {
                                message += args[i] + " ";
                            }   
                            Bukkit.getServer().broadcastMessage(ChatColor.RED + "[S]" + ChatColor.WHITE + "=[" + g + ChatColor.WHITE + "]=" + plName + ChatColor.WHITE + ": " + ChatColor.GOLD + message);
                        } else {
                            sender.sendMessage(ChatColor.RED + "Usage: /shout <your message here>");
                    }
                }
                return true;
            }
        }
        return false;
    }
}
[/S][/i]

 

chat is null. also, don't use static when not needed ;3

 

How do I make it not null? I use PEX.

 

do you have Vault as a dependency?

 

yes

 

do you check what setupChat() returns?

 

no

 

do that. If this returns false, it means chat is null, and it will throw a NPE if you use it any further

 

Boss, I suggest you watch a few NewBoston Java Tutorials first...