From 156b94ae89b0cd23611790f365878d69eacf15ea Mon Sep 17 00:00:00 2001
From: edo999 <edo999@gmail.com>
Date: Tue, 17 Sep 2024 13:10:18 +0100
Subject: [PATCH] =?UTF-8?q?Link=20costumes=20from=20scripts=20=F0=9F=91=94?=
 =?UTF-8?q?.?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 src/components/ScriptCodeInstruction.js | 54 +++++++++++++++++++++++++
 src/lib/parser/parseScriptCode.js       | 24 ++++++-----
 2 files changed, 69 insertions(+), 9 deletions(-)

diff --git a/src/components/ScriptCodeInstruction.js b/src/components/ScriptCodeInstruction.js
index 9c8e104..875b1c5 100644
--- a/src/components/ScriptCodeInstruction.js
+++ b/src/components/ScriptCodeInstruction.js
@@ -7,6 +7,59 @@ const prettifyArguments = (operation) => {
   let args = operation.slice(2);
 
   switch (opCode) {
+    // resourceRoutines
+    case 0x0c:
+    case 0x8c:
+      if (args[0].endsWith('Costume')) {
+        // Note: The link goes to costume set 0. The actual costume set used can't be determined statically.
+        args = [
+          <Link
+            className="underline"
+            to={`/costumes/0/${args[1]}`}>
+            {args[0]}({args[1]})
+          </Link>,
+        ];
+      } else if (args[0].endsWith('Room')) {
+        args = [
+          <Link
+            className="underline"
+            to={`/rooms/${args[1]}`}>
+            {args[0]}({args[1]})
+          </Link>,
+        ];
+      } else if (args[0].endsWith('Script')) {
+        args = [
+          <Link
+            className="underline"
+            to={`/scripts/${args[1]}`}>
+            {args[0]}({args[1]})
+          </Link>,
+        ];
+      }
+      break;
+    // actorOps
+    case 0x13:
+    case 0x53:
+    case 0x93:
+    case 0xd3:
+      if (args[1] === 'Color') {
+        args = [args[0], `${args[1]}(${args[3]}, ${args[2]})`];
+      } else if (args[1] === 'Name') {
+        args = [args[0], `${args[1]}("${args[2]}")`];
+      } else if (args[1] === 'Costume') {
+        // Note: The link goes to costume set 0. The actual costume set used can't be determined statically.
+        args = [
+          args[0],
+          <Link
+            className="underline"
+            to={`/costumes/0/${args[2]}`}>
+            {args[1]}({args[2]})
+          </Link>,
+        ];
+      } else {
+        args = [args[0], `${args[1]}(${args[2]})`];
+      }
+      break;
     case 0x18: // goTo
       args = [
         <Link
@@ -28,6 +81,7 @@ const prettifyArguments = (operation) => {
         args[3],
       ];
       break;
+    // loadRoom
     case 0x72:
       args = [
         <Link
diff --git a/src/lib/parser/parseScriptCode.js b/src/lib/parser/parseScriptCode.js
index f3a3640..21adb45 100644
--- a/src/lib/parser/parseScriptCode.js
+++ b/src/lib/parser/parseScriptCode.js
@@ -141,11 +141,11 @@ const parseScriptCode = (parser, startAddress, parentOffset = 0) => {
       case 0x0c:
       case 0x8c: {
         const resTypes = [
-          0, // Invalid
-          0, // Invalid
+          null, // Invalid
+          null, // Invalid
           'Costume',
           'Room',
-          '0', // Invalid
+          null, // Invalid
           'Script',
           'Sound',
         ];
@@ -169,7 +169,7 @@ const parseScriptCode = (parser, startAddress, parentOffset = 0) => {
         }
 
         const type = resTypes[subOp >> 4];
-        if (type === 0) {
+        if (!type) {
           console.error(`Resource Routines: Invalid type ${type}`);
         }
 
@@ -242,21 +242,27 @@ const parseScriptCode = (parser, startAddress, parentOffset = 0) => {
         const subOp = parser.getUint8();
         switch (subOp) {
           case 0x01:
-            scriptRow.push(`Sound(${subOpArg})`);
+            scriptRow.push(`Sound`);
+            scriptRow.push(subOpArg);
             break;
           case 0x02:
             const value = parser.getUint8();
-            scriptRow.push(`Color(${value}, ${subOpArg})`);
+            scriptRow.push(`Color`);
+            scriptRow.push(subOpArg);
+            scriptRow.push(value);
             break;
           case 0x03:
             const name = parser.getString();
-            scriptRow.push(`Name("${name}")`);
+            scriptRow.push(`Name`);
+            scriptRow.push(name);
             break;
           case 0x04:
-            scriptRow.push(`Costume(${subOpArg})`);
+            scriptRow.push(`Costume`);
+            scriptRow.push(subOpArg);
             break;
           case 0x05:
-            scriptRow.push(`TalkColor(${subOpArg})`);
+            scriptRow.push(`TalkColor`);
+            scriptRow.push(subOpArg);
             break;
           default:
             scriptRow.push(`// Unknown subop: 0x${hex(subOp, 2)}`);