From eaa4f929ca259c7e5436e49289700cb3898dcd09 Mon Sep 17 00:00:00 2001 From: EpicKnarvik97 Date: Fri, 18 Feb 2022 21:04:23 +0100 Subject: [PATCH] Fixes the check for testing if a paid sign already exists --- .../java/net/knarcraft/paidsigns/command/AddCommand.java | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/src/main/java/net/knarcraft/paidsigns/command/AddCommand.java b/src/main/java/net/knarcraft/paidsigns/command/AddCommand.java index bc5da9d..a2cbf82 100644 --- a/src/main/java/net/knarcraft/paidsigns/command/AddCommand.java +++ b/src/main/java/net/knarcraft/paidsigns/command/AddCommand.java @@ -50,13 +50,12 @@ public class AddCommand implements CommandExecutor { try { PaidSign sign = new PaidSign(signName, cost, permission, ignoreCase, ignoreColor); - /* TODO: Rewrite the method for comparing if paid signs are duplicated - for (PaidSign similarSign : manager.getPaidSigns(sign.getCleanId(), sign.getLineIndex())) { - if (sign.matches(similarSign)) { - sender.sendMessage("A paid sign with the same id and line already exists"); + for (PaidSign otherSign : manager.getAllPaidSigns()) { + if (sign.getName().equals(otherSign.getName())) { + sender.sendMessage("A paid sign with the same name already exists"); return false; } - }*/ + } manager.addPaidSign(sign); sender.sendMessage("Successfully added new paid sign"); return true;