From 508baf4f6b2b60fdbcbfa2c4e349af8f442cab42 Mon Sep 17 00:00:00 2001
From: Eric <biederteric.eb@gmail.com>
Date: Mon, 10 Apr 2017 20:06:05 +0200
Subject: [PATCH] If a world is not found, only show the error once

---
 src/main/java/de/epiceric/shopchest/sql/Database.java | 11 ++++++++---
 1 file changed, 8 insertions(+), 3 deletions(-)

diff --git a/src/main/java/de/epiceric/shopchest/sql/Database.java b/src/main/java/de/epiceric/shopchest/sql/Database.java
index 9e94bd1..97c8612 100644
--- a/src/main/java/de/epiceric/shopchest/sql/Database.java
+++ b/src/main/java/de/epiceric/shopchest/sql/Database.java
@@ -21,10 +21,14 @@ import java.text.ParseException;
 import java.text.SimpleDateFormat;
 import java.util.ArrayList;
 import java.util.Calendar;
+import java.util.HashSet;
+import java.util.Set;
 import java.util.UUID;
 
 public abstract class Database {
 
+    private static Set<String> notFoundWorlds = new HashSet<>();
+
     ShopChest plugin;
     Connection connection;
 
@@ -221,8 +225,9 @@ public abstract class Database {
     /**
      * Get all shops from the database
      * @param callback Callback that - if succeeded - returns an array of all shops (as {@code Shop[]})
+     * @param showConsoleMessages Whether console messages (errors or warnings) should be shown
      */
-    public void getShops(final Callback callback) {
+    public void getShops(final boolean showConsoleMessages, final Callback callback) {
         new BukkitRunnable() {
             @Override
             public void run() {
@@ -248,8 +253,8 @@ public abstract class Database {
 
                         if (world == null) {
                             WorldNotFoundException ex = new WorldNotFoundException("Could not find world with name \"" + worldName + "\"");
-                            if (callback != null) callback.callSyncError(ex);
-                            plugin.getLogger().warning(ex.getMessage());
+                            if (showConsoleMessages && !notFoundWorlds.contains(worldName)) plugin.getLogger().warning(ex.getMessage());
+                            notFoundWorlds.add(worldName);
                             plugin.debug("Failed to get shop (#" + id + ")");
                             plugin.debug(ex);
                             continue;