Make sprite and widget sprite overrides more flexible

- Expose NodeCache interface in RuneLite
- Expose Client.getWidgetSpriteCache and reset method on it
- Change setters for sprite and widget overrides to be getters

This allows for more than 1 plugin to use the widget/sprite override API
in future.

Signed-off-by: Tomas Slusny <slusnucky@gmail.com>

runelite-api/src/main/java/net/runelite/api/Client.java

@@ -1394,24 +1394,20 @@ public interface Client extends GameEngine
 	void queueChangedSkill(Skill skill);
 
 	/**
-	 * Sets a mapping of sprites to override.
+	 * Gets a mapping of sprites to override.
 	 * <p>
 	 * The key value in the map corresponds to the ID of the sprite,
 	 * and the value the sprite to replace it with.
-	 *
-	 * @param overrides the sprites to override
 	 */
-	void setSpriteOverrides(Map<Integer, SpritePixels> overrides);
+	Map<Integer, SpritePixels> getSpriteOverrides();
 
 	/**
-	 * Sets a mapping of widget sprites to override.
+	 * Gets a mapping of widget sprites to override.
 	 * <p>
 	 * The key value in the map corresponds to the packed widget ID,
 	 * and the value the sprite to replace the widgets sprite with.
-	 *
-	 * @param overrides the sprites to override
 	 */
-	void setWidgetSpriteOverrides(Map<Integer, SpritePixels> overrides);
+	Map<Integer, SpritePixels> getWidgetSpriteOverrides();
 
 	/**
 	 * Sets the compass sprite.
@@ -1420,6 +1416,13 @@ public interface Client extends GameEngine
 	 */
 	void setCompass(SpritePixels spritePixels);
 
+	/**
+	 * Returns widget sprite cache, to be used with {@link Client#getSpriteOverrides()}
+	 *
+	 * @return the cache
+	 */
+	NodeCache getWidgetSpriteCache();
+
 	/**
 	 * Gets the current server tick count.
 	 *

runelite-api/src/main/java/net/runelite/api/NodeCache.java

@@ -0,0 +1,36 @@
+/*
+ * Copyright (c) 2018, Tomas Slusny <slusnucky@gmail.com>
+ * All rights reserved.
+ *
+ * Redistribution and use in source and binary forms, with or without
+ * modification, are permitted provided that the following conditions are met:
+ *
+ * 1. Redistributions of source code must retain the above copyright notice, this
+ *    list of conditions and the following disclaimer.
+ * 2. Redistributions in binary form must reproduce the above copyright notice,
+ *    this list of conditions and the following disclaimer in the documentation
+ *    and/or other materials provided with the distribution.
+ *
+ * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS" AND
+ * ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED
+ * WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE
+ * DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT OWNER OR CONTRIBUTORS BE LIABLE FOR
+ * ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES
+ * (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES;
+ * LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND
+ * ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
+ * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS
+ * SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+ */
+package net.runelite.api;
+
+/**
+ * Represents a doubly linked node cache.
+ */
+public interface NodeCache
+{
+	/**
+	 * Resets cache.
+	 */
+	void reset();
+}

runelite-client/src/main/java/net/runelite/client/plugins/interfacestyles/InterfaceStylesPlugin.java

@@ -145,7 +145,9 @@ public class InterfaceStylesPlugin extends Plugin
 			}
 		}
 
-		client.setSpriteOverrides(overrides);
+		client.getWidgetSpriteCache().reset();
+		client.getSpriteOverrides().clear();
+		client.getSpriteOverrides().putAll(overrides);
 	}
 
 	private void overrideWidgetSprites()
@@ -168,7 +170,8 @@ public class InterfaceStylesPlugin extends Plugin
 			}
 		}
 
-		client.setWidgetSpriteOverrides(widgetOverrides);
+		client.getWidgetSpriteOverrides().clear();
+		client.getWidgetSpriteOverrides().putAll(widgetOverrides);
 	}
 
 	private SpritePixels getFileSpritePixels(String file, String subfolder)
@@ -292,8 +295,8 @@ public class InterfaceStylesPlugin extends Plugin
 
 	private void removeGameframe()
 	{
-		client.setSpriteOverrides(null);
+		client.getSpriteOverrides().clear();
-		client.setWidgetSpriteOverrides(null);
+		client.getWidgetSpriteOverrides().clear();
 
 		BufferedImage compassImage = spriteManager.getSprite(SpriteID.COMPASS_TEXTURE, 0);
 

runelite-mixins/src/main/java/net/runelite/mixins/SpriteMixin.java

@@ -25,6 +25,7 @@
  */
 package net.runelite.mixins;
 
+import java.util.HashMap;
 import java.util.Map;
 import net.runelite.api.SpritePixels;
 import net.runelite.api.mixins.Copy;
@@ -39,24 +40,23 @@ import net.runelite.rs.api.RSSpritePixels;
 public abstract class SpriteMixin implements RSClient
 {
 	@Inject
-	private static Map<Integer, SpritePixels> spriteOverrides;
+	private static final Map<Integer, SpritePixels> spriteOverrides = new HashMap<Integer, SpritePixels>();
 
 	@Inject
-	private static Map<Integer, SpritePixels> widgetSpriteOverrides;
+	private static final Map<Integer, SpritePixels> widgetSpriteOverrides = new HashMap<Integer, SpritePixels>();
 
 	@Inject
 	@Override
-	public void setSpriteOverrides(Map<Integer, SpritePixels> overrides)
+	public Map<Integer, SpritePixels> getSpriteOverrides()
 	{
-		getWidgetSpriteCache().reset();
+		return spriteOverrides;
-		spriteOverrides = overrides;
 	}
 
 	@Inject
 	@Override
-	public void setWidgetSpriteOverrides(Map<Integer, SpritePixels> overrides)
+	public Map<Integer, SpritePixels> getWidgetSpriteOverrides()
 	{
-		widgetSpriteOverrides = overrides;
+		return widgetSpriteOverrides;
 	}
 
 	@Copy("getSpriteAsSpritePixels")
@@ -68,14 +68,11 @@ public abstract class SpriteMixin implements RSClient
 	@Replace("getSpriteAsSpritePixels")
 	public static RSSpritePixels rl$loadSprite(RSIndexDataBase var0, int var1, int var2)
 	{
-		if (spriteOverrides != null)
+		SpritePixels sprite = spriteOverrides.get(var1);
-		{
-			SpritePixels sprite = spriteOverrides.get(var1);
 
-			if (sprite != null)
+		if (sprite != null)
-			{
+		{
-				return (RSSpritePixels) sprite;
+			return (RSSpritePixels) sprite;
-			}
 		}
 
 		return rs$loadSprite(var0, var1, var2);

runelite-mixins/src/main/java/net/runelite/mixins/WidgetSpriteMixin.java

@@ -49,7 +49,7 @@ public abstract class WidgetSpriteMixin implements RSWidget
 	@Replace("getWidgetSprite")
 	public RSSpritePixels rl$getWidgetSprite(boolean var1)
 	{
-		if (getSpriteId() != -1 && widgetSpriteOverrides != null)
+		if (getSpriteId() != -1)
 		{
 			SpritePixels sprite = widgetSpriteOverrides.get(getId());
 

runescape-api/src/main/java/net/runelite/rs/api/RSClient.java

@@ -645,6 +645,7 @@ public interface RSClient extends RSGameEngine, Client
 	void setCompass(SpritePixels spritePixels);
 
 	@Import("widgetSpriteCache")
+	@Override
 	RSNodeCache getWidgetSpriteCache();
 
 	@Import("oculusOrbState")

runescape-api/src/main/java/net/runelite/rs/api/RSNodeCache.java

@@ -24,9 +24,10 @@
  */
 package net.runelite.rs.api;
 
+import net.runelite.api.NodeCache;
 import net.runelite.mapping.Import;
 
-public interface RSNodeCache
+public interface RSNodeCache extends NodeCache
 {
 	@Import("get")
 	RSCacheableNode get(long id);