Making abbreviations for configs?!

Well I am not sure what the name for it is called yet, but I called abbreviations.

Basically I wanted to figure out how to make for example
p.getName() usable in configs? So like I can make
p.getName() = %player_name, which I can be able to type in config..

 

Ugh...get a string in the config from the path and check if it contains the character sequence of %player_name%, if it does then display the player name in whatever you're doing

 

Try String#replace.

 

Would this be the correct way to do this?

Code:java

    public static String translateToColorCode(String msg){
        return ChatColor.translateAlternateColorCodes('&', msg);
    }
 
    public static String replaceVariables(String msg, Player player)
      {
        msg = msg.replace("%player_name", player.getName());
        return translateToColorCode(msg);
      }

 

Yes. Why is it static though?

 

leon3001
Why not?

 

Assist leon3001 Eh, tried it in-game, it just ignored the whole process of %player_name and the color codes in the config for some odd reason.

Its also static cause it thats what it needed to fix an error.

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

 

Okay, you're probably right when he's writing an utility class.

 

Any help for my messages above?

 

How are you using the methods you showed above?

 

Well, here...
CODE:

Code:java

package me.perezhd.freeze;
 
import java.util.ArrayList;
 
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.event.EventHandler;
import org.bukkit.event.Listener;
import org.bukkit.event.player.PlayerMoveEvent;
import org.bukkit.plugin.java.JavaPlugin;
 
public class Main extends JavaPlugin implements Listener {
 
    ArrayList<Player> frozen = new ArrayList<Player>();
    public static Main plugin;
    public String prefix;
 
    public void onEnable() {
        plugin = this;
 
        Bukkit.getServer().getLogger().info("Plugin Loaded!");
        Bukkit.getServer().getPluginManager().registerEvents(this, this);
 
        getConfig().options().copyDefaults(true);
        reloadConfig();
        saveDefaultConfig();
 
        loadPrefix();
    }
 
    public void onDisable() {
        Bukkit.getServer().getLogger().info("Plugin Disabled");
    }
 
    public void loadPrefix(){
        this.prefix = translateToColorCode(getConfig().getString("prefix"));
    }
    public static String translateToColorCode(String msg){
        return ChatColor.translateAlternateColorCodes('&', msg);
    }
 
    public static String replaceVariables(String msg, Player player)
      {
        msg = msg.replace("%player_name", player.getName());
        return translateToColorCode(msg);
      }
    @EventHandler
    public void onFreezeAll(PlayerMoveEvent e) {
        for (Player online : Bukkit.getOnlinePlayers()) {
            if(frozen.contains(online)) {
                e.setTo(e.getFrom());
        }
    }
    }
 
    @EventHandler
    public void onPlayerMove(PlayerMoveEvent e) {
        Player p = e.getPlayer();
        if(frozen.contains(p)) {
            e.setTo(e.getFrom());
            p.sendMessage(prefix + getConfig().getString("Frozen-Toggled-Message"));
        }
 
    }
    public boolean onCommand(CommandSender sender, Command cmd, String  CommandLabel, String[] args) {
 
        Player p = (Player) sender;
 
        if(cmd.getName().equalsIgnoreCase("freeze")) {
            if(p.hasPermission("pf.freeze")) {
                if(args.length == 0) {
                    p.sendMessage(prefix + getConfig().getString("Specify-Player"));
                    return true;
                }
 
                Player t = Bukkit.getServer().getPlayer(args[0]);
 
                if(t == null) {
                    p.sendMessage(prefix + ChatColor.RED + " Could not find the player " + ChatColor.AQUA + args[0] + ChatColor.RED + "!");
                    return true;
                }
 
                if(frozen.contains(t)) {
                    frozen.remove(t);
                    p.sendMessage(prefix + ChatColor.YELLOW + " " + ChatColor.YELLOW + args[0] + ChatColor.GRAY + " is no longer frozen!");
                    t.sendMessage(prefix + getConfig().getString("Unfreeze-Message"));
                    return true;
                } else {
                    frozen.add(t);
                    p.sendMessage(prefix + ChatColor.YELLOW + " " + ChatColor.YELLOW + args[0] + ChatColor.GRAY + " is now frozen!");
                    t.sendMessage(prefix + getConfig().getString("Freeze-Message") + " You have been frozen by " + ChatColor.YELLOW + p.getName() + ChatColor.GRAY + "!");
                    return true;
                    }
                } else {
                    p.sendMessage(getConfig().getString("No-Permission-Message"));
                    return true;
            }
        }
        if(args[1].equalsIgnoreCase("reload"))
            if(p.hasPermission("pf.freeze.reload")) {
                reloadConfig();
                p.sendMessage(prefix + "You have successfully reloaded the configuration file!");
            }
        return true;
 
    }
 
 
}
 

CONFIG:
http://pastebin.com/dw5zAMgQ

 

PerezHD
I don't see you ever even using the replaceVariables method.

 

.... it says it in the code..

 

That's nice! Call it now.

 

PerezHD

Also, don't leave a public static reference to your plugin in there, that's just lazy.