Config saving to other file.

Author: Moderocky

pom.xml

@@ -6,7 +6,7 @@
 
     <groupId>org.byteskript</groupId>
     <artifactId>byteskript</artifactId>
-    <version>1.0.19</version>
+    <version>1.0.20</version>
     <name>ByteSkript</name>
     <description>A compiled JVM implementation of the Skript language.</description>
 
@@ -113,11 +113,6 @@
             <version>4.13.2</version>
             <scope>test</scope>
         </dependency>
-        <dependency>
-            <groupId>org.panteleyev</groupId>
-            <artifactId>jpackage-maven-plugin</artifactId>
-            <version>1.5.1</version>
-        </dependency>
     </dependencies>
 
 </project>

src/main/java/org/byteskript/skript/lang/syntax/config/SaveConfigEffect.java

@@ -14,6 +14,8 @@
 import org.byteskript.skript.lang.element.StandardElements;
 import org.byteskript.skript.runtime.config.ConfigMap;
 
+import java.util.Map;
+
 @Documentation(
     name = "Save Config",
     description = """
@@ -29,21 +31,23 @@
 public class SaveConfigEffect extends Effect {
 
     public SaveConfigEffect() {
-        super(SkriptLangSpec.LIBRARY, StandardElements.EFFECT, "save config %Object%");
-    }
-    
-    @Override
-    public void preCompile(Context context, Pattern.Match match) throws Throwable {
-        final MethodBuilder method = context.getMethod();
-        method.writeCode(WriteInstruction.getField(System.class.getField("out")));
-        super.preCompile(context, match);
+        super(SkriptLangSpec.LIBRARY, StandardElements.EFFECT, "save config %Object% to %Object%", "save config %Object%");
     }
 
     @Override
     public void compile(Context context, Pattern.Match match) throws Throwable {
         final MethodBuilder method = context.getMethod();
-        method.writeCode(WriteInstruction.cast(CommonTypes.CONFIG));
-        method.writeCode(WriteInstruction.invokeVirtual(ConfigMap.class.getMethod("save")));
+        if (match.matchedPattern == 1) {
+            method.writeCode(WriteInstruction.cast(CommonTypes.CONFIG));
+            method.writeCode(WriteInstruction.invokeVirtual(ConfigMap.class.getMethod("save")));
+        } else {
+            method.writeCode(WriteInstruction.cast(CommonTypes.CONFIG)); // orig, new
+            method.writeCode(WriteInstruction.duplicateDrop2()); // new, orig, new
+            method.writeCode(WriteInstruction.swap());  // new, new, orig
+            method.writeCode(WriteInstruction.cast(CommonTypes.CONFIG)); // new, new, orig
+            method.writeCode(WriteInstruction.invokeVirtual(ConfigMap.class.getMethod("putAll", Map.class))); // new
+            method.writeCode(WriteInstruction.invokeVirtual(ConfigMap.class.getMethod("save")));
+        }
         context.setState(CompileState.CODE_BODY);
     }
 

src/test/resources/configs.bsk

@@ -10,4 +10,13 @@ function test:
             set "thing" from {conf} to "hello there!"
             assert "thing" from {conf} is "hello there!"
         delete hello.csk
+        set {conf2} to a new config
+        add "key: bean" to {conf2}
+        assert "key" from {conf2} is "bean"
+        save config {conf2} to hello.csk
+        set {conf} to hello.csk
+        assert "key" from {conf} is "bean"
+        delete hello.csk
+
+