Bind reward JavaScript placeholders as data

AdvancedCore/pom.xml

@@ -231,6 +231,12 @@
             <version>2.12.2</version>
             <scope>provided</scope>
         </dependency>
+        <dependency>
+            <groupId>org.openjdk.nashorn</groupId>
+            <artifactId>nashorn-core</artifactId>
+            <version>15.7</version>
+            <scope>provided</scope>
+        </dependency>
         <dependency>
             <groupId>org.slf4j</groupId>
             <artifactId>slf4j-simple</artifactId>

AdvancedCore/src/main/java/com/bencodez/advancedcore/api/javascript/JavascriptEngine.java

@@ -1,6 +1,7 @@
 package com.bencodez.advancedcore.api.javascript;
 
 import java.util.HashMap;
+import java.util.Map;
 import java.util.Map.Entry;
 
 import javax.script.ScriptEngine;
@@ -16,13 +17,17 @@
 import com.bencodez.simpleapi.messages.MessageAPI;
 
 public class JavascriptEngine {
-	private HashMap<String, Object> engineAPI;
+	private final HashMap<String, Object> engineAPI;
+	private final HashMap<String, String> placeholders;
+	private OfflinePlayer placeholderPlayer;
 
 	public JavascriptEngine() {
 		engineAPI = new HashMap<>();
+		placeholders = new HashMap<>();
 	}
 
 	public JavascriptEngine addPlayer(AdvancedCoreUser user) {
+		placeholderPlayer = user.getOfflinePlayer();
 		addToEngine("PlayerName", user.getPlayerName());
 		addToEngine("PlayerUUID", user.getUUID());
 		addToEngine("AdvancedCoreUser", user);
@@ -42,6 +47,7 @@ public JavascriptEngine addPlayer(CommandSender player) {
 		addToEngine("CommandSender", player);
 		if (player instanceof Player) {
 			Player p = (Player) player;
+			placeholderPlayer = p;
 			addToEngine("Player", p);
 			addToEngine("PlayerName", p.getName());
 			addToEngine("PlayerUUID", p.getUniqueId().toString());
@@ -58,6 +64,7 @@ public JavascriptEngine addPlayer(CommandSender player) {
 	}
 
 	public JavascriptEngine addPlayer(OfflinePlayer player) {
+		placeholderPlayer = player;
 		addToEngine("Player", player);
 		addToEngine("PlayerName", player.getName());
 		addToEngine("PlayerUUID", player.getUniqueId().toString());
@@ -76,6 +83,7 @@ public JavascriptEngine addPlayer(OfflinePlayer player) {
 
 	public JavascriptEngine addPlayer(Player player) {
 		if (player != null) {
+			placeholderPlayer = player;
 			addToEngine("Player", player);
 			addToEngine("PlayerName", player.getName());
 			addToEngine("PlayerUUID", player.getUniqueId().toString());
@@ -90,6 +98,13 @@ public JavascriptEngine addPlayer(Player player) {
 		return this;
 	}
 
+	public JavascriptEngine addPlaceholders(Map<String, String> placeholders) {
+		if (placeholders != null && !placeholders.isEmpty()) {
+			this.placeholders.putAll(placeholders);
+		}
+		return this;
+	}
+
 	public JavascriptEngine addToEngine(HashMap<String, Object> engineAPI) {
 		if (engineAPI != null && !engineAPI.isEmpty()) {
 			this.engineAPI.putAll(engineAPI);
@@ -119,7 +134,7 @@ public boolean getBooleanValue(String expression) {
 	}
 
 	public Object getResult(String expression) {
-		if (!expression.equals("")) {
+		if (expression != null && !expression.equals("")) {
 			if (!AdvancedCorePlugin.getInstance().getOptions().isJavascriptEngineEnabled()) {
 				return null;
 			}
@@ -128,6 +143,9 @@ public Object getResult(String expression) {
 				AdvancedCorePlugin.getInstance().debug("Failed to process javascript, engine == null");
 				return null;
 			}
+
+			String preparedExpression = JavascriptPlaceholderBinder.bind(expression, placeholderPlayer, placeholders, this);
+
 			engine.put("Bukkit", Bukkit.getServer());
 			engine.put("AdvancedCore", AdvancedCorePlugin.getInstance());
 			engine.put("Console", Bukkit.getConsoleSender());
@@ -142,7 +160,7 @@ public Object getResult(String expression) {
 			}
 
 			try {
-				return engine.eval(expression);
+				return engine.eval(preparedExpression);
 			} catch (ScriptException e) {
 				AdvancedCorePlugin.getInstance().getLogger().warning(
 						"Error occoured while evaluating javascript, turn debug on to see stacktrace: " + e.toString());

AdvancedCore/src/main/java/com/bencodez/advancedcore/api/javascript/JavascriptEngineHandler.java

@@ -274,6 +274,55 @@ private ScriptEngine getServerProvidedEngine() {
 		}
 	}
 
+	/**
+	 * Gets or prepares a Nashorn classloader for parser-only use. This is separate
+	 * from the active ScriptEngine so a server-provided Rhino/GraalJS engine can
+	 * still use Nashorn's parser for safe placeholder context detection.
+	 *
+	 * @return a classloader containing Nashorn's parser API, or null if unavailable
+	 */
+	public ClassLoader getOrCreateNashornParserClassLoader() {
+		if (nashornClassLoader != null) {
+			try {
+				Class.forName("org.openjdk.nashorn.api.tree.Parser", false, nashornClassLoader);
+				return nashornClassLoader;
+			} catch (ClassNotFoundException | LinkageError ignored) {
+			}
+		}
+		if (plugin == null) {
+			return null;
+		}
+
+		URLClassLoader loader = createParserClassLoader(PRIMARY_NASHORN_VERSION, ASM_VERSION_FOR_PRIMARY);
+		if (loader == null) {
+			loader = createParserClassLoader(FALLBACK_NASHORN_VERSION, ASM_VERSION_FOR_FALLBACK);
+		}
+		if (loader != null) {
+			nashornClassLoader = loader;
+		}
+		return loader;
+	}
+
+	private URLClassLoader createParserClassLoader(String nashornVersion, String asmVersion) {
+		try {
+			List<Path> jars = getOrDownloadJars(nashornVersion, asmVersion);
+			if (jars.isEmpty()) {
+				return null;
+			}
+			URLClassLoader loader = createClassLoader(jars);
+			try {
+				Class.forName("org.openjdk.nashorn.api.tree.Parser", false, loader);
+				return loader;
+			} catch (ClassNotFoundException | LinkageError e) {
+				closeQuietly(loader);
+				return null;
+			}
+		} catch (IOException e) {
+			logDebug(e);
+			return null;
+		}
+	}
+
 	/**
 	 * Gets a downloaded Nashorn engine.
 	 *