Am I doing this correctly?

Hi guys. I made a scoreboard plugin for factions related, and this code works, but it generates an error in the console. Here's the code + error.

Code:java

 
public void updateScores()
{
Bukkit.getServer().getScheduler().scheduleSyncRepeatingTask(this, new Runnable()
{
public void run()
{
for (final Player p : Bukkit.getOnlinePlayers())
{
 
if (!b.containsKey(p) || p.getScoreboard().equals(null) || p.getScoreboard().getObjective(DisplaySlot.SIDEBAR).equals(null))
{
createScoreBoard(p);
b.put(p, 1);
}
 
UPlayer uplayer = UPlayer.get(p);
if (p.getScoreboard().getObjective(DisplaySlot.SIDEBAR).equals(null))
{
p.getScoreboard().registerNewObjective(p.getName(), "dummy");
p.setScoreboard(p.getScoreboard());
}
p.getScoreboard().getObjective(DisplaySlot.SIDEBAR).setDisplayName(ChatColor.DARK_AQUA + p.getName());
Score money = p.getScoreboard().getObjective(DisplaySlot.SIDEBAR).getScore(Bukkit.getOfflinePlayer(ChatColor.RED + "Money"));
Score mpower = p.getScoreboard().getObjective(DisplaySlot.SIDEBAR).getScore(Bukkit.getOfflinePlayer(ChatColor.RED + "MyPower"));
Score fpower = p.getScoreboard().getObjective(DisplaySlot.SIDEBAR).getScore(Bukkit.getOfflinePlayer(ChatColor.RED + "FactionPower"));
Score territory = p.getScoreboard().getObjective(DisplaySlot.SIDEBAR).getScore(Bukkit.getOfflinePlayer(ChatColor.RED + "ClaimedLand"));
territory.setScore(uplayer.getFaction().getLandCount());
if (uplayer.hasFaction() == false)
{
fpower.setScore(0);
}
else
{
fpower.setScore(uplayer.getFaction().getPowerRounded());
}
mpower.setScore(uplayer.getPowerRounded());
money.setScore((int)(economy.getBalance(p.getName())));
p.setScoreboard(p.getScoreboard());
 
}
}
}, 0L, 20L);
}
 

Code:java

 
public static void createScoreBoard(Player p)
{
Scoreboard board = Bukkit.getServer().getScoreboardManager().getNewScoreboard();
Objective obj = board.registerNewObjective(p.getName(), "dummy");
obj.setDisplaySlot(DisplaySlot.SIDEBAR);
p.setScoreboard(board);
}
 

Error:

Code:

17.02 08:50:05 [Server] INFO java.lang.NullPointerException
17.02 08:50:05 [Server] INFO [08:50:05 WARN]: [FactionBoard] Task #2649 for FactionBoard v1.0 generated an exception

So, my question is, what could return null in the code I wrote above?.

 

If it's working on your server but not on other servers, these might be the reason why,

* plugin has been compiled in a different java version than the server

* plugin is using a really different CB build than the server

 

Updated.