Fix optimization issue in material_info

Author: LadyCailin

src/main/java/com/laytonsmith/core/functions/Minecraft.java

@@ -1268,7 +1268,7 @@ public ParseTree optimizeDynamic(Target t, Environment env,
 			if(children.size() < 1) {
 				return null;
 			}
-			if(ArgumentValidation.isNumber(children.get(0).getData())) {
+			if(children.get(0).isConst() && ArgumentValidation.isNumber(children.get(0).getData())) {
 				env.getEnv(CompilerEnvironment.class).addCompilerWarning(fileOptions,
 						new CompilerWarning("Numeric ids are deprecated in " + getName() + ".", t, null));
 			}

src/test/java/com/laytonsmith/core/functions/MinecraftTest.java

@@ -5,6 +5,7 @@
 import com.laytonsmith.abstraction.MCPlayer;
 import com.laytonsmith.abstraction.MCServer;
 import com.laytonsmith.abstraction.enums.MCVersion;
+import com.laytonsmith.core.MethodScriptCompiler;
 import com.laytonsmith.core.Static;
 import com.laytonsmith.core.environments.CommandHelperEnvironment;
 import com.laytonsmith.core.exceptions.ConfigCompileException;
@@ -81,4 +82,14 @@ public void testGetMCVersion() {
 		assertEquals(MCVersion.match(new String[]{"1", "8", "8"}), MCVersion.MC1_8_X);
 		assertEquals(MCVersion.match(new String[]{"3", "1", "4"}), MCVersion.MCX_X);
 	}
+
+	@Test
+	public void testMaterialInfoWithVariable() throws Exception {
+		// material_info's optimizeDynamic should not crash when the first
+		// argument is a variable (IVariable has no @typeof annotation).
+		String script = "assign(@mat, 'STONE') material_info(@mat, 'isBlock')";
+		MethodScriptCompiler.compile(
+				MethodScriptCompiler.lex(script, env, null, true),
+				env, env.getEnvClasses());
+	}
 }