From 268d69c077e77fd223b74cf6d9ab0287ceb2b74f Mon Sep 17 00:00:00 2001
From: Eric <biederteric.eb@gmail.com>
Date: Fri, 17 Mar 2017 14:04:05 +0100
Subject: [PATCH] Only update shops if player has moved

---
 .../listeners/ShopUpdateListener.java         |  2 +-
 .../epiceric/shopchest/utils/ShopUtils.java   | 20 ++++++++++++-------
 2 files changed, 14 insertions(+), 8 deletions(-)

diff --git a/src/main/java/de/epiceric/shopchest/listeners/ShopUpdateListener.java b/src/main/java/de/epiceric/shopchest/listeners/ShopUpdateListener.java
index d751952..ddce5c0 100644
--- a/src/main/java/de/epiceric/shopchest/listeners/ShopUpdateListener.java
+++ b/src/main/java/de/epiceric/shopchest/listeners/ShopUpdateListener.java
@@ -22,7 +22,7 @@ public class ShopUpdateListener implements Listener {
     @EventHandler
     public void onShopUpdate(ShopUpdateEvent e) {
         for (Player p : Bukkit.getOnlinePlayers()) {
-            plugin.getShopUtils().updateShops(p, p.getLocation());
+            plugin.getShopUtils().updateShops(p);
         }
     }
 
diff --git a/src/main/java/de/epiceric/shopchest/utils/ShopUtils.java b/src/main/java/de/epiceric/shopchest/utils/ShopUtils.java
index 7c49132..7473cf1 100644
--- a/src/main/java/de/epiceric/shopchest/utils/ShopUtils.java
+++ b/src/main/java/de/epiceric/shopchest/utils/ShopUtils.java
@@ -23,6 +23,7 @@ import java.util.List;
 public class ShopUtils {
 
     private HashMap<Location, Shop> shopLocation = new HashMap<>();
+    private HashMap<Player, Location> playerLocation = new HashMap<>();
     private ShopChest plugin;
 
     public ShopUtils(ShopChest plugin) {
@@ -227,9 +228,13 @@ public class ShopUtils {
     /**
      * Update hologram and item of all shops for a player
      * @param player Player to show the updates
-     * @param location Location of the player
      */
-    public void updateShops(Player player, Location location) {
+    public void updateShops(Player player) {
+        if (player.getLocation().equals(playerLocation.get(player))) {
+            // Player has not moved, so don't calculate shops again.
+            return;
+        }
+
         if (plugin.getShopChestConfig().only_show_shops_in_sight) {
             HashSet<Material> transparent = new HashSet<>();
             transparent.add(Material.AIR);
@@ -288,23 +293,24 @@ public class ShopUtils {
             }
         } else {
             for (Shop shop : getShops()) {
-                updateShop(shop, player, location);
+                updateShop(shop, player);
             }
         }
+
+        playerLocation.put(player, player.getLocation());
     }
 
     /**
      * Update hologram and item of the shop for a player based on their distance to each other
      * @param shop Shop to update
      * @param player Player to show the update
-     * @param location Location of the player
      */
-    public void updateShop(Shop shop, Player player, Location location) {
+    public void updateShop(Shop shop, Player player) {
         double holoDistSqr = Math.pow(plugin.getShopChestConfig().maximal_distance, 2);
         double itemDistSqr = Math.pow(plugin.getShopChestConfig().maximal_item_distance, 2);
 
-        if (location.getWorld().getName().equals(shop.getLocation().getWorld().getName())) {
-            double distSqr = shop.getLocation().distanceSquared(location);
+        if (player.getLocation().getWorld().getName().equals(shop.getLocation().getWorld().getName())) {
+            double distSqr = shop.getLocation().distanceSquared(player.getLocation());
 
             if (distSqr <= holoDistSqr) {
                 if (shop.getHologram() != null) {