Issue #2652 TacOps BA crits

megamek/src/megamek/common/BattleArmor.java

@@ -589,6 +589,16 @@ public String getMovementAbbr(EntityMovementType mtype) {
     @Override
     public HitData rollHitLocation(int table, int side, int aimedLocation, AimingMode aimingMode,
                                    int cover) {
+        return rollHitLocation(table, side, aimedLocation, aimingMode, cover, false);
+    }
+
+    /**
+     * Battle Armor units can only get hit in undestroyed troopers.
+     *
+     * @param isAttackingConvInfantry Set to true when attacked by CI, as these cannot score TacOps crits
+     */
+    public HitData rollHitLocation(int table, int side, int aimedLocation, AimingMode aimingMode,
+                                   int cover, boolean isAttackingConvInfantry) {
 
         // If this squad was killed, target trooper 1 (just because).
         if (isDoomed()) {
@@ -621,16 +631,13 @@ public HitData rollHitLocation(int table, int side, int aimedLocation, AimingMod
             loc = Compute.d6();
         }
 
-        int critLocation = Compute.d6();
-        // TacOps p. 108 Trooper takes a crit if a second roll is the same
-        // location as the first.
+        // TO:AR p.108: Trooper takes a crit if a second roll is the same location as the first.
         if (game.getOptions().booleanOption(OptionsConstants.ADVCOMBAT_TACOPS_BA_CRITICALS)
-                && (loc == critLocation)) {
+                && (loc == Compute.d6()) && !isAttackingConvInfantry) {
             return new HitData(loc, false, HitData.EFFECT_CRITICAL);
         }
         // Hit that trooper.
         return new HitData(loc);
-
     }
 
     @Override

megamek/src/megamek/common/weapons/infantry/InfantryWeaponHandler.java

@@ -206,4 +206,25 @@ public static double calculateBaseDamage(Entity ae, Mounted weapon, WeaponType w
             return ((InfantryWeapon) wtype).getInfantryDamage();
         }
     }
+
+    @Override
+    protected void initHit(Entity entityTarget) {
+        if ((entityTarget instanceof BattleArmor) && ae.isConventionalInfantry()) {
+            // TacOps crits against BA do not happen for infantry weapon attacks
+            hit = ((BattleArmor) entityTarget).rollHitLocation(toHit.getHitTable(),
+                    toHit.getSideTable(), waa.getAimedLocation(),
+                    waa.getAimingMode(), toHit.getCover(), true);
+            hit.setGeneralDamageType(generalDamageType);
+            hit.setCapital(wtype.isCapital());
+            hit.setBoxCars(roll == 12);
+            hit.setCapMisCritMod(getCapMisMod());
+            hit.setFirstHit(firstHit);
+            hit.setAttackerId(getAttackerId());
+            if (weapon.isWeaponGroup()) {
+                hit.setSingleAV(attackValue);
+            }
+        } else {
+            super.initHit(entityTarget);
+        }
+    }
 }