Merge pull request #3438 from AbstractMelon/hacker-macro-fixes

fix: Correct computer core macro ignoring hacker name

Author: alexanderson1993

server/events/computerCore.js

@@ -68,13 +68,13 @@ const hackerNames = [
 
 App.on(
   "computerCoreAddHacker",
-  ({id, simulatorId, level = Math.round(Math.random() * 9 + 1), cb}) => {
+  ({id, simulatorId, name, level = Math.round(Math.random() * 9 + 1), cb}) => {
     const computerCore = App.systems.find(
       s => s.class === "ComputerCore" && s.simulatorId === simulatorId,
     );
     if (!computerCore) return;
     computerCore.addUser({
-      name: randomFromList(hackerNames),
+      name: name || randomFromList(hackerNames),
       level,
       hacker: true,
     });

src/components/macros/computerCore.tsx

@@ -11,7 +11,7 @@ const ComputerCoreAddHacker: React.FC<MacroConfigProps> = ({
       <Label>
         Name
         <Input
-          defaultValue={args.name}
+          value={args.name || ""}
           onChange={e => updateArgs("name", e.target.value)}
         />
         <small>Leave blank for random hacker name</small>