Rename ADVANCED_CATAPULT bonus to reflect the purpose better

config/bonuses_texts.json

@@ -18,9 +18,9 @@
 		"description": "May Retaliate ${val} extra times"
 	},
 	
-	"ADVANCED_CATAPULT":
+	"CATAPULT_EXTRA_SHOTS":
 	{
-		"name": "Advanced wall breaker",
+		"name": "Additional siege attacks",
 		"description": "Can hit siege walls ${val} extra times per attack"
 	},
 

config/creatures/stronghold.json

@@ -273,7 +273,7 @@
 		{
 			"siegeDoubleAttack" :
 			{
-				"type" : "ADVANCED_CATAPULT",
+				"type" : "CATAPULT_EXTRA_SHOTS",
 				"val" : 1
 			}
 		},

lib/HeroBonus.h

@@ -216,7 +216,7 @@ public:
 	BONUS_NAME(DISGUISED) /* subtype - spell level */\
 	BONUS_NAME(VISIONS) /* subtype - spell level */\
 	BONUS_NAME(NO_TERRAIN_PENALTY) /* subtype - terrain type */\
-	BONUS_NAME(ADVANCED_CATAPULT) /*val - number of additional shots, requires CATAPULT bonus to work*/\
+	BONUS_NAME(CATAPULT_EXTRA_SHOTS) /*val - number of additional shots, requires CATAPULT bonus to work*/\
 	/* end of list */
 
 

server/CGameHandler.cpp

@@ -3961,7 +3961,7 @@ bool CGameHandler::makeBattleAction(BattleAction &ba)
 			else //may need to use higher ballistics level for creatures in future for some cases to match original H3 (upgraded cyclops etc)
 			{
 				sbi = VLC->heroh->ballistics.at(1);
-				sbi.shots += std::max(stack->valOfBonuses(Bonus::ADVANCED_CATAPULT), 0);
+				sbi.shots += std::max(stack->valOfBonuses(Bonus::CATAPULT_EXTRA_SHOTS), 0);
 			}
 
 			auto wallPart = gs->curB->battleHexToWallPart(ba.destinationTile);