Calculator fixes

1. fix log(base,num) giving fn is not defined error
2. fix max() not wokring
3. fix vec2. and vec3. operators not working
4. fix atan2 not working

Author: trulyno

kubejs/server_scripts/utils/calculator.js

@@ -467,7 +467,7 @@ let calculatorDefinitions = (() => {
         implementation: [
           {
             arguments: ["vec2"],
-            fn: (v) => ({ t: "number", v: Math.atan2(v[1], v[0]) }),
+            fn: (v) => ({ t: "number", v: Math.atan2(v.v[1], v.v[0]) }),
           },
           {
             arguments: ["number", "number"],
@@ -566,7 +566,7 @@ let calculatorDefinitions = (() => {
             arguments: ["number", "number"],
             fn: (base, v) => ({
               t: "number",
-              v: fn(JavaMath.log(v.v) / JavaMath.log(base.v)),
+              v: JavaMath.log(v.v) / JavaMath.log(base.v),
             }),
           },
           {
@@ -641,25 +641,27 @@ let calculatorDefinitions = (() => {
         name: "max",
         usage: formatOverloads("max", [[["...values:number[]"]]]),
         description: "returns the largest of the numbers given as input",
-        fn: function () {
-          let args = Array.from(arguments);
-          if (args.length === 0) {
-            throw new Error(
-              "wrong argument count for max: expected 1 or more, got 0",
-            );
-          }
-          if (args.some((arg) => arg.t !== "number")) {
-            throw new Error(
-              "wrong argument types for max: expected all numbers",
-            );
-          }
-          return {
-            t: "number",
-            v: Math.max.apply(
-              null,
-              args.map((arg) => arg.v),
-            ),
-          };
+        implementation: {
+          fn: function () {
+            let args = Array.from(arguments);
+            if (args.length === 0) {
+              throw new Error(
+                "wrong argument count for max: expected 1 or more, got 0",
+              );
+            }
+            if (args.some((arg) => arg.t !== "number")) {
+              throw new Error(
+                "wrong argument types for max: expected all numbers",
+              );
+            }
+            return {
+              t: "number",
+              v: Math.max.apply(
+                null,
+                args.map((arg) => arg.v),
+              ),
+            };
+          },
         },
       },
       {
@@ -983,7 +985,7 @@ let calculatorDefinitions = (() => {
               identifier.length <= 3 &&
               identifier.match(/^[xy]+$/)
             ) {
-              return vecAccessors(value, identifier.v);
+              return vecAccessors(value, identifier);
             }
             throw new Error(
               "non-existing member " + identifier + " for type " + value.t,
@@ -1006,7 +1008,7 @@ let calculatorDefinitions = (() => {
               identifier.length <= 3 &&
               identifier.match(/^[xyz]+$/)
             ) {
-              return vecAccessors(value, identifier.v);
+              return vecAccessors(value, identifier);
             }
             throw new Error(
               "non-existing member " + identifier + " for type " + value.t,
@@ -1318,7 +1320,7 @@ let calculatorExec = (() => {
   function execOverloaded(options, args, error) {
     for (let option of options) {
       if (option.arguments) {
-        if (args.every((arg, i) => option.arguments[i] === arg.t)) {
+        if (args.length === option.arguments.length && args.every((arg, i) => option.arguments[i] === arg.t)) {
           return option.fn.apply(null, args);
         }
       } else {