Merge pull request #146 from battlecode/carrier-attack-fix

iangonzalezh · web-flow · commit aefabd31ab97 · 2023-01-19T21:40:25.000-05:00
Fixed resource loss on carrier attacks
diff --git a/engine/src/main/battlecode/world/robots/InternalCarrier.java b/engine/src/main/battlecode/world/robots/InternalCarrier.java
@@ -33,7 +33,9 @@ public InternalCarrier(GameWorld gw, int id, RobotType type, MapLocation loc, Te
      */
     private void emptyResources() {
         for (ResourceType rType : ResourceType.values()) {
-            this.inventory.addResource(rType, -1*this.inventory.getResource(rType));
+            int amount = -1*this.inventory.getResource(rType);
+            this.inventory.addResource(rType, amount);
+            this.gameWorld.getTeamInfo().addResource(rType, getTeam(), amount);
         }
     }
 
diff --git a/specs/specs.md.html b/specs/specs.md.html
@@ -298,6 +298,7 @@
     - Passive resource gain number fixed
   - Engine Fixes:
     - Fix bug in checking islands held win condition
+    - Carrier attacks correctly deduct resources from teamInfo
 
 - Version 2.0.1 (January 18, 2023)
   - Balance Changes:

Original file line number	Diff line number	Diff line change
`@@ -33,7 +33,9 @@ public InternalCarrier(GameWorld gw, int id, RobotType type, MapLocation loc, Te`
`33`	`33`	`*/`
`34`	`34`	`private void emptyResources() {`
`35`	`35`	`for (ResourceType rType : ResourceType.values()) {`
`36`		`- this.inventory.addResource(rType, -1*this.inventory.getResource(rType));`
	`36`	`+ int amount = -1*this.inventory.getResource(rType);`
	`37`	`+ this.inventory.addResource(rType, amount);`
	`38`	`+ this.gameWorld.getTeamInfo().addResource(rType, getTeam(), amount);`
`37`	`39`	`}`
`38`	`40`	`}`
`39`	`41`