package com.gmail.nossr50.runnables.commands;

import com.gmail.nossr50.datatypes.database.PlayerStat;
import com.gmail.nossr50.datatypes.skills.SkillType;
import com.gmail.nossr50.locale.LocaleLoader;
import com.gmail.nossr50.mcMMO;
import com.gmail.nossr50.util.scoreboards.ScoreboardManager;
import org.bukkit.ChatColor;
import org.bukkit.command.CommandSender;
import org.bukkit.entity.Player;
import org.bukkit.scheduler.BukkitRunnable;

import java.util.List;

/**
 * Display the results of {@link MctopCommandAsyncTask} to the sender.
 */
public class MctopCommandDisplayTask extends BukkitRunnable {
    private final List<PlayerStat> userStats;
    private final CommandSender sender;
    private final SkillType skill;
    private final int page;
    private final boolean useBoard, useChat;

    MctopCommandDisplayTask(List<PlayerStat> userStats, int page, SkillType skill, CommandSender sender, boolean useBoard, boolean useChat) {
        this.userStats = userStats;
        this.page = page;
        this.skill = skill;
        this.sender = sender;
        this.useBoard = useBoard;
        this.useChat = useChat;
    }

    @Override
    public void run() {
        if (useBoard) {
            displayBoard();
        }

        if (useChat) {
            displayChat();
        }

        if (sender instanceof Player) {
            ((Player) sender).removeMetadata(mcMMO.databaseCommandKey, mcMMO.p);
        }
        if(sender instanceof Player)
            sender.sendMessage(LocaleLoader.getString("Commands.mctop.Tip"));
    }

    private void displayChat() {

        if (skill == null) {
            if(sender instanceof Player)
                sender.sendMessage(LocaleLoader.getString("Commands.PowerLevel.Leaderboard"));
            else
                sender.sendMessage(ChatColor.stripColor(LocaleLoader.getString("Commands.PowerLevel.Leaderboard")));
        }
        else {
            if(sender instanceof Player)
                sender.sendMessage(LocaleLoader.getString("Commands.Skill.Leaderboard", skill.getName()));
            else
                sender.sendMessage(ChatColor.stripColor(LocaleLoader.getString("Commands.Skill.Leaderboard", skill.getName())));
        }

        int place = (page * 10) - 9;

        for (PlayerStat stat : userStats) {
            // Format:
            // 01. Playername - skill value
            // 12. Playername - skill value
            sender.sendMessage(String.format("%2d. %s%s - %s%s", place, sender instanceof Player ? ChatColor.GREEN : "",
                    stat.name, sender instanceof Player ? ChatColor.WHITE : "", stat.statVal));
            place++;
        }
    }

    private void displayBoard() {
        if (skill == null) {
            ScoreboardManager.showTopPowerScoreboard((Player) sender, page, userStats);
        }
        else {
            ScoreboardManager.showTopScoreboard((Player) sender, skill, page, userStats);
        }
    }
}

@@ -2394,32 +2394,77 @@
 e - skill value%0A
+            if(sender instanceof Player)%0A    
             send
@@ -2523,56 +2523,68 @@
 ce, 
-sender instanceof Player ? ChatColor.GREEN : %22%22,
+ChatColor.GREEN, stat.name, ChatColor.WHITE, stat.statVal));
 %0A   
@@ -2592,24 +2592,29 @@
         
+else%0A
         
 stat.nam
@@ -2609,66 +2609,81 @@
     
-stat.name, sender instanceof Player ? ChatColor.WHITE : %22%22
+        sender.sendMessage(String.format(%22%252d. %25s - %25s%22, place, stat.name
 , st
@@ -2688,32 +2688,33 @@
 stat.statVal));%0A
+%0A
             plac

Remove ternary operators and use if/else