= Updated plugins to each have their own global context

src/core/lua/Plugin.java

@@ -6,29 +6,61 @@ import java.nio.file.Files;
 import java.nio.file.Path;
 import java.util.stream.Stream;
 
-//To promote flexibility, plugins are lua file with predefined callbacks.
-//They are executed between the initial built-in preprocessing and the command parsing.
+import org.luaj.vm2.Globals;
+import org.luaj.vm2.LuaValue;
+import org.luaj.vm2.lib.jse.JsePlatform;
+
+// To promote flexibility, plugins are lua file with predefined callbacks.
+// They are executed between the initial built-in preprocessing and the command parsing.
+// This class wraps the lua layer - provides all the processing and function calls to the lua binding.
 public class Plugin
 {
 	public Path filepath;
 	public String contents;
+	private Globals globals;
+	private LuaValue func_plugin;
 	
 	public Plugin(Path filepath)
 	{
 		this.filepath = filepath;
 		
-		StringBuilder contentBuilder = new StringBuilder();
 		
+		StringBuilder contentBuilder = new StringBuilder();
+
 		try 
 		{
 			Stream<String> stream = Files.lines(filepath, StandardCharsets.UTF_8);
 			stream.forEach(str -> contentBuilder.append(str).append("\n"));
 			stream.close();
 			contents = contentBuilder.toString();
+			
+			globals = JsePlatform.standardGlobals();
+
+			globals.load(contents).call();
+			func_plugin = globals.get("plugin");
+			
+			String output = null;
 		}
 		catch (IOException e)
 		{
 			PluginLog.Error(e.getMessage());
 		}
 	}
+	
+	public String Run(String args)
+	{
+		try
+		{
+			LuaValue res = func_plugin.call(LuaValue.valueOf(args));
+			
+			if(!res.isnil())
+				return res.toString();
+		}
+		catch(Exception e)
+		{
+			PluginLog.Error("Failed to run plugin from file " + filepath);
+		}
+		
+		return null;
+	}
 }

src/core/lua/PluginLoader.java

@@ -1,42 +0,0 @@
-package core.lua;
-
-import org.luaj.vm2.Globals;
-import org.luaj.vm2.LuaValue;
-import org.luaj.vm2.lib.jse.JsePlatform;
-
-// This class wraps the lua layer - provides all the processing and function
-// calls to the lua binding.
-public class PluginLoader
-{
-	private Globals globals;
-	
-	public PluginLoader()
-	{
-		globals = JsePlatform.standardGlobals();
-		PluginLog.Log("Created new PluginLoader lua environment");
-	}
-	
-	public String Process(Plugin toProcess, String args)
-	{
-		globals.load(toProcess.contents).call();
-		LuaValue pluginFunc = globals.get("plugin");
-		
-		String output = null;
-		
-		try
-		{
-			LuaValue res = pluginFunc.call(LuaValue.valueOf(args));
-			
-			if(res.isnil())
-				output = null;
-			else
-				output = res.toString();
-		}
-		catch(Exception e)
-		{
-			output = null;
-		}
-		
-		return output;
-	}
-}

src/core/lua/PluginManager.java

@@ -9,7 +9,6 @@ import java.util.stream.Stream;
 // Reads in, handles, and manipulates plugins. Plugins are loaded in the order they appear in the folder.
 public class PluginManager
 {
-	public PluginLoader pluginLoader;
 	public final String pluginFolder;
 	public ArrayList<Plugin> plugins;
 
@@ -22,8 +21,7 @@ public class PluginManager
 	{
 		this.pluginFolder = folder;
 		plugins = new ArrayList<Plugin>();
-		this.pluginLoader = new PluginLoader();
-		
+
 		try
 		{
 			try (Stream<Path> paths = Files.walk(Paths.get(this.pluginFolder)))
@@ -37,16 +35,18 @@ public class PluginManager
 		}
 	}
 	
-	public String CallAll(String input)
+	// Runs all plugins, returning when it gets a non-nill result.
+	// Otherwise, returns null.
+	public String RunAll(String input)
 	{
 		for(int i = 0; i < plugins.size(); ++i)
 		{
-			Plugin script = plugins.get(i);
-			String out = pluginLoader.Process(script, input);
+			Plugin plugin = plugins.get(i);
+			String out = plugin.Run(input);
 			
 			if(out != null)
 			{
-				PluginLog.Log("Executed plugin at " + script.filepath);
+				PluginLog.Log("Executed plugin at " + plugin.filepath);
 				return out;
 			}
 		}