From 4fac783c1888069a40d8cddf69e5acf9665da275 Mon Sep 17 00:00:00 2001
From: UniquePassive <prozet@live.se>
Date: Sat, 20 Jan 2018 00:17:35 +0100
Subject: [PATCH] Add a Reset button to config panels for resetting to defaults

---
 .../runelite/client/config/ConfigManager.java | 13 ++++---
 .../client/plugins/config/ConfigPanel.java    | 34 +++++++++++++++----
 2 files changed, 35 insertions(+), 12 deletions(-)

diff --git a/runelite-client/src/main/java/net/runelite/client/config/ConfigManager.java b/runelite-client/src/main/java/net/runelite/client/config/ConfigManager.java
index b22827da7d..4907c80594 100644
--- a/runelite-client/src/main/java/net/runelite/client/config/ConfigManager.java
+++ b/runelite-client/src/main/java/net/runelite/client/config/ConfigManager.java
@@ -112,7 +112,7 @@ public class ConfigManager
 	{
 		for (Object config : getConfigProxies())
 		{
-			setDefaultConfiguration(config);
+			setDefaultConfiguration(config, false);
 		}
 	}
 
@@ -352,7 +352,7 @@ public class ConfigManager
 	 * Initialize the configuration from the default settings
 	 * @param proxy
 	 */
-	public void setDefaultConfiguration(Object proxy)
+	public void setDefaultConfiguration(Object proxy, boolean override)
 	{
 		Class<?> clazz = proxy.getClass().getInterfaces()[0];
 		ConfigGroup group = clazz.getAnnotation(ConfigGroup.class);
@@ -371,10 +371,13 @@ public class ConfigManager
 				continue;
 			}
 
-			String current = getConfiguration(group.keyName(), item.keyName());
-			if (current != null)
+			if (!override)
 			{
-				continue; // something else is already set
+				String current = getConfiguration(group.keyName(), item.keyName());
+				if (current != null)
+				{
+					continue; // something else is already set
+				}
 			}
 
 			Object defaultValue;
diff --git a/runelite-client/src/main/java/net/runelite/client/plugins/config/ConfigPanel.java b/runelite-client/src/main/java/net/runelite/client/plugins/config/ConfigPanel.java
index c6f0c25d86..2cabd873df 100644
--- a/runelite-client/src/main/java/net/runelite/client/plugins/config/ConfigPanel.java
+++ b/runelite-client/src/main/java/net/runelite/client/plugins/config/ConfigPanel.java
@@ -36,6 +36,8 @@ import java.awt.event.WindowAdapter;
 import java.awt.event.WindowEvent;
 import java.util.Map;
 import java.util.TreeMap;
+import java.util.function.Function;
+import java.util.stream.Collectors;
 import javax.swing.JButton;
 import javax.swing.JCheckBox;
 import javax.swing.JColorChooser;
@@ -58,6 +60,7 @@ import javax.swing.SwingConstants;
 import javax.swing.event.DocumentEvent;
 import javax.swing.event.DocumentListener;
 import lombok.extern.slf4j.Slf4j;
+import net.runelite.client.config.Config;
 import net.runelite.client.config.ConfigDescriptor;
 import net.runelite.client.config.ConfigItem;
 import net.runelite.client.config.ConfigItemDescriptor;
@@ -108,12 +111,18 @@ public class ConfigPanel extends PluginPanel
 	final void rebuildPluginList()
 	{
 		Map<String, JPanel> newChildren = new TreeMap<>();
-		configManager.getConfigProxies().stream()
-			.map(configManager::getConfigDescriptor)
-			.filter(configDescriptor -> configDescriptor.getItems().stream()
-				.anyMatch(cid -> !cid.getItem().hidden()))
-			.forEach(cd ->
+		configManager.getConfigProxies()
+			.stream()
+			// Config cannot be key because Proxy does not implement hashCode
+			.collect(Collectors.toMap(configManager::getConfigDescriptor, Function.identity()))
+			.entrySet().stream()
+			.filter(e -> e.getKey().getItems().stream()
+					.anyMatch(cid -> !cid.getItem().hidden()))
+			.forEach(e ->
 			{
+				ConfigDescriptor cd = e.getKey();
+				Config config = e.getValue();
+
 				String groupName = cd.getGroup().name();
 				if (children.containsKey(groupName))
 				{
@@ -123,7 +132,7 @@ public class ConfigPanel extends PluginPanel
 				JPanel groupPanel = new JPanel();
 				groupPanel.setLayout(new BorderLayout());
 				JButton viewGroupItemsButton = new JButton(groupName);
-				viewGroupItemsButton.addActionListener(ae -> openGroupConfigPanel(cd, configManager));
+				viewGroupItemsButton.addActionListener(ae -> openGroupConfigPanel(config, cd, configManager));
 				groupPanel.add(viewGroupItemsButton);
 				newChildren.put(groupName, groupPanel);
 			});
@@ -223,7 +232,7 @@ public class ConfigPanel extends PluginPanel
 		}
 	}
 
-	private void openGroupConfigPanel(ConfigDescriptor cd, ConfigManager configManager)
+	private void openGroupConfigPanel(Config config, ConfigDescriptor cd, ConfigManager configManager)
 	{
 		scrollBarPosition = getScrollPane().getVerticalScrollBar().getValue();
 		removeAll();
@@ -350,9 +359,20 @@ public class ConfigPanel extends PluginPanel
 			add(item);
 		}
 
+		JButton resetButton = new JButton("Reset");
+		resetButton.addActionListener((e) ->
+		{
+			configManager.setDefaultConfiguration(config, true);
+
+			// Reload configuration panel
+			openGroupConfigPanel(config, cd, configManager);
+		});
+		add(resetButton);
+
 		JButton backButton = new JButton("Back");
 		backButton.addActionListener(e -> openConfigList());
 		add(backButton);
+
 		revalidate();
 		getScrollPane().getVerticalScrollBar().setValue(0);
 	}