From 99c582dda89d814b18d8124b69ff23953fed029d Mon Sep 17 00:00:00 2001 From: eueln Date: Sun, 2 Nov 2014 01:08:32 -0500 Subject: [PATCH] Fix issues in CmdOpen A formatted message was missing a format argument, and comparison on Strings was being performed with == --- src/main/java/com/massivecraft/factions/cmd/CmdOpen.java | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/main/java/com/massivecraft/factions/cmd/CmdOpen.java b/src/main/java/com/massivecraft/factions/cmd/CmdOpen.java index e2b0b9e8..4054e83a 100644 --- a/src/main/java/com/massivecraft/factions/cmd/CmdOpen.java +++ b/src/main/java/com/massivecraft/factions/cmd/CmdOpen.java @@ -36,8 +36,8 @@ public class CmdOpen extends FCommand { // Inform for (FPlayer fplayer : FPlayers.getInstance().getOnlinePlayers()) { - if (fplayer.getFactionId() == myFaction.getId()) { - fplayer.msg("%s changed the faction to %s.", open); + if (fplayer.getFactionId().equals(myFaction.getId())) { + fplayer.msg("%s changed the faction to %s.", fme.getName(), open); continue; } fplayer.msg("The faction %s is now %s", myFaction.getTag(fplayer.getFaction()), open);